Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Presentation mode #14

Merged
merged 4 commits into from
Aug 18, 2016
Merged

Presentation mode #14

merged 4 commits into from
Aug 18, 2016

Conversation

vidbina
Copy link
Contributor

@vidbina vidbina commented Jul 4, 2016

Better buttons since @andi-tmh is running into a lot of issues while demoing runway setups on a beamer. The buttons should make the experience less painful, furthermore @andi-tmh propsed a details button which hides the the tabs containing the state tables.

@andi-tmh complained about the buttons not being user-friendly which
explains why I introduced a regular "play" button which should make it
easier to click.
@ongardie-sfdc
Copy link
Collaborator

Thanks, @vidbina. I do like the new buttons.

Do you mind removing the old, now hidden, checkbox? I don't think it's needed in many places.

Before I can merge this and your other PR (#13), we need a CLA (Contributor's License Agreement) from you or The Mobility House, as appropriate. You can find the CLA here; please send the completed form to osscore@salesforce.com. You only need to do this once; it'll be valid for any Salesforce project. Feel free to email that list (which I'm on) with questions. Sorry for the hurdle, but I look forward to merging this and continuing to collaborate on Runway.

@ongardie-sfdc
Copy link
Collaborator

@vidbina got your CLA, thanks. Could you remove that hidden checkbox?

@vidbina
Copy link
Contributor Author

vidbina commented Aug 18, 2016

On it

On Aug 17, 2016 20:40, "Diego Ongaro" notifications@github.com wrote:

@vidbina https://github.com/vidbina got your CLA, thanks. Could you
remove that hidden checkbox?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#14 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUeLgGQfTKj_QCgecVbucqZHpFqpJl9ks5qg7gngaJpZM4JEc2N
.

I just needed some state to indicate if the simulation is running or
not. During the initialization phase I setup the play button to the
starting state (show the play symbol) since I assume runway-browser does
not init to a running sim, furthermore I call toggle(cb) on every click
and check the sim state within toggle to start or stop the sim. I call
the callback with the playing state (true is running, false is stopped) to
take care of updating the UI.
@ongardie-sfdc ongardie-sfdc merged commit 23b59df into salesforce:master Aug 18, 2016
@ongardie-sfdc
Copy link
Collaborator

Merged, thanks @vidbina!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants