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

Little Feature Request: enable --title by default #499

Closed
BURGINABC opened this issue Oct 27, 2012 · 11 comments
Closed

Little Feature Request: enable --title by default #499

BURGINABC opened this issue Oct 27, 2012 · 11 comments

Comments

@BURGINABC
Copy link

@BURGINABC BURGINABC commented Oct 27, 2012

I think it would be more intuitive if fetching the title and using it as the file-name were enabled by default, and instead there were a flag to manually disable it.

Before reading the man-page I didn't even know it had that feature (title fetching), I think it would be more user friendly for people to get that by default instead of having to look in the manual for it, because I imagine most people would pretty much always prefer the video's actual title over a random-looking string of characters, particularly when downloading youtube playlists or otherwise downloading large groups of vids at once.

Just a suggestion for your consideration.

@phihag
Copy link
Contributor

@phihag phihag commented Oct 27, 2012

The main problem is that existing scripts may rely on the current format.

Seems worthwile though, so I'll let this issue open and wait for comments/protest.

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Oct 28, 2012

+1 but add a --id and a kind warning when updating

@phihag
Copy link
Contributor

@phihag phihag commented Nov 26, 2012

--id is added now. We should update the FAQ and wait 6 months before switching the default.

@phihag
Copy link
Contributor

@phihag phihag commented Dec 16, 2012

@FiloSottile How could we warn users who don't pass any options, but still rely on --id being the default? And what purpose would that serve?

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Dec 16, 2012

I was thinking about something like

WARNING: since version xxxx the default filename for videos is its title. If you want to keep the old behavior, use --id.

to be shown if no outtmpl is specified for a couple of releases.

It will not be noticed by everyone, but...

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Dec 16, 2012

Uh! Idea: release notes.
youtube-dl -U is more likely to be run by a human and it is where stuff changes. Also, extra output there isn't really annoying.

So, why not implement the ability to print a line or two of announcement while updating?

We could put it as a nullable field in the update-info.json.

If it's OK for you, I'll add it to the release flow.
We could also announce the new downloads page, or when pip package arrives, etc...

@phihag
Copy link
Contributor

@phihag phihag commented Dec 16, 2012

I managed to still have Internet access (still in Germany atm) - I'll probably not be able to answer.

I'm a definite -1 on printing anything by default, but I like the update notes idea.
Can't we just integrate that in the versions.json that we're going to use anyways? As in

{
   latest: "2012.12.12",
   versions: [
      '2012.12.12': {
          'url': 'http://youtube-dl.org/downloads/2012.12.12/youtube-dl',
          'url.exe': 'http://youtube-dl.org/downloads/2012.12.12/youtube-dl.exe',
          'url.src': 'http://youtube-dl.org/downloads/2012.12.12/youtube-dl-src-2012.12.12.tar.gz',
          'notes': ['The default output template will switch to a more human-readable format soon. Use --id for the old behavior.']
      }
   ]
}

It's not like we'd write many notes at all.

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Dec 16, 2012

Yeah, I explained it badly, I meant to include it in the JSON like in your example.
So, going with the update notes!

@hickford
Copy link

@hickford hickford commented Jan 26, 2013

Yes please, saving the download with a meaningful name would be much more useful. While it's easy enough to rename a single download by hand when you forgot --title, renaming a whole playlist of downloads is unpleasant.

@phihag phihag closed this in 08b2ac7 Apr 28, 2013
@surve5
Copy link

@surve5 surve5 commented May 31, 2013

can i show the progress bar on php page while the video gets downloaded.help me plz

@phihag
Copy link
Contributor

@phihag phihag commented May 31, 2013

@surve5 Please open a new issue - otherwise, it will be impossible to follow the discussion. Also, can you clarify what you mean by php page? At the very least, post a link to your code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.