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

Small refactor & update in modlist.menu, server_browser.menu #874

Closed
wants to merge 0 commits into from

Conversation

eve-v0
Copy link
Contributor

@eve-v0 eve-v0 commented Sep 8, 2024

Update modlist.menu
Refactor
Doesnt change visuals

Update server_browser.menu
Makes text in "Join" button allign to middle/north which looks better
image
image

@github-actions github-actions bot added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Sep 8, 2024
Copy link
Contributor

@ASpoonPlaysGames ASpoonPlaysGames left a comment

Choose a reason for hiding this comment

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

Code looks good (there isn't really any code per-say but yeah the diff looks good)

@ASpoonPlaysGames ASpoonPlaysGames removed the needs code review Changes from PR still need to be reviewed in code label Sep 8, 2024
Copy link
Contributor

@Zanieon Zanieon left a comment

Choose a reason for hiding this comment

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

Good finding.

@@ -407,11 +407,11 @@ resource/ui/menus/modlist.menu

labelText "#CLEAR_FILTERS"
wide 100
xpos -15
ypos -55
xpos 0
Copy link
Contributor

Choose a reason for hiding this comment

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

What does this change for the mod list button?
I don't see any change with and without the PR:

Without

ui-mod-main

With

ui-mod-pr

@@ -2554,6 +2554,7 @@ resource/ui/menus/mods_browse.menu
xpos -17
ypos -57
zpos 90
textAlignment north
Copy link
Contributor

Choose a reason for hiding this comment

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

I wanted to suggest center for this parameter, but the resulting position is the same as north it seems ¯\_(ツ)_/¯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs testing Changes from the PR still need to be tested
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants