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

Feature/merge source and extensions screen on desktop #535

Merged
merged 8 commits into from
Jan 2, 2024
Merged

Feature/merge source and extensions screen on desktop #535

merged 8 commits into from
Jan 2, 2024

Conversation

schroda
Copy link
Collaborator

@schroda schroda commented Jan 2, 2024

closes #10

- remove margin - virtuoso has issues calculating the height otherwise
- change group title styling, fontSize caused issues
There was an issue with the height calculation of the virtuoso list, which apparently was caused by the conditional rendering of the group title and item as list items.
Due to this, while scrolling down, the virtuoso list caused an overflow, which rendered the scrollbar, which in turn caused the list items to shrink in width.
The end result of this was, that the list items "jumped" slightly on the initial render of the items, due to the scrollbar appearing and disappearing
@schroda schroda merged commit db65b9d into Suwayomi:master Jan 2, 2024
1 check passed
@schroda schroda deleted the feature/merge_source_and_extensions_screen_on_desktop branch January 2, 2024 22: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.

[Feature Request] Merge sources and extensions screen
1 participant