Skip to content

Commit 9443688

Browse files
authored
fix(modeling): various fixes and performance tweaks to modeling
* feat(modeling): allow appendArc, appendBezier, appendPoints to closed paths * fix(modeling): corrected snap of path2 and path3 when closed * docs(modeling): changes from reviews * fix(modeling): small fix to parameter checks * perf(modeling): small performace changes * fix(modeling): corrected close, reverse, and transform functions in geometries * perf(modeling): changed extrudeLinearGeom2 to use preallocated mat4 and vec3 * build(all): changed linting to use eslint directly * style(all): changes based on lint * style(all): small fixes for consistent return values * style(all): changes for consistent default case statements * style(all): changes for consistent assignment statements * style(all): changes for consistent blocks
1 parent 7c4b445 commit 9443688

59 files changed

Lines changed: 1571 additions & 4307 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

eslint.config.js

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
import js from "@eslint/js"
2+
import globals from "globals"
3+
import { defineConfig, globalIgnores } from "eslint/config"
4+
5+
export default defineConfig([
6+
globalIgnores(["**/dist/", "packages/desktop", "packages/io/scad-deserializer", "packages/vtree"]),
7+
{
8+
files: ["**/*.{js}"],
9+
plugins: { js },
10+
extends: ["js/recommended"],
11+
languageOptions: { globals: {...globals.browser, ...globals.node} }
12+
},
13+
{
14+
"rules": {
15+
// enforce consistent arrow functions
16+
"func-style": [
17+
"error",
18+
"expression"
19+
],
20+
"arrow-body-style": [
21+
"error",
22+
"as-needed"
23+
],
24+
"arrow-parens": [
25+
"error",
26+
"always"
27+
],
28+
"arrow-spacing": "error",
29+
"prefer-arrow-callback": "error",
30+
31+
// enforce consistent code
32+
"consistent-return": ["error"],
33+
"curly": ["error", "multi-line"],
34+
"default-case": ["error"],
35+
"eqeqeq": ["error", "smart"],
36+
"no-array-constructor": ["error"],
37+
// "no-loop-func": ["error"],
38+
"no-multi-assign": ["error"],
39+
// "no-param-reassign": ["error"],
40+
"no-var": ["error"],
41+
}
42+
}
43+
])

jsdoc/assets/static/jscad-docs.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
function addOpenClassToOpenNavItems () {
1+
const addOpenClassToOpenNavItems = () => {
22
const file = window.location.pathname.split('/').pop().replace(/\.html/, '')
33
document.querySelectorAll('nav > ul > li > a').forEach((parent) => {
44
if (parent.parentNode.querySelectorAll('ul li').length) {

0 commit comments

Comments
 (0)