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

Improve efile i18n #79402

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

Procyonae
Copy link
Contributor

@Procyonae Procyonae commented Jan 28, 2025

Summary

None

Purpose of change

Fixes #79371
The onto/off of messages appear to be the wrong way around

Describe the solution

Hopefully makes it more pleasant and possible to translate
Fixes onto/off of order

Describe alternatives you've considered

The "Merge the parts together" commit is probably unnecessary but seems nicer

Testing

Checked the move message log messages were right (I couldn't get copying to work period in order to test it)

Additional context

… if out of bounds, add translator comments

Maybe slightly overkill but it seemed silly splitting these up just to glue them together in the same function
@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jan 28, 2025
@ShnitzelX2
Copy link
Contributor

This is good, just make sure that "move onto/off" make sense in the UI. Example for selecting "move onto":

Before these changes:
image
After:
image

It was confusing because moving files ONTO a device means that you're selecting files on devices to move OFF of in the UI and vice versa. In retrospect, this might not be necessary, you could just say "move files onto (device)" instead.

@GuardianDll GuardianDll marked this pull request as draft February 6, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot add i18n to some spliced string
2 participants