Opened 12 years ago
Closed 12 years ago
#10393 closed Patch - Feature (fixed)
MythUISpinBox::AddSelection
Reported by: | Owned by: | stuartm | |
---|---|---|---|
Priority: | minor | Milestone: | 0.27 |
Component: | MythTV - User Interface Library | Version: | Master Head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
Allow to add special label for certain values.
This is used by the Frontend Settings UI, to associate label for some values.
Use case with PlayBack? group settings:
- Skip ahead (seconds) with a range from 0 to 200, where 0 is the "Default" value (I suppose it will skip by and amount of second defined by on other setting)
- Time stretch (speed x 100) with a range from 45 to 200, a special value "Default" (value = 0) is added (I suppose it will use the default time stretch defined by on other setting)
Attachments (1)
Change History (6)
Changed 12 years ago by
Attachment: | mythuispinbox.diff added |
---|
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
Component: | MythTV - General → MythTV - User Interface Library |
---|---|
Owner: | set to stuartm |
Status: | new → assigned |
Version: | Unspecified → Master Head |
Stuart, I assume you would want to look at this
comment:3 Changed 12 years ago by
Type: | Bug Report - General → Patch - Feature |
---|
comment:4 Changed 12 years ago by
Milestone: | unknown → 0.27 |
---|---|
Status: | assigned → accepted |
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.
comment:5 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
Sorry I incorrectly fill the ticket, it is suppose to be a new feature for MythUI