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

[BUGFIX beta] Add preventDefault option to link-to and action. #3855

Merged
merged 1 commit into from Dec 9, 2013

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Dec 4, 2013

Resolves #3737

handled internally and we do not want to take the browser to a new URL (for
example).

If you need to override this behavior specifiy `preventDefault=false` in
Copy link
Contributor

Choose a reason for hiding this comment

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

s/specifiy/specify

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch, fixed in 35a85d4.

@zurt
Copy link
Contributor

zurt commented Dec 4, 2013

+1

@wagenet
Copy link
Member

wagenet commented Dec 8, 2013

@rjackson Can you rebase this so we can get the tests to pass? Thanks!

@rwjblue
Copy link
Member Author

rwjblue commented Dec 8, 2013

@wagenet - Rebased!

@rwjblue
Copy link
Member Author

rwjblue commented Dec 8, 2013

@wagenet - Failure on beta run is due to gemfile mismatch between beta and master. I can update ember-dev to ensure that bundle install is called after checkout of each branch. Does that make sense?

@wagenet
Copy link
Member

wagenet commented Dec 8, 2013

@rjackson I can merge this manually for now, but it wouldn't hurt to update ember-dev to prevent this in the future.

@rwjblue
Copy link
Member Author

rwjblue commented Dec 8, 2013

@wagenet - DONE: emberjs/ember-dev#111. Next time we update ember-dev in the Gemfile.lock we'll pull this in.

@rwjblue
Copy link
Member Author

rwjblue commented Dec 8, 2013

@wagenet - The changes from emberjs/ember-dev#111 have been pulled in now, and this has been rebased. Should be good now.

wagenet added a commit that referenced this pull request Dec 9, 2013
[BUGFIX beta] Add preventDefault option to link-to and action.
@wagenet wagenet merged commit fbd4f5a into emberjs:master Dec 9, 2013
@rwjblue rwjblue deleted the add_prevent_default_option branch December 11, 2013 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Input of type="file" breaks if ancestor element has {{action}}.
3 participants