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

Officer UI #309

Merged
merged 8 commits into from
Dec 20, 2023
Merged

Officer UI #309

merged 8 commits into from
Dec 20, 2023

Conversation

joshua-rdrgz
Copy link
Collaborator

Description

This pull request introduces several enhancements by @abelaba and myself to the officer search functionality, including the integration of the current repository's data table component for building a search results table for the officer results and the addition of a new bubble chart component for the officer result's data visualization. Additionally, it addresses various visual bugs and implements design updates according to the Figma specifications.

Changes

1. Officer Search Results Table

Utilized the current repository's data table component to construct a search results table for the officer search functionality.

2. Add BubbleChart Component

Created a BubbleChart component for visualizing officer search result data, closely aligned with Figma implementation.

3. Adjust Donate Button Position

Relocated the donate button to the right side of the header component for improved visibility and alignment.

4. Update Search Form Visuals

Adding additional information to each search form input, closely aligned with Figma specifications.

5. Error Alert Dialog Component

Introduced an error alert dialog component to provide users with clear and informative error messages.

Screenshots

Screenshot 2023-12-17 at 2 02 13 PM

Additional Notes

The results table and bubble chart components currently use placeholder data. We'll be working on implementing the API for full functionality this upcoming week, and looking into using the proper schemas for that implementation as well.

@DMalone87
Copy link
Collaborator

Updated the snapshots which should fix the tests. Looks good! And gives us some structure to work with when designing the Officer API.

Copy link
Collaborator

@DMalone87 DMalone87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work!

@DMalone87 DMalone87 merged commit 0488d03 into codeforboston:main Dec 20, 2023
3 checks passed
@joshua-rdrgz joshua-rdrgz deleted the abel-josh/officer-UI branch December 21, 2023 15:28
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