Skip to content

Feat/UI customization - #581

Open
MarioGogogo wants to merge 6 commits into
openRin:mainfrom
MarioGogogo:feat/ui-customization
Open

MarioGogogo wants to merge 6 commits into
openRin:mainfrom
MarioGogogo:feat/ui-customization

Conversation

@MarioGogogo

Copy link
Copy Markdown

No description provided.

- Only JWT_SECRET (plus S3 vars when R2 is not configured) is required now
- GitHub OAuth, NAME and AVATAR fall back to defaults, matching .env.example
  and the server behavior (password-only login when OAuth is not configured)
- Create dist/client automatically so wrangler can start without a prior
  frontend build
…erting to defaults

Clearing a text setting (e.g. site description) in /admin/settings saved
the empty value, but ConfigWrapper.get and getClientConfigWithDefaults
treated "" as unset and resurrected the default/env value, so the old
English default came back after refresh.

- Add ConfigWrapper.getRaw which returns the stored value as-is
- Use getRaw for free-text inputs on the settings page so cleared values
  display as empty (placeholder still shows the default)
- Only fall back to env defaults for client config keys that are not
  stored at all; an explicit empty value now wins over env
- Add regression tests for getClientConfigWithDefaults
- Add two switches (default off) in admin settings under Other Settings,
  styled after the Enable Comments switch
- Add moments.enabled / friends.enabled client config defaults
- NavBar (desktop, compact layout and mobile menu) now shows the Moments
  and Friends tabs only when the corresponding switch is enabled
- Add zh-CN / zh-TW / en / ja translations
…full modes

Add an article width option to the personalization settings card.
Normal keeps the current layout; wide expands the regular width by 30%;
full fills the entire browser width beside the TOC by offsetting the
page padding with negative margins. The left spacer column is hidden
in wide and full modes so the article area can use the whole space.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant