* Update top nav search input to direct to channel panel when channel URL provided

[PikachuEXE]

Pull Request Type
Please select what type of pull request this is:

• Feature Implementation

Related issue
#1218

Description
This PR makes channel URL input in search input box in top nav handled by redirecting to channel view.
Instead of searching with it
Channel URL expected to be like /channel/{channelID}

Screenshots (if appropriate)

Apr-22-2021.10-01-22.mp4

Testing (for code that is not small enough to be easily understandable)

• Enter channel URL (e.g. https://www.youtube.com/channel/UCkRfArvrzheW2E7b6SVT7vQ)
• Press Enter/"Go To" button
• Ensure channel view shown instead of search result view

Desktop (please complete the following information):

• OS: MacOS
• OS Version: 10.15.7
• FreeTube version: 12.0.0 (latest on development actually)

Additional context
Add any other context about the problem here.

[GilgusMaximus]

Good idea, unfortunately 2 out of 4 channel links are not covered by this.
Channel links can be constructed the following ways:
/channel/id
/user/userName
/c/name
/name

That is the reason @Svallinn wanted to reuse the existing extraction code in #1218

[PikachuEXE]

Extracted a function (but not the entire handleYoutubeLink) which tries to extract info from input text if it's Youtube URL
I am assuming the handling of different Youtube URL types might be slightly different in different context

Tested with
https://youtu.be/UlJB3t55u_c (Top nav search box entry & link clicking)
https://www.youtube.com/watch?v=YZCPlbOuxJY (Top nav search box entry & link clicking)
http://www.youtube.com/EEVblog2 (Top nav search box entry & link clicking)
https://www.youtube.com/eevdiscover (Top nav search box entry & link clicking)

https://youtu.be/WDH11NVtxek (Top nav search box entry)
https://www.youtube.com/playlist?list=PLzFTGYa_evXiUoQ4-sWk4-2E2XUbcRLuc (Top nav search box entry & link clicking)

https://www.youtube.com/channel/UCeqUUXaM75wrK5Aalo6UorQ (Top nav search box entry)

[Svallinn]

Since you're using an arrow function for the callback, you can get rid of the v identifier and replace it with this.
Right now, there's a mixture of the two.

[Svallinn]

Should be this.$router.

[Svallinn]

It looks pretty good to me :)

The only thing that's really itching at me is the fact that a lot of the result logic is repeated in the two callbacks used in the getYoutubeUrlInfo dispatches.

But the only thing that came to mind in order to "fix" it was gimmicky hacks like passing this to the dispatch and making use of errors and catch callbacks, so I'm just gonna ignore the itching entirely.

[GilgusMaximus]

The only thing that's really itching at me is the fact that a lot of the result logic is repeated in the two callbacks used in the getYoutubeUrlInfo dispatches.

In theory yes, but when looking at the proportion of code deleted and code added, I would argue this is fair enough. Also it is not like I have never seen stuff like that in other parts of the code / modules before

[GilgusMaximus]

Code looks good in total. So if you could add the changes @Svallinn annotated, I would test it a last time and then merge it

[PikachuEXE]

Requested changes made
The callbacks for getVideoParamsFromUrl in each component are slightly different when handling default and invalid_url cases
e.g. Clicking on a link vs entering a valid but not youtube URL in top nav

Generally I keep this kind of "duplicate" code around (with slight difference) until there is too much copy & paste happening.

For router.push (which can be replaced by this.$router.push)
there is still one occurrence in
https://github.com/FreeTubeApp/FreeTube/blob/development/src/renderer/components/side-nav/side-nav.js#L63

[GilgusMaximus]

Thanks for putting in the work and changes, appreciate it a lot. Works great for all channel types :)