Skip to content

fix: correct ButtonColors and button dimensions on Drift Net Fishing interface#258

Open
orkabyte wants to merge 1 commit intoTorwent:masterfrom
orkabyte:master
Open

fix: correct ButtonColors and button dimensions on Drift Net Fishing interface#258
orkabyte wants to merge 1 commit intoTorwent:masterfrom
orkabyte:master

Conversation

@orkabyte
Copy link

Fixes button detection in the Drift Net Fishing interface. GetButtons() was returning an empty array for the regular buttons (discard/bank/take), making GetButton() non-functional for the primary interface.

Critical Fixes

  1. Wrong ButtonColors: CTS1(2370349, 0) (tolerance 0) was too strict and targeted the wrong color.

BEFORE
CTS1(2370349, 0)
image

AFTER - Corrected to CTS1(5334129, 2)
image

  1. Wrong regular button dimensions: [[36, 36]] → [[33, 34], [34, 34]].

Precision Fix

Overlay button dimensions: [100, 30] → [100, 28]. Corrected to actual measured height. The old value only worked previously due to FindButtons tolerance.

Tests

  1. Main Interface:

BEFORE
image

AFTER
image

  1. Overlay:
image

These corrections have been validated over 50+ hours of live drift net fishing with no detection failures.

Wrong ButtonColors and button dimensions caused GetButtons() to return
an empty array for regular buttons, making GetButton() non-functional.

- Fix ButtonColors: CTS1(2370349,0)/CTS1(1052431,26) → CTS1(5334129,2)/CTS1(657930,20)
- Fix regular button dims: [[36,36]] → [[33,34],[34,34]]
- Fix overlay button dims: [100,30] → [100,28] (precision correction)
@orkabyte orkabyte changed the title Fix button detection on Drift Net Fishing interface fix(driftnet): correct ButtonColors and button dimensions Mar 14, 2026
@orkabyte orkabyte changed the title fix(driftnet): correct ButtonColors and button dimensions fix: correct ButtonColors and button dimensions Mar 14, 2026
@orkabyte orkabyte changed the title fix: correct ButtonColors and button dimensions fix: correct ButtonColors and button dimensions on Drift Net Fishing interface Mar 14, 2026
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.

1 participant