Skip to content
This repository was archived by the owner on Aug 31, 2026. It is now read-only.

feat: allow setting ace editor theme - #3826

Merged
hacdias merged 3 commits into
filebrowser:masterfrom
radam9:dev
Sep 25, 2025
Merged

feat: allow setting ace editor theme#3826
hacdias merged 3 commits into
filebrowser:masterfrom
radam9:dev

Conversation

@radam9

@radam9 radam9 commented Mar 25, 2025

Copy link
Copy Markdown
Contributor

Description
This PR resolves #3799, which gives users the ability to set the syntax highlighting theme for the ace editor.

The user can set the default ace editor theme in the config as follows:

{
    "defaults": {
      "aceEditorTheme": "ace/theme/one_dark"
    }
}

They can also set the theme from the Profile Settings page:
Screenshot from 2025-03-25 22-47-44
Screenshot from 2025-03-25 23-09-11

@radam9
radam9 requested a review from o1egl as a code owner March 25, 2025 22:11
@radam9 radam9 changed the title Dev feat: allow setting ace editor theme Mar 25, 2025
@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions Bot added the Stale label Apr 25, 2025
@radam9

radam9 commented Apr 25, 2025

Copy link
Copy Markdown
Contributor Author

bump

@github-actions github-actions Bot removed the Stale label Apr 26, 2025

@hacdias hacdias 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.

@radam9 thanks for this PR! If you're up for resolving the conflicts, I will gladly merge this.

# Conflicts:
#	cmd/users.go
#	frontend/src/views/files/Editor.vue
#	http/auth.go
@radam9

radam9 commented Sep 24, 2025

Copy link
Copy Markdown
Contributor Author

@radam9 thanks for this PR! If you're up for resolving the conflicts, I will gladly merge this.

@hacdias I just resolved the conflicts 👍🏻

@hacdias hacdias 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.

Thank you!

@hacdias
hacdias merged commit b9787c7 into filebrowser:master Sep 25, 2025
5 of 6 checks passed
@radam9
radam9 deleted the dev branch September 25, 2025 18:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] allow users to set the ace editor syntax highlighting theme.

2 participants