Skip to content

[2.7] bpo-25684: ttk.OptionMenu radiobuttons weren't unique (GH-2276) #2960

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

Merged
merged 2 commits into from
Sep 10, 2017

Conversation

csabella
Copy link
Contributor

@csabella csabella commented Jul 31, 2017

between instances of OptionMenu.
(cherry picked from commit a568e52)

https://bugs.python.org/issue25684

between instances of OptionMenu.
(cherry picked from commit a568e52)
@csabella
Copy link
Contributor Author

@serhiy-storchaka This was a challenge because of the conflicts, but I think it's right. I'm not really comfortable adding my name to the Misc/ACKS for 2.7 since it's just one change, and so many others have made a lot more contributions. I hope that's OK. If not, I'll add it.

@csabella csabella changed the title [2.7] bpo-25684: ttk.OptionMenu radiobuttons weren't unique (#2276) [2.7] bpo-25684: ttk.OptionMenu radiobuttons weren't unique (GH-2276) Jul 31, 2017
@serhiy-storchaka serhiy-storchaka added the type-bug An unexpected behavior, bug, or error label Jul 31, 2017
@terryjreedy terryjreedy changed the title [2.7] bpo-25684: ttk.OptionMenu radiobuttons weren't unique (GH-2276) [2.7] bpo-25684: ttk.OptionMenu radiobuttons weren't uniqu (GH-2276) Jul 31, 2017
@terryjreedy terryjreedy changed the title [2.7] bpo-25684: ttk.OptionMenu radiobuttons weren't uniqu (GH-2276) [2.7] bpo-25684: ttk.OptionMenu radiobuttons weren't unique (GH-2276) Jul 31, 2017
@terryjreedy
Copy link
Member

I edited title to poke Bedevere into rechecking. I had to actually make a change to make that work ;-).

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 think this qualifies you for acks. Up to you. Otherwise, looks good.

@csabella
Copy link
Contributor Author

I've changed Misc/ACKS. My rationalization is that I'll just have to make another change that gets into 2.7. :-)

@csabella
Copy link
Contributor Author

@serhiy-storchaka Did you want this included in the new 2.7 release that's coming up? Thanks.

@Mariatta Mariatta merged commit e1847ea into python:2.7 Sep 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants