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

x/tools/cmd/present: support multiple displays #12355

Closed
adg opened this issue Aug 27, 2015 · 4 comments
Closed

x/tools/cmd/present: support multiple displays #12355

adg opened this issue Aug 27, 2015 · 4 comments
Milestone

Comments

@adg
Copy link
Contributor

@adg adg commented Aug 27, 2015

It's been expressed by @matryer and by others that it would be nice to have a Keynote-style multi-display support. The second display would show speaker notes and the upcoming slide.

My thoughts on how this might be implemented:

  • Introduce the notion of a "session" on the present server side
  • When a client loads a slide presentation, they create a websocket connection to the present server (as they do now), and that connection is tied to a specific session (this is new).
  • The websocket, in addition to carrying snippets to be compiled and their results, will also carry slide navigation instructions. When the client navigates forward or back, this is communicated to the server, and the session object on the server is updated to reflect that state.
  • Additional clients can connect to that session using the unique session id. Any slide navigation instructions are relayed to all connected clients, and those clients reflect that navigation in the user interface.
  • Additional kinds of clients can then be implemented. One of these might be the "presenter view" which includes speaker notes (comments extracted from the slide source) and a preview of the upcoming slides.
  • The default slide view would then include a shortcut key to pop open a presenter view, used in a multi-display context.

cc @robpike

@robpike
Copy link
Contributor

@robpike robpike commented Aug 27, 2015

@rsc

I never wanted this, but I understand the desire and I believe @rsc has done something like this already.

@adg
Copy link
Contributor Author

@adg adg commented Aug 27, 2015

I don't think @rsc's is based on present at all, but I could be wrong.

@matryer
Copy link

@matryer matryer commented Aug 27, 2015

I know a couple of people who didn't use present because they needed the notes. Shame, because there's something neat about gofmt for presentations. Let me know if you want/need some help - I'd be more than willing.

@gopherbot
Copy link

@gopherbot gopherbot commented May 27, 2016

CL https://golang.org/cl/21489 mentions this issue.

jjneely pushed a commit to jjneely/present that referenced this issue Nov 4, 2016
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>
@golang golang locked and limited conversation to collaborators Jun 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.