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

Themes not working #89

Open
fxpar opened this issue May 1, 2022 · 6 comments
Open

Themes not working #89

fxpar opened this issue May 1, 2022 · 6 comments

Comments

@fxpar
Copy link

fxpar commented May 1, 2022

Hi, thanks for this great plugin.
I could not make the theme menu work, either with true or a list of themes.
Tried to download your gh-page branch, but couldn't make it work... which version of reveal.js are you working with?
Currently testing on windows 11.
(Transition menu works fine)

@salomon-chung
Copy link

salomon-chung commented Nov 17, 2022

Hi,

In view-source:https://denehyg.github.io/reveal.js-menu/ I noticed:

[...]
<link
      rel="stylesheet"
      href="lib/reveal.js/dist/theme/black.css"
      id="theme"
/>
[...]

Can you check if you have id="theme" attribute in your index.html?

@fxpar
Copy link
Author

fxpar commented Dec 19, 2022

Hi,
Thank you for your answer.
yes, the id="theme" is in the link tag. I will try to work again with it maybe during the holidays.

@AhmedThahir
Copy link

Any updates?

@gabrielbdornas
Copy link

Hi, I faced the same problem in this repo. I've been using this repo to create presentations inside the presentations folder. All presentations are generated through comping template files that are inside the templates folder.

Before installing the menu plugin I installed the Mermaid plugin. Mermaid plugin install documentation suggests that we use:

<link
  rel="stylesheet"
  href="https://cdn.jsdelivr.net/npm/[email protected]/dist/reset.css"
/>
<link
  rel="stylesheet"
  href="https://cdn.jsdelivr.net/npm/[email protected]/dist/reveal.css"
/>
<link
  rel="stylesheet"
  href="https://cdn.jsdelivr.net/npm/[email protected]/dist/theme/black.css"
  id="theme"
/>

Which confused my local files:

<link rel="stylesheet" href="../../dist/reset.css" />
<link rel="stylesheet" href="../../dist/reveal.css" />
<link rel="stylesheet" href="../../dist/theme/solarized.css" id="theme"/>

When I realized that and remove the link tags suggested by Mermaid everything worked fine.

What I'm trying to suggest, doing your previous plugin installations could not include Mermaid, is that another link tag in your html file could be your problem, confusing the correct ones.

@jpggithub
Copy link

jpggithub commented Oct 12, 2023

Hi
has anyone managed reveal.js-menu to make it work?
Personally, I didn't (reveal.js updated)
EDIT: Sorry, my fault, I didn't check the presence of id="theme" as indicated here and in the documentation! It works when it is added in index.html

@ssl-bio
Copy link

ssl-bio commented May 3, 2024

Hello,
In my case I found that the path to the theme folder was missing in the Reveal.initialize section, more specifically
menu:{themes:true, themesPath:'/path/to/reveal.js/dist/theme/'}
After setting that I'm able to change themes again

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

No branches or pull requests

6 participants