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

Change ng-show to ng-if in button group #1486

Merged
merged 1 commit into from
Jun 6, 2017

Conversation

ZitaNemeckova
Copy link
Contributor

Instead of hiding buttons they will never be created at all. Prevents problems when enter is clicked and calls first button's on-click action.

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

Automation -> Ansible -> Repositories -> Edit any Repository -> Instead of pressing Save press Enter on keyboard
(or any Angular form)

Before:
screen shot 2017-06-05 at 3 22 31 pm

After:
screen shot 2017-06-05 at 3 21 59 pm

@miq-bot add_label bug

Instead of hiding buttons they will never be created at all. Prevents problems when enter is clicked and calls first button's on-click action.
@miq-bot miq-bot added the bug label Jun 5, 2017
@miq-bot
Copy link
Member

miq-bot commented Jun 5, 2017

Checked commit ZitaNemeckova@00b3b54 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@mzazrivec mzazrivec added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 6, 2017
@mzazrivec mzazrivec merged commit 8323d58 into ManageIQ:master Jun 6, 2017
simaishi pushed a commit that referenced this pull request Jun 12, 2017
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit fb0a2ef0035677f86f5add117fd4011278127cbc
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Tue Jun 6 10:18:42 2017 +0200

    Merge pull request #1486 from ZitaNemeckova/ng_show_to_ng_if
    
    Change ng-show to ng-if in button group
    (cherry picked from commit 8323d58cda2653deb620eac8eb20b3d29ad5aab1)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1460791

@ZitaNemeckova ZitaNemeckova deleted the ng_show_to_ng_if branch September 12, 2017 14:20
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.

6 participants