Skip to content

Conversation

epwinchell
Copy link
Contributor

@epwinchell epwinchell commented Dec 13, 2017

Remove pagination styling that is now available in the update PatternFly to v3.31.1, here: ManageIQ/manageiq-ui-classic#2989

@miq-bot miq-bot added the wip label Dec 13, 2017
@epwinchell
Copy link
Contributor Author

@miq-bot add_label refactoring, gaprindashvili/no

@miq-bot
Copy link
Member

miq-bot commented Dec 13, 2017

@epwinchell Cannot apply the following label because they are not recognized: gaprindashvili/no

@epwinchell
Copy link
Contributor Author

@himdel @skateman please test

@epwinchell epwinchell changed the title [WIP] Clean up duplicate pagination styling Clean up duplicate pagination styling Dec 13, 2017
@miq-bot miq-bot removed the wip label Dec 13, 2017
@miq-bot
Copy link
Member

miq-bot commented Dec 13, 2017

Checked commit https://github.com/epwinchell/ui-components/commit/d89fc68f9c74ced272bb76a13689bd73e572aa8c with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@himdel
Copy link
Contributor

himdel commented Dec 15, 2017

LGTM, not seeing any UI breakage 👍

Not merging yet, as this is gaprindashvili/no and there's still a few gaprindashvili/yes PRs.

(The moment we merge this, we'll need to start doing branches if we want to do any more g/yes after.)

@himdel himdel self-assigned this Feb 7, 2018
@himdel
Copy link
Contributor

himdel commented Feb 7, 2018

Well, there is a branch now .. 1.0.* will be from the gaprindashvili branch, 1.1.* from master.. Merging :).

@himdel himdel merged commit f3f2e66 into ManageIQ:master Feb 7, 2018
@himdel himdel added this to the Sprint 79 Ending Feb 12, 2018 milestone Feb 7, 2018
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.

3 participants