Use BusyMessage for price lookup #151

Closed
6ea86b96 wants to merge 1 commit from price-loading-state into master
6ea86b96 commented 2017-05-28 15:57:50 +02:00 (Migrated from github.com)

Maybe better than the current "???"

Before
2017-05-28 at 17 56

After
2017-05-28 at 17 53

Maybe better than the current "???" Before ![2017-05-28 at 17 56](https://cloud.githubusercontent.com/assets/20863631/26529323/f5d6dc60-43ce-11e7-8487-86926e2a755a.jpg) After ![2017-05-28 at 17 53](https://cloud.githubusercontent.com/assets/20863631/26529316/d8873132-43ce-11e7-9408-a4c8dbb6349a.jpg)
kauffj commented 2017-05-28 17:27:10 +02:00 (Migrated from github.com)

This was a conscious choice. I didn't like 40 loading indicators popping up all at once on the landing page.

I do think we can something better than ???.

This was a conscious choice. I didn't like 40 loading indicators popping up all at once on the landing page. I do think we can something better than ???.
kauffj commented 2017-05-28 17:43:23 +02:00 (Migrated from github.com)

I think in general it would be useful to develop a UI indicator for "this information is being refreshed/queried in the background" that is NOT an active indicator. For example, as we persist claim and other information more, this icon could be used when rendering tiles off of "stale" claim info, and likely in other places.

I think in general it would be useful to develop a UI indicator for "this information is being refreshed/queried in the background" that is NOT an active indicator. For example, as we persist claim and other information more, this icon could be used when rendering tiles off of "stale" claim info, and likely in other places.
kauffj commented 2017-05-28 17:43:54 +02:00 (Migrated from github.com)

Possibly icon-refresh?

Possibly `icon-refresh`?

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#151
No description provided.