Skip to content

displays reported teams at the top of view teams page - #338

Merged
michelleyeoh merged 29 commits into
mainfrom
feat/reported-teams
Sep 24, 2025
Merged

displays reported teams at the top of view teams page#338
michelleyeoh merged 29 commits into
mainfrom
feat/reported-teams

Conversation

@phlangiee

Copy link
Copy Markdown
Contributor

Reported Teams header is under View Teams header and above the team cards & contains a toggle to view/unview the reported teams names

reported teams is in the same style as the red report popup & in 4-column (2 column on mobile) grid container (because having them in one vertical collumn made me have to scroll)

@phlangiee phlangiee added the documentation Improvements or additions to documentation label Sep 8, 2025
@phlangiee phlangiee self-assigned this Sep 8, 2025
@michelleyeoh michelleyeoh removed the documentation Improvements or additions to documentation label Sep 8, 2025
…eported teams dropdown

Added the co-pilot suggestion on how to fix the build error
michelleyeoh
michelleyeoh previously approved these changes Sep 24, 2025

@michelleyeoh michelleyeoh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  • Overall, changes are clean and didn't require much changes. For future front-end focused tasks (such as hackdavis.io website), avoid using px and use % for margin, padding, gap (border-radius can use px). Also use global variables for colors
  • Added sorted teams feature

@michelleyeoh
michelleyeoh merged commit d845751 into main Sep 24, 2025
2 checks passed
@michelleyeoh
michelleyeoh deleted the feat/reported-teams branch September 24, 2025 02:04
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.

2 participants