If a player for example tries to drop an entire inventory quickly with the
overlay turned on, the inventory grid will flash for each small mouse movement
if the left mouse key is held down. This change mitigates the issue by adding a
delay option that controls the amount of time the user has to wait after
pressing it before enabling any overlay functionality. Showing the grid still
requires the mouse to be moved ~5px away from the first clicked position.
This delay just delays the activation of the overlay, as the the delay
countdown is started upon item press rather than the exit this ~5px range.
Making it respect the exit is needlessly complex as desired functionality is
reached with the current changeset.
When swapping between bludgeon and bow, onWidgetHiddenChanged is called before
the weapon varbit is set. Since that varbit determines which widgets should be
hidden we would like to make sure the widget is set to hidden even after a
varbit change.
By stripping metadata from the images and compressing them by running
`optipng -o7 -strip all <emoji.png>` the file sizes went from 15kb or 1.1kb
to around 0.2-0.5kb.
Replace the time dependant showing of the inventory overlay with a distance
based one. An item sprite only follows the mouse if the mouse is held
down and dragged ~5 pixels. Thus, the overlay will only show when you are
certain to be dragging an item and the time-delay is unnecessary.
Original PR by Jbleezy.
The wilderness canoe works like a portal teleport as it
is a one way transportation. I opted to use the normal transportation
icon in order to stay consistent with the other canoe stations,
though with a clarifying tooltip that no departures are to be made.
Since the base xp for level 1 is 0 the startLevelXp was initialized from
the current level which made the progress bar calculations wrong as they
were based on the xp earned from the current level base xp and not level 1.
Fix respawn timers for:
- Adamant rocks in Wilderness Resource area
- Coal rocks in Miscellania
The doubled respawn rate of the adamant rocks is a possible Jagex bug as
they are the only type of rock within the Wilderness resource area to do so.
The coal rocks in Miscellania are mined for favour instead of ores
and therefore have a shortened respawn time.
Hiding attack styles primarily relies on a subscription to
onWidgetHiddenChanged but when swapping between bludgeon and bow this event
fires before onVarbitChanged which means that when swapping quickly between
the two, the longrange option will not be hidden. Since the plugin uses info about
the weapon to determine in which skill xp will be gained and ultimately, which widgets
should be hidden, we need an extra call to ProcessWidgets after a weapon switch to ensure
expected behavior.
dekvall