Skip to content

Cleaning up#460

Merged
agarny merged 4 commits intoopencor:mainfrom
agarny:cleaning-up
Mar 17, 2026
Merged

Cleaning up#460
agarny merged 4 commits intoopencor:mainfrom
agarny:cleaning-up

Conversation

@agarny
Copy link
Copy Markdown
Contributor

@agarny agarny commented Mar 17, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 17, 2026 21:15
Copy link
Copy Markdown

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 renderer library build’s CSS layer post-processing and improves the drag-and-drop overlay UX, alongside a dependency/version bump.

Changes:

  • Replace regex-based @layer stripping/unwrapping in the library build with a PostCSS AST-based implementation.
  • Add a fade transition for the drag-and-drop overlay and add a subtle highlight background to the overlay component.
  • Bump package versions and add postcss as a dev dependency (plus lockfile updates).

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/renderer/vite.lib.config.ts Switch CSS @layer post-processing to PostCSS AST traversal in the library build.
src/renderer/src/components/OpenCOR.vue Wrap drag-and-drop overlay in a Vue <Transition> and switch to conditional mount (v-if).
src/renderer/src/components/DragNDropComponent.vue Add a highlight background via a new scoped CSS class.
src/renderer/package.json Bump version and add postcss devDependency for renderer package tooling.
src/renderer/bun.lock Lockfile updates reflecting new dependency and transitive changes.
package.json Bump version and add postcss devDependency at repo root.
bun.lock Lockfile updates reflecting new dependency and transitive changes.

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

You can also share your feedback on Copilot code review. Take the survey.

@agarny agarny merged commit 0332e71 into opencor:main Mar 17, 2026
13 checks passed
@agarny agarny deleted the cleaning-up branch March 17, 2026 21:36
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