Skip to content

updates, minor redesign, sidebar improvements, new tool, and more #447

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ingoau merged 51 commits into from
Aug 11, 2025
Merged

updates, minor redesign, sidebar improvements, new tool, and more #447

Show file tree
Hide file tree
Changes from 48 commits
Commits
Show all changes
51 commits
Select commit Hold shift + click to select a range
b602ad1
update deps
ingoau Aug 8, 2025
74f2e99
Merge pull request #442 from EducationalTools/updates
ingoau Aug 8, 2025
fed3806
Bump paneforge from 1.0.0-next.5 to 1.0.2
dependabot[bot] Aug 8, 2025
c3b7f1e
Bump tailwind-variants from 1.0.0 to 2.1.0
dependabot[bot] Aug 8, 2025
d6d77cc
open sidebar on menu itme click
ingoau Aug 8, 2025
003e0ab
fix typescript issue and remove unused code
ingoau Aug 8, 2025
187692d
fix bullet points issue specifically on chrome when on certain hostnames
ingoau Aug 8, 2025
718fdd0
[With AI] add stopwatch and timer
ingoau Aug 8, 2025
ff10d95
remvoe keyboard shortcuts
ingoau Aug 8, 2025
59f6aa1
Merge pull request #443 from EducationalTools/sidebar-improvements
ingoau Aug 8, 2025
20ae0fe
Merge pull request #444 from EducationalTools/more-tools
ingoau Aug 8, 2025
fdf5cbd
Initial plan
Copilot Aug 8, 2025
41d4c93
Initial plan: Fix isActive condition in app-sidebar.svelte
Copilot Aug 8, 2025
ab4e993
Fix isActive condition for sidebar sub-items to use item.url instead …
Copilot Aug 8, 2025
c8fb20a
Merge pull request #445 from EducationalTools/copilot/fix-677f45b7-76…
ingoau Aug 9, 2025
8cd6be2
Merge pull request #432 from EducationalTools/dependabot/npm_and_yarn…
ingoau Aug 9, 2025
79879d1
Merge pull request #434 from EducationalTools/dependabot/npm_and_yarn…
ingoau Aug 9, 2025
fe805a9
add stopwatch and timer to menu
ingoau Aug 9, 2025
c426c8b
fix mobile experience
ingoau Aug 9, 2025
a88d5b9
update shadcn components
ingoau Aug 9, 2025
c461576
use drawer in sidebar
ingoau Aug 9, 2025
1dc389d
hide sidebar button on mobile
ingoau Aug 9, 2025
6bcb759
fix
ingoau Aug 9, 2025
3c6e65e
change sidebar to match new aesthetic
ingoau Aug 9, 2025
4835895
fix some issues
ingoau Aug 9, 2025
452272d
fix some issues
ingoau Aug 9, 2025
df8ff2f
fix issue again...
ingoau Aug 9, 2025
82e9442
start adding animations
ingoau Aug 10, 2025
374df71
fix mobile padding
ingoau Aug 10, 2025
0d1d052
make padding more consistent
ingoau Aug 10, 2025
0f3a5eb
revert one change
ingoau Aug 10, 2025
f6d527b
change particles configuration
ingoau Aug 10, 2025
cf591a6
change colors
ingoau Aug 10, 2025
fb386f9
change styling and add border
ingoau Aug 10, 2025
8bdaba4
change styling a bit
ingoau Aug 10, 2025
56ffe64
revert some stuff that could break
ingoau Aug 10, 2025
84ef9ed
add border to mobile menu
ingoau Aug 10, 2025
04a1e36
change mobile menu styling
ingoau Aug 10, 2025
2d4a747
fix issue with mobile sidebar spacing
ingoau Aug 10, 2025
f808997
change styling on gmae page a bit
ingoau Aug 10, 2025
f57ddc9
change button radius
ingoau Aug 10, 2025
3ff2dfe
fix specific issue
ingoau Aug 10, 2025
7058a0e
replace comment with bug report
ingoau Aug 10, 2025
66d3465
show 404 when not found
ingoau Aug 10, 2025
b6d56f0
remove unused import
ingoau Aug 10, 2025
58497ef
add branch name and make sidebar wider
ingoau Aug 10, 2025
c498082
Update src/lib/components/play.svelte
ingoau Aug 11, 2025
8a1d431
Merge branch 'main' of https://github.com/EducationalTools/src
ingoau Aug 11, 2025
5637bf4
Fix incorrect import of Bug icon from Lucide
ingoau Aug 11, 2025
a3fff22
load slim particles.js
ingoau Aug 11, 2025
08fcfc0
delete unused code
ingoau Aug 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5,570 changes: 5,570 additions & 0 deletions package-lock.json
View file
Open in desktop

Large diffs are not rendered by default.

33 changes: 18 additions & 15 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,10 @@
},
"devDependencies": {
"@internationalized/date": "^3.8.2",
"@sveltejs/adapter-auto": "^6.0.1",
"@sveltejs/adapter-static": "^3.0.8",
"@sveltejs/kit": "^2.24.0",
"@lucide/svelte": "^0.515.0",
"@sveltejs/adapter-auto": "^6.0.2",
"@sveltejs/adapter-static": "^3.0.9",
"@sveltejs/kit": "^2.27.3",
Comment on lines +18 to +21
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Aug 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

🧩 Analysis chain

Move @lucide/svelte to dependencies (used at runtime/SSR), not devDependencies

@lucide/svelte is imported in app code (e.g., src/lib/components/app-sidebar.svelte). Keeping it in devDependencies can break production installs that only install dependencies (SSR/node adapters). Recommend moving it to dependencies.

-  "devDependencies": {
-    "@lucide/svelte": "^0.515.0",
+  "devDependencies": {
     ...
   },
   "dependencies": {
+    "@lucide/svelte": "^0.515.0",
     ...
   }

Optionally align the version with what worked previously (you removed ^0.525.0). If 0.515.0 is intentional due to compatibility, keep it; otherwise bump.

Verification script (finds usages and shows where the package is declared):

Also applies to: 51-68

🏁 Script executed:

#!/bin/bash
set -euo pipefail
echo "Usages of @lucide/svelte:"
rg -n "@lucide/svelte" -A 2 -B 2 || true
echo
echo "Declared in dependencies:"
jq -r '.dependencies["@lucide/svelte"] // empty' package.json
echo "Declared in devDependencies:"
jq -r '.devDependencies["@lucide/svelte"] // empty' package.json

Length of output: 32026

Move @lucide/svelte to dependencies
It’s imported in production/SSR code (e.g. many src/routes//.svelte and src/lib/.svelte/.ts files), but currently only in devDependencies. This will break installs that skip devDeps.

Locations using @lucide/svelte at runtime:

  • src/routes/**/+page.svelte (multiple icons)
  • src/lib/components/app-sidebar.svelte (GitBranch, ChevronRight, etc.)
  • src/lib/navigation.ts (Home, Wrench, Game, etc.)
    …among many others.

Suggested change in package.json:

   "devDependencies": {
-    "@lucide/svelte": "^0.515.0",
     …
   },
   "dependencies": {
+    "@lucide/svelte": "^0.515.0",
     …
   }

Keep the ^0.515.0 version to match your lockfile or bump if you need the latest.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"@lucide/svelte": "^0.515.0",
"@sveltejs/adapter-auto": "^6.0.2",
"@sveltejs/adapter-static": "^3.0.9",
"@sveltejs/kit": "^2.27.3",
"devDependencies": {
- "@lucide/svelte": "^0.515.0",
"@sveltejs/adapter-auto": "^6.0.2",
"@sveltejs/adapter-static": "^3.0.9",
"@sveltejs/kit": "^2.27.3",
},
"dependencies": {
+ "@lucide/svelte": "^0.515.0",
// …other dependencies
}
🤖 Prompt for AI Agents 
In package.json around lines 18 to 21, @lucide/svelte is listed under
devDependencies but is imported in runtime/SSR code; move it into the
"dependencies" section keeping the version ^0.515.0 (remove it from
devDependencies and add it to dependencies), then update the lockfile by running
the package manager install (e.g., npm install or npm install
@lucide/svelte@^0.515.0) so production installs include the package.

"@sveltejs/vite-plugin-svelte": "^5.1.1",
"@tailwindcss/typography": "^0.5.16",
"@tailwindcss/vite": "^4.1.11",
Expand All @@ -28,39 +29,41 @@
"bits-ui": "2.8.11",
"clsx": "^2.1.1",
"embla-carousel-svelte": "^8.6.0",
"layerchart": "2.0.0-next.31",
"layerchart": "2.0.0-next.27",
"mode-watcher": "^1.1.0",
"paneforge": "1.0.0-next.5",
"paneforge": "1.0.2",
"prettier": "^3.6.2",
"prettier-plugin-svelte": "^3.4.0",
"prettier-plugin-tailwindcss": "^0.6.14",
"svelte": "^5.36.2",
"svelte-check": "^4.3.0",
"svelte": "^5.38.0",
"svelte-check": "^4.3.1",
"svelte-sonner": "^1.0.5",
"sveltekit-superforms": "^2.27.1",
"tailwind-merge": "^3.3.1",
"tailwind-variants": "^1.0.0",
"tailwindcss": "^4.1.11",
"tw-animate-css": "^1.3.5",
"typescript": "^5.8.3",
"tw-animate-css": "^1.3.6",
"typescript": "^5.9.2",
"vaul-svelte": "1.0.0-next.7",
"vite": "^6.3.5"
},
"dependencies": {
"@clerk/backend": "^2.4.3",
"@clerk/themes": "^2.3.3",
"@clerk/backend": "^2.7.0",
"@clerk/themes": "^2.4.6",
"@friendofsvelte/tipex": "0.0.7",
"@lucide/svelte": "^0.525.0",
"@tanstack/match-sorter-utils": "^8.19.4",
"@tsparticles/svelte": "^3.1.1",
"convex": "^1.25.4",
"convex-svelte": "^0.0.11",
"current-git-branch": "^2.0.1",
"dayjs": "^1.11.13",
"formsnap": "^2.0.1",
"jose": "^6.0.12",
"jsonwebtoken": "^9.0.2",
"md5": "^2.3.0",
"posthog-js": "^1.257.0",
"svelte-clerk": "^0.16.0",
"svelte-persisted-store": "^0.12.0"
"posthog-js": "^1.259.0",
"svelte-clerk": "^0.16.5",
"svelte-persisted-store": "^0.12.0",
"tsparticles": "^3.9.1"
}
Comment on lines +51 to 69
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Aug 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

🧩 Analysis chain

current-git-branch belongs in devDependencies (build-time only)

It’s only used in vite.config.ts (build-time). Move to devDependencies to avoid shipping it in production images/lambdas.

-  "dependencies": {
-    ...
-    "current-git-branch": "^2.0.1",
-    ...
-  }
+  "dependencies": {
+    ...
+  },
+  "devDependencies": {
+    ...
+    "current-git-branch": "^2.0.1",
+    ...
+  }

Verification script:

Also applies to: 16-49

🏁 Script executed:

#!/bin/bash
set -euo pipefail
echo "Where current-git-branch is used:"
rg -n "current-git-branch" -A 2 -B 2 || true

Length of output: 1463

Move current-git-branch into devDependencies

current-git-branch is only imported in vite.config.ts (build-time), so it shouldn’t be shipped in production images or lambdas.

• Usage location
– vite.config.ts line 4: import branchName from 'current-git-branch';
• Change in package.json
– Relocate the "current-git-branch": "^2.0.1" entry from dependencies into devDependencies

-  "dependencies": {
-    ...
-    "current-git-branch": "^2.0.1",
-    ...
-  }
+  "dependencies": {
+    ...
+  },
+  "devDependencies": {
+    ...
+    "current-git-branch": "^2.0.1",
+    ...
+  }
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"@clerk/backend": "^2.7.0",
"@clerk/themes": "^2.4.6",
"@friendofsvelte/tipex": "0.0.7",
"@lucide/svelte": "^0.525.0",
"@tanstack/match-sorter-utils": "^8.19.4",
"@tsparticles/svelte": "^3.1.1",
"convex": "^1.25.4",
"convex-svelte": "^0.0.11",
"current-git-branch": "^2.0.1",
"dayjs": "^1.11.13",
"formsnap": "^2.0.1",
"jose": "^6.0.12",
"jsonwebtoken": "^9.0.2",
"md5": "^2.3.0",
"posthog-js": "^1.257.0",
"svelte-clerk": "^0.16.0",
"svelte-persisted-store": "^0.12.0"
"posthog-js": "^1.259.0",
"svelte-clerk": "^0.16.5",
"svelte-persisted-store": "^0.12.0",
"tsparticles": "^3.9.1"
}
"dependencies": {
"@clerk/backend": "^2.7.0",
"@clerk/themes": "^2.4.6",
"@friendofsvelte/tipex": "0.0.7",
"@tanstack/match-sorter-utils": "^8.19.4",
"@tsparticles/svelte": "^3.1.1",
"convex": "^1.25.4",
"convex-svelte": "^0.0.11",
"dayjs": "^1.11.13",
"formsnap": "^2.0.1",
"jose": "^6.0.12",
"jsonwebtoken": "^9.0.2",
"md5": "^2.3.0",
"posthog-js": "^1.259.0",
"svelte-clerk": "^0.16.5",
"svelte-persisted-store": "^0.12.0",
"tsparticles": "^3.9.1"
},
"devDependencies": {
"current-git-branch": "^2.0.1"
}
🤖 Prompt for AI Agents 
In package.json around lines 51 to 68, the "current-git-branch": "^2.0.1" entry
is currently listed under dependencies but is only used at build time in
vite.config.ts; remove the "current-git-branch" entry from the dependencies
object and add the same entry under devDependencies, then run your package
manager to update node_modules and the lockfile (npm install or yarn install) so
the lockfile reflects the move; ensure no other runtime code imports it and
commit the updated package.json and lockfile.

}
Loading