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 the hudbars mod #1047

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Add the hudbars mod #1047

wants to merge 10 commits into from

Conversation

tidle
Copy link
Contributor

@tidle tidle commented Aug 25, 2022

This pull request adds the minetest_hudbars mod, along with the necessary changes to make the other mods play nicely with it.

image

@LoneWolfHT
Copy link
Member

I'm unfortunately not sure this will be a welcome change unless players can choose between the current look and the bars.

@tidle
Copy link
Contributor Author

tidle commented Sep 22, 2022 via email

@LoneWolfHT
Copy link
Member

I've made sure that the old style of bars can be restored by removing the hudbars mod. Not sure how we could make it configurable, maybe add something to the settings menu?

Sorry for the late reply, that would work

@magnetar47
Copy link
Contributor

magnetar47 commented Oct 16, 2022

I've made sure that the old style of bars can be restored by removing the hudbars mod. Not sure how we could make it configurable, maybe add something to the settings menu?

I'm assuming "settings menu" refers to the in-game settings tab in the player's inv GUI, and not the server-side settings? i.e. the intention is to make it configurable per-player?

@tidle
Copy link
Contributor Author

tidle commented Oct 16, 2022

Yes. I'm thinking that there would be a toggle there between the old style bars and the new style. The old ones should probably be the default

@tidle
Copy link
Contributor Author

tidle commented Nov 17, 2022

I added the setting. Old bars are the default, and you need to disconnect and reconnect for the setting to take effect. That's not ideal, but it's better than nothing.

@LoneWolfHT LoneWolfHT self-requested a review November 17, 2022 19:29
@LoneWolfHT
Copy link
Member

Sorry for the delay.

I get this crash when I enable hudbars, relog, disable the setting, and proceed to sprint

2022-12-03 19:03:26: ERROR[Main]: ServerError: AsyncErr: Lua: Runtime error from mod 'sprint' in callback environment_Step(): ...t/bin/../games/capturetheflag/mods/other/sprint/init.lua:52: bad argument #1 to 'hud_change' (number expected, got nil)
2022-12-03 19:03:26: ERROR[Main]: stack traceback:
2022-12-03 19:03:26: ERROR[Main]: 	[C]: in function 'hud_change'
2022-12-03 19:03:26: ERROR[Main]: 	...t/bin/../games/capturetheflag/mods/other/sprint/init.lua:52: in function 'updateHud'
2022-12-03 19:03:26: ERROR[Main]: 	...t/bin/../games/capturetheflag/mods/other/sprint/init.lua:82: in function 'func'
2022-12-03 19:03:26: ERROR[Main]: 	.../lv/minetest/bin/../builtin/profiler/instrumentation.lua:107: in function <.../lv/minetest/bin/../builtin/profiler/instrumentation.lua:100>
2022-12-03 19:03:26: ERROR[Main]: 	/home/lv/minetest/bin/../builtin/game/register.lua:431: in function </home/lv/minetest/bin/../builtin/game/register.lua:417>```

@Emojigit
Copy link
Contributor

It would be good if we can choose between the classic style and the hud bars.

@LoneWolfHT
Copy link
Member

It would be good if we can choose between the classic style and the hud bars.

That's a feature of this PR, choosable per-player

@farooqkz
Copy link
Contributor

farooqkz commented Jul 9, 2024

What is status of this PR?

@LoneWolfHT
Copy link
Member

Needs a crash fix that's probably possible for the merger to do

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.

5 participants