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

Missing class k-list-container of DropDownList's popup element #7327

Closed
grozevg opened this issue May 4, 2023 · 3 comments
Closed

Missing class k-list-container of DropDownList's popup element #7327

grozevg opened this issue May 4, 2023 · 3 comments
Assignees
@ag-petrov
Labels
Bug C: DropDownList FP: Completed Sync status with associated Feedback Item jQuery
Milestone
R3.2023-Increment...

Comments

@grozevg
Copy link
Member

grozevg commented May 4, 2023

Bug report

The DropDownList's popup should have a div with "k-list-container k-popup" classes. Currently, k-list-container is missing. Rendering can be checked here.

Reproduction of the problem

Inspect the rendering of DropDownList's popup

Current behavior

Rendering lacks a class

Expected/desired behavior

Rendering should have an additional class

Environment

All
@github-actions github-actions bot added the FP: Unplanned Sync status with associated Feedback Item label May 4, 2023
@jivanova jivanova assigned kikostadinov and unassigned kikostadinov May 4, 2023
@grozevg
Copy link
Member Author

grozevg commented May 5, 2023

1607704

@github-actions github-actions bot added FP: Planned Sync status with associated Feedback Item and removed FP: Unplanned Sync status with associated Feedback Item labels May 12, 2023
@Dimitar-Goshev
Copy link
Contributor

@Juveniel @TeyaVes I see there is rendering test for opened dropdownlist and other components with popup. And k-list-container is not added to them as well and also there is no animation container. Should the popup-list rendering be the base for the other list "-opened" tests?

@Juveniel
Copy link
Contributor

Yes, we should amend the tests in the themes to include the animation container and the list container as well.

@ag-petrov ag-petrov self-assigned this May 15, 2023
@Dimitar-Goshev Dimitar-Goshev changed the title DropDownList's popup has incorrect rendering after unification Missing class k-list-container of DropDownList's popup element Jun 12, 2023
@github-actions github-actions bot added FP: Completed Sync status with associated Feedback Item and removed FP: Planned Sync status with associated Feedback Item labels Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ag-petrov ag-petrov

Labels
Bug C: DropDownList FP: Completed Sync status with associated Feedback Item jQuery
Projects
None yet
Milestone
R3.2023-Increment.1(19.July.2023)
Development

No branches or pull requests
5 participants
@Dimitar-Goshev @ag-petrov @Juveniel @kikostadinov @grozevg