Skip to content
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

Merged
merged 4 commits into from
Jan 10, 2024
Merged

Style updates #181

merged 4 commits into from
Jan 10, 2024

Conversation

dangrous
Copy link
Contributor

@dangrous dangrous commented Jan 8, 2024

$ 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.

@dangrous dangrous self-assigned this Jan 8, 2024
@dangrous dangrous requested a review from a team January 8, 2024 17:27
@melvin-bot melvin-bot bot requested review from thienlnam and removed request for a team January 8, 2024 17:27
@tgolen
Copy link
Contributor

tgolen commented Jan 8, 2024

Yeah, I agree. I haven't been able to reproduce the problem with the filters again either.

Copy link
Contributor

@thienlnam thienlnam left a 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

.issue-filter {
color: rgb(230, 237, 243)
}
}
Copy link
Contributor

@thienlnam thienlnam Jan 9, 2024

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

Copy link
Contributor Author

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

@dangrous
Copy link
Contributor Author

dangrous commented Jan 9, 2024

updated!

@thienlnam thienlnam merged commit 1476915 into main Jan 10, 2024
3 checks passed
@thienlnam thienlnam deleted the dangrous-filterimprovements branch January 10, 2024 02:12
madmax330 added a commit that referenced this pull request Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants