feat(MainNavigationBar): topSlot support#1518
Conversation
|
I'm reviewing this, it might take a few minutes, I'll add comments and update here when it's done. |
|
I've checked the PR against the requirements in the ticket WEB-2378: ✅ Implement a top slot in the Main Navigation Bar component |
neo-sagittal
left a comment
neo-sagittal
left a comment
There was a problem hiding this comment.
LGTM! Remember you can easily customize how your team reviews code!
|
Size stats
|
|
Deploy preview for mistica-web ready! ✅ Preview Built with commit 1e63bde. |
|
Accessibility report ℹ️ You can run this locally by executing |
There was a problem hiding this comment.
There was a problem hiding this comment.
missing the possibility to custom topBar background color
There was a problem hiding this comment.
missing the possibility to custom topBar background color
yes, you can. Try with something like this:
topSlot={
<ResponsiveLayout backgroundColor="paleviolet">
<Placeholder height={24} />
</ResponsiveLayout>
}
There was a problem hiding this comment.
I found also a bug with variant negative, maybe should I open another ticket for this?
Yes please, open a ticket. Not related with this PR
|
🎉 This PR is included in version 16.54.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
# [16.54.0](v16.53.0...v16.54.0) (2026-03-13) ### Bug Fixes * **Hero:** Restore padding in hero mobile layouts ([#1514](#1514)) ([0513c35](0513c35)) ### Features * **Card:** Add space-between alignment for card slot ([#1516](#1516)) ([0900ebb](0900ebb)) * **Layout:** add extraLargeDesktop breakpoint ([#1503](#1503)) ([229ace4](229ace4)) * **MainNavigationBar:** topSlot support ([#1518](#1518)) ([818748f](818748f))
5 participants
WEB-2378