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

Qt editor alpha #6383

Merged
merged 2 commits into from Jul 10, 2016
Merged

Qt editor alpha #6383

merged 2 commits into from Jul 10, 2016

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented May 8, 2016

Make alpha values accessible through the Qt figure options editor. Requires #6382 (alpha support in matplotlib.colors), closes #6196.

@QuLogic QuLogic added the GUI: Qt label May 8, 2016
@tacaswell tacaswell added this to the 2.1 (next point release) milestone May 8, 2016
@tacaswell
Copy link
Member

@anntzer this needs a rebase after your other PR got merged.

@anntzer
Copy link
Contributor Author

anntzer commented May 12, 2016

Nope, it's still fine.

@tacaswell
Copy link
Member

github/The web UI disagrees with you about this needing a rebase.

@anntzer
Copy link
Contributor Author

anntzer commented May 12, 2016

Sorry, I was confused by your message on the other PR regarding it being pulled in (which is not the case). Anyways, rebased both PRs.

@anntzer anntzer force-pushed the qt-editor-alpha branch 2 times, most recently from 9c01ad3 to abb3e5e Compare May 14, 2016 21:50
@anntzer
Copy link
Contributor Author

anntzer commented May 16, 2016

By the way, I would appreciate if someone did some manual testing of this PR too, because the figure options UI is not unit-tested (and that would not be so easy to do...).

@tacaswell
Copy link
Member

👍 does what it says on the tin.

Looks like this needs a rebase.

@anntzer
Copy link
Contributor Author

anntzer commented Jun 20, 2016

Apparently I also needed to call line.set_alpha explicitly for the alpha value to be set, not sure if something changed recently (I think this wasn't the case before).

@tacaswell
Copy link
Member

If I change those lines back to line.set_color(color) it works as expected for me.

@anntzer
Copy link
Contributor Author

anntzer commented Jun 21, 2016

What did you try? If I call plot([1, 2], alpha=.5) then try to edit the alpha of the line, the alpha is actually not modified (when it's just line.set_color(color)).

@tacaswell
Copy link
Member

Ah, I was starting with a full alpha line and then turning it down.

@QuLogic
Copy link
Member

QuLogic commented Jul 10, 2016

Ping? Milestone changed to match with #6382.

@QuLogic QuLogic modified the milestones: 2.0 (style change major release), 2.1 (next point release) Jul 10, 2016
@anntzer
Copy link
Contributor Author

anntzer commented Jul 10, 2016

Should be good to go? The issue with the need to explicitly call line.set_alpha is an independent one.

@QuLogic QuLogic merged commit f1bab50 into matplotlib:master Jul 10, 2016
@anntzer anntzer deleted the qt-editor-alpha branch July 10, 2016 01:08
@QuLogic
Copy link
Member

QuLogic commented Jul 10, 2016

Hmm, this doesn't backport cleanly. Do you mind opening another PR for it?

@anntzer
Copy link
Contributor Author

anntzer commented Jul 10, 2016

Should I cherry-pick this to upstream/v2.x?

@QuLogic
Copy link
Member

QuLogic commented Jul 10, 2016

Yes, it just doesn't do so cleanly. I'm sure you'll be able to figure it out though.

@anntzer
Copy link
Contributor Author

anntzer commented Jul 10, 2016

I gave it a try; I think that short of rewriting the PR from scratch it's going to be tough to de-interleave it from my other editor-related PRs that went into master but not v2.x (#5469, #6346, #6347, #6366).

@tacaswell
Copy link
Member

@anntzer Go ahead and backport all of the editor code if that is easier.

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.

Qt properties editor discards color alpha
4 participants