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

Add temporary Patch104p text in main menu #558

Merged
merged 3 commits into from
Oct 16, 2021
Merged

Conversation

xezon
Copy link
Collaborator

@xezon xezon commented Oct 12, 2021

closes #155

@xezon xezon added the Task A task for someone to work on label Oct 12, 2021
@commy2
Copy link
Collaborator

commy2 commented Oct 12, 2021

Is it possible to instead use an image?

@xezon
Copy link
Collaborator Author

xezon commented Oct 12, 2021

Do you mean changing the background texture? Yes possible. But not sure if better. Would be more difficult to change text. Here its simple text edit. So can be changed and removed easily anytime.

@commy2
Copy link
Collaborator

commy2 commented Oct 12, 2021

Would be more difficult to change text.

True.

I guess it won't look as bad once STR/CSF files are there.

@xezon
Copy link
Collaborator Author

xezon commented Oct 13, 2021

shot_20211013_214810_1

@commy2
Copy link
Collaborator

commy2 commented Oct 13, 2021

Missing: This looks very professional
Hate the font though.

@xezon
Copy link
Collaborator Author

xezon commented Oct 13, 2021

It's Arial Bold.

@commy2
Copy link
Collaborator

commy2 commented Oct 13, 2021

That's funny. Exactly what I use at work. The aliasing is looks off. Has the font black outlines? It clashes with the black / semi-transparent dark grey backdrop.

Maybe it's just JPG being JPG.

@xezon
Copy link
Collaborator Author

xezon commented Oct 13, 2021

Yes it has a black drop shadow.

@xezon xezon requested a review from commy2 October 16, 2021 09:44
@commy2
Copy link
Collaborator

commy2 commented Oct 16, 2021

Looks much better now:

Screenshot

title

Copy link
Collaborator

@commy2 commy2 left a comment

Choose a reason for hiding this comment

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

lgt my wnd untrained eyes

@commy2 commy2 merged commit f84f199 into main Oct 16, 2021
@commy2 commy2 deleted the xezon-add-mainmenu-patch-text branch October 16, 2021 13:02
xezon added a commit that referenced this pull request Aug 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Task A task for someone to work on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add WIP text to Main Menu
2 participants