RESOLVED FIXED 153699
Web Inspector: Object tree parent items are misaligned
https://bugs.webkit.org/show_bug.cgi?id=153699
Summary Web Inspector: Object tree parent items are misaligned
Devin Rousso
Reported 2016-01-29 21:30:01 PST
Created attachment 270291 [details] [Image] Issue .
Attachments
[Image] Issue (12.52 KB, image/png)
2016-01-29 21:30 PST, Devin Rousso
no flags
[Image] Another Issue (12.46 KB, image/png)
2016-01-29 21:43 PST, Devin Rousso
no flags
Patch (2.32 KB, patch)
2016-01-29 21:47 PST, Devin Rousso
darin: review+
Patch (2.16 KB, patch)
2016-01-30 12:51 PST, Devin Rousso
no flags
Radar WebKit Bug Importer
Comment 1 2016-01-29 21:30:18 PST
Nikita Vasilyev
Comment 2 2016-01-29 21:31:51 PST
Steps to reproduce? Is it a regression?
Devin Rousso
Comment 3 2016-01-29 21:39:25 PST
(In reply to comment #2) > Steps to reproduce? > Is it a regression? I was debugging some Visual sidebar stuff and came across this. Entering the following into the console and expand the first item: WebInspector.cssStyleDetailsSidebarPanel._visualStyleDetailsPanel._groups.margin.properties.marginTop.style.properties It seems like it may be a regression of the recent TreeOutline overhaul, but I'm investigating.
Devin Rousso
Comment 4 2016-01-29 21:43:07 PST
Created attachment 270294 [details] [Image] Another Issue Looks like I missed a testcase for <https://bugs.webkit.org/show_bug.cgi?id=153676>...
Devin Rousso
Comment 5 2016-01-29 21:47:31 PST
Darin Adler
Comment 6 2016-01-30 11:28:13 PST
Comment on attachment 270295 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=270295&action=review > Source/WebInspectorUI/ChangeLog:16 > + This would apply to all .icon elements contained whithin a non-parent item, Typo: within
Devin Rousso
Comment 7 2016-01-30 12:51:05 PST
WebKit Commit Bot
Comment 8 2016-01-30 13:49:38 PST
Comment on attachment 270315 [details] Patch Clearing flags on attachment: 270315 Committed r195910: <http://trac.webkit.org/changeset/195910>
WebKit Commit Bot
Comment 9 2016-01-30 13:49:42 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.