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

Add new header #650

Closed
wants to merge 7 commits into from
Closed

Add new header #650

wants to merge 7 commits into from

Conversation

nikhilwoodruff
Copy link
Contributor

@nikhilwoodruff nikhilwoodruff commented Aug 3, 2023

Fixes #633

image image image

🤖 Generated by Copilot at d6acf05

Summary

🎨🧮📱

This pull request implements a redesign of the PolicyEngine component and its subcomponents, which is the main component of the app. It also improves the app's responsiveness on different screen sizes by adding a tablet display category and a new ResponsiveComponent component. Additionally, it updates the Google Fonts stylesheet link to include the optical size axis for Roboto Serif, which enhances the readability and aesthetics of the text. The changes affect the files src/layout/Responsive.jsx, public/index.html, src/index.js, src/redesign/PolicyEngine.jsx, and src/style/App.css.

Oh we're the crew of the PolicyEngine
We make the app look good and clean
We use Roboto Serif and HoverBackground
And we heave away on the count of three

Walkthrough

  • Update the PolicyEngine component to use a redesigned version that renders a responsive HeaderBar with different elements based on the display category (link, link)
  • Add a tablet category to the DisplaySize object and the determineDisplayCategory function to improve the responsiveness of the app on different devices (link, link)
  • Create a useDisplayCategory custom hook and a ResponsiveComponent component to simplify the logic of rendering different components based on the display category (link)
  • Import the Google Fonts stylesheet with the optical size axis for Roboto Serif to improve the readability and aesthetics of the text (link, link)
  • Use the HoverBackground component to create a color transition effect on hover for the HeaderBar elements (link)

@nikhilwoodruff nikhilwoodruff self-assigned this Aug 3, 2023
@nikhilwoodruff nikhilwoodruff marked this pull request as draft August 3, 2023 11:13
@nikhilwoodruff nikhilwoodruff marked this pull request as ready for review August 8, 2023 13:36
@nikhilwoodruff nikhilwoodruff deleted the nikhilwoodruff/issue633 branch August 8, 2023 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PolicyEngine header
1 participant