[mythtv] Ticket #10393: MythUISpinBox::AddSelection

Xavier Hervy xavier.hervy at gmail.com
Fri Oct 12 09:56:07 UTC 2012


I understand your concern but I can't see a better way in order to implement settings in MythUI. The spinbox  in MythSpinBoxDialog have to be themed large enough so one would fit all.
Thank you for committing it

Xavier 



Sent from Samsung Mobile (i.e sorry for top posting)MythTV <noreply at mythtv.org> wrote:#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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mythtv.org/pipermail/mythtv-dev/attachments/20121012/e20531b3/attachment.html>


More information about the mythtv-dev mailing list