-
Notifications
You must be signed in to change notification settings - Fork 251
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
Feature to disable add files button #132
base: master
Are you sure you want to change the base?
Conversation
…n the input bar. Defaults to true.
</div> | ||
); | ||
} else { | ||
return null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm so this makes the emoji picker icon jump to the left whenever the user starts typing; I think it would be better to always render either the file picker or the send icon. The handler _submitText already just ignores empty messages so I think it would be fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! Just the one rendering detail that I'd like to be changed before merging. Thanks so much for your contribution!
@@ -1,5 +1,6 @@ | |||
coverage/ | |||
demo/dist/ | |||
.idea/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we maybe put this after the .DS_Store
entry, and give that section a title like # OS or editor files
?
Description
Added a prop that decides whether or not to show the add files button in the input bar.
Closes #134.
Checklist
npm start
) to ensure it still functions as expected, and/ornpm run lint
passes (npm run lint -- --fix
will autofix where possible)