Skip to content

fan 界面和渲染的若干问题#73

Open
luqin2007 wants to merge 13 commits intoSinoCraftProject:mainfrom
luqin2007:main
Open

fan 界面和渲染的若干问题#73
luqin2007 wants to merge 13 commits intoSinoCraftProject:mainfrom
luqin2007:main

Conversation

@luqin2007
Copy link
Copy Markdown
Contributor

No description provided.

@luqin2007 luqin2007 requested a review from qyl27 March 29, 2025 12:50
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

可以说明一下为什么要如此处理吗

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

配置文件API的话,我希望引入Configurate,它会让我们的生活更美好它的API很完善

Copy link
Copy Markdown
Contributor Author

@luqin2007 luqin2007 Mar 29, 2025

Choose a reason for hiding this comment

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

主要是,懒得去写neoforge和Fabric两套了,暂时先用这个吧。这个只是个简单的客户端配置读写实现,到时候有了 Configurate 替换下就行。

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

好的,不急

background.getTextureWidth(), background.getTextureHeight(),
TEXTURE.getWidth(), TEXTURE.getHeight());
Component titlePosition = Component.translatable(SBRConstants.Translation.GUI_FAN_SETTING_HUD_POSITION);
Component titleHide = Component.translatable(isHudShow() ? SBRConstants.Translation.GUI_SETTING_HIDE : SBRConstants.Translation.GUI_SETTING_SHOW);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

我一时间竟然没理解这是做什么的,扇子有HUD吗?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

是的,一开始我不是把扇子渲染在游戏界面上了嘛。那个方案没有删只是在编辑界面加了个隐藏按钮

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

我觉得可以直接屏蔽这个hud

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

好的,不急

background.getTextureWidth(), background.getTextureHeight(),
TEXTURE.getWidth(), TEXTURE.getHeight());
Component titlePosition = Component.translatable(SBRConstants.Translation.GUI_FAN_SETTING_HUD_POSITION);
Component titleHide = Component.translatable(isHudShow() ? SBRConstants.Translation.GUI_SETTING_HIDE : SBRConstants.Translation.GUI_SETTING_SHOW);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

我觉得可以直接屏蔽这个hud

@luqin2007
Copy link
Copy Markdown
Contributor Author

已删除hud

@luqin2007
Copy link
Copy Markdown
Contributor Author

有几个用于实现 hud 的组件,是否需要删除

  • 事件:BEFORE_RENDER_HUD,在渲染 HUD 之前触发
  • sinocore.utility.config,简单的客户端 json 配置文件读写工具

@luqin2007 luqin2007 requested a review from qyl27 March 31, 2025 13:07
@qyl27
Copy link
Copy Markdown
Member

qyl27 commented Apr 1, 2025

嗯感觉事件先放着吧,那个config不是说迁移到configurate吗

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

这里对initClient应该放在ClientModInitializer里

@luqin2007
Copy link
Copy Markdown
Contributor Author

嗯感觉事件先放着吧,那个config不是说迁移到configurate吗

等换了再说,到时候直接删了包就行

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.

2 participants