Opened 17 years ago
Closed 17 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 , 17 years ago
| Attachment: | svnmerge-property-renderer-r7676.patch added |
|---|
comment:1 by , 17 years ago
| Milestone: | → 0.12 |
|---|---|
| Owner: | set to |
comment:2 by , 17 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.