Skip to content

Conversation

@ebkr
Copy link
Owner

@ebkr ebkr commented Aug 7, 2025

TODO

  • Add French translation for Extraction text

Differences

Launch parameters modal

The text for this has been changed to show as arguments instead of parameters as this has been an incorrect term for a long time.

image

Code export modal

There was no previous functionality to copy the export code directly.

image

@ebkr ebkr changed the base branch from develop to i18n/profiles August 7, 2025 11:39
@ebkr ebkr force-pushed the i18n/manager-view branch 2 times, most recently from 31384d7 to b3a13f1 Compare August 13, 2025 08:02
@ebkr ebkr requested a review from anttimaki August 13, 2025 08:05
switch (0) {
case modName.value.trim().length:
validationMessage.value = "The mod name must not be empty.";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Similar to comment in an earlier PR in this stack, would it be clearer if the whole translation key was used here?

import {PlatformTranslation} from "./platforms/PlatformTranslation";
import {ProfileSelectionTranslation} from "./pages/ProfileSelectionTranslation";
import {ManagerTranslation} from './pages/ManagerTranslation';
import { DateTimeFormat } from 'src/i18n';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Absolute import

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.

3 participants