-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
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
cosmetic fixes to textfield/select-field #922
Conversation
chrismcv
commented
Jun 23, 2015
- unsetting errorText prop should remove error text
- When selecting an item from select-field e.target.value should be the value not the payload
- Menu calculation didn't work when menuItems are changed/supplied after mount.
@chrismcv Looks like this PR broke SelectField menu: |
apologies... fixed |
@chrismcv Can you rebase please? |
yip done |
@chrismcv - Looks like something weird with the LeftNav now. |
… being actually used. renamed showRowSelectCheckbox to displayRowCheckbox
Sorry again... should be fixed now.... also fixes eslint on windows, and a On Wed, Jun 24, 2015 at 4:13 PM Hai Nguyen notifications@github.com wrote:
|
should be done, |
cosmetic fixes to textfield/select-field
great! Thanks @chrismcv 👍 |
By replacing `@class` annotation with `@__PURE__` we broke rollup internal sourcemap generator. Found the fix here https://github.com/ex-machine/rollup-plugin-ts-treeshaking/blob/master/index.js