#10593 closed enhancement (fixed)
Template enhancement
Reported by: | sdegrande | Owned by: | sdegrande |
---|---|---|---|
Priority: | normal | Component: | DiscussionPlugin |
Severity: | normal | Keywords: | |
Cc: | Radek Bartoň | Trac Release: | 1.0 |
Attachments (4)
Change History (16)
Changed 12 years ago by
Attachment: | forum-list.png added |
---|
Changed 12 years ago by
Changed 12 years ago by
Attachment: | rounded_template.patch added |
---|
comment:1 follow-up: 2 Changed 12 years ago by
Cc: | Radek Bartoň added; Ryan J Ollos removed |
---|---|
Owner: | changed from Radek Bartoň to Ryan J Ollos |
comment:2 follow-up: 4 Changed 9 years ago by
Status: | new → accepted |
---|
Replying to rjollos:
I'll keep this ticket on my radar, but I'd like to avoid forking for 1.0 until we get a few more bugs fixed.
My future self says ... Unacceptable! Supporting old versions of Trac is a waste of time. Branch and support 1.0 and later :)
comment:3 Changed 9 years ago by
sdegrande: would you consider rebasing this patch on the 1.0 branch so that it applies cleanly? Thanks!
comment:4 Changed 9 years ago by
Replying to rjollos:
Replying to rjollos:
I'll keep this ticket on my radar, but I'd like to avoid forking for 1.0 until we get a few more bugs fixed.
My future self says ... Unacceptable! Supporting old versions of Trac is a waste of time. Branch and support 1.0 and later :)
Well, I think that you know my humble opinion on that point... Just move on !
Changed 9 years ago by
Attachment: | adapt_template_to_trac_new_style.patch added |
---|
comment:5 Changed 9 years ago by
I attached a rebased version of the patch (attachment:adapt_template_to_trac_new_style.patch), also including the CSS changes to have rounded box around lists and list elements, to mimic new Trac style.
comment:7 follow-up: 10 Changed 9 years ago by
sdegrande: You now have commit access to both the DiscussionPlugin and DownloadsPlugin. Feel free to push patches directly, or post to a ticket first if for any reason you'd like a second opinion.
comment:9 Changed 9 years ago by
Owner: | changed from Ryan J Ollos to sdegrande |
---|
comment:10 Changed 9 years ago by
Replying to rjollos:
sdegrande: You now have commit access to both the DiscussionPlugin and DownloadsPlugin. Feel free to push patches directly, or post to a ticket first if for any reason you'd like a second opinion.
Thanks.
I do not consider myself as a good enough python coder to avoid a code review from an expert if I was to submit some 'complex changes'. So yes, I would first post a ticket.
I'll keep this ticket on my radar, but I'd like to avoid forking for 1.0 until we get a few more bugs fixed. I'm not very proficient in CSS either, so it will take me a bit of time to get this committed.