Skip to content
This repository has been archived by the owner on Mar 25, 2020. It is now read-only.

Hide empty incident cards #73

Closed
jpike88 opened this issue Nov 25, 2017 · 4 comments
Closed

Hide empty incident cards #73

jpike88 opened this issue Nov 25, 2017 · 4 comments
Labels
good first issue
Milestone
v0.5.0

Comments

@jpike88
Copy link
Contributor

jpike88 commented Nov 25, 2017

This looks ugly:

screen shot 2017-11-25 at 3 28 25 pm
@ks888
Copy link
Owner

ks888 commented Nov 26, 2017

Hmm, it could be. How can we improve this?

@ks888
Copy link
Owner

ks888 commented Nov 26, 2017

Is it better to show some message? Here is the example of statuspage.io.
screen shot 2017-11-26 at 18 19 22

@jpike88
Copy link
Contributor Author

jpike88 commented Nov 26, 2017 •
edited
Loading

Perhaps consolidate them into a single card too, like this:

Nov 21 2017 - Nov 26 2017

No incidents reported.

@jpike88
Copy link
Contributor Author

jpike88 commented Nov 26, 2017

Or just add the message, but remove each of the card borders so it looks just like statuspage.io

@ks888 ks888 mentioned this issue Jan 5, 2018
Merged
@ks888 ks888 closed this as completed in #82 Jan 5, 2018
@ks888 ks888 added this to the v0.5.0 milestone Jan 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
good first issue
Projects
None yet
Milestone
v0.5.0
Development

No branches or pull requests
2 participants
@ks888 @jpike88