Re: Suggestion: Branch/tag-dialog change

Hans-Emil Skogh wrote:

> I would like to suggest a modification to the "Copy (Branch / Tag)"-dialog.
> In my opinion we should change the order (and default selection) of the
> radio button choices. I would suggest to move "Specific version in
> repository" to the top and make it default with the current revision of
> the selected WC entered in the revision field. "HEAD revision in the
> repository" should be moved to the bottom.

I don't like the idea of moving controls around. We had *exhausting*
discussions about the branch/tag and merge dialog already on how to do
it best. And I really don't want to get there again...

> The reason I'm suggesting this is because it's an unsafe working
> practice to make a tag from repository-HEAD since there's no way to know
> what you are tagging. With branching it's not as bad, but still a "worst
> practice"*.
> I would even go as far as wanting to provide some kind of warning for
> using "HEAD revision in the repository", but it might be enough to move
> it to the bottom of the choices. Having it as the default choice is,
> well, bad. :-)

While I agree with you that in general, creating a branch/tag directly
from HEAD is bad (someone else could commit while you don't expect it),
I'm not sure if we should fill in the revision of the working copy. As
you might know, there just is no single working copy revision. We'd have
to do the same as SubWCRev does now (find the highest 'last committed'
revision), which takes a while to compute.

Thoughts?

> I would also suggest changing some texts to clarify:
> "From WC at URL:" -> "From URL:" or "From URL in repository:" or "Origin
> URL:" or similar. The path is showing the source path in the repository
> that the copy will be made from. Mentioning the WC there is confusing.
> "Repository" ("box" caption) -> "Copy" or "Details" or similar maybe?
> "Repository" is to broad and unclear.

Again (see above) I don't want to change any text or controls in that
dialog unless there's a *very, very good reason* for it.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org