Bring the ability to add tabs to your bank for quick access to search tags.
Features:
* Adding a tab
* Selecting and deselecting a tab
* Changing a tab's icon
* Reordering tabs
* Scrolling of tabs (With mouse wheel, ingame buttons, and dragging an item/tab over ingame buttons)
* Saving scroll position
* Dragging an item to a tab to tag the item
* Tagging placeholders
* Removing only a tab
* Removing a tab and deleting all tags from said tab
* Faster bank search when using tag tabs
Closes#1205Closes#4426
Supersedes / Closes#4862
Supersedes / Closes#3750
Can close#4082.
Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
Co-authored-by: Tomas Slusny <slusnucky@gmail.com>
As all submitted tasks are now wrapped in this class automatically,
manual wrapping is no longer necessary.
Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
By default, executor service eats all exceptions and forwards them to
failed futures, so in order to actually see what happened one must check
the resulting future, what is not ideal. Instead, log every exception
from scheduled service using slf4j logger.
Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
This moves the item search locally and does a contains check instead of
requiring item searches to contain full item names.
Fixes assert fail introduced in b72bda1c75 from
fetching item compositions on the executor thread in the !price command.
Fixes an issue where the chat notifications plugin would not split word highlights with input as `some,highlight` even though they are supposed to be valid
Now uses the correct varp from the quest list completion state script (1358).
Additionally, allows the infobox to show if the quest is in progress to allow users to check favor during the last part of the quest.
Closes#1980
VarClientInt(2) is set to `1` whenever a tooltip is present, meaning we
should never render a tooltip when it is set that way. Some menu options
(world map dungeon links) have a null widget associated with them, and
do not utilize the TOOLTIP_TIMEOUT VarClientInt, so this check is
necessary to avoid adding a duplicate tooltip.
Fixesrunelite/runelite#3278
This is regression by PR that introduced 3 step cryptic clues, as check
for highlighting single object was removed.
Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
As it is set in check for LocatioClueScroll it is not necessary to set
it also in object clue scroll check.
Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
For better readability, change the big if blocks in findClueScroll()
method to early returns instead.
Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
