Skip to content
This repository has been archived by the owner on Jan 8, 2019. It is now read-only.

Stream list has no loading spinner #1309

Closed
lennartkoopmann opened this issue May 20, 2015 · 4 comments
Closed

Stream list has no loading spinner #1309

lennartkoopmann opened this issue May 20, 2015 · 4 comments
Labels
ux
Milestone
1.1.0

Comments

@lennartkoopmann
Copy link
Contributor

The stream list shows the "No streams configured" message while it is loading. This is confusing and should instead show a spinner.
@lennartkoopmann lennartkoopmann added this to the 1.1.0 milestone May 20, 2015
@florianpopp
Copy link

Works for me on graylog-web-interface-1.1.0-SNAPSHOT-20150520133756.tgz

@lennartkoopmann
Copy link
Contributor Author

Not working for me on latest master. Are we talking about the same thing? See recording:

streamsspinner

@florianpopp
Copy link

Ups, I am not getting to that page at all but directly to the streams page.

@lennartkoopmann
Copy link
Contributor Author

I delayed the loading by 1000ms in the code to show it. You are probably not seeing the loading at all because it is so fast on your local machine.

Production setups under load or accessed via a slower connection can easily take a short moment to load the streams. Long enough to see the "No streams configured" message and be confused. :)

@dennisoelkers dennisoelkers self-assigned this May 21, 2015
dennisoelkers added a commit that referenced this issue May 27, 2015
@dennisoelkers
Not initializing streams with an empty array upfront.
f863d54 
Fixes #1309.
dennisoelkers added a commit that referenced this issue May 28, 2015
@dennisoelkers
Not initializing streams with an empty array upfront.
fd8563d 
Fixes #1309.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
ux
Projects
None yet
Milestone
1.1.0
Development

No branches or pull requests
3 participants
@lennartkoopmann @dennisoelkers @florianpopp