Skip to content

Setup dev environment#159

Merged
10 commits merged intomainfrom
feature/dev-env
Apr 24, 2026
Merged

Setup dev environment#159
10 commits merged intomainfrom
feature/dev-env

Conversation

@SharonStrats
Copy link
Copy Markdown
Contributor

Setup dev environment so I can test when changing styles.

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

Adds a local webpack-based development harness for the folder-pane package, enabling running the pane standalone in a browser with CSS support and Solid login UI.

Changes:

  • Introduces webpack dev server config + shared module rules (JS/TS via Babel, CSS + CSS modules, .ttl as source assets).
  • Adds a dev/ standalone app (HTML/JS/context stub) and a global CSS file for local styling overrides.
  • Adds npm start and required webpack-related dev dependencies.

Reviewed changes

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

Show a summary per file
File Description
webpack.module.rules.mjs Defines loader rules for JS/TS, CSS (incl. CSS modules), and .ttl assets.
webpack.dev.config.mjs Adds a development webpack config using HtmlWebpackPlugin + Node polyfills and module rules.
package.json Adds start script and webpack-related devDependencies.
dev/index.js Implements the dev entrypoint rendering folderPane and wiring a minimal outliner adapter + login status UI.
dev/index.html Provides the dev page template used by HtmlWebpackPlugin.
dev/dev-global.css Adds dev-only global styling overrides and accessibility-focused base styles.
dev/context.js Provides a minimal context object and fetcher for the dev harness.

💡 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.

Comment thread dev/context.js
Comment thread dev/index.js Outdated
Comment thread dev/index.js
Comment thread package.json
Comment thread webpack.dev.config.mjs Outdated
SharonStrats and others added 5 commits March 13, 2026 21:18
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@timea-solid timea-solid left a comment

Choose a reason for hiding this comment

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

Looks great.

@timea-solid timea-solid moved this from In review to Ready in SolidOS NLNet UI Mar 29, 2026
@bourgeoa bourgeoa closed this pull request by merging all changes into main in 1687fa7 Apr 24, 2026
@github-project-automation github-project-automation Bot moved this from Ready to Done in SolidOS NLNet UI Apr 24, 2026
@SharonStrats SharonStrats deleted the feature/dev-env branch April 24, 2026 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants