Page MenuHomePhabricator

[web] Restyle filters button
ClosedPublic

Authored by inka on Feb 15 2023, 5:22 AM.
Tags
None
Referenced Files
F3365238: D6740.id23463.diff
Mon, Nov 25, 7:00 AM
F3365214: D6740.id22818.diff
Mon, Nov 25, 6:55 AM
F3365131: D6740.id22644.diff
Mon, Nov 25, 6:43 AM
F3365096: D6740.id22617.diff
Mon, Nov 25, 6:33 AM
F3364574: D6740.diff
Mon, Nov 25, 4:21 AM
Unknown Object (File)
Wed, Nov 20, 5:03 PM
Unknown Object (File)
Wed, Nov 20, 4:20 PM
Unknown Object (File)
Wed, Nov 20, 4:20 PM
Subscribers

Details

Summary

Issue: https://linear.app/comm/issue/ENG-2964/restyle-the-filters-button
Restyling the filters button. Note that in the comments, it has been decided that we'll use a different icon than in the designs.
For now keeping the button after opening the filter panel. This will be changed in the future

Test Plan

Run web app, checked that the button displays correctly, and pressing it opens the filter panel

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka held this revision as a draft.

Add hover, change colours since the designs have changed

inka published this revision for review.Feb 16 2023, 1:06 AM
inka edited the summary of this revision. (Show Details)
inka edited the test plan for this revision. (Show Details)

We might add a short transition to the hover state. Looks great!

This revision is now accepted and ready to land.Feb 16 2023, 5:32 AM
This revision was automatically updated to reflect the committed changes.