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

Fix hovering issue in the date range component #420

Closed
metas-rc opened this issue Feb 24, 2017 · 3 comments
Closed

Fix hovering issue in the date range component #420

metas-rc opened this issue Feb 24, 2017 · 3 comments

Comments

@metas-rc
Copy link
Member

metas-rc commented Feb 24, 2017

Type of issue

  • Bug

Current behavior

  • When using a date range component ( a filter) the following happens:
    ** The firstdate is selected => both the dateFrom and dateTo fields are set with this date : OK
    ** The second date is selected => both the dateFrom and dateTo fields are set with this date : NOK
    ** If you hover on other dates, the first date is also modified in the field: NOK
  • This might be confusing from a user perspective

Expected behavior

  • Since the first date was set, leave the date Field value as is .

Steps to reproduce

Step 1: Selected first date as February 1:
image

Step 2: decided to set the second date on March 3:
image

Step 3: Clicked on March 3:

image

Step 4: Move the cursor to March 10:
image

Step 5: Move the cursor out of the date component ( all back to normal):

image

Is this easy to fix?
Please let us know about the complexity of fixing this.

@damianprzygodzki
Copy link
Contributor

First, it is not easy to fix, it is a totally different behavior than that one.
Second, the current behavior is as every date range picker is doing, it is to selecting range, and do not chosing two dates separetely.

@cadavre
Copy link
Contributor

cadavre commented Feb 24, 2017

This is how used library works: http://www.daterangepicker.com/

It's not perfect because it doesn't enable you to change once selected date on left and right sides of calendar. In fact this is not "input and calendar" + "input and calendar" but "input + input and calendar".

What I suggest to do is: enable autoApply – it will close picker after selection of 2nd date, and to change dates you'd need to open it one more time and select new range.

OR

Disable inputs for dates – this will leave only calendar to select from and to. But this will on the other hand disable possibility to see dates at once first next to other.

Both are not perfect but you can decide:

  1. Leave as is
  2. Use autoApply with clear on click.
  3. Disable inputs.

I would say #2 will not confuse user.

@teosarca
Copy link
Member

closing because it seems there is no solution for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants