Skip to content

[SelectField] Add callback signatures to docs; improve other props.#3924

Merged
oliviertassinari merged 1 commit intomui:masterfrom
theosherry:SelectField_docs
May 6, 2016
Merged

[SelectField] Add callback signatures to docs; improve other props.#3924
oliviertassinari merged 1 commit intomui:masterfrom
theosherry:SelectField_docs

Conversation

@theosherry
Copy link
Copy Markdown
Contributor

  • PR has tests / docs demo, and is linted.
  • Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
  • Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Hey, the next one for #3096.

Comment thread src/SelectField/SelectField.js Outdated
* If true, the width will automatically be set according to the
* items inside the menu.
* To control this width in css instead, leave this prop to `false`.
* To control the width in CSS instead, leave this prop to `false`.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

'leave this prop set to false'?

@mbrookes
Copy link
Copy Markdown
Member

mbrookes commented Apr 27, 2016

Thanks @theosherry - sorry for the delay in reviewing, I've been away, and then tied up with work. Looks good apart from one possible clarification.

@mbrookes mbrookes added PR: Needs Revision docs Improvements or additions to the documentation. and removed PR: Needs Review labels Apr 27, 2016
@theosherry
Copy link
Copy Markdown
Contributor Author

@mbrookes No problem 😄

@mbrookes
Copy link
Copy Markdown
Member

mbrookes commented May 5, 2016

Thanks. 👍

@oliviertassinari oliviertassinari merged commit 6a4a8bb into mui:master May 6, 2016
@oliviertassinari
Copy link
Copy Markdown
Member

@theosherry great PR regarding the docs, as always 👍.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants