Launcher icon indicating copy to clipboard operation
Launcher copied to clipboard

NOISSUE Add Install Legacy Fabric Button

Open thecatcore opened this issue 3 years ago • 10 comments

To merge along side MultiMC/meta#9. Closes #4408

thecatcore avatar May 06 '22 23:05 thecatcore

I disagree with how this is done, you're making it confuse Fabric with Legacy Fabric. They're not the same thing, and shouldn't be treated as such. See https://github.com/MultiMC/Launcher/issues/4408#issuecomment-1006104947

kb-1000 avatar May 07 '22 10:05 kb-1000

Is it really required when basically it installs upstream FabricLoader + Legacy Fabric Intermediary? 🤔

thecatcore avatar May 07 '22 12:05 thecatcore

I disagree with how this is done, you're making it confuse Fabric with Legacy Fabric. They're not the same thing, and shouldn't be treated as such. See #4408 (comment)

They're not the same thing

Dude they use exactly the same loader, the only thing different is the intermediary file.

PseudoDistant avatar May 07 '22 14:05 PseudoDistant

The button reads 'Install Fabric', now I would take that to mean Fabric - not a fork there-of.

This should be implemented as Quilt has just been, not by masquerading a third-party project as the upstream one.

jamierocks avatar May 07 '22 15:05 jamierocks

@jamierocks you misunderstand. It's not a fork, we literally use upstream Fabric Loader. The only difference is the Intermediary being fed to it.

PseudoDistant avatar May 07 '22 16:05 PseudoDistant

I don't misunderstand, I'm well aware of what Legacy Fabric is and isn't - and what Fabric is and isn't.

The button doesn't read 'Install Fabric Loader', it reads 'Install Fabric' - it currently does as I would expect and installs the Fabric Loader, and requires Fabric's intermediaries. These are part of the package that Fabric's installer would install, and together really form part of what most would consider Fabric.

Installing intermediaries from a third-party isn't what I would consider installing Fabric, it would be installing Legacy Fabric. Just because the package happens to use Fabric Loader, doesn't make the package what most would understand to be Fabric.

jamierocks avatar May 07 '22 17:05 jamierocks

Updated accordingly to submitted suggestions.

thecatcore avatar May 08 '22 08:05 thecatcore

oops fixed a typo...

thecatcore avatar May 17 '22 20:05 thecatcore

Forgive me if I read this wrong, but wouldn't it make more sense to not disable the button but hide it on newer versions and hide fabric on older ones? Since neither will ever be available on versions where the other exists and they work as de-facto "equivalents" for another, adding a disabled button seems to me like unneeded clutter and this approach also avoids the problems mentioned by jamierocks.

JFronny avatar Jun 13 '22 07:06 JFronny

How would I do it?

thecatcore avatar Jun 13 '22 10:06 thecatcore

Why was this closed?

ImperatorStorm avatar Sep 21 '22 18:09 ImperatorStorm