Skip to content

Reading Options page#241

Merged
eshellman merged 13 commits intodevfrom
JohannesSeikowsky-patch-3
Feb 3, 2026
Merged

Reading Options page#241
eshellman merged 13 commits intodevfrom
JohannesSeikowsky-patch-3

Conversation

@JohannesSeikowsky
Copy link
Contributor

@JohannesSeikowsky JohannesSeikowsky commented Jan 30, 2026

Kicking off important work on Issue #223.
We really should make it MUCH easier for new users to figure out how to start reading Gutenberg books.

This is a new page that clearly lays out the different options people have for reading. Focus is on clarity, practicality and also brevity (as far as brevity is possible) given peoples' limited attention and patience online.

Essentially trying to make it as straightforward as possible for new users to understand what exactly they can do to start reading our books.

Bibrec also needs work to incorporate something like this. Next stop.

@JohannesSeikowsky
Copy link
Contributor Author

Screenshot_2026-01-30_18-56-53

@JohannesSeikowsky
Copy link
Contributor Author

Screenshot_2026-01-30_18-57-01

@JohannesSeikowsky
Copy link
Contributor Author

etc

@JohannesSeikowsky JohannesSeikowsky changed the title Read Options page Reading Options page Jan 30, 2026
@eshellman eshellman changed the base branch from master to dev January 30, 2026 20:18
@eshellman
Copy link
Collaborator

should we do something with mobile.html?

@eshellman
Copy link
Collaborator

could you update your branch from master? thanks

@JohannesSeikowsky
Copy link
Contributor Author

JohannesSeikowsky commented Jan 30, 2026

should we do something with mobile.html?

to be honest, I don't really think so. That page would require a serious amount of editing (it's significantly too wordy) and it may even be a bit confusing to have two pages that cover similar grounds and have essentially the same purpose. As of now mobile.html is still being linked to from "/help/". I'd suggest to either leave it like that or just take the page out.

could you update your branch from master? thanks

I guess you mean to write "from dev"? sure.

@Rowan-McKereghan
Copy link
Contributor

I think this is a good start. I know from experience that even though these instructions seem simple, people (especially older people who tend to use PG) will still get turned around – if there's a way to embed a video walkthrough for kindle, libby, etc. at some point that would be ideal imo

@JohannesSeikowsky
Copy link
Contributor Author

JohannesSeikowsky commented Jan 31, 2026

making it simple/straightforward is really the main thing I tried to do (not very easy to do btw).

I think video walkthroughs are a very good idea!

@eshellman
Copy link
Collaborator

This will be a big improvement!!

@eshellman
Copy link
Collaborator

When I tried updating the branch to view it with current css, there were merge conflicts.

@JohannesSeikowsky
Copy link
Contributor Author

hm not sure why that is. Technically this PR is pretty straightforward. Can you resolve the conflicts?

JohannesSeikowsky and others added 3 commits February 1, 2026 01:25
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
JohannesSeikowsky and others added 7 commits February 1, 2026 01:25
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
Co-authored-by: Eric Hellman <eric@hellman.net>
@JohannesSeikowsky
Copy link
Contributor Author

JohannesSeikowsky commented Feb 2, 2026

@eshellman I can't recreate your merge conflicts.

@JohannesSeikowsky
Copy link
Contributor Author

In fact Github tells me "No conflicts with base branch"
lmk if something needs to be done here, but looks fine to me.

@eshellman
Copy link
Collaborator

The problem was that you re-used a branch name. I was unable to test the current, same-named branch until I explicitly deleted the previous branch

@JohannesSeikowsky
Copy link
Contributor Author

oh I see. sorry - didn't notice.

@JohannesSeikowsky
Copy link
Contributor Author

all good now?

@eshellman eshellman merged commit f03f621 into dev Feb 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants