Skip to content

refactor: simplify sidepanel and remove notes module#25

Merged
Rawdyrathaur merged 7 commits into
mainfrom
refactor/sidepanel-updates
May 25, 2026
Merged

refactor: simplify sidepanel and remove notes module#25
Rawdyrathaur merged 7 commits into
mainfrom
refactor/sidepanel-updates

Conversation

@Rawdyrathaur
Copy link
Copy Markdown
Owner

  • Remove notes feature (App.tsx, NoteEditor, Toolbar, index.html, main.tsx)
  • Remove StickyNotes and StudyFolderPanel components
  • Add EmptyState component
  • Update sidepanel App and TabList components
  • Update manifest and vite config

- Remove notes feature (App.tsx, NoteEditor, Toolbar, index.html, main.tsx)
- Remove StickyNotes and StudyFolderPanel components
- Add EmptyState component
- Update sidepanel App and TabList components
- Update manifest and vite config

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors the extension toward a simplified sidepanel-focused tab manager by removing the standalone notes/study-folder modules and adding a new empty-state experience plus bulk tab actions.

Changes:

  • Removed the standalone notes entry point/components and sidepanel Sticky Notes/Study Folder components.
  • Added EmptyState and updated sidepanel App navigation/content flow.
  • Updated manifest/Vite inputs and adjusted tab list/button styling.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tab-story/vite.config.ts Removes the notes HTML entry from extension build inputs.
tab-story/manifest.json Removes notes web-accessible resource and adds favicon resource/permission.
tab-story/src/sidepanel/App.tsx Refactors sidepanel navigation, empty state, and bulk save/delete actions.
tab-story/src/sidepanel/main.tsx Imports DB and exposes it on window.
tab-story/src/sidepanel/components/EmptyState.tsx Adds the empty-state UI for no saved tabs.
tab-story/src/sidepanel/components/TabList.tsx Adjusts list action button sizing/spacing.
tab-story/src/sidepanel/styles/global.css Adds tooltip positioning for below-oriented toolbar buttons.
tab-story/src/sidepanel/components/StickyNotes.tsx Removes the Sticky Notes sidepanel component.
tab-story/src/sidepanel/components/StudyFolderPanel.tsx Removes the Study Folder panel component.
tab-story/src/notes/App.tsx Removes standalone notes app.
tab-story/src/notes/main.tsx Removes standalone notes entry point.
tab-story/src/notes/index.html Removes standalone notes HTML entry.
tab-story/src/notes/components/NoteEditor.tsx Removes rich notes editor.
tab-story/src/notes/components/Toolbar.tsx Removes notes toolbar.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tab-story/src/sidepanel/main.tsx Outdated
Comment thread tab-story/src/sidepanel/App.tsx Outdated
Comment thread tab-story/src/sidepanel/App.tsx Outdated
Comment thread tab-story/src/sidepanel/styles/global.css Outdated
Rawdyrathaur and others added 6 commits May 26, 2026 01:48
- Add queries configuration to prevent auto-detection of unsupported languages
- Restrict to JavaScript and TypeScript only

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
fix this

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
fix it

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
fix it

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
fix it

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
- Remove .github/codeql-config.yml
- Remove .github/workflows/codeql.yml
- CodeQL not needed for this TypeScript/React project

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Rawdyrathaur Rawdyrathaur merged commit 923043e into main May 25, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants