hide svk:merge and other human unreadable properties in the source browser and changeset view

[jonas]

Fixed in [2075].

[mrowe]

IMO this "fix" is invalid. Hiding certain properties is more confusing than displaying those that aren't "human readable". It is impractical and confusing for Trac to keep a list of non-human-readable properties and explicitly filter them out. Trac should simply display what is in the repository.

[eblot]

IMHO, this kind of filter is handy, because users as myself really don't want to bother with the internal keys of the repository back end, as they provide no more info than noise.

However, I agree on one point: it's impractical for Trac to keep a list of such properties, and I think a more 'natural' location for this definition would be in the INI project file, so that admins can select which properties they wish to hide, if any.

[cboos]

I agree with Mark, I think the svk:merge should not be arbitrarily hidden.
I think a solution similar to the one I suggested for #1601 could be used here.

A generic HiddenPropertyRenderer could be selected in the trac.ini for the
non-human-readable properties, and this would be more flexible than being
hard-coded in the source.

OTOH, a specific SvkPropertyRenderer can be written to display
the svk:merge property in a more pleasant way.

[cmlenz]

How about just making this configurable as eblot suggested, for example:

[browser]
hide_properties = svk:merge, svn:eol-style

This should affect both the browser and the changeset view.

I don't think we should have components for displaying properties, at least not at this point. We really want to finish up 0.9 for a beta release, so a simple fix is the only thing that'll have a real chance to get in.

[anonymous]

Can the properties section be made to fold/unfold on page? If there's a property, readable or not, some would want to see it, say, when troubleshooting things.

[eblot]

fold/unfold sounds like Javascript. There have already been a couple of ML theads about this topic 8)

Or, the page could have a 'property' HTML division to select which properties should be shown, something similar to the options available for the timeline:
I share the opinion of cmlenz: make something simple enough so that it can integrate into 0.9 release, and postpone a more flexible solution for 1.0 or later.

[cboos]

Configurable list of hidden properties

[cboos]

Ok, let's post-pone the custom property renderers,
and in the meantime implement cmlenz's suggestion.

Would attachment:browser_hide_properties.diff​ be OK?

[cmlenz]

The patch looks good.

I think after it has been applied, we should close this ticket. #1601 covers the property renderers, no?

[cboos]

Above patch applied in [2076]. I'll add a ref on #1601.