Skip to content

Make the ConfigWidget method protected#4913

Closed
LeoDreamer2004 wants to merge 1 commit into
GregTechCEu:1.20.1from
LeoDreamer2004:leo/public-init
Closed

Make the ConfigWidget method protected#4913
LeoDreamer2004 wants to merge 1 commit into
GregTechCEu:1.20.1from
LeoDreamer2004:leo/public-init

Conversation

@LeoDreamer2004

Copy link
Copy Markdown
Contributor

What

Changed the abstract init method in ConfigWidget to protected for addon mods to override.

Implementation Details

protected!

AI Usage

The usage of AI tools requires honest disclosure, please refer to our AI Policy.

The purpose of this section is to not guilt trip users who engage with these tools but to give us a better understanding of how they were used as well to what extent.
Users who contribute AI written code are held responsible for the quality of the code as well as understanding the implemented code written by these tools.

**All machine code MUST be reviewed by the pull request submitter before opening. **

  • No AI driven tools were used for this pull request.
  • Yes AI driven tools were used for this pull request.

If you answered no and it is discovered the pull request in question used AI tooling, users will be restricted from submitting future pull requests

Outcome

Addons will be able to override, like integration for Mekanism gases which I'm working on.

@LeoDreamer2004 LeoDreamer2004 requested a review from a team as a code owner June 7, 2026 17:36
@github-actions github-actions Bot added the 1.20.1 label Jun 7, 2026
@jurrejelle

Copy link
Copy Markdown
Contributor

Unfortunately we will replace this LDLib UI code with our new ModularUI UI framework code, so we're not looking to make any changes to the old system in the meantime. Apologies and thanks for the PR!

@jurrejelle jurrejelle closed this Jun 7, 2026
@screret

screret commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

speaking of mek gases, there's a (very, very dead) ""working"" branch with the recipe capability code for those.

@screret

screret commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

nvm branch's been deleted, PR is here tho.

@LeoDreamer2004

Copy link
Copy Markdown
Contributor Author

:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants