Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Fix calendar icon to fit properly after field #1428 #1499

Merged
merged 3 commits into from
Jan 18, 2018
Merged

Conversation

wiadev
Copy link
Contributor

@wiadev wiadev commented Jan 11, 2018

  • open calendar by default after double-click

#1428

- open calendar by default after double-click
@metas-mk metas-mk self-requested a review January 12, 2018 05:39
<Datetime
closeOnTab={true}
renderDay={this.renderDay}
renderInput={this.renderInput}
onBlur={this.handleBlur}
onFocus={this.handleFocus}
{...this.props}
open={true}
Copy link
Member

Choose a reason for hiding this comment

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

@wiadev seems as if the datepickers are now always initially opened and don't close anymore.

see the following example
image

<Datetime
closeOnTab={true}
renderDay={this.renderDay}
renderInput={this.renderInput}
onBlur={this.handleBlur}
onFocus={this.handleFocus}
{...this.props}
open={true}
Copy link
Member

Choose a reason for hiding this comment

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

Also in edit mode. After doubleclick on the datefield to switch into edit more, the datepicker does not dissappear after selection anymore.
image

Copy link
Member

@metas-mk metas-mk left a comment

Choose a reason for hiding this comment

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

Hi @wiadev can u please check the quick feedback about datepicker now remaining open and not closing anymore..

Copy link
Member

@metas-mk metas-mk left a comment

Choose a reason for hiding this comment

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

Found this new Issue that appears after these changes. Just doubleclick on date field in Subtab and check console.

image

@wiadev
Copy link
Contributor Author

wiadev commented Jan 15, 2018

@metas-mk updated

- Remove z-index: -1 from meta-date-picker icon
@wiadev
Copy link
Contributor Author

wiadev commented Jan 17, 2018

@metas-mk can you check out this, please? I might need this integrated into #1387.

@wiadev
Copy link
Contributor Author

wiadev commented Jan 17, 2018

please, check #1507, first before this PR.

@metas-mk
Copy link
Member

Quick local check OK. Integrating.

@metas-mk metas-mk merged commit b0b00de into master Jan 18, 2018
@metas-mk metas-mk deleted the dev-1428 branch January 18, 2018 08:20
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.

2 participants