-
Notifications
You must be signed in to change notification settings - Fork 983
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
Fixes #26781 - DateTimePicker placement (#6750) #6757
Fixes #26781 - DateTimePicker placement (#6750) #6757
Conversation
Issues: #26781 |
@@ -1,4 +1,4 @@ | |||
.bootstrap-datetimepicker-widget.dropdown-menu.top { | |||
.bootstrap-datetimepicker-widget.dropdown-menu { |
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.
Please change this into:
.bootstrap-datetimepicker-widget.dropdown-menu.top {
margin-top: -10px;
}
.bootstrap-datetimepicker-widget.dropdown-menu.bottom {
margin-top: 10px !important;
}
.bootstrap-datetimepicker-widget.dropdown-menu {
max-width: none;
otherwise the popover arrow is inside the input instead of being under it
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.
Please see #6761, this is PR for stable branch, we need to do it first to develop.
#6756 was merged, is this still wip? or is it ready to go to the stable branch? |
6450925
to
a234a80
Compare
#6761 merged as well, this is ready now. |
one of the js snapshot test failures is related, could you please fix that? (Report Template AutoComplete › rendering › renders report template with date input) |
Yeah, it was related, my bad I've missed that! |
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.
Thanks @ezr-ondrej !
Cherry-pick for #6750 blocked by #6756. We need to cherry-pick both or neither.
I mark it WIP till it is in.