Closed
Bug 1447751
Opened 7 years ago
Closed 7 years ago
Remove the grippy in TB after it's removal in bug 1446157
Categories
(Thunderbird :: General, enhancement)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 61.0
People
(Reporter: Paenglab, Assigned: Paenglab)
References
Details
Attachments
(1 file)
|
6.67 KB,
patch
|
jorgk-bmo
:
review+
Fallen
:
review+
|
Details | Diff | Splinter Review |
Bug 1446157 removes the grippy binding. Actually we show no grippy in TB and it's no problem to remove the code in calendar, chat and TB.
|
Assignee | |
Comment 1•7 years ago
|
||
Jörg, the sanitizeDialof part is a port of https://reviewboard.mozilla.org/r/229346/diff/1#index_header.
Florian, chat has two occurrencies of grippy. The first has display:none which already makes all grippies in main window not shown, also the today pane splitter grippy.
Philipp, as wrote above the today pane grippy is already not shown. The event-box grippies don't use the grippy binding and are safe as they are.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8961059 -
Flags: review?(philipp)
Attachment #8961059 -
Flags: review?(jorgk)
Attachment #8961059 -
Flags: review?(florian)
|
||
Comment 2•7 years ago
|
||
Comment on attachment 8961059 [details] [diff] [review]
no-grippy.patch
Thanks, looks good.
Attachment #8961059 -
Flags: review?(jorgk) → review+
|
||
Updated•7 years ago
|
Attachment #8961059 -
Flags: review?(philipp) → review+
|
|
Assignee | |
Comment 3•7 years ago
|
||
Bug 1446157 is in autoland and lands on next merge.
Florian, please can you review the chat part?
Flags: needinfo?(florian)
|
|
||
Comment 4•7 years ago
|
||
Or I have to steal this trivial review ;-)
|
|
||
Updated•7 years ago
|
Attachment #8961059 -
Flags: review?(florian)
|
|
||
Updated•7 years ago
|
Target Milestone: --- → Thunderbird 61.0
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/d424678da2d1
Remove the grippy in TB after its removal in bug 1446157. r=jorgk,philipp
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
|
|
||
Updated•7 years ago
|
Flags: needinfo?(florian)
You need to log in
before you can comment on or make changes to this bug.
Description
•