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

Html specs corrections 6 #5230

Merged
merged 11 commits into from
Dec 2, 2024
Merged

Html specs corrections 6 #5230

merged 11 commits into from
Dec 2, 2024

Conversation

TeyaVes
Copy link
Contributor

@TeyaVes TeyaVes commented Nov 13, 2024

targets: https://github.com/telerik/kendo-themes-private/issues/246

This PR targets the following items from the related issue:

Splitter

  • some suites (Angular, React) are having k-touch-action-none/ touch-action: none; class/style on the splitbar, do we want to add that class in the reference rendering, or add a style in the themes? -
    the touch-action: none style is added to the themes and the utility class is no longer needed. It needs to be removed from the specs.

Gantt

  • simplify templates, remove tasks
  • check if k-selectable can be removed
  • TreeList does not have k-grid-aria-root

PivotGrid/Checkbox

  • missing k-sr-only on the empty-cell
  • the class on the chip list should be k-column-fields, instead of k-filter-fields
  • k-pivotgrid-content cell value wrapper, do we need this, check if it can be removed.
    I think that we should leave this wrapper with the class- in this way we can target the text inside the cell, without affecting the toggle icon which might be present. Similarly, we have a wrapper for the text inside pivotGrid headers- .k-pivotgrid-header-title

Inputs

  • should opened inputs be k-focused?
    The purpose of the templates are to give guidelines for the static reference rendering of the componets. Adding states to inputs under certain conditions should not be in the scope of the specs. Thus, the inputs when a dropdown is opened will currently remain unchanged.

AI Prompt

  • some products use k-white-space-pre-line utility in the card body. Is this okay, should we consider adding those util styles in the component. -  
    I think that we should not add such a class as a part of the rendering or a style in the component, as it imposes certain styles to the card body content and we do not want that- the client should be responsible for that
  • The toolbar buttons does not have k-toolbar-button classes. Should they be added for consistency with other components?

@TeyaVes TeyaVes self-assigned this Nov 13, 2024
@TeyaVes TeyaVes force-pushed the html-specs-corrections-6 branch 3 times, most recently from 82d18d9 to d6b54ce Compare November 14, 2024 12:09
@TeyaVes TeyaVes added the Test issues related to our visual / ci tests label Nov 14, 2024
@TeyaVes TeyaVes marked this pull request as ready for review November 14, 2024 12:30
@TeyaVes TeyaVes requested a review from a team as a code owner November 14, 2024 12:30
@TeyaVes TeyaVes requested review from kendo-bot and a team and removed request for a team and kendo-bot November 14, 2024 12:30
@TeyaVes TeyaVes added this to the 2024 Q4 (Nov) milestone Nov 15, 2024
@inikolova inikolova self-requested a review November 18, 2024 14:13
Copy link
Contributor

@epetrow epetrow left a comment

Choose a reason for hiding this comment

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

LGTM

@inikolova
Copy link
Contributor

Looks OK :) Just one suggestion from me - simplify Gantt templates - should be chore commit;

@TeyaVes
Copy link
Contributor Author

TeyaVes commented Nov 19, 2024

Looks OK :) Just one suggestion from me - simplify Gantt templates - should be chore commit;

I've updated the commit message to chore() instead of fix()

@TeyaVes TeyaVes force-pushed the html-specs-corrections-6 branch 2 times, most recently from 9b245bd to f721dc5 Compare November 20, 2024 15:20
@TeyaVes TeyaVes merged commit 3b261c3 into develop Dec 2, 2024
27 checks passed
@TeyaVes TeyaVes deleted the html-specs-corrections-6 branch December 2, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Test issues related to our visual / ci tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants