Patch #3815
closed
Format links in Timelog
0%
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
Related issues
Updated by Mischa The Evil over 14 years ago
I support this "patch". Personally I'd like to see textilisation of the timelog entries' descriptions too.
Any other opinions?
Updated by Alex Bevilacqua over 14 years ago
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)
Updated by Eric Davis over 14 years ago
- 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.
Updated by Henrik Ammer over 14 years ago
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
textilizablesoon, 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.
Updated by Eric Davis over 14 years ago
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.
Updated by Mischa The Evil about 14 years ago
- 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...