Skip to content
This repository has been archived by the owner on Aug 10, 2024. It is now read-only.

Scroll performance is absolutely miserable #9

Open
ciriousjoker opened this issue Apr 3, 2022 · 4 comments
Open

Scroll performance is absolutely miserable #9

ciriousjoker opened this issue Apr 3, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@ciriousjoker
Copy link

ciriousjoker commented Apr 3, 2022

When I open the plugins tag, it takes around 2 mins to load the list and whenever I scroll, it takes 10 seconds to "apply" the new scroll offset. Sadly, it's completely unusable because of it.

A virtual scroller seems to be necessary here.

That being said, if you don't expand the plugin section it's usable.

@MingweiSamuel MingweiSamuel added the bug Something isn't working label Apr 4, 2022
@shaun-wild
Copy link

Performance in general is extremely poor, this app is next to unusable for me.

@ciriousjoker
Copy link
Author

ciriousjoker commented Jul 24, 2022

I believe chrome supports the element by now. Perhaps this can be an easy 2 line fix.

Edit:
Nevermind that thing has been abandoned.

Theresare simple J's libraries for that though. It doesn't even need to look fancy or support accurate scrollbars, being usable is totally enough.

@JustArion
Copy link

The performance has been greatly improved to being functional to use, though sadly I get an error while trying to build the project.

[nextron] Clearing previous builds
[nextron] Building renderer process
info  - Using webpack 5. Reason: Enabled by default https://nextjs.org/docs/messages/webpack5
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db

Why you should do it regularly:
https://github.com/browserslist/browserslist#browsers-data-updating
Failed to compile.

./layouts/main/index.tsx:7:23
Type error: Cannot find module '@state/Theme' or its corresponding type declarations.

   5 |
   6 | import Header from "@components/Titlebar";
>  7 | import ThemeAtom from "@state/Theme";
     |                       ^
   8 |
   9 | import Styled from "./assets/styled";
  10 |
info  - Checking validity of types .

Though yarn dev works fine and allows us to see the performance improvements.

Please publish a release or fix the build error.

@bayu01
Copy link

bayu01 commented Mar 1, 2023

I would love a rewrite of this plugin. pretty please n_n
Even with all the slowness it works great for exploratory purposes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants