Skip to content

Comments

team status bar fix#158

Merged
sago007 merged 2 commits intoOpenArena:masterfrom
Bishop-333:Team-Status-Bar-fix
May 30, 2023
Merged

team status bar fix#158
sago007 merged 2 commits intoOpenArena:masterfrom
Bishop-333:Team-Status-Bar-fix

Conversation

@Bishop-333
Copy link
Contributor

small fix to shift the colored bar to prevent it from overwriting the score

@Bishop-333
Copy link
Contributor Author

Before :
shot0002
After :
shot0001

@Bishop-333
Copy link
Contributor Author

Bishop-333 commented Apr 14, 2022

look at the score at the bottom right
This is for the classic UI

@sago007 sago007 merged commit 905b539 into OpenArena:master May 30, 2023
@ghost
Copy link

ghost commented May 30, 2023

Did you test this with cg_drawteamoverlay right/bottom? A long time ago I realized that this needed a lot more code to fix it for real. I'm not sure if this is still the case with the current OA status.

@Bishop-333
Copy link
Contributor Author

If I remember right this little change is sufficient since the "y" is reused but honestly cannot test for now because when I launch with ioquake3 it tells me "bad cgame system trap" and with quake3e it tells me "re_addrefentitytoscene bad retype" I'll try to fix this and tell you

@The-Gig
Copy link
Contributor

The-Gig commented May 31, 2023

Tomorrow the change should appear in a new nightly build here https://files.poulsander.com/~poul19/public_files/oa/dev088/gamecode_nightly/, but I will not have access to a PC for a few days..

@Bishop-333
Copy link
Contributor Author

Bishop-333 commented May 31, 2023

In fact @KuehnhammerTobias you was right, it need more changes. Here's the needed changes to work well with the cg_drawteamoverlay I forget to update the pull request since its 1 year ago : Bishop-333@8b9c559
@The-Gig idk if its better to do the fix before doing the nightly
(my errors were caused by cg_leienhancement without the openarena engine i just realized)

@Bishop-333
Copy link
Contributor Author

#172

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