Skip to content

add outliner from folder pane#7

Merged
SharonStrats merged 4 commits into
mainfrom
feat/richer-outliner
Jul 16, 2026
Merged

add outliner from folder pane#7
SharonStrats merged 4 commits into
mainfrom
feat/richer-outliner

Conversation

@SharonStrats

@SharonStrats SharonStrats commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

I have tested this locally for source-pane, contacts-pane, folder-pane of course, and also solid-panes

I did get a babel error but i think it was do to local symlinking.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 minimal getOutliner implementation to the pane sandbox context so panes (notably folder-pane) can render outliner-driven elements when running in the sandbox.

Changes:

  • Provides a sandbox getOutliner implementation that can render simple table rows/cells and subject/object links.
  • Updates sandbox context creation to use the new getOutliner instead of returning null.
  • Adds solid-ui utils import (used for labeling nodes).

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

Comment thread src/pane-sandbox.ts Outdated
Comment thread src/pane-sandbox.ts Outdated
Comment thread src/pane-sandbox.ts Outdated
SharonStrats and others added 2 commits July 16, 2026 20:36
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>

@NoelDeMartin NoelDeMartin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It looks good to me, only check the copilot review for some improvements.

@SharonStrats SharonStrats moved this from In review to Ready in SolidOS NLNet UI Jul 16, 2026
@SharonStrats
SharonStrats merged commit 761b46f into main Jul 16, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Ready to Done in SolidOS NLNet UI Jul 16, 2026
@SharonStrats
SharonStrats deleted the feat/richer-outliner branch July 16, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants