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

showoff a little using Dr. Nikola's Vendetta #2063

Merged
merged 10 commits into from Sep 15, 2015
Merged

showoff a little using Dr. Nikola's Vendetta #2063

merged 10 commits into from Sep 15, 2015

Conversation

@ralsina
Copy link
Member

ralsina commented Sep 11, 2015

This branch tries to show off some of the per-story / per-site customizations Nikola allows, using
a longish text shown as an e-book.

  • Use custom template to make it book-like
  • Use hyphenation to make text layout better
  • Use JS animations to make it scroll smoothly
  • Use FlowType to keep fonts optimally readable
  • 2/1 column adaptive layout (try to resize window)
  • Google Fonts for nicer look

Looks better if you have pyphen installed and enable typogrify filters for HTML.

It's an example, not a real ebok reader. It's good enough :-)

Still to do:

  • Write a document explaining how this was done
  • Announce it in a nice place
  • Tune scrollbar (pain!)
  • Remove the fixed viewport height
  • The second image sometimes loses aspect ratio.
  • On clicking on the book, make sure page scrolls so it's fully visible.
  • On window resizing, make sure we always display on a page boundary

Add more TODOs if you notice anything / want anything

ralsina added 2 commits Sep 11, 2015
@Kwpolska
Copy link
Member

Kwpolska commented Sep 11, 2015

Tune scrollbar (pain!)

What do you plan to do?

Use JS animations to make it scroll smoothly

NO NO NO
Don’t fuck with users’ scrolling. Instead, go to chrome://flags and enable smooth scrolling in your web browser.

@ralsina
Copy link
Member Author

ralsina commented Sep 11, 2015

@Kwpolska mostly just make the scrollbar less conspicuous. Maybe just use overflow tricks to hide it.

I only meant making it scroll smoothly when the user clicks to "turn the page", it's already there, check it out.

@Kwpolska
Copy link
Member

Kwpolska commented Sep 11, 2015

As of right now:

  1. The title font looks ugly IMO
  2. There should be a next/previous page button, clicking there is not immediately obvious
  3. Images get squished at smaller window sizes
  4. The project gutenberg links leads to a 403 forbidden error (PS. they use CherryPy!)
  5. I think we might need to add some ©/PG info there
  6. There should be more details on how this page works
@ralsina
Copy link
Member Author

ralsina commented Sep 11, 2015

@Kwpolska

  1. Yes, it needs a better title font, this was to remind me to get one (although it was something this sort of pulp would be typeset with!)
  2. well, it's how all ebook readers work, but sure, why not :-)
  3. They shoudn't after last commit
  4. That's probably temporary. It used to work, and it's done in the way they suggest
  5. Maybe, I did keep all the PG frontmatter
  6. Totally, I want to write a whole tutorial
@Kwpolska
Copy link
Member

Kwpolska commented Sep 11, 2015

  1. My Kindle 4 doesn’t, it has actual physical buttons that I can press!
  2. They still do.
@ralsina
Copy link
Member Author

ralsina commented Sep 11, 2015

Oh, a fellow kindle-with-buttons fan! I am tempted to get a voyage just to get those back but with a better screen :-)

@Kwpolska
Copy link
Member

Kwpolska commented Sep 11, 2015

(The Kindle 4 is awesome, but it’s painful with a 23-character WiFi password…)

ralsina added 4 commits Sep 11, 2015
…use Gentium black for titles
@Kwpolska
Copy link
Member

Kwpolska commented Sep 15, 2015

What’s the status of this?

@ralsina
Copy link
Member Author

ralsina commented Sep 15, 2015

After we publish the tutorial in the site, or at least decide it's final, I can backport it here.

ralsina added a commit that referenced this pull request Sep 15, 2015
showoff a little using Dr. Nikola's Vendetta
@ralsina ralsina merged commit 6984ccb into master Sep 15, 2015
0 of 3 checks passed
0 of 3 checks passed
continuous-integration/appveyor Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
@ralsina ralsina deleted the dr-nikola-gets-it branch Sep 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.