Skip to content
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

Spotlight "learn more" links don't work #431

Open
2 tasks done
demonbane opened this issue Jun 29, 2024 · 3 comments
Open
2 tasks done

Spotlight "learn more" links don't work #431

demonbane opened this issue Jun 29, 2024 · 3 comments
Assignees
Labels
bug Something isn't working in-progress Accepted, at least some code is implemented or pending push

Comments

@demonbane
Copy link

Preflight Checklist

  • Running microsoft-edge:https://google.com redirects successfully
  • Microsoft Edge is selected in any "How do you want to open this?" box

Install Type

New Install

Install Mode

Active Mode

Steps to reproduce

In newer versions of Windows, there's a desktop shortcut for "Learn about this picture". Right-clicking on it creates a modal window with a "Learn more" button. Clicking that button opens the link in my default browser, but it produces a 400 error. This appears to be due to some of the URL parameters being omitted when opened in the default browser.

For this particular background, my default browser was opened to:

https://www.bing.com/spotlight?spotlightid=DS_WallsNorthYorkshire

While with redirect turned off, Edge was opened to:

https://www.bing.com/spotlight?spotlightid=DS_WallsNorthYorkshire&q=Yorkshire+Dales%2C+England&FORM=MC13ER

Opening the Edge URL in my browser worked just fine, so I believe MSEdgeRedirect might just be stripping out the extended parameters after &.

✔️ Expected Behavior

No response

❌ Actual Behavior

No response

Microsoft Windows version

24H2 Build 26100.994

Other Software

No response

@demonbane demonbane added the bug Something isn't working label Jun 29, 2024
@rodrigosambadesaa
Copy link

Now they work but I'm not being redirected to the search engine of my choice.

@rcmaehl
Copy link
Owner

rcmaehl commented Sep 10, 2024

I can get this added, but you'll lose the overview spotlight page

Note to self q=Yorkshire+Dales%2C+England

rcmaehl added a commit that referenced this issue Sep 10, 2024
@rcmaehl rcmaehl self-assigned this Sep 10, 2024
@rcmaehl rcmaehl added the in-progress Accepted, at least some code is implemented or pending push label Sep 10, 2024
@rcmaehl
Copy link
Owner

rcmaehl commented Nov 7, 2024

TODO: Add UI Elements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working in-progress Accepted, at least some code is implemented or pending push
Projects
None yet
Development

No branches or pull requests

3 participants