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

fix(select): add isPlain variation/prop to Select.tsx, snapshot tests, and integration by jenny-s51 · Pull Request #2588 · patternfly/patternfly-react · GitHub

fix(select): add isPlain variation/prop to Select.tsx, snapshot tests, and integration - #2588

Merged
redallen merged 5 commits into
patternfly:masterfrom
jenny-s51:iss2371
Jul 31, 2019
Merged

fix(select): add isPlain variation/prop to Select.tsx, snapshot tests, and integration#2588
redallen merged 5 commits into
patternfly:masterfrom
jenny-s51:iss2371

Conversation

Copy link
Copy Markdown
Contributor

What: Closes #2371.

jenny-s51 changed the title fix(select): add isPlain prop to Select and snapshot tests WIP: fix(select): add isPlain prop to Select and snapshot tests Jul 25, 2019

Copy link
Copy Markdown
Collaborator

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

jenny-s51 changed the title WIP: fix(select): add isPlain prop to Select and snapshot tests fix(select): add isPlain prop to Select and snapshot tests Jul 25, 2019
jenny-s51 requested review from kmcfaul and tlabaj July 25, 2019 15:24
tlabaj previously requested changes Jul 25, 2019

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

Looking good. I would update the demo as well. It is good practice to verify that the props work in integration.

jenny-s51 changed the title fix(select): add isPlain prop to Select and snapshot tests fix(select): add isPlain variation/prop to Select.tsx, snapshot tests, and integration Jul 29, 2019
jenny-s51 requested a review from tlabaj July 29, 2019 19:22
tlabaj requested a review from mcoker July 30, 2019 14:47
jenny-s51 dismissed tlabaj’s stale review July 30, 2019 15:01

Resolved in latest commit 🙂

jenny-s51 requested a review from redallen July 30, 2019 15:02
redallen merged commit 0809b28 into patternfly:master Jul 31, 2019

mcoker 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

👍

jenny-s51 deleted the iss2371 branch August 6, 2019 17:51
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.

PF4 Select: The isPlain prop is missing

6 participants


Back | FazBrowse Home | New Git URL