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

Manga reader fixes #2942

Closed
wants to merge 12 commits into from
Closed

Conversation

therobbiedavis
Copy link
Collaborator

@therobbiedavis therobbiedavis commented May 6, 2024

Fixed

  • Fixed: Fixed an issue where one some screens images would stack instead of being side by side while using the double setting in the manga reader.
  • Fixed: Fixed an issue where the cover and last page would not be positioned correctly when using the double (Mange) setting in the manga reader. (Fixes Double page/width images are misaligned when Double (Manga) view #2891)
  • Fixed: Fixed the issue with review cards provider and score extending outside it's card boundary.
  • Fixed: Fixed an issue with double scrollbar when viewing saved smart filters

Changes

  • Changes: Changed how scaling works in the manga reader to be more inline with the expected behavior. Height will scale to viewport height even if it is beyond the image's original dimensions. Width will scale to viewport width even if it is beyond the image's original dimensions. Original will show the image at its intensic dimensions, no scaling is applied. (Potentially fixes "Image scale" to "width" not working as expected #2669)

Width (large resolution):
image

Width (mobile, portrait):
image

Width (mobile, landscape)
image

Height (large resolution):
image

Height (mobile, portrait):
image

Height (mobile, landscape)
image

Original (large resolution):
image

Original (mobile, portrait):
image

Original (mobile, landscape):
image

# Fixed
- Fixed: Fixed an issue where one some screens images would stack instead of being side by side while using the double setting in the manga reader.
@therobbiedavis therobbiedavis added this to In progress in v0.8 - PDF & Comic Love via automation May 6, 2024
@therobbiedavis therobbiedavis added the bug Something isn't working label May 6, 2024
@therobbiedavis therobbiedavis changed the title Fixing stacking issue with double on manga reader Fixing stacking and position issue manga reader May 6, 2024
@therobbiedavis therobbiedavis changed the title Fixing stacking and position issue manga reader Manga reader fixes May 6, 2024
@majora2007
Copy link
Member

Reported back cutoff issues via discord, but overall this is looking MUCH better aligned to how I think the reader should function.

@majora2007 majora2007 added the UI Primarily UI impacting label May 7, 2024
"dynamic viewport size allows the content you design to fit exactly within the viewport, irrespective of the presence of dynamic browser interfaces."
https://developer.mozilla.org/en-US/docs/Web/CSS/length#dynamic

dvh is available for all major browsers: https://caniuse.com/viewport-unit-variants
@majora2007 majora2007 moved this from In progress to Done, Not Pushed in v0.8 - PDF & Comic Love May 14, 2024
@majora2007
Copy link
Member

This will be taken care of in #1426 as I needed to test 2 different PRs at once.

@therobbiedavis
Copy link
Collaborator Author

therobbiedavis commented May 20, 2024

@majora2007 I think you have the wrong reference, #1426 is over two years old. I think it should be #2960.

@majora2007 majora2007 closed this May 22, 2024
v0.8 - PDF & Comic Love automation moved this from Done, Not Pushed to Done May 22, 2024
@therobbiedavis therobbiedavis deleted the bugfix/double-reader-stacking-fix branch May 22, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working UI Primarily UI impacting
Projects
2 participants