Skip to content

[ATLAS-313] feat: add redirection from banner cover in Downloads to their entry page - #406

Open
codeon89 wants to merge 1 commit into
towerwatchman:nightlyfrom
codeon89:atlas/313/download-banner-redirect-to-entry-page
Open

codeon89 wants to merge 1 commit into
towerwatchman:nightlyfrom
codeon89:atlas/313/download-banner-redirect-to-entry-page

Conversation

@codeon89

@codeon89 codeon89 commented Sep 8, 2026

Copy link
Copy Markdown

What this changes

Simplify some logic and update the following:

  • Clicking banner cover in Downloads now show the library / catalog entry page.
  • Clicking build chip (e.g. Full Archive) now open the game thread on source site.
  • Change language for "Back to Library" to "Back" on existing back buttons, as well as logo and page title tooltip to "Home" (initial change before ongoing back button logic update).

Why

#313

Targetted change from issue:

  • Downloads from browse mode (new title downloads) does not open the browse mode when clicking it. --> it now go to the library/browse entry for downloaded/installed records.
  • The Download tasks in the download queue should have a link to open the source thread for quicker access for verification. --> decided to use the build chip since the title highlight is too conspicuous. Clicking build chip now go to source thread if exists.

How it was tested

  • Added tests that cover this change
  • For a fix: the regression test fails against the unfixed code
  • npm run check passes locally
  • New functions and IPC handlers have comments explaining why
  • CHANGELOG.md updated
  • This PR targets nightly, not main

AI assistance

  • No AI was used on this change
  • AI was used on this change

Tool and model: VSCode auto free

What it wrote:

  • electron/ipc/games.js
  • Tests

What you verified yourself:

  • Verify test cases & confirm behavior on the client.

IPC changes

  • added: get-catalog-entry

Anything the reviewer should know

…age. Add redirection from build description chip to their thread page
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