-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Style updates #181
Style updates #181
Conversation
Yeah, I agree. I haven't been able to reproduce the problem with the filters again either. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good, just one small comment
src/css/content.scss
Outdated
.issue-filter { | ||
color: rgb(230, 237, 243) | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we are missing a new line after 575
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added! that's a confusing way of displaying it haha
updated! |
$ https://github.com/Expensify/Expensify/issues/334627
This updates the design to match @dubielzyk-expensify's requests.
@tgolen I couldn't reproduce the resetting of the filters. Might that have been fixed with your recent 1.3.6 update? It looks like these remain static even when the new data is fetched.