Bug Tracker

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#10601 closed bug (fixed)

nitpicks

Reported by: anonymous Owned by: Rick Waldron
Priority: low Milestone: None
Component: core Version: 1.7b2
Keywords: Cc:
Blocked by: Blocking:

Description

*There are three if statements that are not properly formatted (spaces missing). Try searching the regular expression if+\s+\(+[a-z] to find them.

*options.duration == Infinity could use a ===

*The spacing for (-Math.cos(p*Math.PI)/2) is lacking

Change History (21)

comment:1 Changed 7 years ago by Rick Waldron

Owner: set to Rick Waldron
Status: newassigned

comment:2 Changed 7 years ago by Rick Waldron

#10602 is a duplicate of this ticket.

comment:3 Changed 7 years ago by Rick Waldron

Component: unfiledcore
Priority: undecidedlow

The regular expression +\s+\(+[a-z] doesn't exist.

comment:4 Changed 7 years ago by Rick Waldron

options.duration == Infinity Will never accidentally match anything else

comment:6 Changed 7 years ago by Rick Waldron

Resolution: fixed
Status: assignedclosed

Cleanup spacing. Fixes #10601

Changeset: 9ff4f26cf9165378347455f92ffaec3c3d6d460b

comment:7 Changed 7 years ago by Rick Waldron

Cleanup spacing in core.js. Fixes #10601

Changeset: da062c54f40bdeaf0dbd726a7930d191ac893079

comment:8 Changed 7 years ago by Rick Waldron

Cleanup spacing in ajax.js. Fixes #10601

Changeset: bad83b39cea9317b1c12673b1b5707a84c3feb9f

comment:9 Changed 7 years ago by Rick Waldron

Cleanup spacing in attributes.js. Fixes #10601

Changeset: 9ea21faf8feaae78fa6391bb9fb777fd8c4b0d26

comment:10 Changed 7 years ago by Rick Waldron

Cleanup spacing in css.js. Fixes #10601

Changeset: 4fb7202e0a96c03d3571ffddc3c57481226b0ae0

comment:11 Changed 7 years ago by Rick Waldron

Cleanup spacing in effects.js. Fixes #10601

Changeset: 0b4ac1d2be17a35edaba2609ae61469714f922e3

comment:12 Changed 7 years ago by Rick Waldron

Cleanup spacing in events.js. Fixes #10601

Changeset: f0cf49fee2201718a56fe09d7a8be641f7775dc3

comment:13 Changed 7 years ago by Rick Waldron

Cleanup spacing in manipulation.js. Fixes #10601

Changeset: 0804cdfe9a6e89cf8277971feac1f4322346a750

comment:14 Changed 7 years ago by Rick Waldron

Cleanup spacing in offset.js. Fixes #10601

Changeset: 33e98834fe209ab615379b0dc3e50976ebda283a

comment:15 Changed 7 years ago by Rick Waldron

Cleanup spacing in outro.js. Fixes #10601

Changeset: d00e29a65eb299ded5896bd84f413b2215548bd5

comment:16 Changed 7 years ago by Rick Waldron

Cleanup spacing in queue.js. Fixes #10601

Changeset: 79875457c08451033d1a618cb7d3984c8055fc3a

comment:17 Changed 7 years ago by Rick Waldron

Cleanup spacing in support.js. Fixes #10601

Changeset: b723d49d0d465410128f22bd27f4fc9a35023be0

comment:18 Changed 7 years ago by Rick Waldron

Cleanup spacing in traversing.js. Fixes #10601

Changeset: 27d099d47168d823430a1709946263f849639d4c

comment:19 Changed 7 years ago by Dave Methvin

Merge pull request #565 from rwldrn/10601

Cleanup spacing. Fixes #10601

Changeset: 736d4d770633720493afe1fefd6f135095c5b5f7

comment:20 Changed 7 years ago by anonymous

Forgot spacing for stuff like

jQuery.clone(ret.fragment)

comment:21 Changed 7 years ago by Rick Waldron

jQuery.clone(ret.fragment) is inside of parens, which is allowed.

Note: See TracTickets for help on using tickets.