Skip to main content

Bug Tracker

Side navigation

#10601 closed bug (fixed)

Opened October 27, 2011 06:41PM UTC

Closed October 27, 2011 07:49PM UTC

Last modified October 29, 2011 02:19PM UTC

nitpicks

Reported by: anonymous Owned by: rwaldron
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

Attachments (0)
Change History (21)

Changed October 27, 2011 06:57PM UTC by rwaldron comment:1

owner: → rwaldron
status: newassigned

Changed October 27, 2011 06:58PM UTC by rwaldron comment:2

#10602 is a duplicate of this ticket.

Changed October 27, 2011 07:01PM UTC by rwaldron comment:3

component: unfiledcore
priority: undecidedlow

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

Changed October 27, 2011 07:03PM UTC by rwaldron comment:4

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

Changed October 27, 2011 07:07PM UTC by rwaldron comment:5

Changed October 27, 2011 07:49PM UTC by Rick Waldron comment:6

resolution: → fixed
status: assignedclosed

Cleanup spacing. Fixes #10601

Changeset: 9ff4f26cf9165378347455f92ffaec3c3d6d460b

Changed October 27, 2011 07:49PM UTC by Rick Waldron comment:7

Cleanup spacing in core.js. Fixes #10601

Changeset: da062c54f40bdeaf0dbd726a7930d191ac893079

Changed October 27, 2011 07:49PM UTC by Rick Waldron comment:8

Cleanup spacing in ajax.js. Fixes #10601

Changeset: bad83b39cea9317b1c12673b1b5707a84c3feb9f

Changed October 27, 2011 07:49PM UTC by Rick Waldron comment:9

Cleanup spacing in attributes.js. Fixes #10601

Changeset: 9ea21faf8feaae78fa6391bb9fb777fd8c4b0d26

Changed October 27, 2011 07:49PM UTC by Rick Waldron comment:10

Cleanup spacing in css.js. Fixes #10601

Changeset: 4fb7202e0a96c03d3571ffddc3c57481226b0ae0

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:11

Cleanup spacing in effects.js. Fixes #10601

Changeset: 0b4ac1d2be17a35edaba2609ae61469714f922e3

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:12

Cleanup spacing in events.js. Fixes #10601

Changeset: f0cf49fee2201718a56fe09d7a8be641f7775dc3

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:13

Cleanup spacing in manipulation.js. Fixes #10601

Changeset: 0804cdfe9a6e89cf8277971feac1f4322346a750

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:14

Cleanup spacing in offset.js. Fixes #10601

Changeset: 33e98834fe209ab615379b0dc3e50976ebda283a

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:15

Cleanup spacing in outro.js. Fixes #10601

Changeset: d00e29a65eb299ded5896bd84f413b2215548bd5

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:16

Cleanup spacing in queue.js. Fixes #10601

Changeset: 79875457c08451033d1a618cb7d3984c8055fc3a

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:17

Cleanup spacing in support.js. Fixes #10601

Changeset: b723d49d0d465410128f22bd27f4fc9a35023be0

Changed October 27, 2011 07:50PM UTC by Rick Waldron comment:18

Cleanup spacing in traversing.js. Fixes #10601

Changeset: 27d099d47168d823430a1709946263f849639d4c

Changed October 27, 2011 07:50PM UTC by Dave Methvin comment:19

Merge pull request #565 from rwldrn/10601

Cleanup spacing. Fixes #10601

Changeset: 736d4d770633720493afe1fefd6f135095c5b5f7

Changed October 29, 2011 01:38PM UTC by anonymous comment:20

Forgot spacing for stuff like

jQuery.clone(ret.fragment)

Changed October 29, 2011 02:19PM UTC by rwaldron comment:21

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