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-shellbar): new event added #8197

Merged
merged 4 commits into from
Feb 1, 2024
Merged

Conversation

yanaminkova
Copy link
Member

New event search-button-click is added, which allows different actions to be triggered
when it is fired on the search button inside the ui5-shellbar.
If its preventDefault method is called, the default action of the search button
(expanding/collapsing the input) won`t be triggered, but let the users trigger their own.

Related: #6273

@kineticjs kineticjs self-requested a review January 31, 2024 08:51
@kineticjs
Copy link
Contributor

I do not see any issues. The only thing I want to verify before merge is that architects also agree with the new API field names, as those cannot be undone once added. In particular, the "isVisible" field. I also prefer shorter and concise names. Let's just double-check with them and I'd suggest to merge right after.

@yanaminkova yanaminkova merged commit 4e864e9 into main Feb 1, 2024
8 checks passed
@yanaminkova yanaminkova deleted the shellbar-button-event branch February 1, 2024 12:44
tsanislavgatev pushed a commit that referenced this pull request Feb 20, 2024
New event search-button-click is added, which allows different actions to be triggered
when it is fired on the search button inside the ui5-shellbar.
If its preventDefault method is called, the default action of the search button
(expanding/collapsing the input) won`t be triggered, but let the users trigger their own.

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

Successfully merging this pull request may close these issues.

None yet

2 participants