-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Sizing covers #12
Comments
I don't feel ready to weigh into discussing this question because I don't have a good sense of how we envision connecting unglue.it, gitensite, and github at an over architectural scale (along with attendant goals). A lot of my thinking has been driven by the short term goal of producing 50 books. That's why, for example, we just store a cover.jpg in its respective repo: for the pragmatical reason of making it easy to build a book using travis. To decide on where to store covers and how to resize them, we have issues like:
|
should leave it out of travis. other websites will want to scale differently |
Generally, the full versions are way too big for an epub. You probably want I don't know the answers below.
|
@jenny8lee You should know that we had decided to write the big covers to the 50 repos as cover.jpg at this point. (for example: https://github.com/GITenberg/Adventures-of-Huckleberry-Finn_76/blob/master/cover.jpg) and building that epub based on the large cover. |
I suggest writing a reusable django app for this. It would contain a model On Thu, Jul 30, 2015 at 12:40 PM, Raymond Yee [email protected]
|
that's pretty much what I was thinking. |
In #5 it was decided to use large covers, we can resize later. I suggest this can be done in Unglue.it or in giten_site.
In Giten_site these are stored as version controlled repository assets. In unglue.it, covers are stored in S3 using django file storage. I expect it's not practical to use (duplicate) repo assets and instead we should make sizing code for unglue.it and deplow on gitensite as well.
Thoughts?
The text was updated successfully, but these errors were encountered: