#5511 closed bug (fixed)

Opened April 13, 2010 12:32PM UTC

Closed October 04, 2012 01:37AM UTC

Demos: Don't filter source, display it all

Reported by: jzaefferer Owned by:
Priority: major Milestone: 1.9.0
Component: [meta] ui.demos Version: 1.8
Keywords: Cc:
Blocked by: Blocking:
Description

The View Source filter is broken anyway, see http://jqueryui.com/demos/autocomplete/#combobox

But more important, not showing the required CSS and JS dependencies is really backwards, see http://www.coldfusionjedi.com/index.cfm/2010/4/12/Playing-with-jQuery-UIs-Autocomplete-Control

The only thing I'll point out is - and this bugs me about all the jQuery UI demos, I'm going to show you the full source behind the code. It really bugs me that their View Source content never shows the included JavaScript and CSS commands. Yes, it is implied, but I know I had a hard time with this when I first began to use jQuery UI. When it comes to docs, I think it's always safe to assume less.

The filtered "overhead" is just about 10 lines anyway.

For this to be feasible, all resources must point at the CDN files.

Attachments (0)
Change History (4)

Changed April 13, 2010 01:17PM UTC by jzaefferer comment:1

description: The View Source filter is broken anyway, see http://jqueryui.com/demos/autocomplete/#combobox \ \ But more important, not showing the required CSS and JS dependencies is really backwards, see http://www.coldfusionjedi.com/index.cfm/2010/4/12/Playing-with-jQuery-UIs-Autocomplete-Control \ \ {{{The only thing I'll point out is - and this bugs me about all the jQuery UI demos, I'm going to show you the full source behind the code. It really bugs me that their View Source content never shows the included JavaScript and CSS commands. Yes, it is implied, but I know I had a hard time with this when I first began to use jQuery UI. When it comes to docs, I think it's always safe to assume less.}}} \ \ The filtered "overhead" is just about 10 lines anyway. → The View Source filter is broken anyway, see http://jqueryui.com/demos/autocomplete/#combobox \ \ But more important, not showing the required CSS and JS dependencies is really backwards, see http://www.coldfusionjedi.com/index.cfm/2010/4/12/Playing-with-jQuery-UIs-Autocomplete-Control \ \ {{{The only thing I'll point out is - and this bugs me about all the jQuery UI demos, I'm going to show you the full source behind the code. It really bugs me that their View Source content never shows the included JavaScript and CSS commands. Yes, it is implied, but I know I had a hard time with this when I first began to use jQuery UI. When it comes to docs, I think it's always safe to assume less.}}} \ \ The filtered "overhead" is just about 10 lines anyway. \ \ For this to be feasible, all resources must point at the CDN files.
milestone: 1.9 → 1.next

Changed October 19, 2010 04:03PM UTC by scottgonzalez comment:2

milestone: 1.next → 1.9
priority: critical → major

Changed February 27, 2012 03:45PM UTC by jzaefferer comment:3

status: new → open

Changed October 04, 2012 01:37AM UTC by scottgonzalez comment:4

resolution: → fixed
status: open → closed

Fixed in the new site.