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

Paste rendering settings on selected wells #3311

Merged
merged 3 commits into from
Jan 7, 2015

Conversation

dominikl
Copy link
Member

@dominikl dominikl commented Jan 5, 2015

See Ticket: 12441

To test, check the following rendering settings workflow with Plate/Wells (expected to work like Dataset/Images)

  • Modify the rendering settings of a particular well (of a plate you are not the owner of, important for the last check)
  • Copy the rendering settings
  • Select a subset of other wells and paste the rendering settings
  • Reset some of the wells by "Set Original and Save"
  • Reset some of the wells by "Set Owner's and Save"

--no-rebase

@sbesson sbesson added the develop label Jan 6, 2015
@dominikl
Copy link
Member Author

dominikl commented Jan 7, 2015

As @will-moore noticed, if multiple images/wells are selected, the acquisition and preview tab should be disabled. I'll fix that in another PR, already created a ticket for it 12694

@will-moore
Copy link
Member

All working as described.
A couple of minor points: Probably should disable Acquisition and Preview tabs when multiple wells selected (as discussed). Also, selecting a range of wells would be much easier if Shift-click was supported.
But functionality above works fine. Good to merge.

@jburel
Copy link
Member

jburel commented Jan 7, 2015

Merging since RFE will be done in another PR.

jburel added a commit that referenced this pull request Jan 7, 2015
Paste rendering settings on selected wells
@jburel jburel merged commit 27e0564 into ome:develop Jan 7, 2015
@sbesson sbesson added this to the 5.1.0-m4 milestone Feb 17, 2015
@dominikl dominikl deleted the paste_well_rndsettings branch March 19, 2015 15:09
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.

4 participants