react-native-readium icon indicating copy to clipboard operation
react-native-readium copied to clipboard

[Bug] Scroll setting is not working properly on Android

Open M-HARIS-97 opened this issue 2 years ago • 10 comments
trafficstars

Please check the following video;

https://user-images.githubusercontent.com/57429370/207814193-abd50882-ebb3-4183-883a-e340d9d67b6b.mp4

M-HARIS-97 avatar Dec 15 '22 08:12 M-HARIS-97

@M-HARIS-97 please create a reproduction in the example app. I haven't seen this before.

jspizziri avatar Dec 15 '22 10:12 jspizziri

@jspizziri This epub has overflow-x: hidden / overflow-x: visible CSS for the <body> tag. That is what is causing this.

M-HARIS-97 avatar Dec 20 '22 11:12 M-HARIS-97

@M-HARIS-97 glad you sorted it out.

jspizziri avatar Dec 20 '22 12:12 jspizziri

This particular issue should be fixed in the next version of the Kotlin toolkit, see https://github.com/readium/kotlin-toolkit/pull/294

mickael-menu avatar Dec 20 '22 13:12 mickael-menu

@mickael-menu , wow! Thanks for letting us know! 🙏

How did this come across your radar in this project?

jspizziri avatar Dec 20 '22 14:12 jspizziri

I found this wrapper thanks to you actually 😄 I'm subscribed to the activity now.

Feel free to come participate in our bi-weekly Zoom calls or API discussions on Slack and the Readium repos. The more the merrier!

mickael-menu avatar Dec 20 '22 14:12 mickael-menu

@mickael-menu I just might do that! Where can I find the invite links?

jspizziri avatar Dec 20 '22 15:12 jspizziri

You should receive a Slack invitation by mail soon, let me know if you don't. Our Zoom calls are notified on the #general channel but we're taking a break until end of January to figure out what's the best timing and cast.

mickael-menu avatar Dec 20 '22 15:12 mickael-menu

@M-HARIS-97 kotlin-tooklit 2.3.0 has been released. As mentioned above the fix for this issue is contained in that release.

What remains to be done is to upgrade this library to use that version. Based on the migration guide it looks like there are several changes we'll need to make (specifically to how preferences work) in order to upgrade.

I encourage you to investigate and put together a PR if you need this fix soon. As for our team, it will probably be a while before it falls on our roadmap.

jspizziri avatar Jan 03 '23 17:01 jspizziri

The changes regarding the Preferences API and the HTTP server are optional in 2.3.0. However the Decoration.extras and PDF support sections in the migration guide are required to upgrade.

mickael-menu avatar Jan 04 '23 08:01 mickael-menu

@M-HARIS-97 i've just merged an upgrade to [email protected], which should resolve this issue.

jspizziri avatar Jul 26 '24 18:07 jspizziri