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

Revise AppBar spacer classes and styles #4854

Open
inikolova opened this issue Jan 19, 2024 · 0 comments
Open

Revise AppBar spacer classes and styles #4854

inikolova opened this issue Jan 19, 2024 · 0 comments
Labels
Enhancement New feature of an existing functionality or an improvement of an existing functionality.

Comments

@inikolova
Copy link
Contributor

In the AppBar tests we use k-spacer k-spacer-sized classes for the spacer:
https://github.com/telerik/kendo-themes/blob/develop/packages/html/src/appbar/tests/appbar.tsx#L42

However, the issue for "Unify rendering appbar" suggests to use k-appbar-spacer k-appbar-spacer-sized classes instead:
#4092

The component implementation in jQuery, Angular, React and Vue follows the suggested rendering in the aforementioned issue (i.e. usesk-appbar-spacer k-appbar-spacer-sized classes), however the implementation in Blazor relies on k-spacer k-spacer-sized. We should revise the styles for .k-spacer / .k-spacer-sized / .k-appbar-spacer / .k-appbar-spacer-sized and unify their usage per suites as they serve the same purpose.

@inikolova inikolova added the Enhancement New feature of an existing functionality or an improvement of an existing functionality. label Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature of an existing functionality or an improvement of an existing functionality.
Projects
None yet
Development

No branches or pull requests

1 participant