Skip to content

Restructure table to div - #472

Merged
SharonStrats merged 5 commits into
stagingfrom
task/restructure
Oct 23, 2026
Merged

SharonStrats merged 5 commits into
stagingfrom
task/restructure

Conversation

@SharonStrats

@SharonStrats SharonStrats commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Change the table to div and some addtional styles to improve the layout and also the status area colors.

Copilot AI balanced review requested due to automatic review settings October 22, 2026 06:49
@SharonStrats SharonStrats self-assigned this Oct 22, 2026
@SharonStrats SharonStrats moved this to In review in SolidOS NLNet UI Oct 22, 2026

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.

🟡 Changes recommended

Hiding root horizontal overflow while retaining a 375px minimum width makes content unreachable on narrower viewports.

1 open issue
What changed in this PR

This PR updates Mashlib’s shared stylesheet to adjust page overflow and give status messages a subtler appearance.

Changes:

  • Hides horizontal overflow on the root and body while retaining vertical scrolling.
  • Restyles status messages with full-width sizing, softer backgrounds, and a light shadow.
File Description
src/​styles/​mash.css Updates document overflow and status-message styling.

Review effort: Balanced

Comment thread src/styles/mash.css
@SharonStrats SharonStrats changed the title style improvements Restructure table to div Oct 22, 2026
…@5.0.2-2 pane-registry@5.0.2-0 solid-panes@5.1.0-1) (latest: rdflib@2.4.1)
…@5.0.2-3 pane-registry@5.0.2-1 solid-panes@5.1.0-2) (latest: rdflib@2.4.1)
@SharonStrats
SharonStrats merged commit dfdba85 into staging Oct 23, 2026
9 checks passed
@SharonStrats
SharonStrats deleted the task/restructure branch October 23, 2026 08:40
@github-project-automation github-project-automation Bot moved this from In review to Done in SolidOS NLNet UI Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants