Ticket #7129 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Datepicker: Can't minify with YUI Compressor

Reported by: jimconte Owned by:
Priority: minor Milestone: 1.8.12
Component: ui.datepicker Version: 1.8.10
Keywords: Cc:
Blocking: Blocked by:

Description

Hi, Thanks for an excellent framework.

I have found a bug regarding comments placed at the following: \JQuery-ui-1.8.10\jquery-ui-1.8.10.custom.js line 9178 \JQuery-ui-1.8.10\jquery.ui.datepicker.js line 1086

notice the developer used in his comments " E.g. 31/02/* "

where the string [forwardslash]/[asterisk] " /* " is likely interpreted as the beginning of a block comment by our minifier.

Our minifier: Yahoo.Yui.Compressor.dll

Thank you for your time and effort.

-Jim Conte

Change History

comment:1 Changed 3 years ago by jimconte

Fix: E.g. 31/02/00

Last edited 3 years ago by jimconte (previous) (diff)

comment:2 Changed 3 years ago by scott.gonzalez

  • Summary changed from Our minifier breaks on comments to Datepicker: Can't minify with YUI Compressor

comment:3 Changed 3 years ago by Scott González

  • Status changed from new to closed
  • Resolution set to fixed

Datepicker: Changed a comment that causes problems with YUI Compressor. Fixes #7129 - Datepicker: Can't minify with YUI Compressor.

Changeset: 6d0164571647e169c9674a2853f3223caf188e7e

comment:4 Changed 3 years ago by Scott González

Datepicker: Changed a comment that causes problems with YUI Compressor. Fixes #7129 - Datepicker: Can't minify with YUI Compressor. (cherry picked from commit 6d0164571647e169c9674a2853f3223caf188e7e)

Changeset: 2b4a902e00b076661dc99528c88536e9460d415a

comment:5 Changed 3 years ago by scott.gonzalez

  • Milestone changed from 1.9 to 1.8.12
Note: See TracTickets for help on using tickets.