Bug 34993
| Summary: | Web Inspector: not updating the elements attributes when it's opened | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Alberto Calvo <intemperie> |
| Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Major | CC: | burg, bweinstein, joepeck, keishi, knorton, pfeldman, pmuellr, rik |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Mac (Intel) | ||
| OS: | OS X 10.6 | ||
| URL: | http://tapbots.com/software/pastebot/#overview | ||
Alberto Calvo
Hi there.
Just found a bug on the Inspector, when I was complaining about the lack of this feature. Tim Hatcher pointed me that Inspector should update the tree, but it doesn't. Not when the Inspector is opened but when it's closed. In example, you toggle a div's display mode via JS. The tree doesn't update the inline styles if the Inspector is opened. However, if it's closed, it updates normally. Both latest nightly and Safari 4.0.4.
You could check it out on the URL I pointed out, triggering the Demo Video.
Also, it would be nice to add some highlighting on this dynamic changes a-la Firebug (I guess I should open another ticket for this).
Cheers,
Alberto
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Pavel Feldman
Could you please provide more details? I am interested in exact scenario, div you are talking about, expected style values before / after the action and the actual ones. Are you saying div's attribute is not getting updated?
Alberto Calvo
Yes, <div id="video-container" style="visibility: hidden; zoom: 1; opacity: 1; "> doesn't get updated when you click on "Watch the video", while it triggers visibility to visible. Same for any other similar tests I did with other pages/situations.
Anthony Ricaud
Updating attributes was introduced in http://trac.webkit.org/changeset/53232 via https://bugs.webkit.org/show_bug.cgi?id=20162.
Anthony Ricaud
Doesn't work in nightly 53415 from January 18th.
Anthony Ricaud
This regressed with bug 33915.