Opened 14 years ago
Closed 14 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)
by , 14 years ago
| Attachment: | mythuispinbox.diff added |
|---|
comment:1 by , 14 years ago
comment:2 by , 14 years ago
| Component: | MythTV - General → MythTV - User Interface Library |
|---|---|
| Owner: | set to |
| Status: | new → assigned |
| Version: | Unspecified → Master Head |
Stuart, I assume you would want to look at this
comment:3 by , 14 years ago
| Type: | Bug Report - General → Patch - Feature |
|---|
comment:4 by , 14 years ago
| 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 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |

Sorry I incorrectly fill the ticket, it is suppose to be a new feature for MythUI