[Bugs] #3406 sugar-artwork NORM: Theme new spinbutton correctly

[Sugar Labs Bugs]

#3406: Theme new spinbutton correctly
------------------------------+---------------------------------------------
    Reporter:  erikos         |          Owner:  erikos           
        Type:  defect         |         Status:  assigned         
    Priority:  Normal         |      Milestone:  0.96             
   Component:  sugar-artwork  |        Version:  Git as of bugdate
    Severity:  Major          |       Keywords:  12.1.0           
Distribution:  OLPC           |   Status_field:  Assigned         
------------------------------+---------------------------------------------

Comment(by garycmartin):

 Replying to [comment:7 erikos]:
 > Gary, I have a question on this one: In the above screenshot after
 applying Manuels patch we have the following layout: (value|-|+). Should
 we try to get it like (value(-|+) ?

 OK so first, Screenshot_spin_button.png does look cleaner than the
 original. I'm Not so convinced about adding an extra curve in a
 (value(-|+) layout. Just to confirm, is it possible to put the value in
 the centre or is that a no go from the implementation side? See attached
 SpinButton_value_centre.png for a quick mockup, I also mocked up an image
 where the value is in a curved container called
 Screenshot_spin_button_curve_value.png

 >And I presume there should be a | between the '-' and the '+'.

 If the '-' and '+' do have to go together, a 2px vertical grey line can
 separate them, see attached mockup
 SpinButton_smaller_with_grey_separator.png

 Lastly, looking at the screenshots so far, the widget total width is
 getting even larger not smaller, one of the issues with the new gtk3
 SpinButton was that it was so much wider than the previous version (e.g
 Abacus activity need to re-work all it's SpinButton UI). See
 SpinButton_size_animation.gif for an animation overlapping the screen
 grabs, even with the smaller font the widget still gets even wider! ;)

 >
 > And then when pressing the buttons it gets all white which I think is
 not the intended lookalike.
 >
 > (marking this one normal priority wise only as the buttons are
 functional, but not nice)

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/3406#comment:8>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system