-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
Remove the outer div in the datepicker #19
Labels
Comments
@lucasnsborges, you may want to take a stab at this. |
Yes, @erickne. This is probably the reason. Sorry about that. 😕 If you want to take a stab at this, feel free to do it, I'd appreciate a lot! |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
alexrsagen
added a commit
to Konsept-IT/react-semantic-ui-datepickers
that referenced
this issue
Feb 14, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
🌈 Feature
Right now, we have an outer
div
in the datepicker with thefield
class wrapping theInput
component and the calendar. This approach has some issues, especially related to how the library behaves in forms.We must remove this
div
and implement the rendering of the calendar in another way. I'm not sure how this is supposed to work, but maybe these implementations can help:https://github.com/CompuIves/codesandbox-client/blob/master/packages/app/src/app/components/ContextMenu/index.js
https://github.com/CompuIves/codesandbox-client/blob/master/packages/app/src/app/components/Portal/index.js
The text was updated successfully, but these errors were encountered: