-
Notifications
You must be signed in to change notification settings - Fork 231
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 bug where margin is inconsistent on .table inside .table-responsive #1254
Conversation
[merge] |
[Test]ing while waiting on the merge queue |
Even though travis was happy when the commit was pushed, jenkins is now saying the dist for main.css isn't right. May need to try clean/install/build and see if you get a diff |
[merge] |
[test] |
[merge] |
1 similar comment
[merge] |
@rhamilto looks like its failing on dist/main.css |
Fixes #1187 The relevant bug fix lines are 218-240. The rest of the changes in _tables.less are refactoring and cleanup to align with CSS code standards. Also adding .table-bordered to the instances that lacked it so .tables are styled consistently.
Evaluated for origin web console test up to b740404 |
[merge] |
Evaluated for origin web console merge up to b740404 |
Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1096/) (Base Commit: 76d914e) |
Origin Web Console Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1095/) (Base Commit: 42c67c2) |
Fixes #1187
The relevant bug fix lines are 218-240. The rest of the changes in
_tables.less are refactoring and cleanup to align with CSS code
standards.
Also adding .table-bordered to the instances that lacked it so .tables
are styled consistently.