RESOLVED FIXED 47822
Web Inspector: Inspect element tooltip obscures element metrics
https://bugs.webkit.org/show_bug.cgi?id=47822
Summary Web Inspector: Inspect element tooltip obscures element metrics
Vernon Thommeret
Reported 2010-10-18 08:17:20 PDT
Created attachment 71038 [details] An image of the inspect element tooltip obscuring element metrics I've attached a picture of what I'm referring to. I'm not sure when the tooltip was introduced, but the image shows an issue that I've run into since it was added — it's not possible to visually see the bottom-margin of an element if either the height of the margin or the width of the element is smaller than the tooltip. I've artificially enlarged element in the second half of the image to show the obscured margin. I'm not sure that the tooltip is terribly useful (possibly the width and height?) but could possibly the tooltip be made translucent or an give to remove it altogether. I like to use the inspector visually — the whether something is the result of a margin or padding, and it's connection to other elements, but recently this tooltip has just removed that visibility.
Attachments
An image of the inspect element tooltip obscuring element metrics (9.70 KB, image/png)
2010-10-18 08:17 PDT, Vernon Thommeret
no flags
[PATCH] Suggested solution (4.31 KB, patch)
2010-11-11 10:37 PST, Alexander Pavlov (apavlov)
yurys: review+
Alexander Pavlov (apavlov)
Comment 1 2010-11-11 10:37:17 PST
Created attachment 73623 [details] [PATCH] Suggested solution - The tooltip is laid out relative to the margins box rather than the content box. - The tooltip should obscure the node highlight only if there are no other layout alternatives.
Alexander Pavlov (apavlov)
Comment 2 2010-11-12 09:56:05 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/inspector/InspectorController.cpp Committed r71922
WebKit Review Bot
Comment 3 2010-11-12 12:29:14 PST
http://trac.webkit.org/changeset/71922 might have broken GTK Linux 32-bit Debug The following tests are not passing: fast/text/international/plane2.html
Note You need to log in before you can comment on or make changes to this bug.