-
Notifications
You must be signed in to change notification settings - Fork 605
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 text overflow on Catalog tiles w/ long names #669
Fix text overflow on Catalog tiles w/ long names #669
Conversation
@sosiouxme fyi /hold |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@jeff-phillips-18 I know @SamiSousa wanted to update patternfly-react as well. We might try to pull this together into one PR to make it easier on @sosiouxme. Are there any other fixes in patternfly-react we might need? We need to figure out how to automate this since we update dependencies so frequently. I'm going to open a task. |
patternfly-react 2.21.3 also has a fix we would want (side modal overlay animation on firefox) |
c255868
to
9eee194
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
9eee194
to
70687a6
Compare
@sosiouxme Updated package.json to patternfly-react-extensions 2.9.8 to pickup some other necessary fixes. |
Sorry folks, was not around much today. @jeff-phillips-18 please include this update to the sources file:
|
Fix is in patternfly-react-extensions version 2.9.8 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1640054
70687a6
to
a21a3f7
Compare
Updated |
/lgtm |
Service catalog tests will be broken until #670 merges |
/hold cancel |
jenkins rebuild |
Fix is in patternfly-react-extensions version 2.9.6
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1640054