Skip to content
This repository has been archived by the owner on May 15, 2021. It is now read-only.

1465 Add "background div" to allow HTML backgrounds #44

Open
nickshine opened this issue Apr 29, 2019 · 4 comments
Open

1465 Add "background div" to allow HTML backgrounds #44

nickshine opened this issue Apr 29, 2019 · 4 comments

Comments

@nickshine
Copy link
Member

This feature would be a nice addition, but maybe using <template> for the html that is acting as the background to avoid having to put style="visibility: hidden;" explicitly.

@nickshine
Copy link
Member Author

nickshine commented Apr 29, 2019

To do:

  • implement in to latest reveal.js code
  • properly document examples as it is a little less straight-forward than the data-background-image or data-background-video types.

@nickshine
Copy link
Member Author

hakimel#1465

@cujarrett
Copy link

I’d lean towards documenting it but don’t feel strongly either way.

@LongLiveCHIEF
Copy link

I've started to wonder how much styling the core of reveal should have in 2.0, and whether or not all "theme" styling should be plugins based on hooks the core implements.

"Theme" being anything that isn't basic css required to handle presentation functionality.

For now, as I've been reviewing, most styling ones that seem basic, I put on the "planned" list, and figure they can be broken out if we make all "theme" based stuff implementable via plugins.

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

No branches or pull requests

3 participants