Skip to content

Conversation

@lineargraph
Copy link
Member

@lineargraph lineargraph commented Jan 30, 2024

TODO:

  • Fix Warning showing outside of skyblock
  • Fix Warning showing while tablist isnt loading (first 1-2s after joining a world
  • Maybe show warning inside of the relevant UI. For example in the skill overlay, show a warning inside of the skill overlay, instead of a notification
  • Maybe only show the warning if you actually have skill related stuff in the overlays enabled
  • Fix /neustats Has Advanced Tab
  • Fix Warning showing after enabling the widget https://imgur.com/7V7Aynx.gif
  • Fix Warning showing if you have Hide Empty And Locked Forge Slots enabled, tablist shows this '§9§lForges: §r§f(0/5)'

lineargraph and others added 4 commits January 30, 2024 13:36
Todo: Add patterns for all the other widgets.

An Example implementation can be seen in MiningOverlay.java. If the requested Widget is not present, a TablistTutorial task will be added to the queue, which informs the user that they need to enable it.

Otherwise (hopefully) only the requested information will be returned, removing the need to constantly check which part of the tab list is currently being parsed.
# Conflicts:
#	src/main/java/io/github/moulberry/notenoughupdates/util/ItemUtils.java
@Lulonaut
Copy link
Contributor

other TODO I forgot: obviously change all other tab list parsers over to this new API

Also attempted to (but failed) to fix the skills being detected
@jani270
Copy link
Collaborator

jani270 commented Mar 13, 2024

https://i.imgur.com/ovS5Kdp.png Shows outside of skyblock lol

@jani270 jani270 linked an issue Mar 13, 2024 that may be closed by this pull request
…blist

Most api calls probably don't care about the current region, so the getWidgetLines function shouldn't care either. For an explicit region, getWidgetLinesInRegion can be used.
@lineargraph lineargraph marked this pull request as ready for review March 16, 2024 17:55
Fix warning showing right after enabling the widget
This fix assumes the skyblock player is capable of not disabling the widget after enabling it.

Fix /neustats Has Advanced Tab
Instead added /neustats tablist, which copies the entire Tablist to the clipboard.
Deleted all SBInfo#hasNewTab usages in the process.

Also converted the MiningOverlay to show the warning inside the overlay, and only if the relevant lines are actually enabled.
@lineargraph lineargraph merged commit c09c456 into NotEnoughUpdates:master Apr 10, 2024
@lineargraph lineargraph deleted the alphatutorial branch April 10, 2024 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EquipmentOverlay: IOOBE: Index: -1

3 participants