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

ebook situation #225

Closed
cedounet opened this issue Sep 9, 2023 · 12 comments
Closed

ebook situation #225

cedounet opened this issue Sep 9, 2023 · 12 comments

Comments

@cedounet
Copy link
Collaborator

cedounet commented Sep 9, 2023

Let's use that as a master bug.

There are several problems with our ebooks.

  1. They are too big so nobody uses them on Kindle. There is an action about that Add version with smaller, B&W images for Kindle #216
  2. The whole (sans)-serif thing i think is useless. From my experience the reader opens with last selected font...
    Remove sans serif ebook from makefile #228
  3. We support 3 formats, two of them are outdated. I have no problem keeping them, if they are actually used of course. Remove azw3 and mobi ebook format from build #242
  4. The list of flowchart does not work (despite 2 entries). This might explain why you needed to do ruby magic for the website. Add list of flowcharts in ebook #240
  5. Some stuff don't show up greatly (degres, some tables, ...) Fix issue with degrees display in flowcharts #223

Might sound like a rant, I promise it is not, just me not being a user of ebooks I need guidance on what makes sense to do and somebody testing it.

Opinions/thoughts ?

C/

@cedounet
Copy link
Collaborator Author

cedounet commented Sep 11, 2023

@hendricius
Copy link
Owner

  1. I don't have a kindle myself. But my dad tried with the .epub version and he could use this tool to send it over (https://www.amazon.com/gp/sendtokindle). It supports max 200mb, so that should be good.

  2. This was a request originally from someone who had a hart time reading the pdf with serifs. I am no expert on accessibility, but I propose we keep it for now?

  3. Agreed - let's remove .mobi and azw3. We can update the instructions on the website for kindle users? It should save a lot of build time.

  4. At least for the website it works currently.

  5. Agreed. I will see if PR Fix issue with degrees display in flowcharts #223 fixes it and test a bit more.

@cedounet
Copy link
Collaborator Author

cedounet commented Sep 12, 2023

  1. pdf sans serif makes sense. Plenty of reasons not to like serif on low res screens… or other reasons. We should keep it.

The sans serif ebook I am more puzzled about as ebook will choose the font it wants and its size.

@cedounet
Copy link
Collaborator Author

  1. At least for the website it works currently.

Yes agree.. you did a good job at working around the issues for the website, just that does not solve the ebook. And we should fix them at the root if we can :)

@cedounet
Copy link
Collaborator Author

Agreed - let's remove .mobi and azw3. We can update the instructions on the website for kindle users? It should save a lot of build time.

I will do that in 2 steps... First remove downloads links and see if somebody is unhappy... If not I will remove the code in the makefile.

@cedounet
Copy link
Collaborator Author

Agreed - let's remove .mobi and azw3. We can update the instructions on the website for kindle users? It should save a lot of build time.

I will do that in 2 steps... First remove downloads links and see if somebody is unhappy... If not I will remove the code in the makefile.

#227

cedounet added a commit that referenced this issue Sep 13, 2023
As per #225
point 2.  The reader program will allow to change the font, no point
building it twice.
@cedounet
Copy link
Collaborator Author

  1. pdf sans serif makes sense. Plenty of reasons not to like serif on low res screens… or other reasons. We should keep it.

The sans serif ebook I am more puzzled about as ebook will choose the font it wants and its size.

#228

hendricius pushed a commit that referenced this issue Sep 15, 2023
* Add missing pic dependency on quick_ebook target

* Remove sans-serif ebooks from the build

* Remove sans-serif ebook from website and README

As per #225
point 2.  The reader program will allow to change the font, no point
building it twice.

* Remove sans_serif mk4 file
@hendricius
Copy link
Owner

@cedounet thanks! I agree, I checked on some ebook readers and they indeed ship their own fonts mostly. So it makes no sense to have 2 versions for epub. Only for the PDF.

@cedounet
Copy link
Collaborator Author

Look like we are done with the easy ones :D

@cedounet
Copy link
Collaborator Author

The list of flowchart does not work (despite 2 entries). This might explain why you needed to do ruby magic for the website.

#240

@cedounet
Copy link
Collaborator Author

cedounet commented Nov 10, 2023

Update:

the rest is addressed...

@cedounet
Copy link
Collaborator Author

Closing now as we are down to 2 issues #184 #183 which are non-critical and tracked somewhere else...

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

When branches are created from issues, their pull requests are automatically linked.

2 participants