Bookreader should use full UI (not embed) on Android tablets

Bug #782301 reported by George
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Internet Archive BookReader
Fix Released
Medium
mangtronix

Bug Description

From Dan Scott:

Mozilla/5.0 (Linux; U; Android 3.0.1; en-us; GT-P7510 Build/HRI83)
AppleWebKit/534.13 (KHTML, like Gecko) Version/4.0 Safari/534.13

The results are that I get the embedded view in the online reader (I
get the same on my Android phone), and particularly troubling is that
there are no images in the embedded view.

mangtronix (mang)
Changed in bookreader:
importance: Undecided → Medium
Revision history for this message
mangtronix (mang) wrote : Re: Bookreader uses Embed view on Android tablets

Confirmed on Motorola Blur. The embed view *should* be used for Android phones, but not for tablets (this is what we do on iOS).

On the Motorola Blur when starting in embed view the first few pages appear but then scrolling to new pages does not cause the image to load. After going to the home screen and then back to the web browser the page images load normally, including scrolling to new pages. Likely a view setup or browser event issue related to lazy loading of page images.

summary: - Bookreader uses Embed view on Samsung Galaxy
+ Bookreader uses Embed view on Android tablets
mangtronix (mang)
Changed in bookreader:
milestone: none → codesprint
status: New → Confirmed
assignee: nobody → mangtronix (mang)
Revision history for this message
mangtronix (mang) wrote :

Removing special logic for Android devices. Some tablet devices use Android with Mobile Safari and browsers lie or are inaccurate about window.screen.width. There doesn't appear to be a general/robust way to determine the screen resolution of an Android browser on first load.

All Android devices will get the default BookReader behaviour (full ui unless ?ui=embed specified).

mangtronix (mang)
summary: - Bookreader uses Embed view on Android tablets
+ Bookreader should use full UI (not embed) on Android tablets
Changed in bookreader:
status: Confirmed → Triaged
mangtronix (mang)
Changed in bookreader:
milestone: codesprint → cleanup
mangtronix (mang)
Changed in bookreader:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.