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

Add possibility to print speaker notes #1518

Closed

Conversation

wasabideveloper
Copy link

Print speaker notes on a separate page after each slide if print-pdf and print-notes are given in url.
reveal-print-notes

@hakimel
Copy link
Owner

hakimel commented May 23, 2016

Hey! Thanks for the PR but we already support printing notes to PDF. This is made possible via the shareNotes config option which was added in reveal.js 3.2.0. With showNotes set to true, the notes will be visible on top of the presentation and be included in exports. https://github.com/hakimel/reveal.js#share-and-print-speaker-notes

You can test this on the demo page, slide 10 has some notes: http://lab.hakim.se/reveal-js/demo.html?print-pdf&showNotes=true

@hakimel hakimel closed this May 23, 2016
@wasabideveloper
Copy link
Author

I know. But shareNotes is not suitable for much lines of notes. So it's a nice possibility to print the notes to a separate page. Often people like to print two versions of their presentation: One with notes and one without. So it's much more comfortable using a url parameter to print the notes instead of changing the config every time.

@hakimel
Copy link
Owner

hakimel commented May 23, 2016

You're right that the current notes design doesn't allow for very long notes. We should try to improve that in the existing solution rather than adding a secondary way to print. Perhaps an additional config value that pushes printed notes to a separate page? Alternatively we could move notes to the right of the slide as a vertical column that spans the full height of the page and about 20-30% of the width.

All config parameters can be set as url params, so you can just add ?showNotes=true to the URL. This works with other properties too, four example you can turn on auto-sliding every five seconds by adding ?autoSlide=5000.

@wasabideveloper
Copy link
Author

I didin't know about the possibility of setting config params by the url.
Under this circumstances I propose to generalize the printNotes feature and add a notesLayout option.

notesLayout: 'none' // no notes shown
notesLayout: 'default' // = 'none'
notesLayout: 'inline' // like the current 'showNotes' behaviour
notesLayout: 'side' // Slide size resized to 70% and a notes bar of 30% width on the right
notesLayout: 'separatePage' // On the following page

This layoutNotes option simply should include a css file from css/print/notes/so the user can create it's own. E.g:

notesLayout: 'notesWithCommentLines' // will include css/print/notes/notesWithCommentLines.css

@egrosclaude
Copy link

I'm a teacher and I'm VERY much interested in the alternative "notes" you
mention. In fact, I've Q&D hacked the 'notes' plugin to turn reveal into
some sort of a courseware app with handsome results. But alas, I'm no js
programmer, so my solution is just an ugly hack. If I had the said plugin,
well written by an expert, and then, the ?showLecture-or-something
parameter, this would be... just Heaven.

On Mon, May 23, 2016 at 7:15 AM, Hakim El Hattab notifications@github.com
wrote:

You're right that the current notes design doesn't allow for very long
notes. We should try to improve that in the existing solution rather than
adding a secondary way to print. Perhaps an additional config value that
pushes printed notes to a separate page? Alternatively we could move notes
to the right of the slide as a vertical column that spans the full height
of the page and about 20-30% of the width.

All config parameters can be set as url params, so you can just add
?showNotes=true to the URL. This works with other properties too, four
example you can turn on auto-sliding every five seconds by adding
?autoSlide=5000.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#1518 (comment)

Eduardo Grosclaude
Universidad Nacional del Comahue
Neuquen, Argentina

@hakimel
Copy link
Owner

hakimel commented May 26, 2016

I like the idea of a layout option for notes. I went ahead and implemented at least the start of it in 3111d3b.

Instead of a new config value, I'm re-using the existing showNotes option. Here's a list of what values you can set showNotes to:

Value Description
nil Notes aren't shown.
false Notes aren't shown.
true Notes are shown with default layout ("inline").
string Notes are shown with the given layout. The layout is added as data-layout="yourLayoutName" on the .speaker-notes element so that you can easily attach any custom styles.

For now, the only supported layout is separate-page which will make notes appear on separate pages in PDF exports.

@wasabideveloper
Copy link
Author

👍 That's great!

hbrysiewicz pushed a commit to hbrysiewicz/reveal.js that referenced this pull request Oct 7, 2016
benabraham pushed a commit to benabraham/reveal.js that referenced this pull request Nov 12, 2016
KroArtem pushed a commit to KroArtem/UltraSparcT2Plus that referenced this pull request Nov 21, 2016
SteveDesmond-ca pushed a commit to SteveDesmond-ca/presentations that referenced this pull request Apr 19, 2017
derheap pushed a commit to derheap/reveal.js that referenced this pull request Feb 22, 2018
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