| Summary: | Web Inspector: Two lines in CSS rule are hidden until resize or click | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Timothy Hatcher <timothy> | ||||||||||
| Component: | Web Inspector | Assignee: | Jonathan Wells <jonowells> | ||||||||||
| Status: | RESOLVED FIXED | ||||||||||||
| Severity: | Normal | CC: | graouts, joepeck, timothy, webkit-bug-importer | ||||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||||
| Version: | 528+ (Nightly build) | ||||||||||||
| Hardware: | All | ||||||||||||
| OS: | All | ||||||||||||
| Attachments: |
|
||||||||||||
Created attachment 233178 [details]
Bug
Created attachment 233179 [details]
After Resize
This is a bug in CodeMirror 4.1. This is fixed in CodeMirror 4.2. I will update. Created attachment 233212 [details]
[PATCH] Attempted fix, update to CodeMirror 4.2.0
Comment on attachment 233212 [details]
[PATCH] Attempted fix, update to CodeMirror 4.2.0
Looks good to me. Do PrettyPrinting tests continue to pass?
The second one fails: /* PASS: js-tests/block-comment.js */ /* FAIL: js-tests/single-statement-blocks.js */ /* PASS: js-tests/switch-case-default.js */ However I think this has been the case maybe since the original CodeMirror update. This can be fixed in another bug. This might need a rebaseline. |
Created attachment 233177 [details] Test Case The color and background-color on the attached page's icon don't show up in the rule sidebar until some refresh happens.