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

BZ#1446408 -filter tooltip overflow #753

Merged

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented May 11, 2017

@AllenBW AllenBW added this to the Sprint 61 Ending May 22, 2017 milestone May 11, 2017
@AllenBW AllenBW requested a review from chalettu May 11, 2017 12:05
@chriskacerguis chriskacerguis self-assigned this May 11, 2017
@miq-bot
Copy link
Member

miq-bot commented May 11, 2017

Checked commit AllenBW@84c19c7 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@serenamarie125
Copy link

LGTM. Have you guys already bumped your version of Angular PF? I think it was just released on 5/10th (https://github.com/patternfly/angular-patternfly/releases/tag/v3.24.0)

@chriskacerguis
Copy link
Contributor

Yup, it was bumped as part of this PR.

@chriskacerguis
Copy link
Contributor

@serenamarie125 could you please add your approval?

Copy link

@serenamarie125 serenamarie125 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the contribution to PF as well

@chriskacerguis chriskacerguis merged commit af8b601 into ManageIQ:master May 12, 2017
@AllenBW AllenBW deleted the BZ/#1446408-filtertooltipoverflow branch May 12, 2017 16:27
@simaishi
Copy link
Contributor

simaishi commented Jun 9, 2017

@AllenBW there is a conflict backporting to Fine branch. Please confirm ^3.24.0 is what we'd like for Fine branch.

++<<<<<<< HEAD
 +    "angular-patternfly": "3.23.2",
++=======
+     "angular-patternfly": "^3.24.0",
++>>>>>>> af8b601... Merge pull request #753 from AllenBW/BZ/#1446408-filtertooltipoverflow

@AllenBW
Copy link
Member Author

AllenBW commented Jun 9, 2017

@simaishi yep! ^3.24.0 is what we want, ( or probably more like 3.24.0 without ^) the release docs here indicate the desired fix is incedlud

@simaishi
Copy link
Contributor

@AllenBW so just 3.24.0 (without ^) for angular-patternfly, and keep ~ for patternfly and bootstrap-switch ?

@simaishi
Copy link
Contributor

simaishi commented Mar 6, 2018

Removed fine/yes as this is no longer needed as per BZ.

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