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.

Sign up for GitHub

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

Jump to bottom

[ui5-menu]: before-open is not correctly #10711

Open
1 task done
nnaydenow opened this issue Jan 29, 2025 · 1 comment · May be fixed by #10898
Open
1 task done

[ui5-menu]: before-open is not correctly #10711

nnaydenow opened this issue Jan 29, 2025 · 1 comment · May be fixed by #10898
Assignees
@TeodorTaushanov
Labels
bug This issue is a bug in the code Low Prio TOPIC RD

Comments

@nnaydenow
Copy link
Contributor

Bug Description

If ui5-menu is rendered as initially opened the before-open event is not fired.

Affected Component

ui5-menu

Expected Behaviour

No response

Isolated Example

https://sap.github.io/ui5-webcomponents/nightly/play/#eyJpbmRleC5odG1sIjp7Im5hbWUiOiJpbmRleC5odG1sIiwiY29udGVudCI6IjwhLS0gcGxheWdyb3VuZC1mb2xkIC0tPlxuPCFET0NUWVBFIGh0bWw-XG48aHRtbCBsYW5nPVwiZW5cIj5cblxuXG5cbjxoZWFkPlxuICAgIFxuICAgIDxzdHlsZT5cbiAgICAgICo6bm90KDpkZWZpbmVkKSB7XG4gICAgICAgIGRpc3BsYXk6IG5vbmU7XG4gICAgICB9XG5cbiAgICBodG1sIHtcbiAgICAgIGZvcmNlZC1jb2xvci1hZGp1c3Q6IG5vbmU7XG4gICAgfVxuICAgIDwvc3R5bGU-XG5cbiAgICBcbiAgICA8c3R5bGU-XG4gICAgICAqOm5vdCg6ZGVmaW5lZCkge1xuICAgICAgICBkaXNwbGF5OiBub25lO1xuICAgICAgfVxuXG4gICAgaHRtbCB7XG4gICAgICBmb3JjZWQtY29sb3ItYWRqdXN0OiBub25lO1xuICAgIH1cbiAgICA8L3N0eWxlPlxuXG4gICAgPG1ldGEgY2hhcnNldD1cIlVURi04XCI-XG4gICAgPG1ldGEgbmFtZT1cInZpZXdwb3J0XCIgY29udGVudD1cIndpZHRoPWRldmljZS13aWR0aCwgaW5pdGlhbC1zY2FsZT0xLjBcIj5cbiAgICA8dGl0bGU-U2FtcGxlPC90aXRsZT5cbjwvaGVhZD5cblxuPGJvZHkgc3R5bGU9XCJiYWNrZ3JvdW5kLWNvbG9yOiB2YXIoLS1zYXBCYWNrZ3JvdW5kQ29sb3IpOyBoZWlnaHQ6IDM1MHB4XCI-XG4gICAgPCEtLSBwbGF5Z3JvdW5kLWZvbGQtZW5kIC0tPlxuXG5cdDx1aTUtYnV0dG9uIGlkPVwiYnRuT3BlblwiPk9wZW4gTWVudTwvdWk1LWJ1dHRvbj4gPGJyLz5cblx0PHVpNS1tZW51IGlkPVwibWVudVwiIG9wZW4gb3BlbmVyPVwiYnRuT3BlblwiPlxuXHRcdDx1aTUtbWVudS1pdGVtIHRleHQ9XCJOZXcgRmlsZSAoc2VsZWN0aW9uIHByZXZlbnRlZClcIj48L3VpNS1tZW51LWl0ZW0-XG5cdDwvdWk1LW1lbnU-XG4gICAgPCEtLSBwbGF5Z3JvdW5kLWZvbGQgLS0-XG4gICAgPHNjcmlwdCB0eXBlPVwibW9kdWxlXCIgc3JjPVwibWFpbi5qc1wiPjwvc2NyaXB0PlxuPC9ib2R5PlxuXG48L2h0bWw-XG48IS0tIHBsYXlncm91bmQtZm9sZC1lbmQgLS0-XG4ifSwibWFpbi5qcyI6eyJuYW1lIjoibWFpbi5qcyIsImNvbnRlbnQiOiIvKiBwbGF5Z3JvdW5kLWhpZGUgKi9cbmltcG9ydCBcIi4vcGxheWdyb3VuZC1zdXBwb3J0LmpzXCI7XG4vKiBwbGF5Z3JvdW5kLWhpZGUtZW5kICovXG5pbXBvcnQgXCJAdWk1L3dlYmNvbXBvbmVudHMvZGlzdC9NZW51LmpzXCI7XG5pbXBvcnQgXCJAdWk1L3dlYmNvbXBvbmVudHMvZGlzdC9NZW51SXRlbS5qc1wiO1xuaW1wb3J0IFwiQHVpNS93ZWJjb21wb25lbnRzL2Rpc3QvTWVudVNlcGFyYXRvci5qc1wiO1xuXHRcdG1lbnUuYWRkRXZlbnRMaXN0ZW5lcihcInVpNS1iZWZvcmUtb3BlblwiLCAoKSA9PiB7XG5cdFx0XHRjb25zb2xlLmxvZyhcInVpNS1iZWZvcmUtb3BlbiBpcyBmaXJlZFwiKVxuXHRcdH0pXG5cdFx0bWVudS5hZGRFdmVudExpc3RlbmVyKFwidWk1LW9wZW5cIiwgKCkgPT4ge1xuXHRcdFx0Y29uc29sZS5sb2coXCJ1aTUtb3BlbiBpcyBmaXJlZFwiKVxuXHRcdH0pIn19

Steps to Reproduce

...

Log Output, Stack Trace or Screenshots

No response

Priority

None

UI5 Web Components Version

nightly

Browser

Chrome

Operating System

No response

Additional Context

No response

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@nnaydenow nnaydenow added bug This issue is a bug in the code Low Prio TOPIC B labels Jan 29, 2025
@tsanislavgatev tsanislavgatev self-assigned this Feb 4, 2025
@tsanislavgatev tsanislavgatev removed their assignment Feb 4, 2025
@tsanislavgatev
Copy link
Contributor

Hi colleagues,

this issue is also reproducible with responsive popover only. Can you please check it?

link

Best Regards,
Tsanislav

@TeodorTaushanov TeodorTaushanov self-assigned this Feb 18, 2025
@TeodorTaushanov TeodorTaushanov linked a pull request Feb 19, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TeodorTaushanov TeodorTaushanov

Labels
bug This issue is a bug in the code Low Prio TOPIC RD
Projects
Maintenance - Topic RD
Status: New Issues
Milestone
No milestone
3 participants
@TeodorTaushanov @tsanislavgatev @nnaydenow