Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Issue 518 - Fix react dates regression in dash==0.41.0 #519

Merged
merged 4 commits into from
Apr 11, 2019

Conversation

Marc-Andre-Rivet
Copy link
Contributor

@Marc-Andre-Rivet Marc-Andre-Rivet commented Apr 11, 2019

Fixes #518

Marc-André Rivet and others added 2 commits April 11, 2019 12:14
@@ -0,0 +1,3 @@
.DateInput {
overflow: hidden;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The react-dates component looks bad if the element is not very wide -- our previous usage and defaults tend to display incorrectly, proper overflow fixes this on our side for now.

@alexcjohnson
Copy link
Collaborator

One weird effect shows up in percy - the datepicker arrow disappeared?
diff:
Screen Shot 2019-04-11 at 2 04 07 PM
new image:
Screen Shot 2019-04-11 at 2 04 16 PM
all the other visual diffs look excellent AFAICT

@Marc-Andre-Rivet
Copy link
Contributor Author

@alexcjohnson Thanks for catching that. The solution is the problem. Looking at this again.

@Marc-Andre-Rivet
Copy link
Contributor Author

Using box-sizing allows the ^ to be visible and the main elements to be bounded correctly / the border to appear.

Copy link
Collaborator

@alexcjohnson alexcjohnson left a comment

Choose a reason for hiding this comment

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

We have a winner! 💃

@Marc-Andre-Rivet Marc-Andre-Rivet merged commit 082165d into master Apr 11, 2019
@Marc-Andre-Rivet Marc-Andre-Rivet deleted the issue518-date-regression branch April 11, 2019 22:44
@byronz byronz mentioned this pull request Apr 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DatePickerSingle and DatePickerRange style regression in dash==0.41.0
2 participants