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

Fix bug in speaker notes that broke navigating to next slide #2634

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

Conversation

michael-kerscher
Copy link
Collaborator

Change this selector to use the ~= selector to test if a white space separated word "prev" or "next" is contained

Fixes a speaker notes bug that did not allow going to the next slide in the speaker notes.
The reason for that is that the "rel" attribute contained "prev" respective "next prefetch".

See: https://developer.mozilla.org/en-US/docs/Web/CSS/Attribute_selectors#attrvalue_2

This fixes part of #2004 when going to the right (containing "prefetch").

…e speaker notes.

The reason for that is that the "rel" attribute contained "prev" respective "next prefetch".
Change this selector to use the ~= selector
See: https://developer.mozilla.org/en-US/docs/Web/CSS/Attribute_selectors#attrvalue_2
Copy link
Collaborator

@djmitche djmitche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!!

@randomPoison
Copy link
Collaborator

What behavior exactly is this fixing? I tried it locally and navigating to the right still breaks the speaker notes (speaker notes become disconnected from the main slides, and the speaker note section re-appears at the bottom of the regular notes). Was there a different issue that this fixes?

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

Successfully merging this pull request may close these issues.

3 participants