Opened 12 years ago

Closed 12 years ago

#7073 closed defect (fixed)

Missing/wrong keybindings in start-early/end-late widgets

Reported by: gigem Owned by: stuartm
Priority: minor Milestone: 0.22
Component: MythTV - General Version: head
Severity: high Keywords:
Cc: Ticket locked: no


The keybindings for the start-early widget in the new schedule editor are backwards. LEFT should move the start time earlier and RIGHT should move it later.

For both the start-early and end-late widgets (and probably anything else derived from the same base widget), the ability to move in big increments using PAGEUP and PAGEDOWN seems to have been lost.

Change History (4)

comment:1 Changed 12 years ago by stuartm

Milestone: unknown0.22
Owner: changed from Isaac Richards to stuartm
Status: newaccepted
Version: unknownhead

The keybindings are correct, what is backwards is the contents of the list, the spinbox widget currently works from negative (left) to positive values (right). The start/end offsets work contrary to logical behaviour, a negative start offset means a later start time etc. I'll have to work around this in the widget by allowing a reverse ordering.

I'll override PAGEUP/PAGEDOWN for spinboxes to offer similar/identical behaviour to the old widgets used in the Scheduled Editor. They currently move through the items a 'page' at a time, where a page is the number of visible items, since the spinboxes in current themes only show a single item it only moves by one.

comment:2 Changed 12 years ago by stuartm

(In [21902]) Allow a spinbox to run from high to low when the supplied range calls for it. Refs #7073

comment:3 Changed 12 years ago by stuartm

(In [21903]) Reverse the range arguments to the spinbox for start offset. Negative values for start offset cause a recording to start late and vice-versa, which is confusing. This change causes the spinbox to display them in the logical order. Refs #7073

comment:4 Changed 12 years ago by stuartm

Resolution: fixed
Status: acceptedclosed

(In [22092]) Support paging by multiples of the step value in MythUISpinBox, defaults to multiples of 5 but the figure can be defined as an optional argument to SetRange?(). Requires a make clean. Fixes #7073

Note: See TracTickets for help on using tickets.