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

Styling fix for Bootstrap switch on Tenant Quota screen #2583

Merged
merged 1 commit into from
Nov 3, 2017
Merged

Styling fix for Bootstrap switch on Tenant Quota screen #2583

merged 1 commit into from
Nov 3, 2017

Conversation

epwinchell
Copy link
Contributor

@epwinchell epwinchell commented Oct 31, 2017

This PR fixes a bug on the "Manage quotas for tenant" screen, where bootstrap switch was malformed because of a css setting that forced word breaks in TDs. This fix excludes the "table-view-pf-select" class (used for checkboxes, switches and icons) from using word-break and also cleans up extraneous markup on the screen. The problem is exposed when the UI is set to use the Japanese language.

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

Before
screen shot 2017-10-31 at 12 05 46 pm

After
screen shot 2017-10-31 at 12 05 17 pm

@epwinchell
Copy link
Contributor Author

@miq-bot add_label bug, formatting/styling, gaprindashvili/yes

@miq-bot
Copy link
Member

miq-bot commented Oct 31, 2017

Checked commit https://github.com/epwinchell/manageiq-ui-classic/commit/fd29407a7fa9a8577965835d8a687f33eeabca98 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@epwinchell
Copy link
Contributor Author

@miq-bot assign @h-kataria

@mzazrivec mzazrivec added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 3, 2017
@mzazrivec mzazrivec assigned mzazrivec and unassigned h-kataria Nov 3, 2017
@mzazrivec mzazrivec merged commit 1f8bfff into ManageIQ:master Nov 3, 2017
@epwinchell
Copy link
Contributor Author

@miq-bot add_label fine/yes

simaishi pushed a commit that referenced this pull request Nov 6, 2017
 Styling fix for Bootstrap switch on Tenant Quota screen
(cherry picked from commit 1f8bfff)

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

simaishi commented Nov 6, 2017

Gaprindashvili backport details:

$ git log -1
commit 5de611f6aa79a54ae11d15cd280eb2db663ba7ae
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Fri Nov 3 13:08:09 2017 +0100

    Merge pull request #2583 from epwinchell/1495190
    
     Styling fix for Bootstrap switch on Tenant Quota screen
    (cherry picked from commit 1f8bfff09f93ccc44fd66e75a8a9b3c195b3e740)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1509422

simaishi pushed a commit that referenced this pull request Nov 20, 2017
 Styling fix for Bootstrap switch on Tenant Quota screen
(cherry picked from commit 1f8bfff)

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

Fine backport details:

$ git log -1
commit 8f8de1dcfd37c0d00d5829c39537281f7b7d9876
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Fri Nov 3 13:08:09 2017 +0100

    Merge pull request #2583 from epwinchell/1495190
    
     Styling fix for Bootstrap switch on Tenant Quota screen
    (cherry picked from commit 1f8bfff09f93ccc44fd66e75a8a9b3c195b3e740)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1509423

@epwinchell epwinchell deleted the 1495190 branch March 20, 2019 15:23
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