Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 336169

Summary:

[Tree] Add loading feedback for VIRTUAL Trees

Product:

[RT] RAP

Reporter:

Ralf Sternberg <rsternberg>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

enhancement

Priority:

CC:

tbuschto

Version:

1.4

Target Milestone:

1.5 M1

Hardware:

OS:

Linux

Whiteboard:

Bug Depends on:

342978

Bug Blocks:

338196

Attachments:

Description	Flags
Initial draft	none
Improved version	none

Description Ralf Sternberg

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

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

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

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

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

2011-02-15 10:34:39 EST

Created attachment 189009 [details]
Improved version

Comment 6 Ivan Furnadjiev

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

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

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.