Page 1 of 1

[0.16.2] No macro expansion in letter tooltip

Posted: Tue Dec 12, 2023 10:11 pm
by pango
As seen in a TalonHatesNames' stream, the tooltip of a Dark Brotherhood warning doesn't have its macros expanded:
we give you =S.01_ days.jpg
we give you =S.01_ days.jpg (138.35 KiB) Viewed 2355 times

Re: [0.16.2] No macro expansion in letter tooltip

Posted: Tue Dec 12, 2023 10:52 pm
by Interkarma
This is expected, macros are no longer expanded in quest letter tooltips.

https://github.com/Interkarma/daggerfal ... 541f335c69

Some macros require world lookups from location data and resulted in the game loading world just by user mousing around the inventory. This isn't very noticeable for unmodded game, but caused major stutters in games with heavily modded world data.

I have also limited tooltip to final line, which addresses the problem for letter in screenshot.

Re: [0.16.2] No macro expansion in letter tooltip

Posted: Fri Feb 16, 2024 3:19 pm
by pango
As seen on an TalonHatesNames stream, S0000010 letter has a broken tooltip.
Last line of the letter is sender name, not some location, so I don't know if that's the same limitation or an actual bug:
S0000010 broken tooltip.jpg
S0000010 broken tooltip.jpg (119.36 KiB) Viewed 1840 times

Re: [0.16.2] No macro expansion in letter tooltip

Posted: Mon Feb 19, 2024 2:10 am
by Kab the Bird Ranger
IK disabled all macro expansion in this tooltip, so it is the same issue.

I'm thinking we could probably replace macros with [...] for this tooltip. IK mentions that expanding them was too expensive, but not expanding them shows "debug" looking info.

Re: [0.16.2] No macro expansion in letter tooltip

Posted: Tue Mar 26, 2024 8:22 pm
by Interkarma
I went a bit too far disabling all macros in tooltips, but at the time was feeling overwhelmed by things and that was the fastest fix.

The problem macros are specifically any that reference location data as these were triggering delays, especially in modded world data setups. Macros like the one in this tooltip (which is most of them) are perfectly fine.

A small refinement would be to allow all macros except the problematic location ones. A proper fix would be to find a way to prevent the mouseover lag for tooltips that require location data lookups.