playing around with the styles a bit.#2
Draft
coldnebo wants to merge 5 commits intofreerobby:mainfrom
Draft
Conversation
Owner
|
@coldnebo Thanks for this. I mostly like these changes aesthetically. Only nit is I would avoid the all-uppercase because I think it makes it harder to read (can't make out the shape of the word as well). I don't recall all the ways I tried to format things, but I do remember it being challenging to nail the screen + print/PDF views in any elegant way. If there is a cleaner way to do it with column CSS that would be great (sorry I can't be more helpful -- it's been ages since I attempted something like that). Appreciate the work here! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Here's my early work. what do you think? @freerobby @eilifjohansen
style inspired by Scott Pace and Marc Nathanson of EastCoastAero Club [example PDF], which I find to be a nice balance of tight layout and legibility.
I included @eilifjohansen branch with the dots update. I'm considering doing a revamp of the columns using the css3 column support, but don't know if you tried that already. I'd like the checklists to flow to the page length, with a 3 column layout preferred... any more or less starts to look strange. The problem is the fixed height of the current way the columns are being done, but column CSS may now have a way around that, I don't know, it's been a while since I tried 3 col layouts... a lot has changed. ideas welcome!
This isn't a clean PR yet, so I'm keeping it a draft until the other PR gets merged or so.
UPDATE:
Screenshot:

Example PDF:
aviation-checklist.pdf
The tradeoff I made was the print version looks ok, but the screenshot has really long columns. Not sure how to avoid that or break it up, but if the use is primarily print, maybe it's ok?
It's also likely that this tweaking looks good on my system only (linux/chrome/webfonts). the Screenshot and PDF should provide guidance on what I see.