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

PR: Pressing Enter in the "Replace with" entry of Find/Replace replaces the currently selected text #4027

Merged
merged 4 commits into from
Feb 25, 2017

Conversation

andfoy
Copy link
Member

@andfoy andfoy commented Jan 22, 2017

Fixes #4010

@andfoy andfoy added this to the v3.2 milestone Jan 22, 2017
@andfoy andfoy self-assigned this Jan 22, 2017
@andfoy andfoy changed the title Pressing Enter in the "Replace with" entry of Find/Replace replaces the currently selected text PR: Pressing Enter in the "Replace with" entry of Find/Replace replaces the currently selected text Jan 22, 2017
@ccordoba12
Copy link
Member

@andfoy, great work! But from now on I'm going to ask you all to add tests for all PRs you do.

So please add a test for this one :-)

@ccordoba12 ccordoba12 modified the milestones: v3.2, v3.3 Feb 14, 2017
@ccordoba12
Copy link
Member

ccordoba12 commented Feb 25, 2017

@andfoy, this needs a merge with 3.x to solve conflicts. Please do it so I can merge it :-)

@andfoy
Copy link
Member Author

andfoy commented Feb 25, 2017

@ccordoba12 This PR is ready to go!

@ccordoba12
Copy link
Member

The failure in Travis is unrelated to this work. I fixed it already in PR #4201.

@ccordoba12 ccordoba12 merged commit fcc2deb into spyder-ide:3.x Feb 25, 2017
ccordoba12 added a commit that referenced this pull request Feb 25, 2017
@andfoy andfoy deleted the enter_replace_text branch June 14, 2019 19:10
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.

2 participants