[Aksel.nav.no] Move Aksel-brand theme into app-router for use #3668
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
There was a problem hiding this comment.
Pull Request Overview
This PR moves the Aksel-brand theme into the app-router for broader usage. Key changes include updating theme color mappings for Aksel-specific components.
Comments suppressed due to low confidence (2)
aksel.nav.no/website/app/_ui/global-search/GlobalSearch.hit.tsx:17
- [nitpick] The updated theme name 'aksel-brand-teal' may be inconsistent with other theme values (e.g., 'brand-blue'). Consider aligning the naming convention across all themes for clarity.
aksel_artikkel: "aksel-brand-teal",
aksel.nav.no/website/app/_ui/global-search/GlobalSearch.hit.tsx:18
- [nitpick] The updated theme name 'aksel-brand-pink' may be inconsistent with other theme values (e.g., 'brand-blue'). Consider aligning the naming convention across all themes for clarity.
aksel_blogg: "aksel-brand-pink",
KenAJoh
added a commit
that referenced
this pull request
Mar 18, 2025
* root layout * ✨ Added draftmode * ✨ Setup boilerplate * ✨ Setup grouping-dir * ✨ Implemented header * 💄 Made header sticky * 🔒 Content security policy updated * 🚚 Moved favicon to appdir * 🔥 restore old frontpage * 🚚 Moved sidebar to appdir * ✨ Implemented sidebar loading * ✨ Doc lyout * 🐛 Add id for skiplink * ✨ Added new footer * 💄 Move skiplink further away * 💄 Add more padding bottom sidebar * ✨ Refactor header and sidebar components for improved layout and structure * 📝 Uncomment some code * 🔥 Removed user server directive from component and layouts * 🐛 Center footer * 🐛 Re-added support for chromatic connect * 🔥 removed browser-token * Update aksel.nav.no/website/app/_ui/footer/Footer.tsx Co-authored-by: Copilot <[email protected]> * [Aksel.nav.no] Ported over global search to app-router (#3663) * 🚚 Moved search-code closer to app router * ✨ Added button and modal * ⚡ Initial loading for recent articles * ⚡ Query data from the server * ✨ Generates parsed search-results * ⚡ Added debounce * ✨ Render hits * ♻️ Limit articles for each field to 20 * 🐛 Revert form-layout to initial view * 🐛 Build searchindex before website * ✨ Pre-select text when re-opening search * ✨ Enhance global search UI with improved layout and styling * ✨ Refactor global search components to improve search functionality and UI * ✨ Improve global search functionality and UI responsiveness * ✨ Update global search UI and functionality with improved input handling and styling * ✨ Refactor global search dialog and add Kbd component for keyboard shortcuts * ⚡ removed reduntant suspense * ✨ Refactor global search components for improved styling and functionality * ✨ Refactor global search results rendering for improved performance and readability * ✨ Update global search dialog and form for improved styling and accessibility * ✨ Refactor global search components for improved layout and styling * ✨ Refactor GlobalSearchHitCollection for improved styling and accessibility * 🎨 Update global search configuration and styles for improved clarity and aesthetics * 🎨 Update layout and styles for global search components for improved aesthetics and readability * ✨ Refactor global search utilities and update styles for improved clarity and maintainability * 🔧 Refactor global search queries and implement caching for improved performance and data retrieval * ⚡ Preload cache for search * 🔧 Refactor fetchArticles function to improve caching logic and remove unnecessary logging * 🔥 Removed user server directive from component * 🔥 Removed comments * ⚡ Explicitly declare server-only component * Update aksel.nav.no/website/app/_ui/global-search/GlobalSearch.fetch.ts Co-authored-by: Copilot <[email protected]> * Update aksel.nav.no/website/app/_ui/global-search/GlobalSearch.fetch.ts Co-authored-by: Copilot <[email protected]> * 📝 Rename sanitiezedata --------- Co-authored-by: Copilot <[email protected]> * [Aksel.nav.no] Move Aksel-brand theme into app-router for use (#3668) * 💄 Added global aksel-theme tokens * ✨ Implement new Aksel-brand for theming * [Aksel.nav.no] Updated MobileMenu for new ds-pages (#3670) * 💄 Added global aksel-theme tokens * ✨ Implement new Aksel-brand for theming * ✨ Initial setup mobile-menu * ✨ V1 mobile-menu component * 💄 Update mobile navigation and global search responsiveness; adjust overlay color opacity * 💄 Refine modal backdrop animation timing and opacity for smoother transitions * 💄 Adjust pressed-sidebar state --------- Co-authored-by: Copilot <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
https://github.com/navikt/team-aksel/issues/788
Component Checklist 📝
@navikt/core/css/config/_mappings.js)@navikt/core/css/tokens.json)@navikt/aksel-stylelint/src/deprecations.ts)@navikt/core/react/src/index.tsand@navikt/core/react/package.json)@navikt/core/css/index.css)<Component>: <gitmoji?> <Text>.E.g. "Button: ✨ Add feature xyz.")