Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

DEV: Remove all empty state icons #849

Closed
4 of 6 tasks
catrobson opened this issue Feb 1, 2018 · 3 comments
Closed
4 of 6 tasks

DEV: Remove all empty state icons #849

catrobson opened this issue Feb 1, 2018 · 3 comments
Assignees
@AdamJ
Labels
code/HTML&CSS implementation of HTML/CSS
Milestone
Sprint 144-3

Comments

@catrobson
Copy link
Collaborator

catrobson commented Feb 1, 2018 •
edited by AdamJ
Loading

For the empty states across the product, we currently use an icon. These have caused confusion for users who think these are actionable. Remove these icons.

Let's see if we can capture a list of all locations below as well so we're tracking this & it can be tested on PR.

  • User Homepage/Dashboard
  • Space Dashboard
  • Planner list
  • Codebases list
  • Pipelines list (currently looks like it needs a empty state?)
  • Deployments? (Need to talk with Michael Celedonia about what is desirable)
@catrobson catrobson added this to the Backlog milestone Feb 1, 2018
@catrobson catrobson changed the title Remove all empty state icons DEV: Remove all empty state icons Feb 1, 2018
@catrobson catrobson mentioned this issue Feb 1, 2018
Open
12 tasks
AdamJ added a commit to AdamJ/fabric8-planner that referenced this issue Feb 13, 2018
@AdamJ
fix(emptystate): remove Empty State icon
25617a6 
Remove the Empty State icon from the Planner list, per fabric8-ui/fabric8-ux#849
@AdamJ AdamJ mentioned this issue Feb 13, 2018
Merged
@AdamJ
Copy link
Collaborator

AdamJ commented Feb 13, 2018

@catrobson Currently, there are no empty states for Pipelines, Applications, Environments and Deployments. Do you want to leave them as is, or spin up a separate story to have empty states created for those pages?

AdamJ added a commit to AdamJ/fabric8-ui that referenced this issue Feb 13, 2018
@AdamJ
fix(cards): Remove Empty State icons
c840e0c 
Remove all Empty State icons from the empty Card states

related to fabric8-ui/fabric8-ux#849
AdamJ added a commit to AdamJ/fabric8-ui that referenced this issue Feb 13, 2018
@AdamJ
fix(cards): Remove Empty State icons
78c6ff6 
Remove all Empty State icons from the empty Card states

related to fabric8-ui/fabric8-ux#849
@AdamJ AdamJ mentioned this issue Feb 13, 2018
Merged
AdamJ added a commit to AdamJ/fabric8-ui that referenced this issue Feb 13, 2018
@AdamJ
fix(cards): Remove Empty State icons
9439419 
Remove all Empty State icons from the empty Card states

related to fabric8-ui/fabric8-ux#849
@AdamJ AdamJ self-assigned this Feb 13, 2018
@AdamJ AdamJ modified the milestones: Backlog, Sprint 144-3 Feb 13, 2018
@AdamJ AdamJ added the code/HTML&CSS implementation of HTML/CSS label Feb 13, 2018
joshuawilson pushed a commit to fabric8-ui/fabric8-ui that referenced this issue Feb 13, 2018
@AdamJ @joshuawilson
fix(cards): Remove Empty State icons
c8ab9f4 
Remove all Empty State icons from the empty Card states

related to fabric8-ui/fabric8-ux#849
nimishamukherjee pushed a commit to fabric8-ui/fabric8-planner that referenced this issue Feb 13, 2018
@AdamJ @nimishamukherjee
fix(emptystate): remove Empty State icon (#2437)
007338a 
Remove the Empty State icon from the Planner list, per fabric8-ui/fabric8-ux#849
@catrobson
Copy link
Collaborator Author

@mindreeper2420 Applications, Environments are going away, so no need to change them. Pipelines should probably have an empty state, so I'll create a story to look at that.

@AdamJ
Copy link
Collaborator

AdamJ commented Feb 14, 2018

@catrobson ok, sounds good. I'll close this then, as the two PRs have been merged.

@AdamJ AdamJ closed this as completed Feb 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@AdamJ AdamJ

Labels
code/HTML&CSS implementation of HTML/CSS
Projects
None yet
Milestone
Sprint 144-3
Development

No branches or pull requests
2 participants
@catrobson @AdamJ