Does launcher icon size slider have to support odd numbers?

[George Karavasilev]

Unity has that handy launcher icon size slider in the Appearance pane that ... well, it control the launcher icon size.
However it also supports making the launcher icon size odd numbers which results in icons looking blurry like this:
http://i.imgur.com/LtVAW3G.png
And with even numbers that doesn't happen:
http://i.imgur.com/mWuqJb5.png
Is it sane to keep the odd numbers as an option then?
Unity tweak Tool (which you may know I designed [almost entirely]) doesn't support odd numbers to prevent the user from making his launcher have blurred icons.

[Stephen M. Webb (bregma)]

Yes, there's a an old bug about that somewhere, but no decision on how to resolve it.

This would make an excellent papercut for someone to fix.

[George Karavasilev (kokoto-java)]

Simplest solution - make the slider jump between 2 rather then 1, we did exactly this for Unity Tweak Tool. If there is a ,ui or .glade file for the appearance window somewhere it is only a matter of adjusting an attribute for the gtkslider from glade or from manually editing the file with text editor (it uses xml syntax). Where is the source three of that appearance pane?

[asmoore82 (asmoore82)]

I think I've found it in the unity-control-center project: panels/appearance/appearance.ui

[George Karavasilev (kokoto-java)]

Me too, sadly restrciting a gtk scale step to step between two and not one isn't as easy as restricting a spinbutton (as in unity Tweak Tool). And GTK docs are being not really helpful.

[George Karavasilev (kokoto-java)]

I've fixeed the bug. In midnight. ;D
Evidence: https://dl.dropboxusercontent.com/u/2656555/Screencast%202014-03-14%2000%3A01%3A17.mp4
Should I make a MR?

[Stephen M. Webb (bregma)]

@George

Please go ahead and make a merge proposal. We will have the idea reviewd by the Unity UI design team once they have something to play with.

[George Karavasilev (kokoto-java)]

@Stephen

Done - https://code.launchpad.net/~kokoto-java/unity-control-center/fix-lp-1292112/+merge/211032

[Adolfo Jayme Barrientos (fitojb)]

Dupe of an earlier bug - moving everything to there if you don't mind :D