SM: Restore the datetimepicker bindings/styles after their removal in bug 1428849

RESOLVED FIXED in seamonkey2.56

Status

enhancement
RESOLVED FIXED
2 years ago
Last year

People

(Reporter: Paenglab, Assigned: stefanh)

Tracking

unspecified
seamonkey2.56
Dependency tree / graph

SeaMonkey Tracking Flags

(seamonkey2.56 fixed)

Details

Attachments

(2 attachments, 4 obsolete attachments)

Reporter

Description

2 years ago
Bug 1428849 removes bindings and styles for the datetimepicker. The only usage is the birthday field in AB.
Reporter

Updated

2 years ago
Blocks: 1428849
Assignee

Comment 1

2 years ago
Posted patch WIP (obsolete) — Splinter Review
Not tested yet.
Assignee: nobody → stefanh
Status: NEW → ASSIGNED
Assignee

Comment 2

2 years ago
Posted patch Another WIP (obsolete) — Splinter Review
Note to self: I missed Modern's cardDialog.css, but I should use a css file in content instead.
Attachment #8942470 - Attachment is obsolete: true
Assignee

Comment 3

2 years ago
Richard, I want the suite css files in another location - you're OK with this? This can go in before the suite-specific parts (provided that IanN is OK with this approach).
Attachment #8942475 - Attachment is obsolete: true
Attachment #8942517 - Flags: review?(richard.marti)
Assignee

Comment 4

2 years ago
Posted patch Part 2: Use the c-c bindings (obsolete) — Splinter Review
This assumes Part 1.
Attachment #8942518 - Flags: review?(iann_bugzilla)
Reporter

Comment 5

2 years ago
Comment on attachment 8942517 [details] [diff] [review]
Part 1: Add suite-specific paths in datetimepicker.xml

Looks good, thanks.
Attachment #8942517 - Flags: review?(richard.marti) → review+

Comment 6

Last year
Comment on attachment 8942518 [details] [diff] [review]
Part 2: Use the c-c bindings

>+++ b/suite/common/communicator.css
>+
>+/******* datepicker *******/
>+datepicker[type="popup"] {
>+  -moz-binding: url("chrome://communicator/content/bindings/datetimepicker.xml#datepicker-popup");
>+}
>+
>+datepicker[type="grid"] {
>+  -moz-binding: url("chrome://communicator/content/bindings/datetimepicker.xml#datepicker-grid");
>+}

Don't we also need to add bindings for:
datepicker {
  -moz-binding: url('chrome://communicator/content/bindings/datetimepicker.xml#datepicker');
}

timepicker {
  -moz-binding: url('chrome://communicator/content/bindings/datetimepicker.xml#timepicker');
}

Also if we have moved the bindings to suite/common, shouldn't datetimepicker.dtd be in a similar area with adjustments made to datetimepicker.xml

r=me with those addressed.
Attachment #8942518 - Flags: review?(iann_bugzilla) → review+
Assignee

Comment 7

Last year
The timepicker binding wasn't migrated, but I can add the datepicker one.
Assignee

Updated

Last year
Attachment #8944296 - Attachment description: For check-in (r=IanN) Part 2: Use the c-c bindings and adjust location of datetimepicker.dtd → Part 2 (r=IanN): Use the c-c bindings and adjust location of datetimepicker.dtd
Assignee

Updated

Last year
Attachment #8942518 - Attachment is obsolete: true
Assignee

Comment 9

Last year
This touches the c-c top dir and also mail/, so I'm sure Jorg wants to land both these attachments.
Keywords: checkin-needed
Assignee

Comment 10

Last year
Actually, I'm going to change one ifdef so the indentation matches what I do in common/src/customizeToolbar.xul in bug 1429862.
Keywords: checkin-needed
Assignee

Comment 12

Last year
Jorg, I'm sure you want to land these 2 attachments.
Keywords: checkin-needed

Comment 13

Last year
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/428b9f503c28
SM: Restore the datetimepicker bindings/styles after their removal in bug 1428849, part 1: Add suite-specific paths in datetimepicker.xml. r=Paenglab
https://hg.mozilla.org/comm-central/rev/2ef0cd0692a2
SM: Restore the datetimepicker bindings/styles after their removal in bug 1428849, part 2: Use the c-c bindings and adjust location of datetimepicker.dtd. r=IanN
Status: ASSIGNED → RESOLVED
Closed: Last year
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

Last year
Target Milestone: --- → Seamonkey2.56
Assignee

Updated

Last year

Comment 14

Last year
Pushed by stefanh@inbox.com:
https://hg.mozilla.org/comm-central/rev/469744d061c1
correct inconsistent use of quotation marks
Assignee

Comment 15

Last year
(In reply to Pulsebot from comment #14)
> Pushed by stefanh@inbox.com:
> https://hg.mozilla.org/comm-central/rev/469744d061c1
> correct inconsistent use of quotation marks

(I did a c&p from comment #6 and missed the fact that I then used ' instead of ")
You need to log in before you can comment on or make changes to this bug.