minimonth and timepicker should be scrollable

RESOLVED FIXED

Status

Calendar
Internal Components
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: Joey Minta, Assigned: Joey Minta)

Tracking

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

13 years ago
I had a friend over using my computer last week.  She'd never used Linux before but intuitively assumed that using the scrollwheel on the mini-calendar display in the system tray would change the displayed month.  We should implement this feature, especially in the minimonth, to cut down on the number of clicks necessary to move the displayed month.
(Assignee)

Comment 1

13 years ago
Created attachment 204226 [details] [diff] [review]
implement scrolling

Implements scrolling in all of our date and time widgets.
Assignee: base → jminta
Status: NEW → ASSIGNED
Attachment #204226 - Flags: first-review?(mvl)
I just tested this patch and have to admit that I really like the scrolling functionality in the minimonth / date picker to change to the next/prev. month.

But in the time picker the scrolling functionality (as implemented by this patch) seems to be for nothing. Yes, I can scroll in the time picker and the new field is highlighted; but I (or the user) still have to click on the hour/minute field to select and save it. 

If the selected time would be updated while scrolling (allowing open time picker - scroll - close time picker action) this would be great.
(Assignee)

Comment 3

13 years ago
Comment on attachment 204226 [details] [diff] [review]
implement scrolling

removing review request to take up ssitter's comment
Attachment #204226 - Attachment is obsolete: true
Attachment #204226 - Flags: first-review?(mvl)
(Assignee)

Comment 4

13 years ago
Created attachment 204280 [details] [diff] [review]
implement scrolling v2

Updated to reflect ssitter's comment.  Scrolling in the timepicker now dynamically updates the textbox.  Scrolling the month does not update the textbox because the selection does not carry over.
Attachment #204280 - Flags: first-review?(mvl)
(Assignee)

Comment 5

13 years ago
Created attachment 204289 [details] [diff] [review]
implement scrolling v3

The previous version didn't fire a 'change' event if the time had been adjusted via a scroll.  This patch fixes that, by routing all changes involving the grid through the 'onpopuphiding' event.
Attachment #204280 - Attachment is obsolete: true
Attachment #204289 - Flags: first-review?(mvl)
Attachment #204280 - Flags: first-review?(mvl)
Comment on attachment 204289 [details] [diff] [review]
implement scrolling v3

r=mvl
Attachment #204289 - Flags: first-review?(mvl) → first-review+
(Assignee)

Comment 7

13 years ago
patch checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.