Closed Bug 1730408 Opened 3 years ago Closed 3 years ago

Support <input type=time> and <input type=date > in SeaMonkey

Categories

(SeaMonkey :: Themes, enhancement)

enhancement

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: frg, Assigned: frg)

References

(Blocks 1 open bug)

Details

(Whiteboard: SM2.53.10)

Attachments

(2 files, 2 obsolete files)

The support for these input types is unfortunately implemented in the backend (toolkit) and frontend (browser) so we need to port some parts of Bug 1288591 and followups. Not 100% a theme bug but close.

Attached patch WIP-1730408-datetimepicker.patch (obsolete) — Splinter Review

wip patch. Modern needs further styling. Mostly copied from toolkit and merged with the current implementation.

Blocks: 1284504
Attached patch 1730408-datetimepicker.patch (obsolete) — Splinter Review

I think this one is reviewable to get it into 2.53.10. njsg did the Modern styling. Maybe the button bar is a bit bulky. Drop down marker is a ^ char and so a bit too high. When you select a month/year the popup has white chars in Windows but otherwise i think seem to fit Modern and Classic uses the toolkit style.

Attachment #9240802 - Attachment is obsolete: true
Attachment #9246044 - Flags: review?(iann_bugzilla)
Attachment #9246044 - Flags: approval-comm-release?
Attachment #9246044 - Flags: approval-comm-esr60?

I didn't see it before, but there is a reset button, which is enabled/shown when the date input field has some value, but needs chrome://global/skin/icons/input-clear.svg which is not present in modern.

New version with the clear icon. Think it can stay as is. Maybe change the color from gray to a Modern bluish?

Attachment #9246044 - Attachment is obsolete: true
Attachment #9246044 - Flags: review?(iann_bugzilla)
Attachment #9246044 - Flags: approval-comm-release?
Attachment #9246044 - Flags: approval-comm-esr60?
Attachment #9246159 - Flags: review?(iann_bugzilla)
Attachment #9246159 - Flags: approval-comm-release?
Attachment #9246159 - Flags: approval-comm-esr60?

Comment on attachment 9246159 [details] [diff] [review]
1730408-datetimepicker.patch

[Triage Comment]
LGTM r/a=me
Any further tweaks can go into a follow-up bug

Attachment #9246159 - Flags: review?(iann_bugzilla)
Attachment #9246159 - Flags: review+
Attachment #9246159 - Flags: approval-comm-release?
Attachment #9246159 - Flags: approval-comm-release+
Attachment #9246159 - Flags: approval-comm-esr60?
Attachment #9246159 - Flags: approval-comm-esr60+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/6c45a869f54d
Add picker for date and time inputs. r=IanN

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

Followup. Add some dupes to the allowed dupes list.

trivial already in the prerelease for a week so r/a=me.

Attachment #9247457 - Flags: review+
Attachment #9247457 - Flags: approval-comm-release+
Attachment #9247457 - Flags: approval-comm-esr60+
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/5b8474af1a33
Add duplicate modern theme files for datepicker to allow list. r=me
Whiteboard: SM2.53.10
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: