-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
b602ad1
update deps
ingoau 74f2e99
Merge pull request #442 from EducationalTools/updates
ingoau fed3806
Bump paneforge from 1.0.0-next.5 to 1.0.2
dependabot[bot] c3b7f1e
Bump tailwind-variants from 1.0.0 to 2.1.0
dependabot[bot] d6d77cc
open sidebar on menu itme click
ingoau 003e0ab
fix typescript issue and remove unused code
ingoau 187692d
fix bullet points issue specifically on chrome when on certain hostnames
ingoau 718fdd0
[With AI] add stopwatch and timer
ingoau ff10d95
remvoe keyboard shortcuts
ingoau 59f6aa1
Merge pull request #443 from EducationalTools/sidebar-improvements
ingoau 20ae0fe
Merge pull request #444 from EducationalTools/more-tools
ingoau fdf5cbd
Initial plan
Copilot 41d4c93
Initial plan: Fix isActive condition in app-sidebar.svelte
Copilot ab4e993
Fix isActive condition for sidebar sub-items to use item.url instead …
Copilot c8fb20a
Merge pull request #445 from EducationalTools/copilot/fix-677f45b7-76…
ingoau 8cd6be2
Merge pull request #432 from EducationalTools/dependabot/npm_and_yarn…
ingoau 79879d1
Merge pull request #434 from EducationalTools/dependabot/npm_and_yarn…
ingoau fe805a9
add stopwatch and timer to menu
ingoau c426c8b
fix mobile experience
ingoau a88d5b9
update shadcn components
ingoau c461576
use drawer in sidebar
ingoau 1dc389d
hide sidebar button on mobile
ingoau 6bcb759
fix
ingoau 3c6e65e
change sidebar to match new aesthetic
ingoau 4835895
fix some issues
ingoau 452272d
fix some issues
ingoau df8ff2f
fix issue again...
ingoau 82e9442
start adding animations
ingoau 374df71
fix mobile padding
ingoau 0d1d052
make padding more consistent
ingoau 0f3a5eb
revert one change
ingoau f6d527b
change particles configuration
ingoau cf591a6
change colors
ingoau fb386f9
change styling and add border
ingoau 8bdaba4
change styling a bit
ingoau 56ffe64
revert some stuff that could break
ingoau 84ef9ed
add border to mobile menu
ingoau 04a1e36
change mobile menu styling
ingoau 2d4a747
fix issue with mobile sidebar spacing
ingoau f808997
change styling on gmae page a bit
ingoau f57ddc9
change button radius
ingoau 3ff2dfe
fix specific issue
ingoau 7058a0e
replace comment with bug report
ingoau 66d3465
show 404 when not found
ingoau b6d56f0
remove unused import
ingoau 58497ef
add branch name and make sidebar wider
ingoau c498082
Update src/lib/components/play.svelte
ingoau 8a1d431
Merge branch 'main' of https://github.com/EducationalTools/src
ingoau 5637bf4
Fix incorrect import of Bug icon from Lucide
ingoau a3fff22
load slim particles.js
ingoau 08fcfc0
delete unused code
ingoau File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
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.
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:
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:
…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
🤖 Prompt for AI Agents