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

Incorrect rendering of Grid's pager buttons arrows in RTL #7581

Closed
IvanDanchev opened this issue Nov 7, 2023 · 2 comments
Closed

Incorrect rendering of Grid's pager buttons arrows in RTL #7581

IvanDanchev opened this issue Nov 7, 2023 · 2 comments
Assignees
Labels
Bug C: Grid FP: Completed Sync status with associated Feedback Item jQuery SEV: Medium
Milestone

Comments

@IvanDanchev
Copy link
Contributor

IvanDanchev commented Nov 7, 2023

Bug report

Regression in R2 2023

Reproduction of the problem

  1. Compare the rendering of the "Next Page", "Last Page", "Previous Page" and "First Page" buttons in RTL between the MVC Grid and the jQuery and Core Grids:

https://demos.telerik.com/aspnet-mvc/grid/right-to-left-support
https://demos.telerik.com/kendo-ui/grid/right-to-left-support
https://demos.telerik.com/aspnet-core/grid/right-to-left-support

Current behavior

Incorrect rendering of the buttons.

Expected/desired behavior

Buttons rendered consistently with the jQuery and Core Grids.

Environment

  • Kendo UI version: 2023.3.1010
  • jQuery version: x.y
  • Browser: [all]
@gyankov
Copy link

gyankov commented Nov 8, 2023

This is actually a problem in the client side pager. As there is no reliable way to determine whether the component is within a k-rtl element on the server. While there is the kendo.support.isRtl API on the client. They actually swap the icons on the client when in RTL already, it should also be done when elements are rendered on the server:

@IvanDanchev
Copy link
Contributor Author

The issue is a regression introduced with R2 2023 (v2023.2.606)

@ag-petrov ag-petrov self-assigned this Nov 13, 2023
@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 Nov 23, 2023
@Dimitar-Goshev Dimitar-Goshev added Next LIB FP: Unplanned Sync status with associated Feedback Item and removed FP: Planned Sync status with associated Feedback Item labels Nov 23, 2023
@github-actions github-actions bot added FP: Completed Sync status with associated Feedback Item and removed FP: Unplanned Sync status with associated Feedback Item labels Nov 23, 2023
@Dimitar-Goshev Dimitar-Goshev changed the title Incorrect rendering of the Grid's pager buttons in RTL Incorrect rendering of Grid's pager buttons arrows in RTL Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug C: Grid FP: Completed Sync status with associated Feedback Item jQuery SEV: Medium
Projects
None yet
Development

No branches or pull requests

4 participants