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

SideNavigation: overflow behavior doesn't work #8242

Closed
1 task done
Lukas742 opened this issue Feb 5, 2024 · 3 comments · Fixed by #8948
Closed
1 task done

SideNavigation: overflow behavior doesn't work #8242

Lukas742 opened this issue Feb 5, 2024 · 3 comments · Fixed by #8948
Assignees
Labels

Comments

@Lukas742
Copy link
Collaborator

Lukas742 commented Feb 5, 2024

Bug Description

The overflow behavior doesn't work, it just shows an empty popover. Furthermore the overflow icon is not imported by the component, so it just shows an empty item.

Affected Component

SideNavigation

Expected Behaviour

No response

Isolated Example

https://stackblitz.com/edit/github-permf7-o6sev6?file=index.html

Steps to Reproduce

  1. Go to StackBlitz
  2. Click on the empty item
  3. See an empty popover

Log Output, Stack Trace or Screenshots

image

Priority

None

UI5 Web Components Version

1.22.0

Browser

Chrome

Operating System

No response

Additional Context

No response

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@Lukas742 Lukas742 added the bug This issue is a bug in the code label Feb 5, 2024
@yanaminkova yanaminkova self-assigned this Feb 5, 2024
@yanaminkova
Copy link
Member

Hello @SAP/ui5-webcomponents-topic-rd,

Could you please have a look at this issue about the overflow functionality of the ui5-side-navigation component.

Thanks and Regards,
Yana

@dimovpetar
Copy link
Contributor

Issue with missing overflow icon is resolved by #8736 in branches 2.0 and 1.24.
I have updated the StackBlitz sample https://stackblitz.com/edit/github-permf7-xhnkwr?file=package-lock.json,package.json,index.html.

@dimovpetar
Copy link
Contributor

Hello @Lukas742 ,

Issue is fixed with #8948 in branch 2.0 and will be downported to 1.24 with #8954.

Best regards,
Petar

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

Successfully merging a pull request may close this issue.

6 participants