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

Layout corruption in Arabic Right-To-Left #516

Open
horimasumi opened this issue Apr 8, 2016 · 14 comments
Open

Layout corruption in Arabic Right-To-Left #516

horimasumi opened this issue Apr 8, 2016 · 14 comments

Comments

@horimasumi
Copy link

■This issue is a:

• Bug report

■ Observed behavior

Page corruption occurs as shown in the attached file while using Arabic Right-To-Left (writing mode + page progression direction).
The sample EPUB can be found in the following URL.

IDPF EPUB3 Samples Project, EPUB 3 Samples Table:
https://drive.google.com/uc?export=download&id=0B9g8D2Y-6aPLUUkxa2VYbURqVjA

The trouble occurs in the following conditions:

  1. When using Chrome browser.
  2. When the text amount of page.xhtml is large and exceeds to the next page.
  3. The layout can be corrected by going back and forward a page.

■ Product

• Readium Chrome extension ** latest official version available from the Chrome Web Store
• Readium cloud reader app ** local build ### Web browser Tested only with Chrome Version 49.0.2623.87 (64-bit)

@rkwright
Copy link
Member

rkwright commented Apr 8, 2016

Verified this in the specified book in the latest build from develop.

  1. Open Regime-Anti-Cancer
  2. Page to left a couple of pages with window large enough for a two-page spread.
  3. Result:
    image

Verified on OSX 10.10 with development build (2.23.0-alpha) in Chrome Version 49.0.2623.110 (64-bit)

@danielweck
Copy link
Member

danielweck commented May 25, 2016

something to try:

https://github.com/readium/readium-shared-js/blob/develop/js/views/reflowable_view.js#L779

@ -45905,9 +45919,15 @@ var ReflowableView = function(options, reader){

     function showBook()
     {
+        console.log("PATCHED by OZ");
+        var currOpacity = _currentOpacity;
         if (_currentOpacity != -1)
         {
-            _$epubHtml.css('opacity', _currentOpacity);
+            _$epubHtml.hide();
+            setTimeout(function() {
+                _$epubHtml.show();
+                _$epubHtml.css('opacity', currOpacity);
+            }, 50);
         }
         _currentOpacity = -1;
     }

thanks Helicon!

@danielweck
Copy link
Member

Related: readium/readium-shared-js#282

@da70
Copy link
Contributor

da70 commented Jun 1, 2016

We are able to reproduce this bug with the sample EPUB referenced in this ticket. Had to advance many pages before coming back in order to correct the layout.

  • ReadiumJS Viewer 9c341d64a9c14f81952c6caf015453bbef65d303 (with cherry-picked commit 3946d54c594b2a3a44564caa0478b64b375a9aaa applied to submodule readium-js) - cloud reader build.
  • Chrome version 50.0.2661.94 (64-bit)
  • Mac OS X 10.10.5

3 columns:

3-columns

2 columns, after advancing many pages and coming back:

2-columns

@rkwright
Copy link
Member

rkwright commented Jun 1, 2016

I can confirm that I was also able to reproduce it easily with the same document.
Using current build (2.23.0-alpha) from develop on OSX 10.11.5 on my MBP.
Note that I was able to reproduce it with either a single column or two column layout . See image below.

arabic_err

@mahag
Copy link

mahag commented Sep 21, 2016

Any fix?

@ghost
Copy link

ghost commented Oct 25, 2016

Any new update to this issue?

@mahag
Copy link

mahag commented Oct 25, 2016

am still waiting for a fix too

@janbolat
Copy link

Any updates?

@janbolat
Copy link

janbolat commented Apr 25, 2017

I am unable to reproduce this bug on OSX (Firefox). Test link:
https://readium.firebaseapp.com/?epub=https%3A%2F%2Fcdn.rawgit.com%2FIDPF%2Fepub3-samples%2Fmaster%2F30%2Fregime-anticancer-arabic&goto=%7B%22idref%22%3A%22content%22%2C%22elementCfi%22%3A%22%2F4%2F2%2F2%2F2%2C%2F1%3A0%2C%2F1%3A1%22%7D

screenshot from 2017-04-25 12 31 37

Hi @danielweck,
I have three columns on Ubuntu 14.04, Chrome Version 56.0.2924.87 (64-bit).
Also having similar problems with my own books on local server.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants