Well, almost everything is in the title : if the daterange field has unlimited values, then litepicker popup will show up only for the first one.

I'll try to provide a patch for that one.

CommentFileSizeAuthor
#2 date_range_picker-3110119-1.patch2.93 KBKojo Unsui

Comments

Kojo Unsui created an issue. See original summary.

Kojo Unsui’s picture

Kojo Unsui commented
StatusFileSize
new date_range_picker-3110119-1.patch2.93 KB

Since there is a fieldset with form-wrapper class for each set of values, I used it as top selector, it returns an object with all the fieldsets. Then it was not necessary to change the logic below.

var $dateRangeWidgets = $(context).find('.field--type-daterange .form-wrapper');

I also added an undefined test line 13 to remove an error, and removed a bunch of console.log (hope you won't mind that one).

I tested for limited values (a single value, three values). And for unlimited values.
I tested with following admin themes : Claro, Seven, and Adminimal.

So far, looks good.

Kojo Unsui’s picture

Kojo Unsui commented
Status: Active » Needs review
webmestre’s picture

webmestre commented

Hi,
thank you for helping.

When apply the patch, I got the following message:

can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/js/litepicker.js b/js/litepicker.js
|index 7cd1626..3f207ab 100644
|--- a/js/litepicker.js
|+++ b/js/litepicker.js
--------------------------
k4v’s picture

k4v commented

I just pushed a fixed for this, thanks for your help :).

k4v’s picture

k4v commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.