Edgewall Software
Modify

Opened 16 years ago

Closed 16 years ago

#7814 closed enhancement (fixed)

[patch] Property renderer for svnmerge properties

Reported by: ebray Owned by: ebray
Priority: normal Milestone: 0.12
Component: version control Version: devel
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

When properties such as svnmerge-blocked and especially svnmerge-integrated are set on a branch, the values of those properties will often get very long. And because svnmerge doesn't insert spaces in the list of revision numbers, the entire property value will run on a single, long line.

This SvnMergePropertyRenderer simply inserts zero-width spaces between revisions and revision ranges after the commas so that it can be broken up into multiple lines. It's a very simplistic approach, but it's working for me in FF3 and IE6.

Sure this could be provided as a plugin, but I think that the use of svnmerge is still common enough that I think this ought to be provided by Trac. If the general consensus is against that though I'll just release it as a plugin.

Attachments (1)

svnmerge-property-renderer-r7676.patch (839 bytes ) - added by ebray 16 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 by Remy Blank, 16 years ago

Milestone: 0.12
Owner: set to ebray

This sounds like an excellent idea. svnmerge-* properties are hidden on t.e.o, probably due to the problem exposed here.

I'll test and commit the patch.

comment:2 by Remy Blank, 16 years ago

Resolution: fixed
Status: newclosed

Patch applied in [7688].

I have integrated the renderer into SubversionPropertyRenderer instead of having a separate renderer, and I have also included the new svn:mergeinfo introduced in Subversion 1.5.

I have also taken the liberty of adding you to the THANKS file, I hope you don't mind. Actually, I was surprised that you were not already listed there!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain ebray.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from ebray to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.