Skip to content

Allow setting image clims in Qt options editor. #6347

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 2, 2016
Merged

Allow setting image clims in Qt options editor. #6347

merged 1 commit into from
May 2, 2016

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Apr 29, 2016

In this implementation, (cmin, cmax) is swapped if cmin > cmax because it's not so easy to impose the constraint using formlayout...

@anntzer
Allow setting image clims in Qt options editor.
2f28508 
Currently, `(cmin, cmax)` is swapped if `cmin > cmax` because it's not
easy to impose the constraint using formlayout.
@tacaswell tacaswell merged commit ff0012e into matplotlib:master May 2, 2016
@tacaswell
Copy link
Member

👍 thanks! At somepoint can you write up a unified whats_new entry for all of the improvements you have been making to the config window?

@QuLogic QuLogic added this to the 2.1 (next point release) milestone May 2, 2016
@QuLogic QuLogic added the GUI: Qt label May 2, 2016
@anntzer anntzer deleted the image-options-clims branch May 2, 2016 04:02
@anntzer
Copy link
Contributor Author

anntzer commented May 2, 2016

Something like the following?

Improvements for the Qt figure options editor
---------------------------------------------

Various usability improvements were implemented for the Qt figure options
editor, among which:
- Line style entries are now sorted without duplicates.
- The colormap and normalization limits can now be set for images.
- Line edits for floating values now display only as many digits as necessary
  to avoid precision loss.  An important bug was also fixed regarding input
  validation using Qt5 and a locale where the decimal separator is ",".
- Use a shorter, more user-friendly name in the axes selector, and do not crash
  if there are no axes.

If that looks good to you, can you PR it in?

@tacaswell tacaswell mentioned this pull request May 2, 2016
Closed
This was referenced Jul 10, 2016
Merged
Closed
Merged
@QuLogic QuLogic modified the milestones: 2.0 (style change major release), 2.1 (next point release) Jul 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
GUI: Qt
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@anntzer @tacaswell @mdboom @QuLogic