[mythtv-commits] Ticket #10393: MythUISpinBox::AddSelection

[MythTV]

#10393: MythUISpinBox::AddSelection
---------------------------------------------+-----------------------------
 Reporter:  Xavier Hervy <xavier.hervy@…>    |          Owner:  stuartm
     Type:  Patch - Feature                  |         Status:  accepted
 Priority:  minor                            |      Milestone:  0.27
Component:  MythTV - User Interface Library  |        Version:  Master Head
 Severity:  medium                           |     Resolution:
 Keywords:                                   |  Ticket locked:  0
---------------------------------------------+-----------------------------
Changes (by stuartm):

 * status:  assigned => accepted
 * milestone:  unknown => 0.27


Comment:

 Xavier, I like this and I have been meaning to implement something similar
 for a long time. The only thing which is troubling me is the substitution
 of a label for the default/special value. It would look good, and clearly
 there is a need for any special values to be given a label to explain
 their meaning, but from a theme perspective spinboxes would have to become
 much wider to accommodate that label - Default is more than twice as wide
 as 200, and translations even more so (e.g. Spanish is 'Por defecto').
 This ultimately spoil some effects, leaving spinboxes appearing too wide
 when showing values such as '1' and making something like that shown in
 this image impossible -
 http://miffteevee.co.uk/imagebin/spinbox_concept.png

 I can't think of a way to avoid that though and it's not exactly a problem
 with the patch so much as the code that uses it.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10393#comment:4>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center