There is a bug in the diff page where the titles are not printing out properly.

I also took the liberty of doing some code cleanup while i was in there.

CommentFileSizeAuthor
#1 features-diff_component_titles-1429408-1.patch1.27 KBjoelcollinsdc

Comments

joelcollinsdc’s picture

joelcollinsdc commented
StatusFileSize
new features-diff_component_titles-1429408-1.patch1.27 KB
joelcollinsdc’s picture

joelcollinsdc commented
Status: Active » Needs review
joelcollinsdc’s picture

joelcollinsdc commented

anyone? this is a reaaaallly trivial patch.

matglas86’s picture

matglas86 commented
Status: Needs review » Reviewed & tested by the community

I made the exact same changes for myself. Tested and ready. :) Please commit.

mpotter’s picture

mpotter commented
Status: Reviewed & tested by the community » Needs work

This patch does not apply to the 7.x-1.x-dev version.

Edited: Bah, nevermind, had the wrong version myself.

mpotter’s picture

mpotter commented
Status: Needs work » Reviewed & tested by the community

Seems to be a good patch and is definitely useful. Will try to get it committed soon.

mpotter’s picture

mpotter commented
Status: Reviewed & tested by the community » Patch (to be ported)

Committed 2353da7

mpotter’s picture

mpotter commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
joelcollinsdc’s picture

joelcollinsdc commented
Status: Patch (to be ported) » Closed (fixed)

AFAIK it wasn't a bug in 6.x. thanks!