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)
Change History (3)
by , 16 years ago
Attachment: | svnmerge-property-renderer-r7676.patch added |
---|
comment:1 by , 16 years ago
Milestone: | → 0.12 |
---|---|
Owner: | set to |
comment:2 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
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!
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.