Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add filter button to controls #26

Open
3 tasks
vjeffrey opened this issue Apr 11, 2019 · 5 comments
Open
3 tasks

add filter button to controls #26

vjeffrey opened this issue Apr 11, 2019 · 5 comments
Labels

Comments

@vjeffrey
Copy link

vjeffrey commented Apr 11, 2019

User Story

In today's version of Automate, we have these handy-dandy filter buttons on the nodes list and profiles list.

Example:
Screen Shot 2019-04-10 at 19 24 49
the filter button is the one just to the left of the scan results button.

With the (soon-arriving) addition of deep filtering, we expect that more users will want to filter by control. Instead of making them type up the control every time, we could add the same filter button to the places in the ui where we list controls.
Those are:

  • scan results (accessible from reporting nodes list and reporting profiles list)
  • reporting profile details (accessible by clicking on a profile in reporting profiles list)
  • reporting node details (accessible by clicking on a node in reporting nodes list)

required: ux designs

Definition of Done

filter buttons on all control lists

Aha! Link: https://chef.aha.io/epics/SH-E-459

@vjeffrey vjeffrey added compliance reporting This issue or pull request applies to the compliance reporting part of Automate automate-ui labels Apr 11, 2019
@jonong1972
Copy link

jonong1972 commented Apr 11, 2019

What we can do is utilize the "3 Dot" icon (the more actions) and display a list of things you can do. in this case:

  • Use as filter...
  • Scan results or View error

@vjeffrey
Copy link
Author

the three views where we display control lists are:

scan results:
Screen Shot 2019-04-11 at 12 03 12

node details:
Screen Shot 2019-04-11 at 12 03 26

profile details:
Screen Shot 2019-04-11 at 12 03 41

@jonong1972 could you please expand on your ux idea for this given those three lists are formatted in slightly different ways?

@vjeffrey
Copy link
Author

(i'm assuming you mean the + button and scan results buttons on those views would get replaced with the control menu, but want to confirm)

@vjeffrey vjeffrey added the ui label Apr 19, 2019
@vjeffrey vjeffrey added this to the enhanced search and filtering for client runs and compliance reporting milestone May 16, 2019
@vjeffrey
Copy link
Author

this whole thing should be made easier by the controls list work

@vjeffrey vjeffrey removed this from the enhanced search and filtering for client runs and compliance reporting milestone Jun 20, 2019
@jonong1972 jonong1972 self-assigned this Jul 29, 2019
@tarablack01 tarablack01 self-assigned this Aug 5, 2019
@jonong1972 jonong1972 assigned jonong1972 and unassigned tarablack01 Aug 5, 2019
@jonong1972 jonong1972 added the ux label Aug 13, 2019
@jonong1972
Copy link

jonong1972 commented Aug 22, 2019

For the sake of getting this enhancement out, we can just add the functionality now, and then retro fit the buttons into the combined ellipses button. I opened another issue for that work.
#1331
https://app.zenhub.com/workspaces/ui-team-5cba23a3b14fdc05970c8f87/issues/chef/automate/1331

@AnkurMundhra AnkurMundhra added Epic and removed compliance reporting This issue or pull request applies to the compliance reporting part of Automate ui automate-ui ux labels Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants