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

feat: Fix invalid "pId" attribute in AddButton (Bootstrap-4) #2046

Merged
merged 3 commits into from
Sep 24, 2020

Conversation

Xtremilicious
Copy link
Collaborator

Reasons for making this change

Fixes #2043

  1. <AiOutlinePlus> contains a "pId" attribute out of the box which is deemed invalid by React and throws a warning. Since this is a problem related to the used third-party package itself (ref), I superseded react-icons/ai with react-icons/bs which doesn't throw any such warnings.

  2. Decreased the margin-bottom of the DescriptionField widget as it was uneven as compared to the other themes.

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests

@Xtremilicious
Copy link
Collaborator Author

@epicfaace Kindly have a look at this :)

@epicfaace epicfaace merged commit 2c41db3 into rjsf-team:master Sep 24, 2020
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.

[Bootstrap-4] Warning: React does not recognize the pId prop on a DOM element.
2 participants