Can you update the description to talk about the changes to the browser compatibility list?

Let's put an Oxford Comma after description and change that to "expiration date and time".

For chaining, the preferred format would be:

this._$datePicker =
    $(dedent`
        ...
    `)
    .prepend(this.options.descriptorText);


Though if we're looking to build this and put something in it at the same time, we should probably just use a template, or build the DOM elements incrementally (since we have to find the input, we might as well do the latter).

Ship It!

Ship It!

Looks really good! These changes are small.

We probably don't need to worry about changes here. This is important for transpiling code, but the usage of datetime-local doesn't really impact that. It's true that datetime-local may not work on older browsers, but that's probably okay. People who need to be on older browsers that need to edit this can put in a date manually, if needed.

No need for dedent here, since there's only one line. You can simply have a string on the next line.

This will be 5.0.2 now.

No need for dedent here.

For repeat index lookups, let's assign to a variable. Helps group together related assertions, and reduces visual noise.

I think what Christian meant was:

this._$datePicker = $(
    '<span class="rb-c-date-inline-editor__picker">'
)


We also don't need the closing </span> here.

Same here.

Ship It!