For a fragment: allow to show a separate note defined in it #1636
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When a slide has several fragments it could be convenient to define a note for each of them. In this case we need to show only this specific note defined in a fragment and not others. General note of a slide shouldn't be also shown, as a more specific one should have greater relevance in this case.
|
Great idea 🙌 |
hakimel
added a commit
that referenced
this pull request
Jan 31, 2017
|
^ made a few changes to this, to make it work correctly when stepping to and from a slide with multiple fragment notes. Also added support for notes in an attribute |
pm-nsimic
pushed a commit
to chriwu/ps-tec-2017
that referenced
this pull request
May 1, 2017
* resolves hakimel#1324 allow display property to be configured - allow display property apply to current slide to be configured * add 'npm build' script that launch grunt * Allow switching off preview links The data-preview-link=false tag can be used to switch off preview links * Support sass extension for custom themes * Shift `dependencies` to `devDependencies`. The `reveal.js` npm module is typically installed to access the CSS/JS assets for Reveal.js. For those that want to run the plugins, they would clone the whole repository. Hence, it doesn’t make sense to include those as `dependencies`, which will increase download times. Closes hakimel#1734. * Allow background images without protocols to have query hashes/parameters. Closes hakimel#1603. * hakimel#1770, resume playback for paused media. * hakimel#1770, fixed function name. * hakimel#1770, added second argument for setAttribute * hakimel#1770, set 'data-paused-by-reveal' attribute only for the media elements, which are not already in paused state. * support complex markdown support <textarea data-template> `some code` </textarea> * Remove 'history: true' from index.html Readme says the default is 'history: false', so it may be surprising for users - it's easy to miss the contradicting setting in index.html * wait for in-slide video/audio to load before playing, reuse same autoplay logic for slide backgrounds * prevent autoplaying backgrounds from playing if slide is hidden * fix race condition by updating backgrounds after slide visibility * works with video, defaults to (same as for images) * remove unused argument * make backgrounds (iframes) interactive when the foreground is empty * 2017 * merge info links in index * fix empty pages in pdf exports (closes hakimel#1804) * additional fix for empty pages in pdf exports hakimel#1804 * adjust pdf margins instead of heights to fix hakimel#1804 * typo * note about showNotes in PDF exports hakimel#1800 * fix markdown typo, update comment hakimel#1780 * recommend using textarea wrapper for inline markdown hakimel#1780 * make background videos work on ipad hakimel#1778 * improved fragment notes support hakimel#1636 * warning about cube/page being deprecated hakimel#1774 * adjust comment for display config value hakimel#1325 * added toggleHelp function this way a key can be given ability to toggleHelp on and off. Previously Reveal.showHelp could open the help screen, but no way to close it. * added override parameter to toggleHelp function * update deserialize regex to work with decimals * docs for hakimel#1816 and remove showHelp * phantom export throws error if reveal.js isn't present * fix overview rendering issues in latest chrome hakimel#1649 * allow configuration of which view slide number appears on - fixes hakimel#1791 * correct config name in README * note about deploying multiplex server with now hakimel#1830 * tweaks to showSlideNumber config hakimel#1833 * avoid npe on iframe postMessage * print-pdf using callback iso timer Use window.callPhantom icw page. onCallback to wait for pdf-ready event. From: http://stackoverflow.com/a/28925479/129269 * print-pdf layout slide contents to fit stretch elements layout() and thereby layoutSlideContents() is never invoked when isPrintingPdf() is true so stretch elements are not sized correctly. This is ensured now by invoking the layoutSlideContents from SetupPDF(). There seems no need to scale down like in other PR's is suggested. * Fix video background autoplay on iOS * add autoPlayMedia config option, overrides individual autoplay settings * allow fullscreen for background iframes * new cli and in-browser pdf printing compatible fix for hakimel#1804 * make pdf page height offset configurable * replace cdn.mathjax.org with cdnjs cdn.mathjax.org is shutting down: https://www.mathjax.org/cdn-shutting-down/ This changes the default MathJax URL in the math plugin, as well as references in README.md and test/examples/math.html * formatting * formatting * clarify meaning of getProgress() value * expose getPastSlideCount() via public API This can be used by plugins which are concerned with helping the presenter with their pace. * clarify getTotalSlides() returns the number of slides * add new getSlides() API This can be used by plugins which are concerned with helping the presenter with their pace. * Allow popup window access to Reveal API * extract time display code into new _displayTime() function This will allow us to reuse the display code for displaying an additional pacing timer. * use opacity for muted clock elements This allows us to use different colours for different timers, which will be useful when we add a pacing timer. * don't mute minutes when hours is unmuted * avoid deleting existing classes when muting time elements and make muting work for negative values * don't show negative signs inside minutes/seconds elements * round decreasing timers to mirror increasing timers * document clock and timer in speaker notes window * optionally display pacing advice based on slide timings Add an option to display advice on whether the current pace of the presentation is on track for the right timing (shown as green), and if not, whether the presenter should speed up (shown as red) or has the luxury of slowing down (blue). The pacing timer can be enabled by configuring by the `defaultTiming` parameter in the `Reveal` configuration block, which specifies the number of seconds per slide. 120 can be a reasonable rule of thumb. Timings can also be given per slide `<section>` by setting the `data-timing` attribute. Both values are in numbers of seconds. When the option is enabled, clicking on the timers will reset the timer to the beginning of the current slide, i.e. as if pacing was perfectly on track, not to zero as if the presentation had just begun. * minor tweak for hakimel#1564 * 3.5.0 * * updated big picture slide * added schema image * added deployment texts * * update big picture svg * adjusted big picture styles * moved authors and company slides to the end * * * added styles for codelistings * tweaked dockerfile * added kubernetes cheatsheet * moved tldr slide to end of presentation * * added version number to html source * added config file for CircleCI integration * added deploy.sh * * updated status badge * troubleshooting deploy script * * bugfix as suggested in https://goo.gl/W1XnSk * * changed circle badges in readme * finetuning deploy.sh * moved sha label to span inside presentation * * optimized Dockerfile to leverage docker caching * * added key ‘v’ to toggle display of GIT sha in upper right corner * * added PS_VERSION_NUMBER to big picture slide * shortened stateflow texts * * removed unneeded files * * added link to github project on tldr * * changed port for k8-service * wip * * finetuning deployment * debug * * editorial finetuning * * extended deploy script to update dyndns with public cluster ip address * * added shell scripts for starting and stopping the kubernetes (k8) cluster on GCP * moved it all to subfolder k8 * * huge commit. The best. No seriously, it’s way to big and it can easily serve as a bad example. * * added seniority * finetuning claims on last slide * controls are now hidden * * extended collaboration slide * * added samples for collaboration * * replaced slide bgnd by collage * * modified collage * swapped slide order * added notes * reduced size of leafs bgnd * * tweakd tldr slide * * replaced the bgnd image on the collaboration slide - new one has some effects applied * * ignoring .vscode folder * * added data-timing values for every slide * extended readme section of the repo * * changed the way how the schema image is built into the slide * * windows millenium. and why you may never have heard of it
derheap
pushed a commit
to derheap/reveal.js
that referenced
this pull request
Feb 22, 2018
This was referenced Apr 6, 2020
|
Hi. I was trying out this feature but it doesnt work in the sense when I added the tags in different fragments, only the first fragment notes were displayed and others were ignored. Is it something wrong in Im doing or is the framework still looking for first occurance of inside section tag and ignoring all subsequent ones.Also, It will be nice if possibility for timed highlighting of speaker notes (change in color or bold text etc.) can be implemented. As it helps in adhering to a timed script for a presentation. More specifically, if there is a video in the presentation,. voice over becomes much more easier with this. Im sorry if this is something which already exists but I couldnt locate it. Best regards |
|
This doesn't work optimally if there are multiple current fragments - it only pulls the notes for the first one. That's particularly troublesome if the notes are only in the second fragment. using |
R0bes
pushed a commit
to R0bes/Terraform-Presentation
that referenced
this pull request
Jun 7, 2021
For a fragment: allow to show a separate note defined in it
R0bes
pushed a commit
to R0bes/Terraform-Presentation
that referenced
this pull request
Jun 7, 2021
|
Hello, can someone please give an example how achieve this effect, i couldn't(t find any in documentation |
sachac
added a commit
to sachac/reveal.js
that referenced
this pull request
Oct 9, 2023
* js/controllers/notes.js: Use only the current fragment's notes if showing a fragment. * js/reveal.js: Update notes whenever fragments are shown or hidden. Related to: - hakimel@f496613 - hakimel#1636
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When a slide has several fragments it could be convenient to define a note for each of them. In this case we need to show only this specific note defined in a fragment and not others. General note of a slide shouldn't be also shown, as a more specific one should have greater relevance in this case.