Bug 18650 – Errors/warnings in Inspector should be visible outside of Resources

• Component: Web Inspector (Deprecated)
• Status: RESOLVED FIXED

[Matt Lilek]

I don't know if there is already something in mind for this, but there should definitely be some sort of visual indication of errors on the page in the inspector (Syntax errors, undefined values, etc) without looking at the Resources pane.  Perhaps the icon on the console button could be red (and then switch back to black/blue until another error occurs)?

[Adam Roben (:aroben)]

I think we could do something similar to Firebug/Xcode and put an error/warning count in the status bar.

[Adam Roben (:aroben)]

*** Bug 19029 has been marked as a duplicate of this bug. ***

[Adam Roben (:aroben)]

<rdar://problem/5931102>

[Adam Roben (:aroben)]

Created attachment 21144 [details]
patch v1 with ChangeLog

[Adam Roben (:aroben)]

Created attachment 21145 [details]
screenshot with attachment 21144 [details] applied

[Adam Roben (:aroben)]

I think ideally we'd show the error/warning count all the way on the right. But I couldn't figure out how to do that and make the DOM breadcrumb aware of the space taken up by the count. Maybe Tim will know how to do this.

[Adam Roben (:aroben)]

Comment on attachment 21144 [details]
patch v1 with ChangeLog

I forgot to include the changes to localizedStrings.js in this patch, but I've fixed that locally.

[Timothy Hatcher]

Hmm. I agree showing the all the way to the right would be best.

So ElementsPanel.js:517:

        function crumbsAreSmallerThanContainer()
        {
            var rightPadding = 20;
            return ((crumbs.totalOffsetLeft + crumbs.offsetWidth + rightPadding) < window.innerWidth);
        }

That rightPadding value could be dynamic based on the size of the error/waring count container.

[Timothy Hatcher]

Comment on attachment 21144 [details]
patch v1 with ChangeLog

I have a revised patch to put the counts on the right of the status bar.

[Timothy Hatcher]

Landed in r33525.