Enhancement to the svn:externals property dialog
GoogleCodeExporter opened this issue · comments
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:
This issue was closed by revision r22098.
Original comment by tortoisesvn on 14 Oct 2011 at 8:09
- Changed state: Fixed
- Added labels: ****
- Removed labels: ****
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
- Added labels: ****
- Removed labels: ****
Attachments:
This issue was updated by revision r22129.
Original comment by tortoisesvn on 18 Oct 2011 at 8:14
- Added labels: ****
- Removed labels: ****