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

Fix/library manga grid infinite item size on category switch #378

Merged
merged 3 commits into from
Jun 17, 2023
Merged

Fix/library manga grid infinite item size on category switch #378

merged 3 commits into from
Jun 17, 2023

Conversation

schroda
Copy link
Collaborator

@schroda schroda commented Jun 17, 2023

No description provided.

schroda added 3 commits June 17, 2023 22:21
This reverts commit 48d559e

This fixed (and worked as expected in dev mode) the very rare issue in dev mode (you had to load the page with a small screen size and increase the screen size before the grid and grid items were rendered) and caused it frequently in production build in the Library grid when switching categories
Hook only run once, which could lead to an issue where the grid item width was calculated to be infinite (e.g. in SearchAll due to strict-mode behaviour in the dev mode).
Could be that this is only an issue in dev mode due to react-strict mode.
@schroda schroda merged commit 0f029cb into Suwayomi:master Jun 17, 2023
@schroda schroda deleted the fix/library_manga_grid_infinite_item_size_on_category_switch branch June 17, 2023 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant