Closed Bug 1196878 Opened 10 years ago Closed 10 years ago

Security error unreadable with dark theme

Categories

(DevTools :: Netmonitor, defect)

defect
Not set
normal

Tracking

(firefox43 fixed)

VERIFIED FIXED
Firefox 43
Tracking Status
firefox43 --- fixed

People

(Reporter: jryans, Assigned: grapherd, Mentored)

Details

(Whiteboard: [good first bug][lang=css])

Attachments

(2 files, 1 obsolete file)

The error text in the security panel is basically unreadable in the dark theme. See attached screenshot.
Mentor: jryans
Whiteboard: [good first bug][lang=css]
Hi Ryan, I want to work on this bug. Could you tell me where to make the change? I am checking the code and didn't found a css rule where to add the change for the color, so I think it's necessary create a new one. Added to it, could you tell me what is the color (variable with the color) you want for the text? Thanks
Flags: needinfo?(jryans)
Attached patch Bug 1196878.patch (obsolete) — Splinter Review
fixed Bug 1196878, add .theme-dark #security-error-message for font color.
Attachment #8651511 - Flags: review?(jryans)
(In reply to Giovanny Gongora [:gioyik] from comment #1) > Hi Ryan, > > I want to work on this bug. Could you tell me where to make the change? I am > checking the code and didn't found a css rule where to add the change for > the color, so I think it's necessary create a new one. Added to it, could > you tell me what is the color (variable with the color) you want for the > text? It looks like grapherd jumped ahead with a patch for this one, so it's probably best to look for another bug to try. In general though, you can use the Browser Toolbox[1] to inspect the tools and tweak style like regular web content. The various DevTools theme variables are listed on MDN[2]. As for which one... I wouldn't really know myself until I've looked at other similar text types. I'll be doing that momentarily as part of review. [1]: https://developer.mozilla.org/en-US/docs/Tools/Browser_Toolbox [2]: https://developer.mozilla.org/en-US/docs/Tools/DevToolsColors
Flags: needinfo?(jryans)
Assignee: nobody → grapherd
Status: NEW → ASSIGNED
Comment on attachment 8651511 [details] [diff] [review] Bug 1196878.patch This change looks good to me! However, can you correct the commit message to say "r=jryans" at the end? Also, I'll run a try build to ensure it's safe to land: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e8354fa7104
Attachment #8651511 - Flags: review?(jryans) → feedback+
Attached patch 1196878.patchSplinter Review
fixed reviewer in patch commit
Attachment #8652870 - Flags: review?(jryans)
Comment on attachment 8652870 [details] [diff] [review] 1196878.patch Review of attachment 8652870 [details] [diff] [review]: ----------------------------------------------------------------- Great, this looks good to me. The try run looks good too, so let's mark it for landing. Thank you for working on this!
Attachment #8652870 - Flags: review?(jryans) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
I have reproduced this bug on Nightly 43.0a1 (2015-08-20) on ubuntu 14.04 LTS, 32 bit! The bug's fix is now verified on Latest Beta 43.0b1! Build ID: 20151103023037 User Agent: Mozilla/5.0 (X11; Linux i686; rv:43.0) Gecko/20100101 Firefox/43.0 [bugday-20151104]
I have reproduced this bug on Nightly 43.0a1 (2015-08-20) on windows 8.1 , 64 bit! The bug's fix is now verified on Latest Beta 48.0b9! Build ID: 20160718142219 User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:48.0) Gecko/20100101 Firefox/48.0 [bugday-20160727]
Status: RESOLVED → VERIFIED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: