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 button tag to generic object form #6226

Merged
merged 1 commit into from
Sep 23, 2019
Merged

Add button tag to generic object form #6226

merged 1 commit into from
Sep 23, 2019

Conversation

epwinchell
Copy link
Contributor

@epwinchell epwinchell commented Sep 23, 2019

This PR replaces a div with a button tag for Delete buttons so that tabbing works properly

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

@epwinchell
Copy link
Contributor Author

@miq-bot assign @h-kataria

@epwinchell
Copy link
Contributor Author

@miq-bot add_label bug, formatting/styling, hammer/yes, ivanchuk/yes

@miq-bot
Copy link
Member

miq-bot commented Sep 23, 2019

Checked commit https://github.com/epwinchell/manageiq-ui-classic/commit/52c843ec48d4b5a6501001fbfdfec6d7edf41df7 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@h-kataria h-kataria added this to the Sprint 121 Ending Sep 30, 2019 milestone Sep 23, 2019
@h-kataria h-kataria merged commit 523d547 into ManageIQ:master Sep 23, 2019
simaishi pushed a commit that referenced this pull request Nov 1, 2019
Add button tag to generic object form

(cherry picked from commit 523d547)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1767823
@simaishi
Copy link
Contributor

simaishi commented Nov 1, 2019

Ivanchuk backport details:

$ git log -1
commit 51443513b2c175e1b60e1e63e2eb698ac8858b2b
Author: Harpreet Kataria <[email protected]>
Date:   Mon Sep 23 14:24:14 2019 -0400

    Merge pull request #6226 from epwinchell/1519494
    
    Add button tag to generic object form
    
    (cherry picked from commit 523d547b5d45ff869e382b119bc94ab6f5ca957e)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1767823

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.

5 participants