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

New theme: Diamonds #1074

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

New theme: Diamonds #1074

wants to merge 1 commit into from

Conversation

gnclmorais
Copy link

This theme is 95% based on simple.scss, the big change are the arrows.
I was never fond of reveal.js’s arrow, and always thought some kind
of squares would look muche better.

So, for the sake of checking how these themes work, I did this.
reveal.js doesn’t play along well with what I’ve done… That’s why
you can find a couple of !important on diamonds.scss.
Maybe a simple refactor would increase themes customisation…?

This theme is 95% based on simple.scss, the big change are the arrows.
I was never fond of reveal.js’s arrow, and always thought some kind
of squares would look muche better.

So, for the sake of checking how these themes work, I did this.
reveal.js doesn’t play along well with what I’ve done… That’s why
you can find a couple of !important on diamonds.scss.
Maybe a simple refactor would increase themes customisation…?
@bnjmnt4n
Copy link
Collaborator

bnjmnt4n commented Mar 8, 2016

@hakimel what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants