Skip to content

Store Page button opens the link in a new window instead of the webview #863

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
arielj opened this issue Jan 3, 2022 · 4 comments · Fixed by #1149
Closed

Store Page button opens the link in a new window instead of the webview #863

arielj opened this issue Jan 3, 2022 · 4 comments · Fixed by #1149
Assignees

Comments

@arielj
Copy link
Collaborator

arielj commented Jan 3, 2022

Describe the bug

When clicking the Store Page link of a game, it opens in a new window. It's weird since the Store link on the sidebar opens the store in the webview inside the main Heroic window.

the Check Compatibility link behaves the same.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the Tools tag of any game
  2. Click on Store Page or Check Compatibility
  3. The page will open in a new window

Expected behavior

I would expect at least the store page to open in the webview, the same way the Store link of the sidebar works.

EDIT: this issue now applies only to the Store Page button, not the compatibility button

Desktop (please complete the following information):

  • OS: Linux Mint 20.02
  • Heroic Version: Main branch
  • Game: any
  • Wine/Proton version: -
  • Hardware specs (RAM, CPU, GPU): -
@flavioislima
Copy link
Member

The Webview component accepts a url as parameter so we can perhaps route to the store with the url.

In theory is a simple fix.

@flavioislima
Copy link
Member

But to be honest, I think would be better to keep as it is today.
Because will be weird to open protondb on the STORE page. so we would need to have another Webview route, this would add unnecessary complexity imo.

@arielj
Copy link
Collaborator Author

arielj commented Jan 3, 2022

@flavioislima What about only doing this fix for the Store Page link? It's taking the user to the Store, it should behave like the Store link of the sidebar.

I agree the protondb url is ok if it stays the same (it's a bit weird, but it's also weired if it ends up in the Store screen)

@flavioislima
Copy link
Member

Yes, should be an easy fix then. Makes sense in this case.

@arielj arielj changed the title Store Page / Check compatibility opens the link in a new window instead of the webview Store Page button opens the link in a new window instead of the webview Jan 9, 2022
@arielj arielj self-assigned this Mar 21, 2022
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 a pull request may close this issue.

2 participants