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

bpo-30723: Enhancement to IDLE parenmatch extension #2306

Merged
merged 8 commits into from
Jun 28, 2017
Merged

bpo-30723: Enhancement to IDLE parenmatch extension #2306

merged 8 commits into from
Jun 28, 2017

Conversation

wohlganger
Copy link
Contributor

added highlight both parens option.
added name to acks file.

added highlight both parens option.
@mention-bot
Copy link

@wohlganger, thanks for your PR! By analyzing the history of the files in this pull request, we identified @terryjreedy, @kbkaiser and @Yhg1s to be potential reviewers.

@wohlganger wohlganger changed the title trivial - made FLASH_DELAY configure for all styles. "trivial" - made FLASH_DELAY configure for all styles. Jun 20, 2017
@mlouielu
Copy link
Contributor

mlouielu commented Jun 21, 2017

I think this changed isn't trivial, you will need to first add the issue on bugs.python.org. Nosy @terryjreedy for this is a IDLE PR.

@wohlganger
Copy link
Contributor Author

Added to bugs.python.org . ID# 30723

@terryjreedy terryjreedy self-requested a review June 21, 2017 19:53
@terryjreedy terryjreedy self-assigned this Jun 21, 2017
@terryjreedy terryjreedy changed the title "trivial" - made FLASH_DELAY configure for all styles. bpo-30723: Enhancement to IDLE parenmatch extension Jun 21, 2017
@wohlganger
Copy link
Contributor Author

Minor change committed to fix conflict with bpo-29910.

Copy link
Member

@terryjreedy terryjreedy left a comment

Choose a reason for hiding this comment

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

I made comments about the initial PR on bpo-30723 and made desired changes myself. Still need news.

@terryjreedy terryjreedy merged commit fae2c35 into python:master Jun 28, 2017
terryjreedy pushed a commit to terryjreedy/cpython that referenced this pull request Jun 28, 2017
…lp (pythonGH-2306)

* Add 'parens' style to highlight both opener and closer.
* Make 'default' style, which is not default, a synonym for 'opener'.
* Make time-delay work the same with all styles.
* Add help for config dialog extensions tab, including parenmatch.
* Add new tests.
Original patch by Charles Wohlganger.
(cherry picked from commit fae2c35)
terryjreedy added a commit that referenced this pull request Jun 28, 2017
…lp (GH-2306) (#2460)

* Add 'parens' style to highlight both opener and closer.
* Make 'default' style, which is not default, a synonym for 'opener'.
* Make time-delay work the same with all styles.
* Add help for config dialog extensions tab, including parenmatch.
* Add new tests.
Original patch by Charles Wohlganger.
(cherry picked from commit fae2c35)
@wohlganger wohlganger deleted the parens branch June 28, 2017 12:58
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.

5 participants