mortenn / BrowserPicker

Smart browser selector for Windows
MIT License
244 stars 17 forks source link

How does Browser Picker add arguments ? #83

Closed rp1231 closed 1 year ago

rp1231 commented 2 years ago

I'm using it to send a youtube url to a media player(potplayer)

It works fine if there's no argument. But it misbehaves if I add the /add argument in browserpicker to add the url to the playlist.

I can open a url in potplayer fine with this command:

potplayermini64 {youtube url} /add
mortenn commented 1 year ago

Hi, process launching is handled here

Ie. the arguments get added in front of the url, without a space.

rp1231 commented 1 year ago

Thanks, Any way to add a space after the argument?

I think that's what is causing the problem.

mortenn commented 1 year ago

You should just be able to add the space behind the argument in the UI, I believe

rp1231 commented 1 year ago

I tried that out but it doesn't seem to be working for some reason. It gets added but with a lot(a lot) of extra files as well....... So its working but not properly......

Is there some sort of debug mode for the app?

mortenn commented 1 year ago

There is not a debug mode in the app as such, but if you really want to debug it, you could grab visual studio community edition from Microsoft and download the source code. Right clicking the app project you can go into properties and set the command line arguments to a url to test with and then hit F5 to launch it in debug mode.

I'll refrain from writing an entire howto debug .net apps in visual studio in this issue though :) I am sure there are plenty of good intros on youtube if you want a crash course.

Whatever may be going on though, it seems clear that we should extend the advanced settings to enable greater control over the executed command line.

rp1231 commented 1 year ago

Thanks for the detailed reply. I was asking just in case something like this was already implemented so I could try and see if I could decipher something from it. Not really a programmer, so the visual studio thing would be a little too advanced for me.