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

Fix Qt4 figure editor color setting and getting #2275

Merged
merged 5 commits into from Aug 14, 2013

Conversation

mspacek
Copy link
Contributor

@mspacek mspacek commented Aug 3, 2013

This fixes #2274. It also permits only MPL colors to be entered in the color lineedit boxes. Previously, it seems Qt4 was dealing with any named colours entered into the lineedits, instead of MPL. This probably came out the same most of the time, but was less than ideal. Now, all colors are passed through MPL, which returns an unambiguous hex string for Qt to use. Also, there was some code duplication in terms of checking for valid colors. Also, when editing the color lineedits, the corresponding color buttons now only update when the lineedit loses focus, not on every keypress. Finally, the mass of Qt objects entered directly into the namespace have been cleaned up.

@mspacek
Copy link
Contributor Author

mspacek commented Aug 3, 2013

I just force pushed a shorter, cleaner set of commits.

@mspacek
Copy link
Contributor Author

mspacek commented Aug 7, 2013

Any particular reason why Travis failed? I'd be surprised if there was any test coverage at all for the qt4 editor.

@mdboom
Copy link
Member

mdboom commented Aug 7, 2013

That's a failure that crops up most of the time, but I haven't been able to reproduce. Benign as far as this PR goes.

@mdboom
Copy link
Member

mdboom commented Aug 7, 2013

This looks good on first blush. Since this is GUI-related changes, I think I'd like someone else who uses the Qt editor on a regular basis (I don't personally) to look this over and make sure all of their use cases are still met. Maybe we can recruit someone on the matplotlib-users mailing list for this?

@mspacek
Copy link
Contributor Author

mspacek commented Aug 7, 2013

Sure, I'll try sending a message to the user list.

@mspacek
Copy link
Contributor Author

mspacek commented Aug 7, 2013

BTW, ipython was having similar problems with Travis, minrk thought it was due to timeouts:

ipython/ipython#3641
ipython/ipython#3647 (comment)

@mspacek
Copy link
Contributor Author

mspacek commented Aug 14, 2013

@mdboom I posted a message to the user list a week ago, but no one's replied:

http://article.gmane.org/gmane.comp.python.matplotlib.general/33391

I've been using this branch for about 2 weeks now without any problems.

@mdboom
Copy link
Member

mdboom commented Aug 14, 2013

Ok -- no need to further hold this up.

mdboom added a commit that referenced this pull request Aug 14, 2013
Fix Qt4 figure editor color setting and getting
@mdboom mdboom merged commit daf7efb into matplotlib:master Aug 14, 2013
@mspacek
Copy link
Contributor Author

mspacek commented Aug 14, 2013

Thanks!

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.

Figure editor incorrectly captures color properties
3 participants