Skip to main content

Search and Top Navigation

#7129 closed bug (fixed)

Opened March 18, 2011 05:48AM UTC

Closed March 18, 2011 12:26PM UTC

Last modified March 18, 2011 12:28PM UTC

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:
Blocked by: Blocking:
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

Attachments (0)
Change History (5)

Changed March 18, 2011 05:56AM UTC by jimconte comment:1

_comment0: E.g. 31/02/001300427841656185

Fix: E.g. 31/02/00

Changed March 18, 2011 12:23PM UTC by scottgonzalez comment:2

summary: Our minifier breaks on commentsDatepicker: Can't minify with YUI Compressor

Changed March 18, 2011 12:26PM UTC by Scott González comment:3

resolution: → fixed
status: newclosed

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

Changeset: 6d0164571647e169c9674a2853f3223caf188e7e

Changed March 18, 2011 12:28PM UTC by Scott González comment:4

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

Changed March 18, 2011 12:28PM UTC by scottgonzalez comment:5

milestone: 1.91.8.12