Closed Bug 1523861 Opened 10 months ago Closed 9 months ago

Remove toggle filter button and always show the filter buttons toolbar

Categories

(DevTools :: Console, enhancement, P2)

enhancement

Tracking

(firefox67 fixed)

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: nchevobbe, Assigned: koroknay, Mentored)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

The toggle filter button is created in devtools/client/webconsole/components/FilterBar.js#261-266, so we can remove the whole dom.button block here.

It means we can remove the onClickFilterBarToggle method too (See FilterBar.js#105-107 and FilterBar.js#50).

Going further, we can remove the filterBarToggle action that was called in onClickFilterBarToggle (See devtools/client/webconsole/actions/ui.js#25-33,114), the type we were using for it (devtools/client/webconsole/constants.js#18) and its reducer case (devtools/client/webconsole/reducers/ui.js#40-41).

Then, we can always show the filter buttons toolbar by removing the if in devtools/client/webconsole/components/FilterBar.js#303-305

Then, some tests hide/show the toolbar (See this search result), so we should modify them to remove the parts where we hide the bar (since it can't be hidden anymore).

Blocks: 1523864
Priority: -- → P2

Hello, I would be happy to take this on.

Hello Martin, thanks for wanting to help us!

I'm assigning the bug to you.

If it's your first time working on Firefox, I suggest you to read https://docs.firefox-dev.tools/getting-started/ to setup the work environment.

You can also come and talk on our Slack if you're blocked on something 🙂

Assignee: nobody → koroknay
Status: NEW → ASSIGNED

Removes the toggle filter button and makes the filter buttons toolbar permanently shown. At the same time updates some tests which relied on showing and hiding the filter buttons toolbar. Also amends an unrelated test which relied on the non-presence of the filter buttons toolbar.

Hi, this is submitted for review now. Please note that I also had to update an unrelated test (devtools/client/webconsole/test/mochitest/browser_jsterm_no_input_and_tab_key_pressed.js) because it relied on the filter buttons not being there by default :)

Attachment #9041039 - Attachment description: Bug 1523861 - Remove toggle filter button and always show the filter buttons toolbar. r:nchevobbe → Bug 1523861 - Remove toggle filter button and always show the filter buttons toolbar. r=nchevobbe
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/19cdd41c33e6
Remove toggle filter button and always show the filter buttons toolbar. r=nchevobbe
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Depends on: 1529225

Why was the "toggle filter bar" button removed? When using a screen reader to examine devtools console output, the filter bar must be traversed before one gets to the start of the console output. The "toggle filter bar" was a welcome feature because it removed it from the DOM and allowed screen reader users to more quickly find the information they needed.

(In reply to Rich Caloggero from comment #7)

Why was the "toggle filter bar" button removed?

I'm curious about this as well. I guess there was some UX reason, but it's not documented in this bug.

When using a screen reader to examine devtools console output, the filter bar must be traversed before one gets to the start of the console output. The "toggle filter bar" was a welcome feature because it removed it from the DOM and allowed screen reader users to more quickly find the information they needed.

If there was a solid UX reason for removing this, perhaps we could add a role="main" to the output area (class webconsole-output) so screen reader users have a fast way to skip to the output. If that's the decided course here, we can file a new bug.

Flags: needinfo?(nchevobbe)
Blocks: 1530936

Hello Rich, James.

The button was removed as we're redesigning the console toolbar (See Bug 1523842).
There will be 2 "modes": one where the filter buttons are always displayed (possibly in the same row as the filter input), and a "compact" one, where the filters will be put in a <select>, kind of what Chrome does.

This bug was the first step for this redesign, and I'm sorry it's making the developer experience worse for people who use screenreaders.

I will be more than happy to add a role="main" attribute to the output area, so I filed Bug 1530936 for that.

Flags: needinfo?(nchevobbe)
You need to log in before you can comment on or make changes to this bug.