FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(Topology): Fix paddings for topology control bar buttons by jeff-phillips-18 · Pull Request #2635 · patternfly/patternfly-react · GitHub

fix(Topology): Fix paddings for topology control bar buttons - #2635

Merged
tlabaj merged 1 commit into
patternfly:masterfrom
jeff-phillips-18:topology-package
Aug 2, 2019
Merged

fix(Topology): Fix paddings for topology control bar buttons#2635
tlabaj merged 1 commit into
patternfly:masterfrom
jeff-phillips-18:topology-package

Conversation

Copy link
Copy Markdown
Member

What:
Remove css settings that override the paddings for the topology control bar buttons

Additional issues:
Fixes #2627

cc @christianvogt @mceledonia

Copy link
Copy Markdown
Collaborator

PatternFly-React preview: https://patternfly-react-pr-2635.surge.sh

tlabaj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

tlabaj added the PF4 label Aug 2, 2019
tlabaj requested a review from mceledonia August 2, 2019 17:33

mceledonia left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Nice, looks great!

tlabaj merged commit a8aa582 into patternfly:master Aug 2, 2019

Copy link
Copy Markdown
Collaborator

Your changes have been released in:

  • @patternfly/react-topology@2.7.10

Thanks for your contribution! 🎉

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

icon button size look out of place in topology

5 participants


Back | FazBrowse Home | New Git URL