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

Revised draft of interactions section #281

Closed
wants to merge 4 commits into from

Conversation

AutomatedTester
Copy link
Contributor

There are issues in the prose for bits that are not complete but those can be completed as follow ups. The main feedback required is missing sections that have not been covered at all.

You can visit a rendered version of the section on my gh-pages

The Actions API gives you a mechanism to do atomic interactions by sending over
a JSON object that can then be used to <a>dispatch commands</a>.
The commands are then <a data-lt=command-processed>processed</a> by the user agent,
allowing complex interactions to be created.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feels a little unspecific. I think a nicer phrasing would be that it allows you to pipeline interaction actions to be performed atomically on the remote end, or something along those lines.

@andreastt
Copy link
Member

I should point out that I haven’t compared this to existing implementations like Marionette yet. I also haven’t finished reading it, but it’s overall a massive improvement to the status quo.

@AutomatedTester
Copy link
Contributor Author

Landed in cd0fb1b

I have addressed all of Andreas' comments and since there are no other comments from other people I have felt it was better to land and then get people implementing it to report errors.

It also creates the ability for others to submit PRs based on issues they are seeing when implementing.

@AutomatedTester AutomatedTester deleted the interactions branch January 8, 2016 22:08
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.

None yet

2 participants