Closed
Bug 466288
Opened 16 years ago
Closed 15 years ago
Make use of type="number" for textboxes representing integer preferences
Categories
(Calendar :: Preferences, enhancement)
Calendar
Preferences
Tracking
(Not tracked)
RESOLVED
FIXED
1.0b1
People
(Reporter: ssitter, Assigned: Fallen)
Details
Attachments
(1 file, 1 obsolete file)
14.82 KB,
patch
|
dbo
:
review+
|
Details | Diff | Splinter Review |
We should use type="number" for textboxes representing integer preferences to benefit from automatic input check, range checks, spinbuttons, ... https://developer.mozilla.org/En/XUL/Textbox#a-textbox.type
Reporter | ||
Comment 1•16 years ago
|
||
- uses type="number" on general, alarms and connection page - limits event length and snooze time to 24 hours instead of just 999 minutes - limits alarm offset to 32767 to avoid overflow similar to Bug 405650
Assignee: nobody → ssitter
Status: NEW → ASSIGNED
Attachment #349561 -
Flags: review?(daniel.boelzle)
Comment 2•16 years ago
|
||
Stefan, I think, the function validateNaturalNums (http://mxr.mozilla.org/comm-central/source/calendar/base/content/calendar-ui-utils.js#594) should also be removed.
Comment 3•16 years ago
|
||
Comment on attachment 349561 [details] [diff] [review] fix Playing with the patch on Mac, my alarm prefs behave wrong, units toggle in a strange way. Moreover I think validateNaturalNums() is obsolete then, you should remove it.
Attachment #349561 -
Flags: review?(daniel.boelzle) → review-
Reporter | ||
Updated•15 years ago
|
Assignee: ssitter → nobody
Updated•15 years ago
|
Status: ASSIGNED → NEW
Assignee | ||
Comment 4•15 years ago
|
||
This should do it. I went over all <textboxes>. I didn't set a max on the default event length, I don't see why the max should be 1440 (days). No doubt its unreasonable to make the default length that large, but why limit the user?
Assignee: nobody → philipp
Attachment #349561 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #383153 -
Flags: review?(ssitter)
Reporter | ||
Comment 5•15 years ago
|
||
Comment on attachment 383153 [details] [diff] [review] Fix - v2 The max attribute was set to prevent integer overflows as written above. Daniel reviewed and denied the 1st patch, therefore he should review the 2nd too.
Attachment #383153 -
Flags: review?(ssitter) → review?(dbo.moz)
Comment 6•15 years ago
|
||
Philipp, did you spend some testing on this patch?
Assignee | ||
Comment 7•15 years ago
|
||
Yes, I've briefly looked at all number textboxes. I tested at least one of them to see if the max/min range works.
Updated•15 years ago
|
Attachment #383153 -
Flags: review?(dbo.moz) → review+
Comment 8•15 years ago
|
||
Comment on attachment 383153 [details] [diff] [review] Fix - v2 r=dbo
Assignee | ||
Comment 9•15 years ago
|
||
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/08625567a7c7> -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
Assignee | ||
Comment 10•13 years ago
|
||
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
You need to log in
before you can comment on or make changes to this bug.
Description
•