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

BUG#1167-Restores rbac permissiosn and camera eventicon #1168

Merged
merged 1 commit into from
Oct 31, 2017

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Oct 26, 2017

closes #1167

Look camera icon is back! also we have create in the main dropdown and delete in the row kebab

screen shot 2017-10-26 at 12 29 22 pm

screen shot 2017-10-26 at 12 29 13 pm

@Loicavenel @serenamarie125 did i miss anything on this page?

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1507714

@AllenBW AllenBW added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 26, 2017
@miq-bot
Copy link
Member

miq-bot commented Oct 26, 2017

Checked commit AllenBW@4ab516d with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@AllenBW AllenBW changed the title Restores rbac permissiosn and camera eventicon BUG#1167-Restores rbac permissiosn and camera eventicon Oct 26, 2017
@serenamarie125
Copy link

@Loicavenel shouldn't the button group be called "Snapshots"? I think that's what it is called when on the VM level.

screen shot 2017-10-26 at 8 14 39 pm

If the name of the command button group is Snapshots, you should be able to change the menu item labels to be:

  • Create
  • Delete All

regardless, we need to make sure these are consistent when in the Snapshot View versus the VM Details view. Can you include screenies of both of the button group menu items?

@Loicavenel
Copy link

@serenamarie125, Button has to be contextual.

  • VM details: Button Snapshots: View & Create
  • Service Level: on resource line: Button Snapshots: View & Create
  • Snapshot view: I don't think we should call the button SnapShot anymore.. we are in... BUT in place to call it Configuration: we should show the 2 Buttons: Create and Delete All, we have enough place for it.. we save one click :)

And don't forget in Snapshot view we should offer per Snapshot with Kebab menu: Delete (always) and Revert (where it does apply)

@AllenBW
Copy link
Member Author

AllenBW commented Oct 27, 2017

Great discussion! Gonna play parent and suggest that we merge this pr, open an issue to continue the discussion as whats being discussed exceeds the scope of this work. @Loicavenel @serenamarie125 ?

@chriskacerguis
Copy link
Contributor

@serenamarie125 and @Loicavenel thoughts on @AllenBW 's suggestion?

@Loicavenel
Copy link

looks good for me... We should re-discuss the "Configuration" button later.. I think when there is only 2 buttons, it may be better to show them both than have to click on configuration

@serenamarie125
Copy link

This is the tough part with PRs which cover multiples. Approving as not to block progress.

@serenamarie125
Copy link

@miq-bot add_labels ux/approved

@serenamarie125
Copy link

@miq-bot remove_labels ux/review

Copy link

@serenamarie125 serenamarie125 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for bringing back the camera icon!

@chriskacerguis chriskacerguis merged commit 88c608e into ManageIQ:master Oct 31, 2017
@AllenBW AllenBW deleted the bug/#1167-snapshot-view branch October 31, 2017 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Snapshot View, missing camera and create/delete functionality
5 participants