Closed Bug 1360082 Opened 6 years ago Closed 5 years ago
detail is not visible for bare json strings in response viewer
Bug 1360082 - Network monitor: use devtools theme alt color for bare json strings in response viewer
59 bytes, text/x-review-board-request
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0 Build ID: 20170418123106 Steps to reproduce: 1. Open the developer tools. 2. Make a request that returns content-type application/json and has a body of a quoted string by itself. I've put this up for testing: https://jc6e81at9k.execute-api.us-west-2.amazonaws.com/dev/jsonstring 3. Click on the network tab, then the request you just made, then the response tab. Actual results: It's blank, showing only the word "JSON" that always appears at the top. Expected results: It should show the string returned. A bare quoted string is perfectly valid JSON. JSON.stringify will produce one, and JSON.parse will accept one. (I reported this against 54, but it's not actually new.)
It exists but is almost invisible because it is almost white: color: rgb(245, 247, 250); tree-view.css:79 .treeTable .treeRow.selected:not(:hover) * var(--theme-selection-color) If set devtools use dark theme, it is visible.
Status: UNCONFIRMED → NEW
Has STR: --- → yes
Ever confirmed: true
OS: Unspecified → All
Hardware: Unspecified → All
(In reply to YF (Yang) from comment #1) > It exists but is almost invisible because it is almost white: > color: rgb(245, 247, 250); > > tree-view.css:79 > .treeTable .treeRow.selected:not(:hover) * > var(--theme-selection-color) > > > If set devtools use dark theme, it is visible. Seem not for Firefox 54, white text for Fx55 and Nightly 56.
Priority: -- → P2
Hi, I would like to work on this one.
Comment on attachment 8899199 [details] Bug 1360082 - Network monitor: use devtools theme alt color for bare json strings in response viewer https://reviewboard.mozilla.org/r/170470/#review177412 Works for me, thanks for the patch! Honza
Attachment #8899199 - Flags: review?(odvarko) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/bfce5393f345 Network monitor: use devtools theme alt color for bare json strings in response viewer r=Honza
We probably want this to ride the train.
You need to log in before you can comment on or make changes to this bug.