Skip to content

Unify rendering Skeleton #4531

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

Merged
merged 8 commits into from
Jul 24, 2023
Merged

Unify rendering Skeleton #4531

merged 8 commits into from
Jul 24, 2023

Conversation

kikostadinov
Copy link
Contributor

@kikostadinov kikostadinov self-assigned this Jun 7, 2023
@kikostadinov kikostadinov force-pushed the unify-rendering-skeleton branch from 071b8ff to 21d0187 Compare June 7, 2023 13:48
@epetrow epetrow added this to the 2023.3 milestone Jun 8, 2023
@epetrow
Copy link
Contributor

epetrow commented Jun 13, 2023

Also rename the last commit to "chore(html): update captcha spec"

@kikostadinov kikostadinov force-pushed the unify-rendering-skeleton branch 2 times, most recently from 3555190 to db6744a Compare June 13, 2023 14:28
@kikostadinov kikostadinov force-pushed the unify-rendering-skeleton branch from 641529c to 39b0fc9 Compare June 30, 2023 11:54
@kikostadinov kikostadinov requested a review from epetrow June 30, 2023 12:07
@kikostadinov kikostadinov force-pushed the unify-rendering-skeleton branch from be5e752 to ca01d95 Compare July 7, 2023 11:42
@kikostadinov kikostadinov requested a review from Juveniel July 7, 2023 11:42
@epetrow epetrow force-pushed the unify-rendering-skeleton branch from 2135973 to d0c1cef Compare July 12, 2023 14:00
Copy link
Contributor

@JoomFX JoomFX left a comment

Choose a reason for hiding this comment

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

Regarding this commit - please replace the Skeleton rendering (<span className="k-skeleton k-skeleton-text k-skeleton-pulse"></span>) with the actual component (<Skeleton />).

@kikostadinov kikostadinov force-pushed the unify-rendering-skeleton branch from 25649b2 to 6c94f08 Compare July 14, 2023 05:52
@kikostadinov kikostadinov requested a review from JoomFX July 14, 2023 05:55
@epetrow epetrow added C: Skeleton Enhancement New feature of an existing functionality or an improvement of an existing functionality. labels Jul 18, 2023
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.

looks good to me

@Juveniel Juveniel marked this pull request as ready for review July 20, 2023 05:06
@Juveniel Juveniel requested a review from a team as a code owner July 20, 2023 05:06
@epetrow epetrow force-pushed the unify-rendering-skeleton branch from 9980f28 to 5bbb74e Compare July 21, 2023 08:25
@epetrow epetrow force-pushed the unify-rendering-skeleton branch from 5fefde9 to b037326 Compare July 21, 2023 09:29
@epetrow epetrow force-pushed the unify-rendering-skeleton branch from db692e6 to 2928c85 Compare July 21, 2023 09:53
@Juveniel Juveniel merged commit 8fb5fe9 into develop Jul 24, 2023
@Juveniel Juveniel deleted the unify-rendering-skeleton branch July 24, 2023 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Skeleton Enhancement New feature of an existing functionality or an improvement of an existing functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants