Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 386943 - Some suggestions for git-repository page and dynamic content loading
Summary: Some suggestions for git-repository page and dynamic content loading
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: 1.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 1.0 M2   Edit
Assignee: Maciej Bendkowski CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 384955
Blocks:
  Show dependency tree
 
Reported: 2012-08-09 11:47 EDT by Szymon Brandys CLA
Modified: 2012-08-28 09:35 EDT (History)
2 users (show)

See Also:
malgorzata.tomczyk: review+
maciej.bendkowski: review? (Szymon.Brandys)


Attachments
Loading char preview (1.42 MB, image/png)
2012-08-10 06:48 EDT, Maciej Bendkowski CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Szymon Brandys CLA 2012-08-09 11:47:40 EDT
- We were talking about different look of progress indicator, maybe the one we use in "select remote to push" dialog.
- There should be more space between the item label and the indicator
- We also wanted to use the same indicator for section progress and item progress
Comment 1 Maciej Bendkowski CLA 2012-08-10 06:48:21 EDT
Created attachment 219747 [details]
Loading char preview

The loadingAnimation.gif image from Dojo toolkit looks really awful as a progress indicator in this place. The attachment shows the experiment results.
Comment 2 Maciej Bendkowski CLA 2012-08-10 06:54:37 EDT
Proposal: https://github.com/maciej-bendkowski/orion.client/commit/1e83b272c12e96265aa427614f60c7d3af2452c5

With this commit I tried to adopt the "three dots" section look as a progress indicator. Now it should be more consistent with the other indicators. It would be nice to pull the fix into M1.
Comment 3 Malgorzata Janczarska CLA 2012-08-10 10:52:03 EDT
I also think the proposed animation doesn't look good and the pattern with 3 dots looks much better. If we prefer we can create a custom animation.
However I think we should use the same progress indicator in the section and in the other places, so please implement those changes also to the section.js. Please try to use unique css classes and try to avoid custom style in places that may be reused.
Comment 4 Szymon Brandys CLA 2012-08-12 08:38:38 EDT
(In reply to comment #3)
> I also think the proposed animation doesn't look good and the pattern with 3
> dots looks much better. If we prefer we can create a custom animation.

I would just raise a separate bug for Anton to add the section-item progress indicator to the icon pass.
Comment 5 Malgorzata Janczarska CLA 2012-08-21 09:55:12 EDT
I like this approach a lot. It's simple and not intrusive. However I know Szymon is not a found of it, so I'll hold off with committing until he makes the review.
Comment 7 Szymon Brandys CLA 2012-08-28 09:35:23 EDT
Merged. Thanks Maciek.