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: DS4CH layout with header and footer #2167
Conversation
LeoniePeters
commented
Jan 2, 2024
•
edited by rwd
edited by rwd
- Disables i18n for microsite/DS4CH.eu page
- Adds a ds4ch layout type
- Adds a DS4CHPageHeader component with navigation and sidebar navigation on mobile
- Adds a DS4CHPageFooter component
- Adds the Montserrat font
- Adds new stylesheets for the DS4CH brand
- Adds new components to the Europeana style guide under a separate DS4CH section
- Moves scrollToRouteHash from default layout into a mixin, also used by ds4ch layout
- Adds a file route.js under plugins/i18n which declares the routes that are not to be localised, including the DS4CH microsite page
JIRA ticket: EC-6599 |
Preview deployment: https://pr-2167.portal-js.dev.eanadev.org/ |
packages/portal/src/lang/en.js
Outdated
@@ -213,6 +213,15 @@ export default { | |||
"left": "Left", | |||
"right": "Right" | |||
}, | |||
"ds4ch": { | |||
"dataSpaceForCulturalHeritage": "Data space for cultural heritage", | |||
"ds4chHome": "Data space for cultural heritage home page", |
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.
Rename this logoAlt
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.
Done.
> | ||
<img | ||
:src="logoSrc" | ||
:alt="$t('ds4chHome')" |
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.
There is no such key as this.
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.
Updated to use ds4ch.logoAlt
.
// TODO: add i18n head when decided page should be multilingual | ||
|
||
return { | ||
title: APP_SITENAME, |
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.
There is already text for this in the English language file, so use it.
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.
Done.
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 4 New issues |