codeforpdx / PASS

PASS project - with HMIS module integration
MIT License
24 stars 23 forks source link

605/theme updates #623

Closed JaeGif closed 2 months ago

JaeGif commented 2 months ago

This PR:

Resolves #605

Future Steps/PRs Needed to Finish This Work (optional):

This PR is just the beginning of the work that needs to be done to overhaul the UI. With the addition of new themes and fonts, the next steps can be broken into multiple issues. I'd recommend making an issue per page to be re-themed since most pages require the addition of new elements/functionality and rewriting unit tests. Some components (like the DocumentList component) will need major overhauls and might warrant their own issue.

Also should contact the UI team and see if there's any more/other designs incoming before we get too far on that.

leekahung commented 2 months ago

Related to this PR, there still a bit of theming we could update from the Inbox/Outbox buttons, namely the custom styling from React Paginate (see MessageFolder.jsx for the component and PaginationStyles.js for the custom styles and classes used).

Screenshot 2024-04-29 at 11 46 36 AM (2)

I believe Ben at the time made it so that the styling and coloring were in line with what we had before. Think we could do the same thing here with the new theme.

JaeGif commented 2 months ago

Alright, think that touches most of the things that still needed to be addressed. But outside of those, the changes made with the new theme looks great!

Great, I think I've addressed most of the points in the last few commits. Please let me know if there's anything else or if something doesn't fit the requirements. Cheers!