blazingly fast bordered text renderer ⚡ #347
Conversation
|
@RedthMC i think this might already be a feature in polyui. if not maybe we should port it over somehow, because merging this would be silly only for it to get superseded in 5 seconds by twoconfig |
|
@pauliesnug but people are complaining that chatting and polynametag's full shadow are broken |
it’s not… |
|
this seems to be a complete rewrite of the font renderer which will break any mod that mixins into it, meaning this is not great for compatibility.
|
|
@nextdayy after |
|
okay so edit: |
|
as i said, this should be rewritten as a series of mixins into the functions, not a complete rewrite. |
|
additionally, this is incompatible with minecraft 1.17+. |
| @Inject(method = "drawString(Ljava/lang/String;FFIZ)I", at = @At(value = "HEAD"), cancellable = true) | ||
| private void cachedShadow(String text, float x, float y, int color, boolean dropShadow, CallbackInfoReturnable<Integer> cir) { | ||
| if (Preferences.optimizedFontRenderer && dropShadow) { | ||
| cir.setReturnValue((int) BorderedTextRenderer.INSTANCE.drawString(text, x, y, color, TextRenderer.TextType.SHADOW)); |
There was a problem hiding this comment.
do not overwrite this whole method
| import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable; | ||
|
|
||
| @Mixin(value = FontRenderer.class) | ||
| public class FontRendererMixin { |
There was a problem hiding this comment.
transfer all of this to a patcher font renderer mixin as well
it doesnt work with pathcer optimized font renderer |
Description
blazingly fast bordered text renderer ⚡ by using cache
( only forge 1.8.9 and 1.12.2 yet )