search

neo618 / tortoisesvn

Automatically exported from code.google.com/p/tortoisesvn
0 stars 0 forks source link

Enhancement to the svn:externals property dialog #90

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Reported here:
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=28549
13

Currently, the Edit External Panel allows to edit both the peg and the
operative revision, but the operative revision is simply called
"Revision" and is more prominent than the peg revision.

The difference between peg and operative revision is subtle, but the
peg revision is what is most commonly referred to as "Revision", where
as the operative revision is Subversion's special sauce for corner
cases.

Therefore, I think the panel should be changed so that either the peg
revision is more prominent and is called "Revision", or the
distinction between the peg and the operative revision is made
clearer.

Moreover, the List External Panel only shows the operative revision in
a separate column, leaving the peg revision relatively hidden as part
of the URL. I think the peg revision should be displayed prominently
in it's own column, rather than being a suffix of the URL.

See attached pictures of a proposed GUI that would satisfy the need to
make peg revision more prominent.

Original issue reported on code.google.com by tortoisesvn on 14 Oct 2011 at 8:04

Attachments:

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r22098.

Original comment by tortoisesvn on 14 Oct 2011 at 8:09

GoogleCodeExporter commented 9 years ago 
Original poster here.

We're almost there. The GUI looks fine, but the dialog behaviors does not 
reflect the prominence of Peg versus Operative.

I thought of explaining in details, but then I decided to checkout the code and 
express it as a patch, so it will be clearer for everybody involved. Here it is.

PS: I could not compile this, since I don't have the Microsoft Visual C++ 
compiler your're using, plus I never worked with MFC before, but I tried to do 
it with care.

Original comment by francis....@cm-labs.com on 18 Oct 2011 at 12:19

Attachments:

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r22129.

Original comment by tortoisesvn on 18 Oct 2011 at 8:14