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

FinTech Grid poor performance #1020

Closed
zdrawku opened this issue Mar 15, 2022 · 2 comments · Fixed by #1021
Closed

FinTech Grid poor performance #1020

zdrawku opened this issue Mar 15, 2022 · 2 comments · Fixed by #1021
Assignees
@Lipata @simeonoff @valadzhov
Labels
👀 status: in-review

Comments

@zdrawku
Copy link
Contributor

zdrawku commented Mar 15, 2022 •
edited by hanastasov
Loading

Description

Video - https://www.screencast.com/t/LXi2rtoxC

The performance of the Grid is poor compared with what it was before the latest release

  1. Get latest igniteui-cli
  2. Run the FinTech sample
  3. Reproduce the steps from the video
  4. Switching between dark/light themes resolves the issue. Poor performance happens on Light theme only.
@hanastasov hanastasov self-assigned this Mar 15, 2022
@hanastasov
Copy link

Change to Dark Theme it solves the issue

@dobromirts
Copy link

The performance issue is caused by '@include theme($default-palette);' on line 234 in the fintech sample generated from the cli part of the .scss file and could be removed in the generated template in the cli.

@Lipata Lipata transferred this issue from IgniteUI/igniteui-angular-samples May 11, 2022
@Lipata Lipata assigned Lipata and valadzhov and unassigned hanastasov May 11, 2022
valadzhov added a commit that referenced this issue May 16, 2022
@valadzhov
fix(fintech grid): fix slow grid #1020
4f888dc
@valadzhov valadzhov mentioned this issue May 16, 2022
Merged
Lipata added a commit that referenced this issue May 16, 2022
@Lipata
Merge pull request #1021 from IgniteUI/valadzhov/fintech-performance-…
8067217 
…fix-1020

fix(fintech grid): fix slow grid #1020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Lipata Lipata

@simeonoff simeonoff

@valadzhov valadzhov

Labels
👀 status: in-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(fintech grid): fix slow grid #1020 IgniteUI/igniteui-cli
6 participants
@Lipata @simeonoff @valadzhov @zdrawku @hanastasov @dobromirts