On select toolbar "Object lock"-Button resize icon on clicking

Bug #1052645 reported by Alexander Janzen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
John Smith

Bug Description

On clicking on the mentioned toolbar button it resizes itself. Its may be because it is a small image on a bigger button.
This behavior appears on Windows (7/8) and Ubuntu 12.04, so I think it is a UI-bug.
I tried to track the bug, but I couldn't fixed it.

I think it appears on changing the iconId of the button in:
/inkscape/src/widgets/select-toolbar.cpp : toggle_lock(..)

Maybe there is something missing?

Tags: ui

Related branches

tags: added: ui
Revision history for this message
Alexander Janzen (anticultist+launchpad) wrote :
su_v (suv-lp)
Changed in inkscape:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
John Smith (john-smithi) wrote :

Fix committed as r11701.

FYI The problem was in the InkToggleAction widget itself.
When it creates the initial icon it wraps the icon in a GtkAlignment to ensure the icon doesn't resize to fill the whole button's space. But when the button is toggled , the new icon was not wrapped in a GtkAlignment - so was expanding to fill the whole buttons space (might have noticed the icon was blurry). Simple fix is to do the same wrapping when the new icon is set.

Changed in inkscape:
assignee: nobody → John Smith (john-smithi)
status: Confirmed → Fix Committed
Revision history for this message
su_v (suv-lp) wrote :

@John - finally! Many thx :-)
(This has been a small but visually annoying issue which AFAICT started with Inkscape 0.47 - glad you took the time to figure out how to fix it.)

Changed in inkscape:
milestone: none → 0.49
Revision history for this message
su_v (suv-lp) wrote :
tags: added: backport-proposed
Revision history for this message
jazzynico (jazzynico) wrote :

Backported to the branch, revision 9926.

tags: removed: backport-proposed
Changed in inkscape:
milestone: 0.49 → 0.48.4
Ted Gould (ted)
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.