Add 'counter-comments' to the comments page

[micha]

Integrate a member's written comments into the CommentsPage? that lists all the comments he received. This should look similar to the way, CS shows their comments.

I add an image to show what I'm aiming for.

[micha]

Comments in conversation view

[micha]

commited my changes in 636cd63. Hopefully will be released with the new milestone.

[planetcruiser]

Replying to micha:

commited my changes in 636cd63. Hopefully will be released with the new milestone.

let's put it this way, the milestone won't be released before this ticket is closed. :)

i deployed 636cd63 to alpha, but somehow it's lacking css. my comments to others have no special border and appear in the same way as comments for me. forgot to commit a file?

[micha]

A browser reload does the job... ;)

[planetcruiser]

looks really cool. :)

but i think the grey comments need a "from <member> to <other_member>" at the top to make clear that it's a "counter comment"

[planetcruiser]

Replying to planetcruiser:

but i think the grey comments need a "from <member> to <other_member>" at the top to make clear that it's a "counter comment"

micha, what about this? do you think you can add this as well?

i still need to do code review.. will do that tomorrow hopefully.

[micha]

Replying to planetcruiser:

but i think the grey comments need a "from <member> to <other_member>" at the top to make clear that it's a "counter comment"

Would you add this line on top and leave the rest of the comment as is? Or would you just replace the part "from <member>" with "from <member> to <other_member>" ? The latter looks better here and keeps consistency of the UI.

After an answer I'll commit the appropriate changes. I also found minor bugs that I'll fix right away.

[planetcruiser]

Replying to micha:

Would you add this line on top and leave the rest of the comment as is? Or would you just replace the part "from <member>" with "from <member> to <other_member>" ? The latter looks better here and keeps consistency of the UI.

sorry, i wasn't very clear, i also meant the latter one.

After an answer I'll commit the appropriate changes. I also found minor bugs that I'll fix right away.

cool! :)

[micha]

ok, done. see latest commit 85395d5 which also deals with this ticket: http://trac.bewelcome.org/ticket/1579

[crumbking]

an error show up before the "add comment" button

( ! ) Notice: Undefined offset: 0 in /home/www/alpha.bewelcome.org-85395d5/build/members/templates/comments.column_col3.php on line 4
Call Stack
#	Time	Memory	Function	Location
1	0.0002	71152	{main}( )	../index.php:0
2	0.0002	71272	main( )	../index.php:78
3	0.0021	325416	RoxLauncher->launch( )	../index.php:63
4	0.0191	714584	RoxLauncher->chooseAndRunApplication( )	../roxlauncher.php:22
5	0.0242	1242652	RoxFrontRouter->route( )	../roxlauncher.php:68
6	0.0325	1598836	RoxFrontRouter->route_normal( )	../roxfrontrouter.class.php:34
7	0.0361	1893880	RoxFrontRouter->runControllerMethod( )	../roxfrontrouter.class.php:224
8	0.0990	4396956	RoxFrontRouter->renderPage( )	../roxfrontrouter.class.php:351
9	0.1001	4441540	PageWithHTML->render( )	../roxfrontrouter.class.php:392
10	0.1001	4441540	PageWithHTML->printHTML( )	../html.page.php:18
11	0.1013	4448588	PageWithRoxLayout->body( )	../html.page.php:169
12	0.1014	4453848	require( '/home/www/alpha.bewelcome.org-85395d5/templates/shared/roxpage/body.php' )	../roxlayout.page.php:123
13	0.1357	5109180	MemberPage->columnsArea( )	../body.php:21
14	0.1531	5507996	CommentsPage->column_col3( )	../member.page.php:151
15	0.1531	5507996	VisualComponent->__call( )	../visualcomponent.php:0
16	0.1624	6187296	include( '/home/www/alpha.bewelcome.org-85395d5/build/members/templates/comments.column_col3.php' )	../visualcomponent.php:47

[micha]

Just fixed in c9ff12c. Please deploy and test again.

[globetrotter_tt]

some minor ranting ;-): The counter comment is located closer to the next comment than to the comment that it belongs to. Maybe we can increase the bottom margin a bit? Or put a light border around both comments?

[micha]

I'd rather schedule a few css improvements and collect them alltogether in one commit. There's quite a lot layout and CSS stuff that could need an overhaul.

[planetcruiser]

micha:

on  http://alpha.bewelcome.org/members/planetcruiser/comments i see:

"from planetcruiser To - 2011-12-18 09:12:06"

two things:

1. the "To" should be in lower-case to match the "from", right?
2. the username of the comment recipient is not shown. is it the same for you?

[planetcruiser]

taking over

[planetcruiser]

translation "To" -> "to" fixed for all languages. the word is also used in ./build/lastcomments/templates/lastcomments.php - no idea what that is. i hope a lower-case "to" doesn't hurt there ;)

[planetcruiser]

oh, cool:  http://www.bewelcome.org/lastcomments - why is this not linked anywhere? :)

[planetcruiser]

Replying to planetcruiser:

the username of the comment recipient is not shown. is it the same for you?

fixed via  https://gitorious.org/bewelcome/rox/commit/d12d08fb179a4dde491b52b2b9033dd9cd27f87d and  https://gitorious.org/bewelcome/rox/commit/040cc86e218a9a942690fbff26f53fc1a4105449 :)