Align message header with dismiss button for bookmark panel message
Categories
(Firefox :: Messaging System, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: andreio, Assigned: andreio)
References
Details
Attachments
(2 files)
As discovered through bug 1559946 in some locales due to the length of some strings the bookmark panel is a lot wider than in en-US. This was partly addressed in the previous bug but it did not properly account for the length of the title.
Assignee | ||
Comment 1•2 years ago
|
||
With the patch applied.
Previously the dismiss button was absolutely position. The issue with that is the title couldn't expand to full panel width without sometimes overlapping with the dismiss button. We had to limit the width of the title but that was problematic for locales where the panel is a lot wider.
Assignee | ||
Comment 2•2 years ago
|
||
Pushed by aoprea@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/80e608691309 Align message header with dismiss button for CFR bookmark panel message r=r1cky
Comment 5•2 years ago
|
||
Backed out changeset 80e608691309 for causing failures in browser_asrouter_bookmarkpanel.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/fca7c7bdf3ee5fa5dd4d306a5dc09e330609b75b
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=253932530&repo=autoland&lineNumber=4485
[task 2019-06-28T14:45:36.968Z] 14:45:36 INFO - TEST-PASS | browser/components/newtab/test/browser/browser_asrouter_bookmarkpanel.js | We sent a valid message - true == true -
[task 2019-06-28T14:45:36.969Z] 14:45:36 INFO - Console message: [JavaScript Error: "The character encoding of the plain text document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the file needs to be declared in the transfer protocol or file needs to use a byte order mark as an encoding signature." {file: "data:text/plain,test cfr fxa bookmark panel message" line: 0}]
[task 2019-06-28T14:45:36.969Z] 14:45:36 INFO - Console message: [JavaScript Error: "uncaught exception: [fluent][resolver] errors in en-US/newtab-section-header-pocket: ReferenceError: Unknown variable: provider."]
[task 2019-06-28T14:45:36.970Z] 14:45:36 INFO - Buffered messages finished
[task 2019-06-28T14:45:36.970Z] 14:45:36 INFO - TEST-UNEXPECTED-FAIL | browser/components/newtab/test/browser/browser_asrouter_bookmarkpanel.js | Should attach 4 children elements - 3 == 4 - JS frame :: chrome://mochitests/content/browser/browser/components/newtab/test/browser/browser_asrouter_bookmarkpanel.js :: test_fxa_message_shown :: line 41
[task 2019-06-28T14:45:36.971Z] 14:45:36 INFO - Stack trace:
[task 2019-06-28T14:45:36.971Z] 14:45:36 INFO - chrome://mochitests/content/browser/browser/components/newtab/test/browser/browser_asrouter_bookmarkpanel.js:test_fxa_message_shown:41
[task 2019-06-28T14:45:36.971Z] 14:45:36 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1115
[task 2019-06-28T14:45:36.971Z] 14:45:36 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1143
[task 2019-06-28T14:45:36.971Z] 14:45:36 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:1004
[task 2019-06-28T14:45:36.971Z] 14:45:36 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:803
[task 2019-06-28T14:45:36.972Z] 14:45:36 INFO - TEST-PASS | browser/components/newtab/test/browser/browser_asrouter_bookmarkpanel.js | Two fluent files required for translating the message - 2 == 2 -
Assignee | ||
Comment 6•2 years ago
|
||
Mochitests need to be updated. I'll fix up the patch.
Pushed by aoprea@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/921c89bebafe Align message header with dismiss button for CFR bookmark panel message r=r1cky
Comment 8•2 years ago
|
||
bugherder |
Updated•2 years ago
|
Description
•