Just a minor patch to fix a bunch of missing colons in javascript, which may confuse some browsers out there...

CommentFileSizeAuthor
#2 minor-js-fixes-1037358-2.patch1.6 KBsanduhrs
locale_datepicker_js.patch1.76 KBazol

Comments

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Status: Needs review » Reviewed & tested by the community

Just stumbled upond the same issue.
Patch applies and works well.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
StatusFileSize
new minor-js-fixes-1037358-2.patch1.6 KB

Just found the trailing commas, that are not allowed in js syntax.
Fixed that and rerolled the patch.

aspilicious’s picture

aspilicious commented

I agree :)

dawehner’s picture

dawehner
Primary language German
commented
Version: 7.x-dev » 8.x-dev
Issue tags: +Needs backport to D7
dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks!

bfroehle’s picture

bfroehle commented
Issue tags: -Needs backport to D7

Untagging since it's been committed to 7.x

Status: Fixed » Closed (fixed)

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