Fabricators-of-Create / Create

[Fabric Mod] Building Tools and Aesthetic Technology
MIT License
834 stars 188 forks source link

Incompatibility with REI Dark Mode #290

Open surprisedpika opened 2 years ago

surprisedpika commented 2 years ago

Describe the Bug

When viewing recipes through REI with Dark Mode enabled, certain item boxes are not correctly rendered as dark.

Reproduction Steps

  1. Make sure REI and Create are downloaded
  2. On the REI settings (right of search bar) select "Dark Theme"
  3. View any recipe type added by Create using REI

Expected Result

The graphics should all update to a dark theme.

Screenshots and Videos

https://user-images.githubusercontent.com/82579156/168845477-e4786892-6f1b-49ed-98cf-760bca945e0c.mp4

Crash Report or Log

No response

Operating System

Windows 10

Mod Version

0.4.1

Minecraft Version

1.18.2

Other Mods

REI

Additional Context

No response

JieningYu commented 2 years ago

need to redo the textures used for rei because the og texture is white-gray based.

labaredaf commented 2 years ago

I made a resourcepack that """fixes""" it (only use it if REI's dark theme is enabled, as it overrides the textures and would make light theme look weird) Create REI Dark Theme.zip

maximumpower55 commented 2 years ago

I made a resourcepack that fixes it Create REI Dark Theme.zip

Could i pr this?

JieningYu commented 2 years ago

I made a resourcepack that fixes it Create REI Dark Theme.zip

Could i pr this?

but this pack override the original texture which is hacky.

labaredaf commented 2 years ago

I made a resourcepack that fixes it Create REI Dark Theme.zip

Could i pr this?

but this pack override the original texture which is hacky.

Yeah, it's only supposed to be used if REI's dark theme is going to be enabled all the time, in case a full fix is done feel free to use my textures though

JieningYu commented 2 years ago

Need to add a condition to detect if dark mode is enabled.

maximumpower55 commented 2 years ago

Okay, And a fix to allow for both light and dark mode textures would just be a simple if statement

maximumpower55 commented 2 years ago

The more i think about this, this is a issue with upstream (forge) create too, so i don't think we should fix it

maximumpower55 commented 2 years ago

Nevermind im dumb, JEI doesn't have dark mode

Skidamek commented 2 years ago

Nevermind im dumb, JEI doesn't have dark mode

But rei is available for forge too

maximumpower55 commented 2 years ago

Nevermind im dumb, JEI doesn't have dark mode

But rei is available for forge too

Yes but literally no one uses that

electricsteve commented 11 months ago

Nevermind im dumb, JEI doesn't have dark mode

But rei is available for forge too

Yes but literally no one uses that

that isn't true, it is used in modpacks like FTB SB3. Mainly for grouping of items.

electricsteve commented 11 months ago

I made a resourcepack that """fixes""" it (only use it if REI's dark theme is enabled, as it overrides the textures and would make light theme look weird) Create REI Dark Theme.zip

I am currently fixing this issue, so can i use your textures? edit: @labaredaf