1

Resolved

Output simpeler HTML elements for each message

description

The current implementation (v0.2) creates a new Table for each message (while already inside another table). The html also contains a lot of static styles, these can not be overridden by a css file.
 
Suggestion:
 
Instead of using ~30 lines of code to generate a new table use either a Template string or refactor it into different methods, the RenderMessages() is just to long and not easy to read.
 
One solution would be:
 
private const String newMessageTable = "<li><img src=\"{imagesrc}\" alt=\"face\"><caption>{username}</caption><p>{message}</p></li>";
 
And inside your render message you could use 3 replace calls to create your new html.

comments

nickboumans wrote May 6, 2009 at 6:36 AM

Nice replacement! Easy, but verry elegant! Thanks! Will replace it at new release.

wrote May 6, 2009 at 6:36 AM

nickboumans wrote May 6, 2009 at 6:37 AM

update in new release

** Closed by nickboumans 5/5/2009 11:36 PM

nickboumans wrote May 6, 2009 at 6:37 AM

wrote May 6, 2009 at 6:37 AM

wrote Feb 2, 2013 at 5:26 AM

wrote May 14, 2013 at 3:48 AM

wrote May 14, 2013 at 3:48 AM

wrote Jun 4, 2013 at 1:07 AM