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

Riot 3.0.2 is 2 orders of magnitude slower than Riot 2.6 for high CSS tags #2152

Closed
irisjay opened this Issue Dec 10, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@irisjay

irisjay commented Dec 10, 2016

Help us to manage our issues by answering the following:

  1. Describe your issue:
    Let's say there is a tag 'tabs'. This tag has lots of CSS. On the first mount of the tag, it takes around 200 to 300ms for both Riot 2.6.7 and 3.0.2. In Riot 2.6.7, in each subsequent tag mount for 'tabs' is quite fast, with around only 10ms, yet this performance has deteriorated significantly in Riot 3.0.2, taking almost as much time as the initial mount.

  2. Can you reproduce the issue?
    The example for Riot 3: http://plnkr.co/edit/JPGbo8irDf4SjVeNEjRA?p=preview
    The example for Riot 2: http://plnkr.co/edit/JFdEvET0fjdfOU7ZSdga?p=preview
    The example is a page with three tabs. The tabs is the high CSS riot component. When you click on another tab, the router unmount the page and mounts the page with the other tab selected.

  3. On which browser/OS does the issue appear?
    Safari on MacOSX

  4. Which version of Riot does it affect?
    Riot 3.0.2

  5. How would you tag this issue?

  • Question
  • Bug
  • Discussion
  • Feature request
  • Tip
  • Enhancement
  • Performance

@GianlucaGuarini GianlucaGuarini referenced this issue Dec 13, 2016

Closed

100x slower than Riot 2!? #2147

2 of 7 tasks complete

@GianlucaGuarini GianlucaGuarini added fixed and removed to verify labels Dec 13, 2016

@GianlucaGuarini

This comment has been minimized.

Show comment
Hide comment
@GianlucaGuarini

GianlucaGuarini Dec 13, 2016

Member

fixed with riot@3.0.3

Member

GianlucaGuarini commented Dec 13, 2016

fixed with riot@3.0.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment