Skip to content

Conversation

@Elihpmap
Copy link
Contributor

This is a proposition to fix the regression on the use of the tag {url} to specify the position of the url in the argument to call the browser with.
This should also allow a simple fix for this issue by using the following line in the config file instead : chromeKiosk = "C:\Program Files\Google\Chrome\Application\chrome.exe" --app={url}

I added a few more tags too since it wasn't costing anything much and would allow some other interesting uses, you can see the details of them in the proposed modifications to the readme.md.

I plan to use these tags in my own config file by adding the following lines for example:

[browsers]
steam = "%ProgramFiles%\Steam\steam.exe" steam://openurl/{url}
noQueryFF = "%ProgramFiles%\Mozilla Firefox\firefox.exe" "{authority}{path}{fragment}"

[urls]
# open Steam related links in the Steam application
steamcommunity.com* = steam
steampowered.com* = steam
*.steampowered.com* = steam
# strip Amazon tracking queries in shared links
amazon.com* = noQueryFF 
amazon.fr* = noQueryFF 

@nref
Copy link
Owner

nref commented Dec 27, 2024

Hey @Elihpmap , thanks for the contribution. I'll review this ASAP - Christmas family activities and travel are keeping me off the computer.

@Elihpmap
Copy link
Contributor Author

No problem! Take your time and have a great christmas!

@nref nref merged commit 6650b46 into nref:main Jan 5, 2025
1 of 2 checks passed
@nref
Copy link
Owner

nref commented Jan 5, 2025

Very nice contribution. Thanks!

@nref
Copy link
Owner

nref commented Jan 5, 2025

@Elihpmap Elihpmap deleted the Fix{url}Marker branch January 31, 2025 20:18
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.

[Regression] {url} replacement doesn't work anymore

2 participants