The value cannot be more than MAX_VALUE so the previous condition was
never met. Additionally use MAX_QUANTITY constant instead of abusing
char max value which is completely unrelated.
Fix notifications for highlighted drops occuring when a non highlighted drop is
assigned a value tier with the same color as highlighted drops. Check the
highlightedItems list directly instead of doing a color comparison.
Because both level up dialogs and BA high gambles use the same widget
interface, commit 40f032bede caused BA
high gambles and some level up dialogs to trigger screenshots if either
config was enabled in the config, as that would set a flag to capture a
screenshot on the next game tick based on the widget text. This commit
adds config checks to the DIALOG_SPRITE_TEXT branch to ensure the
appropriate config flag must be set for a screenshot to be captured.
this should fix the following known bugs:
- putting a U+2019 in the config causes it to become corrupted and
~double in size every launch
- scripts become assembled incorrectly and the nbsp after your name in
the chatbox becomes incorrect
- the feed panel doesn't show emoji
`RING_OF_ENDURANCE_24844` is the ingame item which can actually be
obtained. `RING_OF_ENDURANCE` is the original ring ID for [the ring
which could not be traded on the Grand Exchange on Darkmeyer's day of
release][1]. This commit corrects the mapping to map to the tradeable
ring.
[1]: https://secure.runescape.com/m=news/darkmeyer-day-one-feedback-changes?oldschool=1
The bank widget can become null before the bank build script finishes by
holding the bank close key as the bank is opened. This commit adds a
null check to avoid this possible NPE.
Invoking the tab separator removal allowed tab separators to be visible
and interactable for a frame. On slower computers, users could click and
navigate to a tab during that period. The removal no longer needs to be
invoked after e4b43d9 so we can just invoke the script run.
jcwhisman