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 comment:1
owner: | → rwaldron |
---|---|
status: | new → assigned |
Changed October 27, 2011 07:01PM UTC by comment:3
component: | unfiled → core |
---|---|
priority: | undecided → low |
The regular expression +\\s+\\(+[a-z] doesn't exist.
Changed October 27, 2011 07:03PM UTC by comment:4
options.duration == Infinity
Will never accidentally match anything else
Changed October 27, 2011 07:07PM UTC by comment:5
Changed October 27, 2011 07:49PM UTC by comment:6
resolution: | → fixed |
---|---|
status: | assigned → closed |
Cleanup spacing. Fixes #10601
Changeset: 9ff4f26cf9165378347455f92ffaec3c3d6d460b
Changed October 27, 2011 07:49PM UTC by comment:7
Cleanup spacing in core.js. Fixes #10601
Changeset: da062c54f40bdeaf0dbd726a7930d191ac893079
Changed October 27, 2011 07:49PM UTC by comment:8
Cleanup spacing in ajax.js. Fixes #10601
Changeset: bad83b39cea9317b1c12673b1b5707a84c3feb9f
Changed October 27, 2011 07:49PM UTC by comment:9
Cleanup spacing in attributes.js. Fixes #10601
Changeset: 9ea21faf8feaae78fa6391bb9fb777fd8c4b0d26
Changed October 27, 2011 07:49PM UTC by comment:10
Cleanup spacing in css.js. Fixes #10601
Changeset: 4fb7202e0a96c03d3571ffddc3c57481226b0ae0
Changed October 27, 2011 07:50PM UTC by comment:11
Cleanup spacing in effects.js. Fixes #10601
Changeset: 0b4ac1d2be17a35edaba2609ae61469714f922e3
Changed October 27, 2011 07:50PM UTC by comment:12
Cleanup spacing in events.js. Fixes #10601
Changeset: f0cf49fee2201718a56fe09d7a8be641f7775dc3
Changed October 27, 2011 07:50PM UTC by comment:13
Cleanup spacing in manipulation.js. Fixes #10601
Changeset: 0804cdfe9a6e89cf8277971feac1f4322346a750
Changed October 27, 2011 07:50PM UTC by comment:14
Cleanup spacing in offset.js. Fixes #10601
Changeset: 33e98834fe209ab615379b0dc3e50976ebda283a
Changed October 27, 2011 07:50PM UTC by comment:15
Cleanup spacing in outro.js. Fixes #10601
Changeset: d00e29a65eb299ded5896bd84f413b2215548bd5
Changed October 27, 2011 07:50PM UTC by comment:16
Cleanup spacing in queue.js. Fixes #10601
Changeset: 79875457c08451033d1a618cb7d3984c8055fc3a
Changed October 27, 2011 07:50PM UTC by comment:17
Cleanup spacing in support.js. Fixes #10601
Changeset: b723d49d0d465410128f22bd27f4fc9a35023be0
Changed October 27, 2011 07:50PM UTC by comment:18
Cleanup spacing in traversing.js. Fixes #10601
Changeset: 27d099d47168d823430a1709946263f849639d4c
Changed October 27, 2011 07:50PM UTC by comment:19
Changed October 29, 2011 01:38PM UTC by comment:20
Forgot spacing for stuff like
jQuery.clone(ret.fragment)
Changed October 29, 2011 02:19PM UTC by comment:21
jQuery.clone(ret.fragment)
is inside of parens, which is allowed.