Typing in Layers Dialog searches hidden/unhidden state instead of layer name

Bug #1219175 reported by Formerly Kevin Yin, now disabled
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
jazzynico

Bug Description

Windows 7, Inkscape 0.48.4, English

Make a document with a few layers. Make some of them hidden. Name one of them "yo".
Press Ctrl Shift L to bring up the Layers Dialog.
Click on a layer inside the Layers Dialog. Type "y": this will bring up a text box. At this point, Inkscape should be searching for a layer starting with the letter "y". Layer "yo" should be selected, but it won't be.
Press Backspace, then press "f". This will highlight the first hidden layer.
Press Backspace, then press "t". This will highlight the first visible layer.

Guess of cause of bug: Inkscape is searching the names in the Hidden/Visible column instead of the Layer Name column.

Tags: layers ui
Revision history for this message
jazzynico (jazzynico) wrote :

Confirmed on Windows XP, Inkscape trunk revision 12501.

tags: added: layers ui
Changed in inkscape:
importance: Undecided → Low
status: New → Triaged
Revision history for this message
jazzynico (jazzynico) wrote :

Patch attached. Tested on Windows XP, Inkscape trunk revision 12501.
Please test!

Changed in inkscape:
assignee: nobody → JazzyNico (jazzynico)
milestone: none → 0.49
status: Triaged → In Progress
tags: added: backport-proposed
Revision history for this message
jazzynico (jazzynico) wrote :

Patch tested on Crunchbang Waldorf (Debian stable) and committed revision 12503.

Changed in inkscape:
status: In Progress → Fix Committed
Revision history for this message
su_v (suv-lp) wrote :

Patch backported to stable <lp:inkscape/0.48.x> tested successfully with r9956 on OS X 10.7.5 (GTK+/X11 2.24.21).

Changed in inkscape:
milestone: 0.49 → 0.48.5
status: Fix Committed → In Progress
Revision history for this message
su_v (suv-lp) wrote :

@JazzyNico - do you have any concerns, or is if ok if I just go ahead and commit the diff to stable 0.48.x?

Revision history for this message
jazzynico (jazzynico) wrote :

@~suv - No problem, feel free to backport!

Revision history for this message
su_v (suv-lp) wrote :

Fix backported to lp:inkscape/0.48.x in revision 9963.

Changed in inkscape:
status: In Progress → Fix Committed
Kris (kris-degussem)
tags: removed: backport-proposed
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.