Closed
Bug 1106792
Opened 10 years ago
Closed 10 years ago
Links to clear history and remove cookies have lost their style
Categories
(Firefox :: Settings UI, defect)
Tracking
()
People
(Reporter: ken.zalewski, Assigned: aryx)
References
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:34.0) Gecko/20100101 Firefox/34.0
Build ID: 20141126041045
Steps to reproduce:
Go to Options -> Privacy.
In the History group box, make sure "Remember history" is selected.
Actual results:
The links for "clear your recent history" and "remove individual cookies" have lost their style. There is no visual indicator that "clear your recent history" and "remove individual cookies" are selectable.
Expected results:
In versions of Firefox prior to 34.0, the "clear your recent history" and "remove individual cookies" links were properly styled, clearly indicating that they are selectable links.
Regression range:
good=2014-08-26
bad=2014-08-27
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=dc352a7bf234&tochange=0753f7b93ab7
Birunthan Mohanathas — Bug 1009370 - Part 1: Move .inline-link style from preferences.css to common.inc.css. r=Unfocused
Blocks: 1009370
Status: UNCONFIRMED → NEW
status-firefox34:
--- → affected
tracking-firefox35:
--- → ?
tracking-firefox36:
--- → ?
tracking-firefox37:
--- → ?
Component: Untriaged → Preferences
Ever confirmed: true
Flags: needinfo?(birunthan)
Keywords: regression
[Tracking Requested - why for this release]:
tracking-firefox34:
--- → ?
Comment 3•10 years ago
|
||
Ken, could you please attach a screenshot of the bug and maybe another one for the expected look?
Flags: needinfo?(ken.zalewski)
Reporter | ||
Comment 5•10 years ago
|
||
Thanks Loic.
Your screenshots match the link style problem that I am seeing. It's very straightforward. The two links were previously styled to appear selectable, and now they are not.
In my opinion, these links should be buttons, to make it clear that they can be clicked and that they will open new dialogs. For now, however, I'll be happy just to see the styled links restored.
Updated•10 years ago
|
Comment 6•10 years ago
|
||
This isn't a severe enough issue to fix in 34. I have tracked for other releases as I think that we should take the fix in 35+.
status-firefox35:
--- → affected
status-firefox36:
--- → affected
status-firefox37:
--- → affected
Flags: firefox-backlog?
Assignee | ||
Comment 7•10 years ago
|
||
Updated•10 years ago
|
Attachment #8536222 -
Flags: review?(dao) → review+
Updated•10 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
Updated•10 years ago
|
Flags: firefox-backlog? → firefox-backlog+
Updated•10 years ago
|
Flags: needinfo?(birunthan)
Assignee | ||
Comment 9•10 years ago
|
||
From #developers:
[2014-12-15 10:22:50] * Archaeopteryx * Tomcat|sheriffduty: hi, do i need a try run for a m-c patch which only reverts some css removal? bug 1106792
[2014-12-15 10:26:51] * Tomcat|sheriffduty * Archaeopteryx: well depened if you think this is trivial and don't break any test then its fine
Attachment #8536222 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 10•10 years ago
|
||
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 11•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 37
Updated•10 years ago
|
Iteration: --- → 37.2
Flags: qe-verify?
Updated•10 years ago
|
Flags: qe-verify? → qe-verify+
QA Contact: camelia.badau
Comment 12•10 years ago
|
||
Can we get a nomination for uplift here? This needs to be nominated before Mon Dec 22 beta.
Flags: needinfo?(archaeopteryx)
Assignee | ||
Comment 13•10 years ago
|
||
Comment on attachment 8537309 [details] [diff] [review]
patch, v2, r=dao
Thank you for the reminder.
Approval Request Comment
[Feature/regressing bug #]: Bug 1009370
[User impact if declined]: Links 'clear your recent history' and 'remove individual cookies' in the Privacy pane of the Preferences window (which we still use for beta and release) won't be recognizable as links
[Describe test coverage new/current, TBPL]: Tested with nightly build
[Risks and why]: minimal (only CSS changes which are reverts)
[String/UUID change made/needed]: none
Flags: needinfo?(archaeopteryx)
Attachment #8537309 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 14•10 years ago
|
||
Comment on attachment 8537309 [details] [diff] [review]
patch, v2, r=dao
Approval Request Comment
[Feature/regressing bug #]: Bug 1009370
[User impact if declined]: Links 'clear your recent history' and 'remove individual cookies' in the Privacy pane of the Preferences window (which we still use for beta and release) won't be recognizable as links
[Describe test coverage new/current, TBPL]: Tested with nightly build
[Risks and why]: minimal (only CSS changes which are reverts)
[String/UUID change made/needed]: none
Attachment #8537309 -
Flags: approval-mozilla-beta?
Updated•10 years ago
|
Attachment #8537309 -
Flags: approval-mozilla-beta?
Attachment #8537309 -
Flags: approval-mozilla-beta+
Attachment #8537309 -
Flags: approval-mozilla-aurora?
Attachment #8537309 -
Flags: approval-mozilla-aurora+
Comment 15•10 years ago
|
||
Verified fixed on Windows 7 64bit, Ubuntu 13.10 32bit and Mac OSX 10.9.5 using latest Nightly 37.0a1 (buildID: 20141221030204).
Status: RESOLVED → VERIFIED
Comment 16•10 years ago
|
||
Comment 18•10 years ago
|
||
Verified fixed on latest DevEdition, build ID: 20141223004006.
Comment 19•10 years ago
|
||
Verified fixed on Firefox 35 Beta 8, buildID: 20141229214612.
You need to log in
before you can comment on or make changes to this bug.
Description
•