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-combobox/multi-combobox): physical items #10051

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MapTo0
Copy link
Member

@MapTo0 MapTo0 commented Oct 18, 2024

Passed items to the ui5-combobox and ui5-multi-combobox are now physically slotted to the internal list of the picker.

@MapTo0
Copy link
Member Author

MapTo0 commented Oct 18, 2024

@dobrinyonkov I have some changes in the list if something is not clear please contact me.

Basically, Since the combobox and the multicombobox are filtering their items internally, I need to render inidividual slots just for filtered items. That breaks the Item Navigation in the list and I had to enhance it

@vladitasev


onEnterDOM(): void {
super.onEnterDOM();
this.setAttribute("accessible-role", "Option");
Copy link
Contributor

@dobrinyonkov dobrinyonkov Oct 18, 2024

Choose a reason for hiding this comment

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

This might not work as it is a property of the ListItem and not the ListItemBase. What you can do is override the listItemAttributes inline template of the ListItemBase.hbs, similarly to the ui5-option

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.

2 participants