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

Fixed Component URLInput not honouring className #13800

Merged
merged 2 commits into from
Feb 14, 2019

Conversation

ashwin-pc
Copy link
Contributor

@ashwin-pc ashwin-pc commented Feb 9, 2019

Description

Closes #13754
This pull request fixes the URIInput Component not honouring. This add the className to the parent div for the component

How has this been tested?

  • Added className to the URLInput component "/packages/format-library/src/link/inline.js"
  • Inspected element in browser for classname from link menu

Before Change:
The class was not appended to the URLInput class "editor-url-input"

After Change
The class was appended to the URLInput class "editor-url-input"

Types of changes]

Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.

Related: WordPress#13754
This pull request fixes the URIInput Component not honouring. This add the className to the parent div for the component
@gziolo gziolo added this to the 5.1 (Gutenberg) milestone Feb 10, 2019
@gziolo gziolo added [Type] Enhancement A suggestion for improvement. Good First Review A PR that's suitable for someone looking to contribute for the first time by reviewing code labels Feb 10, 2019
Copy link
Member

@noisysocks noisysocks left a comment

Choose a reason for hiding this comment

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

This looks good to me!

Could you please add className to the documentation in editor/src/components/url-input/README.md?

@ashwin-pc
Copy link
Contributor Author

This looks good to me!

Could you please add className to the documentation in editor/src/components/url-input/README.md?

The Readme has been updated to reflect the change

Copy link
Member

@noisysocks noisysocks left a comment

Choose a reason for hiding this comment

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

Love it! Thanks for your work on this 👍

@noisysocks noisysocks merged commit 0ede174 into WordPress:master Feb 14, 2019
youknowriad pushed a commit that referenced this pull request Mar 6, 2019
* Fixed Component URLInput not honouring className

Related: #13754
This pull request fixes the URIInput Component not honouring. This add the className to the parent div for the component

* Updated URLInput Readme for className prop
youknowriad pushed a commit that referenced this pull request Mar 6, 2019
* Fixed Component URLInput not honouring className

Related: #13754
This pull request fixes the URIInput Component not honouring. This add the className to the parent div for the component

* Updated URLInput Readme for className prop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Review A PR that's suitable for someone looking to contribute for the first time by reviewing code [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Component URLInput not honouring className prop
3 participants