scaife-viewer / readhomer

in-progress flagship demo project for the Scaife Viewer ecosystem
3 stars 2 forks source link

ability to enter a passage identifier to jump to it #8

Closed jtauber closed 5 years ago

jtauber commented 5 years ago

Note that this might need to be in the reader widget itself so it can be used in cases where there is more than one reader widget

Reference Model: A2

jtauber commented 5 years ago

I think this is basically a text input field that replaces the row of 24 book links

jtauber commented 5 years ago

While the existing Homer Reference Input is useful with the Reader, I think the Homer Reader needs to replace the book row with it too (as mentioned in the previous comment).

Also probably needs a placeholder with an example of the format to use and we'll have to consider how to handle both books in #22

Finally, I think it should submit on hitting return. I shouldn't have to click on lookup (although have the button there is fine as otherwise it might not be clear what to do; but my natural instinct is to hit enter and it doesn't work)

paltman commented 5 years ago

@jtauber so to be clear the action items are:

jtauber commented 5 years ago

"(not sure exactly what)" -> "text input field that replaces the row of 24 book links"

Instead of the row of 24 book links (which can only be used to retrieve a book at a time) put the text input / button from the separate widget there.

jtauber commented 5 years ago

Okay, after discussion, what I propose is:

The text input field replaces the row of 24 book links but there is also a toggle to get the passage reference from the store. If this is toggled, the text input will be disabled (but will still show the passage).

In short, this means each reader component can show its own passage OR get the passage from the store (where it is set by the Homer Reference Input).

With this change, there is no reason for both the Reader and Homer Reader. The 2x2 is still useful (and will be more useful with this change and will fulfill #5). The Parallel reader isn't that interesting until we actually get to #17.

jtauber commented 5 years ago

but see #26