Allow setting image clims in Qt options editor. #6347

Merged
merged 1 commit into from May 2, 2016

Conversation

Projects
None yet
4 participants
Contributor

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 anntzer Allow setting image clims in Qt options editor.
Currently, `(cmin, cmax)` is swapped if `cmin > cmax` because it's not
easy to impose the constraint using formlayout.
2f28508

mdboom added the needs_review label Apr 29, 2016

@tacaswell tacaswell merged commit ff0012e into matplotlib:master May 2, 2016

1 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
coverage/coveralls Coverage decreased (-0.003%) to 69.656%
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

tacaswell removed the needs_review label May 2, 2016

Owner

tacaswell commented May 2, 2016

👍 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 added this to the 2.1 (next point release) milestone May 2, 2016

QuLogic added the GUI/Qt label May 2, 2016

anntzer deleted the anntzer:image-options-clims branch May 2, 2016

Contributor

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment