Skip to content

feat: Add <Open in Desktop App> button in header#267

Merged
xPaw merged 5 commits intoSteamDatabase:masterfrom
nolddor:master
Apr 11, 2026
Merged

feat: Add <Open in Desktop App> button in header#267
xPaw merged 5 commits intoSteamDatabase:masterfrom
nolddor:master

Conversation

@nolddor
Copy link
Copy Markdown
Contributor

@nolddor nolddor commented Apr 11, 2026

Solves #241

This PR introduces a new button (disabled by default) at the top, close to the "Install Steam" button set by Valve.
Consequently, a new option on SteamDB Extension preferences page has been introduced to manage this.

Preview

image

@nolddor
Copy link
Copy Markdown
Contributor Author

nolddor commented Apr 11, 2026

G'day @xPaw let me know if this makes sense or need further refinement.

Cheers!

@xPaw
Copy link
Copy Markdown
Member

xPaw commented Apr 11, 2026

One thing to note, both this and hiding the install button currently won't work on React pages. I just looked at the workshop beta and wishlist pages, their css classes are hashes, but it seems to be at least consistent, but I don't know if it changes between builds.

<a class="_2CYMhC951F4-" href="https://store.steampowered.com/about/">Install Steam</a>

@xPaw xPaw merged commit c83d31d into SteamDatabase:master Apr 11, 2026
1 check passed
@nolddor
Copy link
Copy Markdown
Contributor Author

nolddor commented Apr 11, 2026

I was aware of that, do you have any idea about how to proceed on new React pages?
Also I started to play around to solve #266 until I noticed they was using React.js now...

I don't event know what's the right way to edit the DOM without making React.js to complain about "Hydration Errors" even setting the *.js files at manifest to "run_at": "document_idle" seems inefficient

@xPaw
Copy link
Copy Markdown
Member

xPaw commented Apr 11, 2026

I don't know what the best approach to react pages is either.

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.

2 participants