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 comment:1
| _comment0: | E.g. 31/02/00 → 1300427841656185 |
|---|
Changed March 18, 2011 12:23PM UTC by comment:2
| summary: | Our minifier breaks on comments → Datepicker: Can't minify with YUI Compressor |
|---|
Changed March 18, 2011 12:26PM UTC by comment:3
| resolution: | → fixed |
|---|---|
| status: | new → closed |
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 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 comment:5
| milestone: | 1.9 → 1.8.12 |
|---|
Fix: E.g. 31/02/00