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

Reverse intensity rename Figure #238

Merged
merged 1 commit into from Dec 20, 2017

Conversation

Projects
None yet
3 participants
@dominikl
Member

dominikl commented Aug 16, 2017

Simply changes the 'Reverse Intensity' label to 'Invert', to be consistent with the other clients.

@mtbc

This comment has been minimized.

Member

mtbc commented Oct 1, 2017

Ready for review?

@jburel

This comment has been minimized.

Member

jburel commented Oct 2, 2017

We did not review or merge in case we needed a release before 5.4.0
since this will create an indirect dependency on 5.4.0

@jburel jburel added this to the 3.2.0 milestone Nov 20, 2017

@jburel

This comment has been minimized.

Member

jburel commented Dec 20, 2017

the text is correctly updated. No functionality change
This does not really create a 5.4 dependency from an interaction point of view
it will create a small naming divergence if used in a 5.3 context.
No need to hold off
Merging

@jburel jburel merged commit cfe8bb5 into ome:master Dec 20, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@will-moore will-moore referenced this pull request Jan 24, 2018

Merged

Release blog post 3.2.0 #261

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