Skip to content
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(ui5-side-navigation): enable the web component in popover placement #10701

Merged
merged 19 commits into from
Mar 18, 2025

Conversation

GerganaKremenska
Copy link
Member

@GerganaKremenska GerganaKremenska commented Jan 29, 2025

Introduce a new design property to easily enable OverlayMode for ui5-side-navigation, and provide an example of how to implement it.

JIRA: BGSOFUIRODOPI-3411

@GerganaKremenska GerganaKremenska requested a review from a team February 25, 2025 08:55
@GerganaKremenska GerganaKremenska changed the title chore(ui5-side-navigation): overlay poc chore(ui5-side-navigation): Overlay mode example Feb 25, 2025
@GerganaKremenska GerganaKremenska changed the title chore(ui5-side-navigation): Overlay mode example chore(ui5-side-navigation): enabling componet in popover placment Feb 25, 2025
@GerganaKremenska GerganaKremenska changed the title chore(ui5-side-navigation): enabling componet in popover placment feat(ui5-side-navigation): enabling componet in popover placment Feb 26, 2025
Copy link

@Stoev Stoev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please consider* :
feat(ui5-side-navigation): enable the web component in popover placement

Introduce a new design property to easily enable OverlayMode for ui5-side-navigation, and provide an example of how to implement it.

*(header should be in present imperative tense (https://sap.github.io/ui5-webcomponents/docs/contributing/conventions-and-guidelines/#commit-description))

@GerganaKremenska GerganaKremenska changed the title feat(ui5-side-navigation): enabling componet in popover placment feat(ui5-side-navigation): enable the web component in popover placement Feb 27, 2025
@GerganaKremenska GerganaKremenska marked this pull request as draft February 28, 2025 13:39
Copy link

@Stoev Stoev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor edits.

@s-todorova s-todorova marked this pull request as ready for review March 17, 2025 16:06
@s-todorova s-todorova merged commit a4565fe into main Mar 18, 2025
12 checks passed
@s-todorova s-todorova deleted the sideNavOverlay branch March 18, 2025 12:10
@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This PR is included in version v2.9.0-rc.1 🎉

The release is available on v2.9.0-rc.1

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants