-
Notifications
You must be signed in to change notification settings - Fork 222
Reveal CSS structural update #2598
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
Reveal CSS structural update #2598
Conversation
650975e to
8b0a089
Compare
|
Force pushed to fix typo in commit message. @rbeezer, @davidaustinm, @sean-fitzpatrick - I view this PR as a self contained (and complete) implementation of what I proposed to do. If other people are actively working on different Reveal related improvements that might intersect CSS, I suggest resolving/merging this before they get too far into things. It will clean up the path for them and give them more tools to work with (custom css file for reveal). |
|
I don't currently have anything on the go, and don't expect to in the near future. I look forward to seeing how this turns out! |
|
Dang. Forgot to put the PR number onto the merge commit. So here goes, it is at: d6a94f8 |
|
Thanks for getting this setup! Looks good. Merged, mostly as-is. Just one hunk of whitespace. ;-) I've done that flag thing for a derivative stylesheet. Sometimes I've gone back and ripped it out. It'd be nice if we didn't have to do it here. Maybe with an Would you like to post on -dev to "warn" about CSS changes and to advertise the new option for custom CSS? |
Removes baked in CSS from reveal template and uses version built by cssbuilder script.
Adds pubvar to specify custom css files for revealjs builds.