-
Notifications
You must be signed in to change notification settings - Fork 185
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/non linear spine items #402
base: develop
Are you sure you want to change the base?
Conversation
Conflicts: readium-js
Conflicts: readium-js
Conflicts: readium-js
…"return to primary reading flow" (non-linar spine items)
…imary reading flow from ancillary documents)
Live demonstration: |
Aside from the conflicts noted above and in the related PR 211, is there more work to be done here? |
@rkwright yes, I need to test Media Overlays (make sure the boundaries of the ancillary document are not crossed when audio playback reaches the end of the spine item), and I need to check the general behaviour in the scroll view mode(s). |
Defer out of 0.22 because there is more testing and some minor modifications still needed. |
Not sure why I was not seeing this correctly yesterday but I do see now the Also the shortcut key 0 also seems to work as expected and from my point of view this looks great. I think the shortcut "0" is fine, I think the icon image is fine as well (i.e.: play on the REW and REW to Beginning from the old tape deck days which I think makes a lot of sense) Is there anything else we need to decide before this gets merged into Readium main developers branch? |
I think this is a UI that is a bit easter-eggy in that you need to hover Ric From: AnFengDe [email protected] Not sure why I was not seeing this correctly yesterday but I do see now the Also the shortcut key 0 also seems to work as expected and from my point of I think the shortcut "0" is fine, I think the icon image is fine as well Is there anything else we need to decide before this gets merged into � |
I see, and I guess the same goes for the bottom left corner where the |< "Go Back in Navigation History [9]" pops up when you hover over that area if applicable. If that the case then maybe would we want to make those new buttons show up all the time when you are on a non-linear page to show the user that there is something special here and it helps them get back to where they were? I think everyone understands what the top button in the upper left corner will do right? This will always bring you back no matter how many pages you go down on a non-linear document or even if one non-linear document references another non-linear document and the user then follows that and goes multiple pages in that document that pressing this top left button "Return to Primary Reading Flow [0]" that the user will be brought straight back to where they left off in the original book before they explored the non-linear content. (I assume everyone agrees this is what this button does) I guess my other question is how that 2nd button in the bottom left corner "Go Back in Navigation History [9]" behaves since I am trying to understand what it does especially when there are multiple pages on a non-linear page and maybe what the behavior is when one non-linear page links to another non-linear page which may have multiple pages on it as well. Either this button is not doing what it was meant to do or I am really not understanding what it should do. I thought this would bring you back page by page, and document by document until you get ultimately back to the main document before you started exploring the non-linear spine item. What is a little confusing is that the link Daniel gave http://readium-diagram.surge.sh/?epub=nonLinear starts at the nonLinear page so this button doesn't appear initially and then clicking on the embedded link to the << (8) linear (primary) will cause this button to start showing up in the bottom left corner which when pressed brings you back to the non-linear page. So what was messing me up is I started at the non-linear page, then pressed the link to << (8) linear (primary) then pressed the link (9) non-linear (ancillary) > at which point finally the "Button: Return to Primary reading flow [0]" appears but the "Button: Go Back in Navigational History [9] can be pressed twice, once to go back to the main document and again to bring you back to the non-linear page since this was the first page you were on since the link itself was to this non-linear page originally. I guess that all makes sense but just making sure this is what everyone expects to happen. We can discuss this with a UI/UX employee if you would like but I am trying to understand what we want/need here. Thanks |
User interface for readium/readium-shared-js#211
Fixes:
#188