Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #6743 (closed)

Opened 8 years ago

Closed 8 years ago

Bug: Insight Importer: tool-bar launch

Reported by: wmoore Owned by: jburel
Priority: critical Milestone: OMERO-Beta4.3.2
Component: Insight Version: n.a.
Keywords: n.a. Cc: cxallan, saloynton
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-09-15 (5)

Description

Launching the Insight Importer from the toolbar generates the UI with Project and Dataset choosers in the top right, instead of using the pop-up chooser.

The pop-up chooser is not displayed when images are added to the queue.

Change History (7)

comment:1 Changed 8 years ago by jburel

We had a discussion about that when you were away and the conclusion was:

  • Having 2 UI might be confusing for users.

So the idea was to always display containers in the top right.

comment:2 Changed 8 years ago by wmoore

  • Cc cxallan saloynton added

Couple of points:

  • Seems that the pop-up to choose Project/Dataset? is never displayed now. This seems like we're back to the main problem that guys in our lab were complaining about before: You choose 'Import', you choose an image and hit '>' and then you realise that it's been added to the wrong Project / Dataset in the queue.
  • If I select a Dataset in the tree before hitting the 'Import' button in the toolbar, the selection is ignored. I get Project: 'No Project' and Datast 'New from folder'.

comment:3 Changed 8 years ago by jburel

For second point, are you sure you are running the latest code? I did not notice the problem
I have no strong opinion, just the fact that the 2 UI can potentially confuse users.

comment:4 Changed 8 years ago by wmoore

My preference is we leave it exactly as it was for the last release. People seem to like it, we have had positive feedback from most users. Has anyone complained about the 2 UIs?
Changing something this critical should probably be avoided unless we have a compelling reason.

comment:5 Changed 8 years ago by jburel

I will review that code, later on.
By most users, you mean "users from the lab" otherwise we received 2 comments: martin positive and another user complained.

comment:6 Changed 8 years ago by jburel

  • Remaining Time set to 0.15
  • Status changed from new to accepted

comment:7 Changed 8 years ago by jburel <j.burel@…>

  • Remaining Time changed from 0.15 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed

(In [21fa132dd5c754c938111222912f286ba30282be/ome.git] on branch develop) Review setting of import location (close #6743)

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.75144 sec.)

We're Hiring!