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

Fix: Improve faction icons to show accurate icons in score screen #666 #2594

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

L3-M
Copy link

@L3-M L3-M commented Mar 2, 2025

@DevGeniusCode DevGeniusCode added Minor Severity: Minor < Major < Critical < Blocker GUI For graphical user interface labels Mar 2, 2025
@DevGeniusCode
Copy link
Collaborator

DevGeniusCode commented Mar 2, 2025

Can you elaborate a bit on the work and a screenshot of the textures?

@xezon
Copy link
Collaborator

xezon commented Mar 2, 2025

Nice. Thank you. How does this look in game?

Please add feature comments in INI and a yaml file as described here:
https://github.com/TheSuperHackers/GeneralsGamePatch/blob/main/CONTRIBUTE.md#how-to-contribute-as-developer

@xezon xezon added the Art Is art related label Mar 2, 2025
@L3-M
Copy link
Author

L3-M commented Mar 2, 2025

Screenshot 2025-03-03 010606
Screenshot 2025-03-03 010745

@L3-M L3-M changed the title Improve textures (score screen accurate faction icons) Fix: Improve faction icons to show accurate icons in score screen #666 Mar 2, 2025
@xezon
Copy link
Collaborator

xezon commented Mar 2, 2025

Nice it works. The images itself look a bit bad. Perhaps we can improve them? I think there are such faction logos also somewhere in retail UI textures... not 100% sure.

@L3-M
Copy link
Author

L3-M commented Mar 2, 2025

i found those in SCShellUserInterface512_001 & SCShellUserInterface512_006. if there is a better quality logo. surely!

Ex:
image

@xezon
Copy link
Collaborator

xezon commented Mar 2, 2025

Are these the textures that sit on the entrances of the Command Centers?

@L3-M
Copy link
Author

L3-M commented Mar 2, 2025

I think so

Actually I found that Command Centers use this textures from TexturesZH.big
image

@tintinhamans
Copy link

These ones from the Command Centers look much better! Do we know where the other ones are used, maybe they should be updated with these.

@xezon
Copy link
Collaborator

xezon commented Mar 3, 2025

Actually I found that Command Centers use this textures from TexturesZH.big

Yes. These should allow to create higher quality icons.

@L3-M
Copy link
Author

L3-M commented Mar 3, 2025

Actually I found that Command Centers use this textures from TexturesZH.big

Yes. These should allow to create higher quality icons.

image

@xezon
Copy link
Collaborator

xezon commented Mar 3, 2025

Nice. This is going into right direction.

There are some artifacts:

  • icons have thin outline artifacts
  • 2 icons have white color inside
  • bottom left edge has bright pixel
  • toxin symbol has boxy background on button and looks more blurry than others
  • positioning and sizing can be optimized

How was the background created? With Vector Rectangle + Bevel Emboss + Gradient?

@xezon
Copy link
Collaborator

xezon commented Mar 3, 2025

Btw, I have not checked PSD file, but it looks very large (4MB+). I suspect it has Smart Objects inside? If so, please flatten them after the sizing is nailed. Smart Objects make PSD very large and we always avoid them for this reason.

@xezon
Copy link
Collaborator

xezon commented Mar 3, 2025

And another thought: Texture dimensions need to be reviewed with 4:3 resolution, because all ui was specifically created for this resolution (800 x 600). So if logo's looks squashed in 4:3, then it is not right.

@L3-M
Copy link
Author

L3-M commented Mar 3, 2025

How was the background created? With Vector Rectangle + Bevel Emboss + Gradient?

Yes.

Btw, I have not checked PSD file, but it looks very large (4MB+). I suspect it has Smart Objects inside? If so, please flatten them after the sizing is nailed. Smart Objects make PSD very large and we always avoid them for this reason.

Yeah, I will lower the size in a commit after we agree on the texture.

And another thought: Texture dimensions need to be reviewed with 4:3 resolution, because all ui was specifically created for this resolution (800 x 600). So if logo's looks squashed in 4:3, then it is not right.

This is a 800x600 preview, logos seems fine.

image

@xezon
Copy link
Collaborator

xezon commented Mar 3, 2025

Overall looks pleasing to the eye. It does seem to be a bit stretched wide, no? The toxin, stealth, nuke logo can be centered better. The airforce plane is a bit too tall. The tank logo is perhaps a bit too wide

@Chikinsupu
Copy link

I agree with xezon, the padding of the icons is inconsistent, other than that it looks nice so far

@L3-M
Copy link
Author

L3-M commented Mar 4, 2025

Overall looks pleasing to the eye. It does seem to be a bit stretched wide, no? The toxin, stealth, nuke logo can be centered better. The airforce plane is a bit too tall. The tank logo is perhaps a bit too wide

I agree with you, for me it was ok. I fixed everything, redid the positioning and sizing.
Here is the latest version:

  • Texture:

image

  • Ingame:

sshot001

Copy link
Collaborator

@xezon xezon left a comment

Choose a reason for hiding this comment

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

It looks still a bit squashed in 4:3 but perhaps its ok so that it looks not as stretched in 16:9? A compromise?

Texture = SCShellUserInterface512_011.tga
TextureWidth = 512
TextureHeight = 512
Coords = Left:7 Top:407 Right:51 Bottom:436
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you check if Right and Bottom values are correct? On the ingame image it looks not correctly cut out.

Copy link
Author

Choose a reason for hiding this comment

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

I just checked, and all coords values are correct.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmm that is strange. It looks as if it is off by 1 pixel or so. The edge right and bottom looks different.

@@ -930,3 +930,74 @@ MappedImage Rank_CaptainElite
Status = NONE
End

MappedImage GameinfoToxin ; Patch104p @fix L3-M 03/03/2025 MappedImage For FactionGLAToxinGeneral SideIconImage & EnabledImage
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can use @feature


changes:
- fix: Show accurate faction icons in the score screen
SCShellUserInterface512_011
Copy link
Collaborator

Choose a reason for hiding this comment

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

This looks like it should not be here.

Copy link
Author

Choose a reason for hiding this comment

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

Where should it be?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think just remove it.

Also it looks like it is not just score screen, but also for Observer player info.

labels:
- art
- minor
- GUI
Copy link
Collaborator

Choose a reason for hiding this comment

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

In alphabetical order, lowercase.

- art
- gui
- minor
- v1.0

@xezon
Copy link
Collaborator

xezon commented Mar 4, 2025

How does it look with vanilla factions in the mix?

@L3-M
Copy link
Author

L3-M commented Mar 4, 2025

image

@L3-M
Copy link
Author

L3-M commented Mar 4, 2025

I think it's best to use black textures for the vanilla faction too so they have a same appearance or is it ok. What do you think?

@xezon
Copy link
Collaborator

xezon commented Mar 4, 2025

Hmm the old ones do not mix well with the new ones. The original icons fill more space of the button. And they are not as crisp. I think it makes sense to create 12 buttons with same visual style.

In Control Bar Pro we have a texture that has 3 new buttons for vanilla factions. Perhaps you can reuse something of that. I think it is in the observer texture.

This:
https://github.com/TheSuperHackers/GeneralsControlBar/blob/main/ControlBarProZH/GameFilesEdited/Art/Textures/ControlButtonsPro_512_512.psd

@L3-M
Copy link
Author

L3-M commented Mar 4, 2025

I retextured them with black.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Art Is art related GUI For graphical user interface Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding custom icons for all non-original armies
5 participants