Closed
Bug 1491042
Opened 7 years ago
Closed 7 years ago
Selected tab has almost no padding-left/right in preferences Certificate Manager
Categories
(Toolkit :: Themes, defect, P1)
Tracking
()
VERIFIED
FIXED
mozilla64
| Tracking | Status | |
|---|---|---|
| firefox-esr60 | --- | unaffected |
| firefox62 | --- | unaffected |
| firefox63 | --- | verified |
| firefox64 | --- | verified |
People
(Reporter: Paenglab, Assigned: dao)
References
Details
(Keywords: regression)
Attachments
(2 files)
|
9.13 KB,
image/png
|
Details | |
|
2.96 KB,
patch
|
Paenglab
:
review+
pascalc
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
See screenshot.
STR:
1. Go to Preferences, Privacy & Security.
2. Scroll to the bottom of that page and click the 'View Certificates...' button.
3. In the modal dialog that pops up, notice the missing padding of the selected tab.
| Assignee | ||
Comment 1•7 years ago
|
||
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Attachment #9010190 -
Flags: review?(richard.marti)
| Assignee | ||
Updated•7 years ago
|
Priority: -- → P1
| Reporter | ||
Comment 2•7 years ago
|
||
Comment on attachment 9010190 [details] [diff] [review]
patch
Thanks!
Attachment #9010190 -
Flags: review?(richard.marti) → review+
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a3eb47f81993
Clean up padding, margin, and border overrides for in-content tabs after tabbox.css changes. r=Paenglab
Comment 4•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Updated•7 years ago
|
status-firefox62:
--- → unaffected
status-firefox63:
--- → affected
status-firefox-esr60:
--- → unaffected
| Assignee | ||
Updated•7 years ago
|
Flags: qe-verify+
| Assignee | ||
Comment 5•7 years ago
|
||
Comment on attachment 9010190 [details] [diff] [review]
patch
Approval Request Comment
[Feature/Bug causing the regression]: bug 1446168
[User impact if declined]: see comment 0
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: not yet
[Needs manual test from QE? If yes, steps to reproduce]:
[List of other uplifts needed for the feature/fix]: /
[Is the change risky?]: no
[Why is the change risky/not risky?]: straightforward CSS cleanup
[String changes made/needed]: /
Attachment #9010190 -
Flags: approval-mozilla-beta?
Comment 6•7 years ago
|
||
Comment on attachment 9010190 [details] [diff] [review]
patch
CSS fix for a P1 regression, patch on nightly for a week, uplift approved for 63 beta 10, thanks.
Attachment #9010190 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•7 years ago
|
||
I successfully reproduced the issue on Firefox Nightly 64.0a1 (2018-09-13) under Windows 10 (x64) using STR from Comment 0.
The issue is verified fixed on latest Nightly 64.0a1 (2018-09-25) under Windows 10 (x64).
I'll be verifying on Beta too, as soon as the build with the fix will be available.
Comment 8•7 years ago
|
||
| bugherder uplift | ||
| Assignee | ||
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
Comment 9•7 years ago
|
||
The issue is verified fixed on Firefox Beta 63.0b10 under Windows 10 (x64).
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•