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

Restore the appearance of the "Start Game" tab after #13761 #13769

Merged
merged 3 commits into from Aug 30, 2023

Conversation

grorp
Copy link
Member

@grorp grorp commented Aug 29, 2023

This PR mostly restores the appearance of the "Start Game" tab to how it was before #13761. Since that PR was only meant to migrate the tab to real coordinates, I assume that these visual changes were not intended. (There are more visual changes from that PR, but I have no idea whether they are good or bad.)

Good, master branch before #13761 (commit 7b56daa)
screenshot 1

Bad, current master branch (commit 0cbf96c)
screenshot 2

Good, this PR (commit ef07d89)
screenshot 3

To do

This PR is a Ready for Review.

How to test

Verify that the mainmenu looks closer to how it looked before #13761 again.

Copy link
Member

@rubenwardy rubenwardy left a comment

Choose a reason for hiding this comment

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

I wasn't intending to make it the same, but it's probably a good idea to do so

builtin/mainmenu/tab_local.lua Outdated Show resolved Hide resolved
@grorp
Copy link
Member Author

grorp commented Aug 30, 2023

Applied your suggestion. Also updated the PR to get even closer to the original version of the tab. Two screenshots:

Before #13761
screenshot before

With this PR
screenshot after

The differences should really be minimal now.

@grorp grorp changed the title Fix two "visual regressions" from #13761 Restore the appearance of the "Start Game" tab after #13761 Aug 30, 2023
@grorp grorp requested a review from srifqi August 30, 2023 09:34
@grorp grorp merged commit 4252f9d into minetest:master Aug 30, 2023
2 checks passed
@grorp grorp deleted the vrf branch December 18, 2023 16:38
kawogi pushed a commit to kawogi/minetest that referenced this pull request Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants