Skip to content

Conversation

@janekamata
Copy link
Contributor

@janekamata janekamata commented Jan 25, 2026

ℹ️ Issue

Closes #258

📝 Description

Cleaned up the frontend a bit

  1. Changed logo file name to be more generic
  2. Removed some of the in-line styling and replaced with tailwind where appropriate
  3. Added more colors to the tailwind config for future use
  4. Removed unused css files

✔️ Verification

Checked local build

🏕️ Ways the frontend can be cleaned up

  • Streamline styling based on design style guide (which also abstracts colors and fonts at the same time) with defaults

    • Remove all hex codes and replace with tailwind colors set in config file (for css replacements you can reference theme variables like this)
    • Use global font definitions and remove font specifications at lower levels (notification.css, UserDropdown.css, CalendarDropdown.css, NewGrantModal.tsx); also check for other similar settings like font-size
    • Buttons with different formats (ex. add button animation from Account Info popup to other buttons?)
  • Styling format

    • In-line styling where tailwind classes would be more appropriate (CostBenefitAnalysis.tsx, NewGrantModal.tsx)
    • Separate style sheets from tsx components (Account.tsx, Login.tsx, Profile.tsx, Register.tsx, RegisterLanding.tsx)
    • Improve accessibility/responsiveness (use REM for sizing and certain spacing, and Pixels for header height, borders, certain padding, etc.); see this to learn more
  • Simple code fixes/adds

    • Add BCAN quick links to footer (TODO comment I found in Footer.tsx)
    • Remove or fix AnimatedRoutes.tsx
    • Remove large sections of commented out code (GrantSearch.tsx)
  • Extracting components; see this for more info

    • Long stretches of code that should be separated into smaller components/template partials (CostBenefitAnalysis.tsx, GrantAttributes.tsx, GrantItem.tsx, NewGrantModal.tsx) or utilize loops to reduce repetition

…with tailwind, add black and red to tailwind config
@janekamata janekamata self-assigned this Jan 25, 2026
@janekamata janekamata linked an issue Jan 25, 2026 that may be closed by this pull request
Copy link
Collaborator

@prooflesben prooflesben left a comment

Choose a reason for hiding this comment

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

lgtm would be good to go over this pr in the next standup

Copy link
Contributor

@adityapat24 adityapat24 left a comment

Choose a reason for hiding this comment

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

Looks Good!

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.

DEV - Frontend cleanup

4 participants