1561586 - Align message header with dismiss button for bookmark panel message

Status: RESOLVED FIXED

(Firefox :: Messaging System, task)

Description

[Andrei Oprea [:andreio]]

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.

Comment 1

[Andrei Oprea [:andreio]]

Attachment: Screenshot 2019-06-26 at 13.34.07.png

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.

Comment 2

[Andrei Oprea [:andreio]]

Attachment: Bug 1561586 - Align message header with dismiss button for CFR bookmark panel message

Comment 3

[Release mgmt bot [:sylvestre / :calixte / :marco for bugbug]]

Bugbug thinks this bug is a task, but please change it back in case of error.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/80e608691309
Align message header with dismiss button for CFR bookmark panel message r=r1cky

Comment 5

[Alexandru Michis [:malexandru]]

Backed out changeset 80e608691309 for causing failures in browser_asrouter_bookmarkpanel.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/fca7c7bdf3ee5fa5dd4d306a5dc09e330609b75b

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&fromchange=80e608691309cba09049d7a85d981bf0a06e052b&tochange=fca7c7bdf3ee5fa5dd4d306a5dc09e330609b75b&searchStr=chrome&selectedJob=253936707

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 -

Comment 6

[Andrei Oprea [:andreio]]

Mochitests need to be updated. I'll fix up the patch.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/921c89bebafe
Align message header with dismiss button for CFR bookmark panel message r=r1cky

Comment 8

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/921c89bebafe