manavortex / DailyAutoShare

2 stars 5 forks source link

GUI Tweaks, HUD Scene injection, SecurePostHook for Focused Quest Tracker #52

Closed mychaelo closed 3 years ago

mychaelo commented 3 years ago

This PR should fix these two ancient glitches: image image

Well, not so much 'fix' as circumvent in a different way. I've opted to instead register a callback on hud scene for "StateChange", which will setFontSize() with a 0.5s delay each time the hud scene is shown. This eliminates the quest labels pile-up on your first character load, which happened due to the HUD scene not being immediately ready and showing.

The quest 'labels' (or rather buttons) now use a child Caption control that actually houses the text now. This makes them wrap nicely with ... at the end if they overflow, and kills a few bugs with quest colours resetting on clicks.

A few layout changes were made in XML file, namely the background texture is now ignored for list size calculations, and widths were locked to the default 330, so that the overflowing labels will not affect the list's width. Close button got its own draw tier so that it's easier to click.

Finally, instead of prehooking the ForceAssist, we now secureposthook both ForceAssist and AssistNext functions of the quest tracker. [T] button presses cycling the focused quest were not tracked before, now they're captured, and update the list accordingly.

manavortex commented 3 years ago

Nice one! :)