Skip to content

Encapsulate Legacy Steam Market Behavior and Deprecate JQuery#397

Merged
GODrums merged 7 commits into
masterfrom
fix/comp-with-steam-beta
Jun 9, 2026
Merged

Encapsulate Legacy Steam Market Behavior and Deprecate JQuery#397
GODrums merged 7 commits into
masterfrom
fix/comp-with-steam-beta

Conversation

@GODrums

@GODrums GODrums commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Closes #396

Step 1 of #395

Gets rid of JQuery logic in shared functionality in favor of common DOM APIs.
Encapsulates existing Steam Market components to isLegacySteamMarket.

Resolves all errors when using the extension with the Steam Market Beta enabled.


< />!NOTE< />
Medium Risk
Changes shared injection and styling paths used on every market page; incorrect mode detection could skip features on legacy or leave beta issues partially unfixed.

Overview
Adds Steam Market mode detection (legacy vs React beta) and gates legacy-only UI injection and Google Fonts loading so the extension no longer throws on the Market beta.

Injection decorators now accept an optional guard; market utility belt and listing row components only inject when isLegacySteamMarket() is true. Conflict styling in StyleConflictingElement drops jQuery for querySelectorAll and inline setProperty.

Reviewed by Cursor Bugbot for commit a3f69c3. Configure here.

@GODrums GODrums requested a review from Step7750 June 4, 2026 23:14
@GODrums GODrums self-assigned this Jun 4, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a3f69c3. Configure here.

Comment thread src/lib/components/injectors.ts

@Step7750 Step7750 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Lovely edits to the system, thanks!

@GODrums GODrums merged commit f9041ae into master Jun 9, 2026
2 checks passed
@GODrums GODrums deleted the fix/comp-with-steam-beta branch June 9, 2026 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ensure all errors stemming from the legacy implementations are fixed when the Market Beta is active (no functionality in the new beta mode yet)

2 participants