Ticket #7390 (new task)

Opened 18 months ago

Last modified 4 months ago

[meta] style guideline adherence

Reported by: dante Owned by: dante
Priority: normal Milestone: future
Component: General Version: 1.1.1
Severity: normal Keywords:
Cc:

Description

There are a lot of flubs in our own commitment to style guidelines. This meta ticket is to be used to commit small style guidelines changes. This is mostly for small nits, like spaces between if () and } else { etc.

the guidelines are defined at: http://dojotoolkit.org/developer/StyleGuide

Change History

Changed 18 months ago by dante

(In [14743]) refs #7390 - minor space and tab nits. !strict

Changed 18 months ago by dante

(In [14969]) refs #7390 - extra tabs in Button.js

Changed 15 months ago by elazutkin

In [15808] this line of _base/html.js:

af(node, 1).Enabled = (opacity == 1 ? false : true);

is replaced with this one:

af(node, 1).Enabled = !(opacity == 1);

Why not like this:

af(node, 1).Enabled = opacity != 1;

saving 3 characters and making it easier to understand.

Changed 13 months ago by dante

(In [16200]) refs #7390 - style cleanups

Changed 9 months ago by dante

(In [17513]) refs #7390 - super-minor style patch testing jslint warning

Changed 8 months ago by cb1kenobi

(In [17956]) Added whitespace to _isDocumentOk(), refs #7390.

Changed 8 months ago by bill

(In [17961]) Comment formatting as per dojo style guidelines, refs #7390 !strict.

Changed 8 months ago by bill

(In [17962]) Fix comment formatting as per dojo's style guidelines, and to make the doc parser work. Refs #7390 !strict.

Changed 4 months ago by doughays

(In [20519]) Refs #7390. Minor whitespace style cleanups in dijit/form

Note: See TracTickets for help on using tickets.