Make it easy to copy the download link URL

adamreichold / QMediathekView

Alternative Qt-based MediathekView front-end

GNU General Public License v3.0

17 stars 5 forks source link

Make it easy to copy the download link URL #7

Closed probonopd closed 5 years ago

probonopd commented 5 years ago

Please make it easy to copy the download link URL, e.g., by clicking a "Copy download link(s)" button and selecting the size.

It should be possible to copy multiple download links for more than one episode at once, by selecting multiple ones:

adamreichold commented 5 years ago

Thank you for taking the time to report this. I have two questions though:

If multiple shows are selected and the download link is to be copied, what format should the resulting clipboard content have? One line per link? Comma-separated links? Or some more complex MIME type? Or should they be copied one after the other to be picked up clipboard managers?
Are you aware that you can use an external download manager by specifying a download command in the settings, e.g. I personally use this to interface with a aria2c instance via IPC. Would that resolve your use case as well? I am hesitant to add another button to avoid cluttering the UI. Maybe adding context menu would be more suitable?

probonopd commented 5 years ago

My use case: I want to be able to put the links e.g., into an e-mail.

One line per link with no other markup would be ideal.

I could also imagine this format that would copy all resolutions at once:

# Low resolution
http://.../...
http://.../...
http://.../...

# Mid resolution
http://.../...
http://.../...
http://.../...

# High resolution
http://.../...
http://.../...
http://.../...

adamreichold commented 5 years ago

Agreed, plain text and one link per line seems like the best format. But as indicated above, I think I'd prefer putting this into a context menu instead of adding another button to the details panel...

adamreichold commented 5 years ago

Could you give 0c6f6f7fc78703e72650d58090b0e916c3a8081a a try and tell me if that works for you?

probonopd commented 5 years ago

If the AppImage PR was merged, then I could download it :-)

adamreichold commented 5 years ago

If the AppImage PR was merged, then I could download it :-)

Obviously, but as I said there, I need to learn about this a bit first. But since this being GitHub and such, rebasing your branch should allow you to both the latest changes from master here and the as-of-now unmerged AppImage integration.

probonopd commented 5 years ago

Yes, I will give it a try. Thank you

probonopd commented 5 years ago

Works perfectly :+1: Thank you very much.

works

If you want to try: https://github.com/probonopd/QMediathekView/releases

probonopd commented 5 years ago

If things progress at this rate, this app should seriously be nominated for the Grimme Award. Because it is way better than the sectionalist mess the public TV stations have been producing over all those years.

adamreichold commented 5 years ago

If things progress at this rate, this app should seriously be nominated for the Grimme Award.

Do note that the main work here is that of the MediathekView project which maintains the scrapers and hosts the database snapshots. This here is just an alternative Qt-based frontend (with rather limited functionality compared to the main application).

probonopd commented 5 years ago

I know. That project should have been nominated for the Grimme Award years ago, along with MediathekViewWeb.