biosemantics / etc-site-archived-do-not-use

Source code for the ETC Toolkit web application
http://etc.cs.umb.edu/etcsite/
1 stars 0 forks source link

[UI] etc-dev select a file #638

Closed hongcui closed 7 years ago

hongcui commented 7 years ago

see https://www.screencast.com/t/va0Je6KFsEx

Basically when you have a long list of files in the File Manager, and you are trying to select one file for deletion, you click on the file and the interface refreshes and the file selected is no longer in your view.

etc and etc-dev share the same issue.

hongcui commented 7 years ago

Jin, you can work on the two matrix review issues first. I will see if I could look into this issue.

It doesn't break the system, but it is just very confusion.

danveno commented 7 years ago

This seems to be a bug in the Tree widget. Possibly related to https://github.com/gwtproject/gwt/issues/9475.

danveno commented 7 years ago

A browser event is added to the FileTree when initializing the tree: @Override public void onBrowserEvent(Event event) { if (DOM.eventGetType(event) == Event.ONMOUSEDOWN) { super.onClick(event); TreeNode node = findNode(event.getEventTarget(). cast()); if(!(node.getModel() instanceof FolderTreeItem)) { this.scrollIntoView(node.getModel()); } }

                super.onBrowserEvent(event);
         }