Skip to content

更改主页 UI - #3129

Merged
Glavo merged 6 commits into
HMCL-dev:mainfrom
burningtnt:feature/change-root-page-ui
Oct 26, 2024
Merged

更改主页 UI#3129
Glavo merged 6 commits into
HMCL-dev:mainfrom
burningtnt:feature/change-root-page-ui

Conversation

@burningtnt

Copy link
Copy Markdown
Member

image

Comment thread HMCL/src/main/resources/assets/lang/I18N.properties Outdated
Co-authored-by: 3gf8jv4dv <158490944+3gf8jv4dv@users.noreply.github.com>

@3gf8jv4dv 3gf8jv4dv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1e68198 (#3129), burningtnt/HMCL/actions/runs/9534351876

The current UI design does not mesh well with the English strings changed by this PR. When the user is in English, the last word of Game Instance Settings is blocked. I cannot find a way to make the left sidebar wider.

Preview

@burningtnt

Copy link
Copy Markdown
Member Author

How about changing 'Game Instance Settings' to a shorter phrase?

@3gf8jv4dv

Copy link
Copy Markdown
Contributor

How about changing 'Game Instance Settings' to a shorter phrase?

Sure.

@burningtnt

Copy link
Copy Markdown
Member Author

Sure.

Can you provide any suggestion? I'm not a native Enligsh spaker.

@3gf8jv4dv

Copy link
Copy Markdown
Contributor

Can you provide any suggestion? I'm not a native Enligsh spaker.

Instance Settings is all it takes. Prism Launcher also adopts this description.

@burningtnt

Copy link
Copy Markdown
Member Author

image
Nice. Thank you

@3gf8jv4dv

Copy link
Copy Markdown
Contributor

The current UI design does not mesh well with the English strings changed by this PR. When the user is in English, the last word of Game Instance Settings is blocked. I cannot find a way to make the left sidebar wider.

Off topic. It occurred to me that there are similar issues in several other places in the launcher, but those issues are different from this PR and can basically be solved by modifying the i18n files. Should I create a PR to fix the above issues?

@burningtnt

burningtnt commented Jun 26, 2024

Copy link
Copy Markdown
Member Author

The current UI design does not mesh well with the English strings changed by this PR. When the user is in English, the last word of Game Instance Settings is blocked. I cannot find a way to make the left sidebar wider.

Off topic. It occurred to me that there are similar issues in several other places in the launcher, but those issues are different from this PR and can basically be solved by modifying the i18n files. Should I create a PR to fix the above issues?

You are always welcomed to contribute. Thanks!

@3gf8jv4dv

Copy link
Copy Markdown
Contributor

@Glavo What do you think? I think it is a good change, it matches the launch button in the bottom right corner of the home page, and it lets users know exactly what this button does.

Note that the i18n parts in this PR are outdated — I replaced them with new translations in some recent PRs.

Glavo added 2 commits October 26, 2024 20:57
# Conflicts:
#	HMCL/src/main/resources/assets/lang/I18N.properties
#	HMCL/src/main/resources/assets/lang/I18N_zh.properties
#	HMCL/src/main/resources/assets/lang/I18N_zh_CN.properties
@Glavo
Glavo merged commit 959adf1 into HMCL-dev:main Oct 26, 2024
@burningtnt
burningtnt deleted the feature/change-root-page-ui branch June 9, 2025 12:32
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