Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336169 - [Tree] Add loading feedback for VIRTUAL Trees
Summary: [Tree] Add loading feedback for VIRTUAL Trees
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.4   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: 1.5 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 342978
Blocks: 338196
  Show dependency tree
 
Reported: 2011-02-02 17:13 EST by Ralf Sternberg CLA
Modified: 2012-05-23 07:45 EDT (History)
1 user (show)

See Also:


Attachments
Initial draft (7.59 KB, patch)
2011-02-15 08:46 EST, Ivan Furnadjiev CLA
no flags Details | Diff
Improved version (9.57 KB, patch)
2011-02-15 10:34 EST, Ivan Furnadjiev CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ralf Sternberg CLA 2011-02-02 17:13:07 EST
When VIRTUAL Trees load new items after scrolling, there is just a big empty space before the items appear. Depending on how long it takes to load the items, this can look a bit broken. We could improve the user experience by adding some loading feedback, e.g. a rotating loading image.
Comment 1 Ivan Furnadjiev CLA 2011-02-03 06:08:29 EST
In VIRTUAL Table the loading feedback is items with "..." as text. Why we can't use the same for Tree? The loading feedback should be common for Tree and Table. If we agreed on a "new" feedback, we should port it to Table too.
Comment 2 Ralf Sternberg CLA 2011-02-03 16:23:23 EST
I think a web-like loading image would look much better then these ellipses. It doesn't have to be one image per line, it could also be a spinning wheel image centered over the Tree. Maybe with a shadow. It's just an idea, let's see if it works out...

Yes, Tree and Table should not differ. If we wait until after the Tree-Table merger, we don't have to adapt the Table anymore.
Comment 3 Tim Buschtoens CLA 2011-02-10 16:58:27 EST
I think a static image might be distracting from the fact that the content of the tree is scrolling. Its already hard to tell that you are scrolling because the content doesnt really change (if its all on the same layer and all items have the same content: "..." or nothing). Not to mention if the scroll-events are merged and the tree is really less scrolling and more jumping.

I would rather display a text-placeholder like "(loading item 123)". Its clear what it means, and with the numbers changing you are also aware that you're scrolling. Not sure how to localize it though.
Comment 4 Ivan Furnadjiev CLA 2011-02-15 08:46:47 EST
Created attachment 188992 [details]
Initial draft

We agreed, that we will provide a new feedback with the Tree/Table merge. This will not happen before 1.4 release. Till than, a similar to the current Table loading feedback ( "..." ) should be available in Tree too. This is a initial draft of "..." feedback in Tree. With the current draft the "..." feedback disappears if you hover over the "loading" item. In TreeItemLCA writeClear method has been added in the same way like in TableItemLCA.
Comment 5 Ivan Furnadjiev CLA 2011-02-15 10:34:39 EST
Created attachment 189009 [details]
Improved version
Comment 6 Ivan Furnadjiev CLA 2012-03-29 05:29:30 EDT
Comment on attachment 189009 [details]
Improved version

In the current CVS HEAD, Tree has the same loading feedback ( ... ) like in table.
Comment 7 Tim Buschtoens CLA 2012-03-29 05:53:25 EDT
Perhaps we could simply add a themeable state for unresolved items, then do something with the background in the theme? It still would have to be something that is not identical in every item, or it wouldn't look like scrolling.
Comment 8 Ralf Sternberg CLA 2012-05-23 07:45:08 EDT
Since the Tree-Table merge, VIRTUAL Trees and Tables have a consistent loading feedback (the "..."). This may not be ideal, but since we couldn't agree on a better solution yet, I'll close this bug, as the original problem is fixed.

Once someone comes up with a draft for an improved loading feedback, we can open another enhancement request.