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

Fix for different behavior of items in regards to default values on dialog sample page #3371

Merged
merged 1 commit into from Feb 5, 2018

Conversation

eclarizio
Copy link
Member

This is a simple change to the sample page of a dialog (when you click on it in the accordion list and it shows some basic sample of what it's going to look like in the right cell). We're merely taking into account default values that were not being utilized before. The radio button sample was also just completely incorrect and needed to be modified.

This fixes part of https://bugzilla.redhat.com/show_bug.cgi?id=1540273, the fix for the timepicker not showing up in the service-ui will likely be needed in the ui-components repo.

@miq-bot add_label gaprindashvili/yes, bug, blocker
@miq-bot assign @dclarizio

@miq-bot
Copy link
Member

miq-bot commented Feb 5, 2018

Checked commit eclarizio@d7b7344 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@h-kataria
Copy link
Contributor

looks good.

@dclarizio dclarizio merged commit 4bddbcf into ManageIQ:master Feb 5, 2018
@dclarizio dclarizio added this to the Sprint 79 Ending Feb 12, 2018 milestone Feb 5, 2018
simaishi pushed a commit that referenced this pull request Feb 6, 2018
Fix for different behavior of items in regards to default values on dialog sample page
(cherry picked from commit 4bddbcf)

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

simaishi commented Feb 6, 2018

Gaprindashvili backport details:

$ git log -1
commit 962bc8144a93352fb7c7d112d45b80a6a87b2092
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Mon Feb 5 13:18:56 2018 -0800

    Merge pull request #3371 from eclarizio/BZ1540273
    
    Fix for different behavior of items in regards to default values on dialog sample page
    (cherry picked from commit 4bddbcf4788e76ee6b44fc66c1940f50f160450a)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1542610

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.

None yet

5 participants