Skip to content

Implements ma2html (goodbye icon2html) #6654

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented May 12, 2025

About The Pull Request

Ports /mob/proc/send_appearance and the vis_holder from DaedalusDock/daedalusdock#1029

Why It's Good For The Game

icon2html and getFlatIcon are annoyingly slow procs. so let's. use them less!

Changelog

🆑
refactor: Refactored how icons are displayed for examines and such, so they should be far more performant now.
/:cl:

@Absolucy Absolucy added Process: do not merge don't merge this ffs Code: Performance / Optimization the number going down makes me happy :3 Process: should testmerge PR should be testmerged first labels May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code: Performance / Optimization the number going down makes me happy :3 Process: do not merge don't merge this ffs Process: should testmerge PR should be testmerged first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant