-
Notifications
You must be signed in to change notification settings - Fork 8
feat: Cross-linking of news with operators and countries #168
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
Conversation
c4b0071
to
310f870
Compare
82719d1
to
a505cac
Compare
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.
Pull Request Overview
This PR implements cross-linking between news, operators, and countries by adding new front matter keys and updating the related styling. Key changes include adding an "operator" key to operator and news markdown files, introducing "country" and "operator" lists in news items, and refactoring SASS styles to support the unified design.
Reviewed Changes
Copilot reviewed 32 out of 32 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
content/operator/**/*.md | Added an "operator" key in the front matter for operator pages |
content/news/**/*.md | Added "country" and "operator" keys to link news with operators/countries |
assets/sass/styles.scss | Updated styling for operator links, added margins and flag styling |
assets/sass/sidemenu.scss | Refactored related news list styling under the ".o-related" namespace |
assets/sass/content.scss | Removed global paragraph styling to support localized styling |
archetypes/operator/**/*.md | Updated operator template to include the ContentBaseName as operator |
Comments suppressed due to low confidence (2)
assets/sass/content.scss:31
- [nitpick] Removing the global paragraph styling may lead to inconsistent text presentation across pages. Consider applying context-specific paragraph styles to ensure consistent appearance.
p { margin-bottom: 3.2rem; text-align: justify; }
assets/sass/styles.scss:113
- Renaming from '.o-list__link-wrapper' to '.o-related__operator-wrapper > .o-related__list' may impact template references. Verify that all corresponding templates utilize the updated class name to maintain correct styling.
.o-related__operator-wrapper > .o-related__list {
a505cac
to
c9121c6
Compare
310f870
to
ac4de6d
Compare
c9121c6
to
c0bfcbd
Compare
c0bfcbd
to
62b1e3a
Compare
✅ Deploy Preview for fipguide ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
62b1e3a
to
e7b5696
Compare
e7b5696
to
623b4a8
Compare
Cross-link news, operators and countries. For news, related countries and operators are displayed in the sidebar. For countries and operators, related news are displayed.
A flag is displayed for countries in the sidebar navigation (resolves #105). The PR contains some refactoring to unify all "related" links. The operator and country lists haven been unified.