Bug 149533 - Web Inspector: make it easy to disable sidebar filters if everything is filtered out
Summary: Web Inspector: make it easy to disable sidebar filters if everything is filte...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-09-24 11:13 PDT by BJ Burg
Modified: 2017-05-17 16:27 PDT (History)
5 users (show)

See Also:


Attachments
[Image] Working UI (214.25 KB, image/png)
2015-10-27 17:15 PDT, Matt Baker
no flags Details
[Image] Mockup (150.18 KB, image/png)
2015-10-28 01:06 PDT, Nikita Vasilyev
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description BJ Burg 2015-09-24 11:13:36 PDT
This is especially mysterious on the debugger sidebar.

Two ideas:

 * Placeholder text "No files match (20 files not shown)."
 * Underneath placeholder text, show hyperlinks/buttons to toggle the currently active filters.
Comment 1 Radar WebKit Bug Importer 2015-09-24 11:13:59 PDT
<rdar://problem/22841740>
Comment 2 Matt Baker 2015-10-27 17:15:53 PDT
Created attachment 264178 [details]
[Image] Working UI

WIP patch to follow
Comment 3 Matt Baker 2015-10-27 17:17:26 PDT
(In reply to comment #2)
> Created attachment 264178 [details]
> [Image] Working UI
> 
> WIP patch to follow

The message bubble could probably use a more modern look, so I'm experimenting with removing the text-shadow and box-shadow.
Comment 4 Devin Rousso 2015-10-28 00:15:23 PDT
This looks really nice!  I am assuming that if the user clicks on the "Show # hidden items" then it will remove the filter entirely and show everything?
Comment 5 Nikita Vasilyev 2015-10-28 01:06:58 PDT
Created attachment 264199 [details]
[Image] Mockup

How about:

— Show all files when there are no matches. Showing only "No Filter Results" is just not very useful.
— Add "Show X hidden items" button after the items list
Comment 6 Matt Baker 2015-10-28 13:13:55 PDT
(In reply to comment #4)
> This looks really nice!  I am assuming that if the user clicks on the "Show
> # hidden items" then it will remove the filter entirely and show everything?

Correct. This means different things depending on the content view associated with the sidebar panel. The default behavior can be for NavigationSidebarPanel to simply call:

this._filterBar.filters = null;

but if a derived sidebar panel has custom filters, more work needs to be done. Timeline views filter based on the timeline ruler selection, the frames view filters based on task type and frame duration, and so on.
Comment 7 Matt Baker 2015-10-28 13:15:27 PDT
(In reply to comment #5)
> Created attachment 264199 [details]
> [Image] Mockup
> 
> How about:
> 
> — Show all files when there are no matches. Showing only "No Filter Results"
> is just not very useful.
> — Add "Show X hidden items" button after the items list

I think these are interesting ideas, and think the second would be very useful. Positioning the message in the correct location requires some additional layout logic (see https://bugs.webkit.org/show_bug.cgi?id=150608), so if we tackle this let's do it as a follow up.
Comment 8 Matt Baker 2017-05-17 16:27:47 PDT
This complicates the Debugger sidebar, which has multiple filterable tree outlines (Sources, Breakpoints). I think we should defer pending a re-think of our current sidebar design.