Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update save password button label #5430

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

thornbill
Copy link
Member

Changes
Updates the save password button text to avoid accidental password changes

Issues
Related to #5199

@thornbill thornbill added the enhancement Improve existing functionality or small fixes label Apr 29, 2024
@thornbill thornbill requested a review from a team as a code owner April 29, 2024 06:38
Copy link

sonarcloud bot commented Apr 29, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.1% Duplication on New Code

See analysis details on SonarCloud

@jellyfin-bot
Copy link
Collaborator

Cloudflare Pages deployment

Latest commit bc598c7
Status ✅ Deployed!
Preview URL https://dc00a296.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs
View bot logs

@@ -1347,6 +1347,7 @@
"Saturday": "Saturday",
"Save": "Save",
"SaveChanges": "Save changes",
"SavePassword": "Save Password",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"SavePassword": "Save Password",
"SavePassword": "Save password",

👀

Copy link
Member Author

Choose a reason for hiding this comment

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

I believe most buttons in web use Pascal casing... "Reset Password" directly below this certainly does 😉

Copy link
Member

Choose a reason for hiding this comment

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

I don't like it though 😅

@thornbill thornbill merged commit a5ccf9a into jellyfin:master Apr 29, 2024
10 checks passed
@thornbill thornbill deleted the save-password-text branch April 29, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve existing functionality or small fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants