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

Fix name tweaking not properly applied #830

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Fix name tweaking not properly applied #830

wants to merge 2 commits into from

Conversation

ChromaPIE
Copy link
Contributor

No description provided.

@@ -389,7 +389,7 @@ recipes.addShaped(<thermalexpansion:frame>, [
[<gregtech:meta_item_1:12183>,<gregtech:meta_item_1:12183>,<gregtech:meta_item_1:12183>],
[<gregtech:meta_item_1:12183>,<thermalexpansion:frame:64>,<gregtech:meta_item_1:12183>],
[<gregtech:meta_item_1:12183>,<gregtech:meta_item_1:12183>,<gregtech:meta_item_1:12183>]]);
<thermalexpansion:frame:64>.displayName = "Thermal Machine Casing";
<thermalexpansion:frame>.displayName = "Thermal Machine Casing";
Copy link
Member

Choose a reason for hiding this comment

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

This incorrectly labels "Machine Frame" as "Thermal Machine Casing".

<thermalexpansion:frame:64> and <thermalexpansion:frame> are two different items.

Copy link
Contributor Author

@ChromaPIE ChromaPIE Dec 22, 2021

Choose a reason for hiding this comment

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

This incorrectly labels "Machine Frame" as "Thermal Machine Casing".

<thermalexpansion:frame:64> and <thermalexpansion:frame> are two different items.

<thermalexpansion:frame:64> was originally Device Frame, which was overwritten by L399 <thermalexpansion:frame:64>.displayName = "Thermal Device Casing"; with Thermal Device Casing.

Judging from that, <thermalexpansion:frame>, with its original displayed name Machine Frame, is supposed to be tweaked to Thermal Machine Casing.

Now, see L392 and L399. L392 overwrites the name of <thermalexpansion:frame:64> with Thermal Machine Casing, and L399 double overwrites the name of exactly the same item with Thermal Device Casing, that makes zero sense. If this commit incorrectly labels "Machine Frame" as "Thermal Machine Casing", then I don't get why L399, to which I didn't make any change, correctly labels "Device Frame" as "Thermal Device Casing".

Choose a reason for hiding this comment

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

@NotMyWing :nudge:

@NotMyWing NotMyWing added the enhancement New feature or request label Dec 22, 2021
@ChromaPIE
Copy link
Contributor Author

ChromaPIE commented Nov 29, 2022

Bruh lmao why it's still open. Take ur time to figure all these out mr.dev.

Imagine thumbs down 🤓

@Exaxxion
Copy link
Collaborator

Exaxxion commented Dec 8, 2022

We have a lot of things we're working on and this issue is not of a particularly high priority. The pull request also has merge conflicts with the development branch, so it cannot currently be merged.

@ChromaPIE
Copy link
Contributor Author

@Exaxxion yea i understand. i was just browsing those issues and prs and left some nonsense here. zero offense. take your time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants