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

Clickable labels in policy's event assignment screen #1511

Conversation

mzazrivec
Copy link
Contributor

@mzazrivec mzazrivec commented Jun 8, 2017

Control -> Explorer -> Policies -> [policy] -> Edit policy's event assignment: clicking on event label
would check / uncheck the checkbox as well.

Before, the checkbox labels weren't clickable:
checkboxes-before

After, the checkbox labels are clickable (not really visible from the screenshot):
checkboxes-after

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

@miq-bot
Copy link
Member

miq-bot commented Jun 8, 2017

Checked commit mzazrivec@773fccf with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 1 offense detected

app/views/miq_policy/_policy_details.html.haml

  • ⚠️ - Line 165 - Avoid more than 3 levels of block nesting.

@dclarizio
Copy link

@mzazrivec can you add a screenshot please? Thx, Dan

@dclarizio dclarizio self-assigned this Jun 8, 2017
@mzazrivec
Copy link
Contributor Author

@dclarizio Screenshots added.

@dclarizio dclarizio merged commit 4351809 into ManageIQ:master Jun 9, 2017
@dclarizio dclarizio added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 9, 2017
@simaishi
Copy link
Contributor

simaishi commented Jun 15, 2017

Backported to Fine to avoid conflict backporting #1543. The change made in this PR is re-done in #1543 and will not change the outcome.

$ git log -1
commit 6a258329e9649cf9ae3a01bf4a27caafc37bae7d
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Fri Jun 9 06:56:16 2017 -0700

    Merge pull request #1511 from mzazrivec/clickable_labels_in_policy_event_assignment_screen
    
    Clickable labels in policy's event assignment screen
    (cherry picked from commit 4351809e63c0ec09c1523ec6fcef2a18c049b405)

simaishi pushed a commit that referenced this pull request Jun 15, 2017
…ent_assignment_screen

Clickable labels in policy's event assignment screen
(cherry picked from commit 4351809)
@mzazrivec mzazrivec deleted the clickable_labels_in_policy_event_assignment_screen branch September 11, 2017 06:33
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.

4 participants