Patch #3815
closed
Added by Alex Bevilacqua over 15 years ago.
Updated over 12 years ago.
Description
This addresses the issue I reported earlier (#3810).
I'm not running this on a *nix system, so I don't have diff/patch available atm (though I could just install winmerge I guess :P).
/app/views/timelog/_list.rhtml
26: <td class="comments"><%= textilizable entry.comments %></td>
That's it :D
I support this "patch". Personally I'd like to see textilisation of the timelog entries' descriptions too.
Any other opinions?
I think textilisation of the descriptions is a good idea. (and next time i submit a patch, i'll make sure it's actually a patch :P)
- Category set to Time tracking
- Status changed from New to 7
- Assignee set to Eric Davis
This would be a good addition.
We might need to start looking at the performance of textilizable
soon, my performance statistics are showing a lot of time spent in rendering. Would probably be good to profile post 0.9.
Mischa The Evil wrote:
I support this "patch". Personally I'd like to see textilisation of the timelog entries' descriptions too.
Any other opinions?
There needs to be some restriction on what it outputs though. I personally use a lot of headers when adding comments which would look quite awkward in the timelog.
Eric Davis wrote:
We might need to start looking at the performance of textilizable
soon, my performance statistics are showing a lot of time spent in rendering. Would probably be good to profile post 0.9.
Since Redmine is a tool many will use at the same time textile takes it's time, therefor a lot of projects textiles the content into html that gets saved in a different column in the database table. It's a thought.
Henrik Jönsson wrote:
Since Redmine is a tool many will use at the same time textile takes it's time, therefor a lot of projects textiles the content into html that gets saved in a different column in the database table. It's a thought.
Agree. I opened #4482 for this with my thoughts.
- Category deleted (
Time tracking)
- Status changed from 7 to Closed
- Assignee deleted (
Eric Davis)
This actually is a partly duplicate of issue #202. Therefor and to keep the tracker clean I'll close this one...
Also available in: Atom
PDF