Skip to content

fix(theme): align user menu profile label with password change page#3145

Merged
marevol merged 1 commit into
masterfrom
fix/theme-profile-menu-label
Jun 11, 2026
Merged

fix(theme): align user menu profile label with password change page#3145
marevol merged 1 commit into
masterfrom
fix/theme-profile-menu-label

Conversation

@marevol

@marevol marevol commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

When logged in, the user dropdown menu on the search screen shows a different label depending on the UI: the default JSP view displays "Change Password" (labels.profile), while the bootstrap theme displays "Profile" (nav.profile). Both link to the same /profile page, which only provides password change functionality. This PR unifies the theme menu label with the default view.

Changes Made

  • Updated nav.profile in src/main/webapp/themes/bootstrap/i18n/messages.*.json (16 locales) to match the corresponding labels.profile translation in fess_label_*.properties (e.g. en: Profile -> Change Password, ja: プロファイル -> パスワード変更)
  • No changes to the message key or auth.js; only the translated values were updated

Testing

  • Verified all 16 JSON files remain valid after the update
  • Confirmed no tests rely on the previous "Profile" label text

Breaking Changes

  • None

Additional Notes

  • The "Change Password" wording follows the existing label introduced by Add Change Password page #370, since the /profile page only implements password changes. The theme's own page title (profile.title) already uses "Change Password", so this also resolves an inconsistency within the theme itself.
  • A companion PR applies the same fix to the docuforge theme in the fess-themes repository.

@marevol marevol self-assigned this Jun 10, 2026
@marevol marevol added the task label Jun 10, 2026
@marevol marevol added this to the 15.7.0 milestone Jun 10, 2026
@marevol marevol merged commit 3431135 into master Jun 11, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant