Ticket #6664 (closed defect: fixed)

Opened 14 months ago

Last modified 12 months ago

dijit test pages are misleading regarding styling and themes

Reported by: alex Owned by: dante
Priority: normal Milestone: 1.2
Component: Dijit Version: 1.1.0
Severity: normal Keywords: dijit, tests
Cc: sfoster, dante, phiggins, edencardim@…

Description

the structure of Dijit test-pages doesn't make it clear that users need to add the theme class explicitly on the body of their document, nor does it make clear that you must pull in the css files for the themes you wish to use.

Many users still use the test files as examples of how to set up their pages, and making this clearer would reduce the number of support requests we need to answer. The right solution, I think, is to pull in tundra by default and have the helper pull in an alternate and change the body style instead of having the helper be responsible for the initial application of both.

Change History

Changed 14 months ago by alex

  • cc edencardim@… added

Changed 14 months ago by bill

This has bothered me a long time too, mainly for the reason you mentioned but also because it exacerbates race conditions loading CSS (where the JS starts executing before the CSS has finished loading and bad things happen). If that's doable then let's do it (although I'm not volunteering to modify all the test files myself).

Changed 14 months ago by dante

  • status changed from new to assigned
  • owner set to dante

Changed 14 months ago by dante

(In [13661]) refs #6664 - all "public" dijit tests using _testCommon now use tundra as the default theme. WARNING: this currently breaks tests in nihilo / soria where there is a dijit.byId('something') in an addOnLoad because we're stalling parsing (not in the tundra case ... ) by a few ms. we probably don't need to, but we're fighting an inconsistant css race condition.

need to do dijit/demos, and dojox/* still ... that's next. please svn up dijit/ and check for any regressions based on this checkin. I've tested in ie6, ff2 and safari _most_ every test, but it is admittedly a little overwhelming so extra eyes would be appreciated.

Changed 14 months ago by dante

(In [13662]) refs #6664 - missed tree innard tests, and removes duplicated styles from matrix.

Changed 14 months ago by dante

(In [13663]) refs #6664 - hit all the demos.

Changed 14 months ago by dante

(In [13665]) refs #6664 - hit dojox test files using _testCommon.

Changed 14 months ago by dante

  • status changed from assigned to closed
  • resolution set to fixed

(In [13666]) fixes #6664 - hit every test in dijit and dojox. please reopen if I missed something. I will fix immediately.

Changed 14 months ago by dante

  • status changed from closed to reopened
  • resolution deleted

I forgot dojox/grid/* ...

Changed 14 months ago by dante

  • status changed from reopened to closed
  • resolution set to fixed

(In [13680]) re fixes #6664 - grid was the last little piece to get.

Changed 14 months ago by dante

(In [13694]) refs #6664 - i knew the delayed parsing in the nihilo / soria case would break something. the jist is: when using nihilo / soria, parsing is stalled by 120ms, though it may not have to be. This doesn't affect tundra at all, as no magic is happening there. fixes #6732

Changed 14 months ago by dante

(In [13697]) refs #6664 - accidentally copied the wrong "default dojo.js line" for root dijit tests. tests needing extra locales should be in i18n/ folder. (for clarity)

Changed 14 months ago by peller

(In [13700]) Move Calendar tests using non-default locales to tests/i18n. Refs #6664

Changed 14 months ago by bill

(In [13778]) Remove duplicate import. Refs #6664.

Changed 13 months ago by bill

(In [13944]) <link ... /> is invalid, should be <link ... > (unless you are writing XHTML). Refs #6664, #6887.

Changed 13 months ago by bill

(In [14047]) Make test_TabContainer?theme=soria work. Refs #6664.

Changed 13 months ago by peller

(In [14117]) Put back djConfig.extraLocale definition, since there is a test which explicitly runs in de-de locale. Thanks, Bill, for noticing. Refs #6664

Changed 12 months ago by peller

(In [14190]) Load locales used in this test in the bootstrap, removed in [13661]. Refs #6664

Note: See TracTickets for help on using tickets.