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

Share editing 8790 #396

Merged
merged 5 commits into from
Oct 22, 2012
Merged

Share editing 8790 #396

merged 5 commits into from
Oct 22, 2012

Conversation

will-moore
Copy link
Member

Rebased old branch #386 onto dev_4_4 by cherry-picking all commits.

Will Moore added 5 commits October 10, 2012 12:37
This statement seems responsilble to adding omero.model.FormatI objects to the share
as retrieved by shareService.getContents(sid). These add to the content count but are
not displayed in web because we filter for images only.
According to Josh, the addObjects() method is old and should not be needed anymore but
'no promises - needs testing'.
No ticket for this, but I just found it really annoying not being able to see
which image is which in the basket (when the basket content drop-down has already loaded them)
@gusferguson
Copy link

Tested. Works as expected, but cannot test "choose new exp-date" in expired share aspect yet as I have to wait until the share I set up today expires. Will check that on Monday.

@jburel
Copy link
Member

jburel commented Oct 15, 2012

@gusferguson: did you have a chance to test it today? If so what is the outcome?

@gusferguson
Copy link

Tested expired share this morning - behaved as expected.

@jburel
Copy link
Member

jburel commented Oct 18, 2012

@gusferguson: Ready to merge?

@gusferguson
Copy link

@jburel - I can test today

@gusferguson
Copy link

Tested following build - expiry date and reset still working as expected - no problems from owner or other share users perspectives.

Did notice problem with thumbnails in centre pane using IE in Win 7 - there is nothing separating thumbnails - in Safari on Mac there is a this white line between thumbnails. There is also no other information about the image shown in the list view in contrast to Data list view where you see image name etc next to thumbnails.

Screenshots: squig>team>screenshots>4.5_Sprint1>Gus>Webclient-share-mac.jpg and Webclient-share-win7.jpg

@will-moore will-moore mentioned this pull request Oct 19, 2012
@bpindelski
Copy link

Tested on howe. The share logic from the commits is in place. As Gus reported - the Public data view (middle panel in the web client) is currently unusable. The images are displayed in a single column. There is no indication which image is selected in the tree. There is no border around images.

This PR is ready to merge.
The fix for the Public view image display will be included in #405

jburel added a commit that referenced this pull request Oct 22, 2012
@jburel jburel merged commit a0b657e into ome:dev_4_4 Oct 22, 2012
@will-moore will-moore mentioned this pull request Oct 23, 2012
@pwalczysko
Copy link
Member

The share-editing-public-data-view issues as reported by Blazej and Gus are resolved
on Safari/Mac 10.8, IE8, Firefox and Google Chrome on WindowsXP and IE8 and Firefox on Windows2003. The thumbnails are ordered in row-first column-second manner, are separated nicely and the click on an image in the tree brings you to the proper thumbnail which is marked
Possible further improvements: I would go for more striking color than blue-gray for the marking though. Possibly it would be good if a double-click onto the image in the tree in Public pane brings up a Full Viewer of the image, just like it does in the "Explore" pane.

@jburel
Copy link
Member

jburel commented Mar 22, 2013

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.

None yet

5 participants