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
proposal: x/tools/cmd/present: Support presenter notes on the Go present tool #14654
Comments
Related issue: #12355 |
CC @adg |
Summary of changes (latest): audreylim/go-presenter@6d2d6fe...9b35c29 Technical implementation:
Notes/Concerns:
Please let me know what you think. |
I think the general approach looks fine. We'll need to resolve the issue with Hey @robpike (who has some ownership over the present file format) do you think that |
I think it doesn't matter much but I might choose > or even -> . |
I don't mean to bikeshed, but I would mention the following about
Which is also valid Markdown that gets rendered as:
some reply goes here
However, I looked at some existing talks, and did not find any usages of this markdown idiom there. The one quote I found was actually written as a quote rather than a blockquote, so maybe |
Another options is
I'm actually kinda partial to I too apologize for bike shedding. |
I liked The next best choice seems to be Eg.
My vote goes to |
Okay, let's go with Implementation-wise, I suggest going in this order:
I also suggest, since notes seem to be the focus here, that the flag be |
I would consider issue #12355 closed once this is resolved. The approach described there is overkill, I think. |
I like After seeing @audreylim's analogy to "Person A says: hello", that makes it easy to remember too. |
@adg Noted your suggestions. Minor: With the |
|
CL https://golang.org/cl/21485 mentions this issue. |
This is the first of a series of changes that add support for speaker notes to the Go present tool. This is done by displaying slides with speaker notes on a second window, and synchronizing both windows. Updates golang/go#14654 Change-Id: Ic7b158d1e40f9e7e58d01791c88909f5619ce87f Reviewed-on: https://go-review.googlesource.com/21485 Reviewed-by: Andrew Gerrand <adg@golang.org>
CL https://golang.org/cl/21489 mentions this issue. |
Fixes golang/go#14654 Fixes golang/go#12355 Change-Id: I0c05db624170f7bef5883192b47b618ca343a830 Reviewed-on: https://go-review.googlesource.com/21489 Reviewed-by: Andrew Gerrand <adg@golang.org>
I propose adding support for presenter notes in the Go present tool.
Motivation: Go present is great. I used Go present at the recent GopherCon India and wished there was support for presenter notes. Having a presenter tool, with Go present, can make presentations a lot smoother and calm the presenter's nerves. Presenters don't have to use alternatives with presenter options if they want to use Go present. Best of both worlds.
Demo: http://virtivia.com:27080/1vs4fiwjm924q.mov
An implementation: https://github.com/audreylim/go-presenter#readme
If this proposal is accepted, I can submit a CL with the changes I've done for further review.
EDIT: here are the changes
The text was updated successfully, but these errors were encountered: