From 20e655fd60efe918a58a8c5b870ae25eaedd85a5 Mon Sep 17 00:00:00 2001
From: Sacha Chua <sacha@sachachua.com>
Date: Mon, 9 Oct 2023 11:26:53 -0400
Subject: [PATCH 1/2] Use the current fragment's notes when showNotes: true

* 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:

- https://github.com/hakimel/reveal.js/commit/f496613dd3309a3066642adf1261e9da0578cdda
- https://github.com/hakimel/reveal.js/pull/1636
---
 js/controllers/notes.js | 19 ++++++++++++++++++-
 js/reveal.js            |  9 +++++++++
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/js/controllers/notes.js b/js/controllers/notes.js
index 7256425e46c..55d6943884c 100644
--- a/js/controllers/notes.js
+++ b/js/controllers/notes.js
@@ -101,10 +101,27 @@ export default class Notes {
 			return slide.getAttribute( 'data-notes' );
 		}
 
+		if ( Reveal.getConfig().fragments ) {
+			let fragmentElement = slide.querySelector( '.current-fragment' );
+			if( fragmentElement ) {
+				let fragmentNotes = fragmentElement.querySelector( 'aside.notes' );
+				if( fragmentNotes ) {
+					return fragmentNotes.innerHTML;
+				}
+				else if( fragmentElement.hasAttribute( 'data-notes' ) ) {
+					return fragmentElement.getAttribute( 'data-notes' );
+				}
+			}
+		}
 		// ... or using <aside class="notes"> elements
 		let notesElements = slide.querySelectorAll( 'aside.notes' );
 		if( notesElements ) {
-			return Array.from(notesElements).map( notesElement => notesElement.innerHTML ).join( '\n' );
+			// ignore notes inside of fragments since those are shown individually when stepping through fragments
+			let notes = Array.from(notesElements);
+			if ( Reveal.getConfig().fragments ) {
+				notes = notes.filter( notesElement => notesElement.closest( '.fragment' ) === null );
+			}
+			return notes.map( notesElement => notesElement.innerHTML ).join('\n');
 		}
 
 		return null;
diff --git a/js/reveal.js b/js/reveal.js
index 605d5c841c5..fb1c1a30984 100644
--- a/js/reveal.js
+++ b/js/reveal.js
@@ -148,6 +148,15 @@ export default function( revealElement, options ) {
 		// Register plugins and load dependencies, then move on to #start()
 		plugins.load( config.plugins, config.dependencies ).then( start );
 
+		if ( config.fragments ) {
+			Reveal.on( 'fragmentshown', () => { console.debug('shown'); notes.update(); } );
+			Reveal.on( 'fragmenthidden', () => { console.debug('hidden'); notes.update(); } );
+			Reveal.on( 'slidechanged', () => {
+				fragments.update();
+				notes.update();
+			} );
+		}
+
 		return new Promise( resolve => Reveal.on( 'ready', resolve ) );
 
 	}

From ff688810648ff1fc251e0a2c67a098e06ad82448 Mon Sep 17 00:00:00 2001
From: Sacha Chua <sacha@sachachua.com>
Date: Tue, 10 Oct 2023 15:18:24 -0400
Subject: [PATCH 2/2] Whoops, remove debugging output

---
 js/reveal.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/js/reveal.js b/js/reveal.js
index fb1c1a30984..73da6612182 100644
--- a/js/reveal.js
+++ b/js/reveal.js
@@ -149,8 +149,8 @@ export default function( revealElement, options ) {
 		plugins.load( config.plugins, config.dependencies ).then( start );
 
 		if ( config.fragments ) {
-			Reveal.on( 'fragmentshown', () => { console.debug('shown'); notes.update(); } );
-			Reveal.on( 'fragmenthidden', () => { console.debug('hidden'); notes.update(); } );
+			Reveal.on( 'fragmentshown', () => { notes.update(); } );
+			Reveal.on( 'fragmenthidden', () => { notes.update(); } );
 			Reveal.on( 'slidechanged', () => {
 				fragments.update();
 				notes.update();