Summary: | Web Inspector: Shorthand Properties Getting Deleted When Editing | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bweinstein, joepeck, keishi, pfeldman, pmuellr, rik, timothy, yurys | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Joseph Pecoraro
2010-05-11 17:39:57 PDT
Created attachment 55786 [details]
[PATCH] Easy Fix
Easy!
<radr://7972069> Created attachment 55788 [details]
[TEST]: Sample Test Case
Steps to Reproduce:
1. Open Test Case html file
2. Right Click + Inspect the Text
3. Double Click to Edit the margin value
4. Put cursor by the number
5. Push Up/Down a few times
Expected Results:
Only the margin should change.
Actual Results:
The margin changes, but after a few increments/decrements the border is deleted, without warning! Deselecting and Reselecting the original node shows that it has disappeared.
Comment on attachment 55786 [details]
[PATCH] Easy Fix
Consider landing before the CQ gets to it.
Thanks for fixing it. Any chance we get a test for this? (In reply to comment #5) > Thanks for fixing it. Any chance we get a test for this? Do you mean an inspector test? I gave this only a brief thought at the time but figured since it was such a straight forward fix (if shorthand -> getShorthandValue) that I'd just put up the patch. But, now that you mention it, I think it would be nice to have a patch testing the styles output in general. To catch a case like this style manipulation functions can be run. I'll land this, because I feel its something urgent (this hits me quite a bit) and open up a follow up bug that styles need more coverage. Committed r59210 M WebCore/ChangeLog M WebCore/inspector/front-end/DOMAgent.js r59210 = 0348fb02fde32b1c173f3f2d1e03f3b0d5662603 (refs/remotes/trunk) http://trac.webkit.org/changeset/59210 |