Skip to content

Dn d manager#46

Merged
jburel merged 112 commits intoome:developfrom
jburel:DnD-manager
Jan 6, 2012
Merged

Dn d manager#46
jburel merged 112 commits intoome:developfrom
jburel:DnD-manager

Conversation

@jburel
Copy link
Copy Markdown
Member

@jburel jburel commented Dec 21, 2011

I have now added DnD to the data manager, ready to be tested.
I will add a movie to http://trac.openmicroscopy.org.uk/ome/ticket/1620

chris-allan and others added 30 commits October 27, 2011 16:53
Custom template tag support for static prefix in webadmin, see #7220
Configuring statics in settings.py, see #7219
Custom template tag support for static prefix in feedback, see #7233
Configuring statics in settings.py, see #7219
Custom template tag support for static prefix in webadmin, see #7221
Configuring statics in settings.py, see #7219
That will become configurable in the standard manner.
Accidentally committed while running web unittests
This was the thinner logo without the text at the bottom, as was already being
used in 'feature/django_templates' branch.
Footer separated out: links and copywrite. Webclient links in webclient.
Webclient templates base_container.html and base.html now extend common container3.html
and container2.html respectively. nav.js moved to common static.
TODO: split images.css and layout.css into styles needed by common vv webclient
@jburel
Copy link
Copy Markdown
Member Author

jburel commented Jan 4, 2012

Problem has been fixed see #52 and merged.

@qidane
Copy link
Copy Markdown
Contributor

qidane commented Jan 4, 2012

Tested and generally works.

Some comments:
• If you drop a dataset onto itself (eg by clicking and accidentally dragging a tiny bit) you get a modal dialog saying you cannot move it there - would be better to just not do anything.
• Sometimes when dragging an image you get a green (+) symbol, other times you do not, even though you are dragging and releasing to/from the same folders.
• The modal dialogs generally are very annoying, would be better to use the red on-entry sign on the drag to say you cannot drop there, or make the dropped item just spring back and show a tooltip or something else that vanished without being model
• If you drag a dataset to its parent project, the parent project is closed so you can on longer see the dataset.

@jburel
Copy link
Copy Markdown
Member Author

jburel commented Jan 4, 2012

@qidane: tickets created.

jburel added 4 commits January 4, 2012 14:11
Do not display any message when the user drags an
object and drop it on itself. (close #7720)
Do not display any message and no action
done when the src is dropped on its parent
i.e. a dataset dropped on the project that
already contains it. (close #7723)
Show a "no-entry" cursor when the target
is not valid. (close #7722).
@jburel
Copy link
Copy Markdown
Member Author

jburel commented Jan 4, 2012

@qidane: I have just trigged a new build with your comments implemented. You should be notified by e-mail when the build is completed. If you could give it a shot that will be great.

@jburel
Copy link
Copy Markdown
Member Author

jburel commented Jan 4, 2012

@qidane
Copy link
Copy Markdown
Contributor

qidane commented Jan 5, 2012

Crash on right click, select import on dataset.

@jburel
Copy link
Copy Markdown
Member Author

jburel commented Jan 5, 2012

@qidane: this not related to DnD, the problem has been fixed in PR 51.(build issue) due to missing jar. Any other comments on DnD

@qidane
Copy link
Copy Markdown
Contributor

qidane commented Jan 5, 2012

Problems:
• Inconsistent icon, sometime green plus sometime red no entry when dragging same image over same project. (images sent by email)

@qidane
Copy link
Copy Markdown
Contributor

qidane commented Jan 5, 2012

• If I select an image and drop it over the word "Empty"in a dataset nothing happens. I would expect it to move.
• I sometime get the green plus when dragging a file over another users dataset.
• I sometime get on red or green when dragging a file over another users images, but do get the red when over the dataset the images are in.

@jburel
Copy link
Copy Markdown
Member Author

jburel commented Jan 5, 2012

@qidane:

  • The red is displayed when you mouse over a container you cannot drop the element into.
  • for the green +, I do not have much control over that It does not appear at all in my case, so that might require more investigation.
  • for the empty I will have a look.

@jburel
Copy link
Copy Markdown
Member Author

jburel commented Jan 5, 2012

@qidane: tickets created or re-opened.

jburel added 7 commits January 5, 2012 18:20
Display the no-entry cursor when mousing over
images or empty node (close #7726)
Sprint5 insight backlog. Tested by @qidane, merging.
…nD-manager

Conflicts:
	components/insight/SRC/org/openmicroscopy/shoola/agents/treeviewer/view/TreeViewer.java
	components/insight/SRC/org/openmicroscopy/shoola/agents/treeviewer/view/TreeViewerComponent.java
	components/insight/SRC/org/openmicroscopy/shoola/agents/treeviewer/view/TreeViewerModel.java
	components/insight/SRC/org/openmicroscopy/shoola/util/ui/IconManager.java
@qidane
Copy link
Copy Markdown
Contributor

qidane commented Jan 6, 2012

Tested, all working with exception of appearance of the green plus issue that @jburel marked as known issue.

jburel added a commit that referenced this pull request Jan 6, 2012
Dn d manager. Tested by @qidane, one minor point that we will investigate later. merging
@jburel jburel merged commit ec3edae into ome:develop Jan 6, 2012
chris-allan added a commit to chris-allan/openmicroscopy that referenced this pull request Sep 9, 2015
Add bulk annotations to the main view in OMERO.web
mtbc pushed a commit to mtbc/openmicroscopy that referenced this pull request May 21, 2020
Test 5.3: Port test fixes to 5.3 line  (not secvul specific)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants