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

Waitp 1201 basic web layout #4620

Merged
merged 23 commits into from
Jun 7, 2023

Conversation

alexd-bes
Copy link
Contributor

Issue WAITP-1201: Basic desktop layout

Changes:

  • Added layout files for Modal, AuthModal, Sidebar, menus, TopBar, and general layout
  • Added stories for key elements
  • Added decorator for react router to use in storybook
  • Applied layout to views

color?: CSSStyleDeclaration['color'];
}
) => {
export const EnvBanner = ({ color = '#f39c12' }: { color?: CSSStyleDeclaration['color'] }) => {
const deploymentName = process.env.REACT_APP_DEPLOYMENT_NAME;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is just formatting

Co-authored-by: Tom Caiger <caigertom@gmail.com>
Co-authored-by: Tom Caiger <caigertom@gmail.com>
Copy link
Contributor

@tcaiger tcaiger left a comment

Choose a reason for hiding this comment

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

Thanks for all the changes. It's looking really good now and it's great to get rid of some of that legacy logic.

Copy link
Contributor

@tcaiger tcaiger left a comment

Choose a reason for hiding this comment

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

Thanks for all the changes. It's looking really good now and it's great to get rid of some of that legacy logic.

@alexd-bes alexd-bes merged commit 22053e0 into epic-frontend-rewrite Jun 7, 2023
@alexd-bes alexd-bes deleted the waitp-1201-basic-web-layout branch June 7, 2023 23:53
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