Skip to content

refactor: reduce project bar size - #1679

Merged
maxelkins merged 8 commits into
mainfrom
reduce-project-bar-size
Oct 19, 2026
Merged

refactor: reduce project bar size#1679
maxelkins merged 8 commits into
mainfrom
reduce-project-bar-size

Conversation

@maxelkins

@maxelkins maxelkins commented Oct 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closes: https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/1863
Closes: https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/1877
Closes: https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/1878

Summary

  • reduce the project bar footprint and share its layout and behaviour across standard and Scratch projects
  • use configurable design-system controls for save, download, upload, and project-name actions
  • switch project actions to their accessible icon-only variants when the project bar is narrow while preserving natural wrapping for longer labels
  • add a design-system project-name field to the Projects sidebar, sharing its editing behaviour with the project bar and including dark-mode read-only theming
  • standardise project-bar, save-status, and offline-indicator spacing and colour tokens

Screenshots

Normal user / teacher

Screenshot.2026-10-16.at.12.23.34.mp4

Student

Screenshot.2026-10-16.at.12.27.36.mp4

@maxelkins
maxelkins force-pushed the reduce-project-bar-size branch from 94a29bb to 1e4beba Compare October 14, 2026 14:37
maxelkins
maxelkins commented Oct 14, 2026
View reviewed changes
Comment thread web-component.html Resolved
@maxelkins
maxelkins force-pushed the reduce-project-bar-size branch from 9923bc5 to d59b8c6 Compare October 16, 2026 10:38
@maxelkins
maxelkins force-pushed the reduce-project-bar-size branch from e78c637 to b2fd4a6 Compare October 16, 2026 11:41
@maxelkins
maxelkins force-pushed the reduce-project-bar-size branch from b2fd4a6 to ee56c9d Compare October 16, 2026 12:01
@maxelkins
maxelkins marked this pull request as ready for review October 16, 2026 13:58
maxelkins
maxelkins commented Oct 16, 2026
View reviewed changes
Comment thread src/components/SaveButton/SaveButton.jsx
const previewMode = usePreviewMode();
const isOnline = useIsOnline();

useEffect(() => {

@maxelkins maxelkins Oct 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

SaveButton no longer checks webComponent because the editor now only runs as a web component. This is legacy code

@zetter-rpf
zetter-rpf self-requested a review October 19, 2026 08:48
zetter-rpf
zetter-rpf approved these changes Oct 19, 2026
View reviewed changes

@zetter-rpf zetter-rpf 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.

Nice one, thanks for tidying up things and splitting out into separate commits made it easy to review

@maxelkins
maxelkins merged commit 835e11a into main Oct 19, 2026
12 checks passed
@maxelkins
maxelkins deleted the reduce-project-bar-size branch October 19, 2026 14:26

This branch was successfully deployed

1 active deployment
previews/1679/merge — ddfeaddb Deployed Oct 19, 2026 by maxelkins via deploy-branch / build-deploy #6943
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zetter-rpf zetter-rpf zetter-rpf approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@maxelkins @zetter-rpf