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#1171-Resource and menu action buttons on the same row #1179

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Oct 27, 2017

closes #1171

pf-list-view has provisions for both menu (kebab) and action buttons, opting for our proven in place solution for fear the pf implementation will be a regression in terms of customizability

These two classes create an experience identical to the component options without sacrificing functioning in place code

screen shot 2017-10-27 at 12 19 03 pm

screen shot 2017-10-27 at 12 19 17 pm

pf-list-view has provisions for both menu (kebab) and action buttons, opting for our proven in place solution for fear the pf implementation will be a regression in terms of customizability

These two classes create an experience identical to the component options without sacraficing functioning in place code
@AllenBW AllenBW force-pushed the bug/#1171-service-details-resource-rows branch from 0660a19 to 7f8b47b Compare October 27, 2017 16:20
@AllenBW AllenBW removed the wip label Oct 27, 2017
@AllenBW AllenBW added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 27, 2017
@miq-bot
Copy link
Member

miq-bot commented Oct 27, 2017

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

@chriskacerguis chriskacerguis self-assigned this Oct 27, 2017
@chriskacerguis
Copy link
Contributor

LGTM, waiting for UXD review from @serenamarie125

@chriskacerguis
Copy link
Contributor

@serenamarie125 does this match what was discussed at the meeting?

@chriskacerguis
Copy link
Contributor

@Loicavenel this work for you?

@Loicavenel
Copy link

yes, this is what was in 4.5, back to normal

@chriskacerguis
Copy link
Contributor

Awesome, thanks @Loicavenel just waiting for @serenamarie125 sign off.

@serenamarie125
Copy link

Spacing looks off, but I can open an issue re: spacing

@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.

LGTM with exception of too much spacing before the initial items in the resource list view items. We can follow up with that later.

@chriskacerguis chriskacerguis merged commit 3e5dac6 into ManageIQ:master Oct 31, 2017
@AllenBW AllenBW deleted the bug/#1171-service-details-resource-rows branch October 31, 2017 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants