Skip to content

EZStorage 2のStorage CoreのGUIの挙動を修正#108

Closed
sjcl wants to merge 15 commits intodevfrom
patch-ez-storage2
Closed

EZStorage 2のStorage CoreのGUIの挙動を修正#108
sjcl wants to merge 15 commits intodevfrom
patch-ez-storage2

Conversation

@sjcl
Copy link
Member

@sjcl sjcl commented Jul 8, 2023

AE2のGUIの挙動になるべく近づけました。
#104 の議論を踏まえた修正も含めています。
GTModpackTeam/GregTech-Expert-2#306 の修正内容をすべて含んでいるため、それを削除できます。

@sjcl sjcl requested a review from tier940 July 8, 2023 20:50
@tier940 tier940 added type: feature New feature or request type: refactor Suggestion to refactor a section of code labels Jul 8, 2023
@sjcl
Copy link
Member Author

sjcl commented Jul 9, 2023

dependencies.gradle

gradle.propertiesでdebug_mixinsじゃなくて、特にこだわりが無ければdebug_ez2に変更してください

関連して、そもそもEZ Storage2の修正機能がGTExpert-Coreに含まれているべきではないという話もある気がしますが、どうなんでしょう 🤔

@tier940
Copy link
Member

tier940 commented Jul 9, 2023

完全にド忘れしてますがCoreModでやった理由ってなんでしたっけ...楽だからとかでしたっけ???

関連して、そもそもEZ Storage2の修正機能がGTExpert-Coreに含まれているべきではないという話もある気がしますが、どうなんでしょう

This reverts commit 5190605.
@sjcl
Copy link
Member Author

sjcl commented Jul 9, 2023

完全にド忘れしてますがCoreModでやった理由ってなんでしたっけ...楽だからとかでしたっけ???

色々経緯がありますが、合理的な理由は無いかも。

@miozune
Copy link
Member

miozune commented Jul 10, 2023

@tier940
Copy link
Member

tier940 commented Jul 10, 2023

dev-talksで話していた通り、分割したのでクローズ

@tier940 tier940 closed this Jul 10, 2023
@tier940 tier940 deleted the patch-ez-storage2 branch July 10, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature New feature or request type: refactor Suggestion to refactor a section of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants