Emil Ghitta, QA [:emilghitta] Reporter
	Description • 6 years ago

[Affected versions]:
Firefox 63.0b6 (BuildId:20180913141435)
Firefox 64.0a1 (BuildId:20180913100107)

[Affected platforms]:
Windows 10 64bit.
Ubuntu 16.04 64bit.
macOS 10.13.6

[Preconditions]
Enable the "Bookmarks" policy (with the "Folder" field completed) via the JSON or GPO.

[Steps to reproduce]:
1. Launch Firefox.
2. Access the about:policies#active page.

[Expected result]:
The Bookmarks policy fields are displayed under a dark gray background.

[Actual result]:
All the Bookmarks active policy information is displayed under a dark gray background except for the "Folder" information (which is displayed under a white background).

[Notes]
For further information regarding this issue please observe the attached screenshot.
Please note that this is reproducible only with the last fields for the Bookmarks policy (Reproducible with Favicon or Placement fields as well if they are the last fields for the Bookmark policy).
I didn't managed to reproduce this under other policies (I will try investigating this further).

	:Felipe Gomes (needinfo for replies!)
	Comment 1 • 6 years ago

This policies.json file reproduces the problem

	:Felipe Gomes (needinfo for replies!)
	Comment 2 • 6 years ago

The last row is missing the "odd" color class. I haven't verified the fix, but looking at the code, this seems to be the only case where the color class might accidentally not be added to the row: https://searchfox.org/mozilla-central/rev/dd965445ec47fbf3cee566eff93b301666bda0e1/browser/components/enterprisepolicies/content/aboutPolicies.js#166

so I bet that's the problem there

	Arshad Kazmi [:arshadkazmi42] Assignee
	Comment 3 • 6 years ago

i was looking into this, and trying to reproduce it. I am not sure about this

> [Preconditions]
> Enable the "Bookmarks" policy (with the "Folder" field completed) via the JSON or GPO.


Where can i find this options?

	:Felipe Gomes (needinfo for replies!)
	Comment 4 • 6 years ago

Hello Arshad,

take a look at the beginning of this README file here: https://github.com/mozilla/policy-templates/blob/master/README.md

You need to add a policies.json file to an specific folder inside the Firefox install dir. I already included one example policies.json file in this bug, so what you need to do is to download it and put it in the right location.

After you've done that, to verify that things are working correctly, you can:

- open about:support and see if it says that "Enterprise Policies" are "Active"
- open about:policies to see if the Bookmarks policy was picked up, and see the problem described in this bug

	:Felipe Gomes (needinfo for replies!)
	Comment 6 • 6 years ago

Comment on attachment 9013069 [details]
Bug 1491268 - Bookmarks policy's last field is displayed in same color like all other rows

:Felipe Gomes (needinfo me!) has approved the revision.

	Pulsebot
	Comment 7 • 6 years ago

Pushed by fgomes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b79da889f39b
Bookmarks policy's last field is displayed in same color like all other rows r=Felipe

	Daniel Varga [:dvarga]
	Comment 8 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/b79da889f39b

	Emil Ghitta, QA [:emilghitta] Reporter
	Comment 9 • 6 years ago

This issue is verified fixed using Firefox 64.0a1 (BuildId:20181002220140) on Windows 10 64bit, macOS 10.13.6 and Ubuntu 16.04.64bit.

	:Felipe Gomes (needinfo for replies!)
	Comment 10 • 6 years ago

Comment on attachment 9013069 [details]
Bug 1491268 - Bookmarks policy's last field is displayed in same color like all other rows

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1465942

User impact if declined: about:policies visual polish

Is this code covered by automated tests?: No

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: Yes

If yes, steps to reproduce: (QE has already tested it)

List of other uplifts needed: none

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Simple CSS classes fix

String changes made/needed: none

	Pascal Chevrel:pascalc (PTO until April 26)
	Comment 11 • 6 years ago

Comment on attachment 9013069 [details]
Bug 1491268 - Bookmarks policy's last field is displayed in same color like all other rows

Minimal CSS patch fixing a P2 and verified on nightly, uplift approved for 63 beta 12, thanks.

	Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)
	Comment 12 • 6 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/cfa284805fbf

	Emil Ghitta, QA [:emilghitta] Reporter
	Comment 13 • 6 years ago

This issue is verified fixed using Firefox 63.0b12 (BuildId:20181004174654) on Windows 10 64bit, macOS 10.13.6 and Ubuntu 18.04 64bit.