Skip to content

Allow changing location of the Cockpit folder#2425

Merged
rafaellehmkuhl merged 3 commits into
bluerobotics:masterfrom
rafaellehmkuhl:allow-changing-cockpit-folder
Feb 27, 2026
Merged

Allow changing location of the Cockpit folder#2425
rafaellehmkuhl merged 3 commits into
bluerobotics:masterfrom
rafaellehmkuhl:allow-changing-cockpit-folder

Conversation

@rafaellehmkuhl
Copy link
Copy Markdown
Member

This patch adds a long-waited feature on Cockpit that is the ability to change the default Cockpit folder location.

It adds a new section under /menu/settings/general which shows the current folder location as well as allow changing it. The user is allowed to change it to any desired folder, but if they choose one that does not contains/is called "Cockpit", we ask them if they really want to use the bare one or if they want to use/create a "Cockpit" one under it.

This feature can specially help users that want to use a separate disk for storing heavy stuff like videos and snapshots.

Cap.2026-02-26.at.13.58.42.mp4

Fix #1780.

@rafaellehmkuhl rafaellehmkuhl changed the title Allow changing cockpit folder Allow changing location of the Cockpit folder Feb 26, 2026
Copy link
Copy Markdown
Contributor

@ArturoManzoli ArturoManzoli left a comment

Choose a reason for hiding this comment

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

Everything seems to be working well!

I would suggest that you swap the position of these two elements, keeping the show tutorial manage settings and enable pirate mode right next to the user selection part

Image

@ES-Alexander ES-Alexander added the fast-track Solves an important user complaint - only block for major concerns label Feb 26, 2026
Copy link
Copy Markdown
Contributor

@ES-Alexander ES-Alexander left a comment

Choose a reason for hiding this comment

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

Works fine on macOS - tested video, subtitle, and snapshots 👍

I think it would be better if the interface was more like the other configs on the same page:

  • Show the folder name in a text entry, which can be edited directly
    • We could allow typing/pasting a file path directly into it, with an extra button on the right to open a file selector
    • Alternatively clicking to type (entering focus) could automatically open a file selector
  • Have a reset button (with a tooltip for its purpose) that is only enabled when the folder name is non-default
    • I'm not a fan of the long small clickable text in the current implementation, it's hard to read, hard to click on, and not obvious it even can be clicked on
Image

I think those points are important, but not blockers for this fast-tracked PR if they're overly annoying to do (especially since we're already planning an update to encourage users to select their location of choice).

@rafaellehmkuhl rafaellehmkuhl force-pushed the allow-changing-cockpit-folder branch from 3c18222 to ecfde4e Compare February 26, 2026 20:40
@rafaellehmkuhl
Copy link
Copy Markdown
Member Author

Everything seems to be working well!

I would suggest that you swap the position of these two elements, keeping the show tutorial manage settings and enable pirate mode right next to the user selection part

Done!

@rafaellehmkuhl
Copy link
Copy Markdown
Member Author

I think it would be better if the interface was more like the other configs on the same page:

  • Show the folder name in a text entry, which can be edited directly

    • We could allow typing/pasting a file path directly into it, with an extra button on the right to open a file selector
    • Alternatively clicking to type (entering focus) could automatically open a file selector
  • Have a reset button (with a tooltip for its purpose) that is only enabled when the folder name is non-default

    • I'm not a fan of the long small clickable text in the current implementation, it's hard to read, hard to click on, and not obvious it even can be clicked on

It's a valid discussion. I've opened #2426 for us to better discuss it.

@rafaellehmkuhl rafaellehmkuhl force-pushed the allow-changing-cockpit-folder branch from ecfde4e to 09ab5bd Compare February 26, 2026 20:49
@rafaellehmkuhl rafaellehmkuhl force-pushed the allow-changing-cockpit-folder branch 2 times, most recently from 03a8f9c to 7fef019 Compare February 27, 2026 13:54
@rafaellehmkuhl
Copy link
Copy Markdown
Member Author

@ES-Alexander @ArturoManzoli I have changed only the UI to look more like what Eliot suggested. Can I merge it like this?

image

@rafaellehmkuhl rafaellehmkuhl force-pushed the allow-changing-cockpit-folder branch from 7fef019 to 0f6ef89 Compare February 27, 2026 14:19
@ArturoManzoli
Copy link
Copy Markdown
Contributor

@ES-Alexander @ArturoManzoli I have changed only the UI to look more like what Eliot suggested. Can I merge it like this?

image

Yes, no problem

@rafaellehmkuhl rafaellehmkuhl merged commit f9dee10 into bluerobotics:master Feb 27, 2026
11 checks passed
@rafaellehmkuhl rafaellehmkuhl deleted the allow-changing-cockpit-folder branch February 27, 2026 15:52
@ES-Alexander ES-Alexander added the docs-needed Change needs to be documented label Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-needed Change needs to be documented fast-track Solves an important user complaint - only block for major concerns

Projects

None yet

Development

Successfully merging this pull request may close these issues.

frontend: standalone video folder should be moved to /Documents directory

3 participants