-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(homepageHeader): initial commit #21
base: main
Are you sure you want to change the base?
Conversation
…issue, added missing colors to tailwind config, added missing logos to public images
…ck that github actions updates report to be a pass
…VGR svg imports in Storybook, wrote component test for Content Spotlight
…ed carousel to use live-region to improve accessibility
…text looks centered relative to vertical bar
import profileIcon from '../../../public/icons/HealIcons/Icon-Profile.svg'; | ||
import helpIcon from '../../../public/icons/HealIcons/Icon-Help.svg'; | ||
|
||
const navLinks = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this component should be able to ready config from this file https://github.com/uc-cdis/heal-data-commons/blob/main/config/heal/navigation.json
import gen3Logo from '../../../public/images/gen3.png'; | ||
import ctdsLogo from '../../../public/images/ctds-logo.png'; | ||
import footerNihLogo from '../../../public/images/footer-nih-logo.png'; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use the footer config? https://github.com/uc-cdis/heal-data-commons/blob/main/config/heal/footer.json
Link to JIRA ticket if there is one:
https://ctds-planx.atlassian.net/browse/HP-1746
https://ctds-planx.atlassian.net/browse/HP-1747
DESIGN
Header:
Footer:
IMPLEMENTATION
Header:
Footer:
STORYBOOKS
Header:
Footer:
New Features
Improvements