Comments summary/

Summary

For Milestonev0.085 - refactor query filters

Statusdone?

OpenedApr 22, 2007

Completed100%

CreatedApr 22, 2007 / krisp

ModifiedJun 21, 2007 / xl

View previous 5 versions

Assigned totino

No files uploaded

Did you noticed nonsense in Summary "Reply to Antwort auf Reply to I see..." (Request feedback of persons on items)? Languages are mixing together :)
I propose to name comments like this: "(Xn.Xn+1.Xn+2....) <root_comment_summary>" where Xn is a reply number on nth level.

(1) I see...
  (1.1) I see...
  (1.2) I see...
    (1.2.1) I see...
    (1.2.2) I see...
  (1.3) I see...
(2) nice
etc.


pixtur Apr 22, 2007	hmm... Yeah, I already noticed this behavior. It is not easy to get rid of it, because "Reply" is a text fragment added to the default title of the reply comment. It is completely reasonable that this default titles is changed / overwritten. Besides this I what to get rid of numbers. Having the item-ids to already too much numbers in Streber. There are several solutions for this: "> > xyz" "Re: Re: xyz" no prefix at all Use the default language of the streber-installation to get the prefix. I prefer method 4, although it will be some effort.
tino May 27, 2007	Reply to hmm... I prefer solution number 3! Actually the comments are intendend - I think that's enough. Take a look at the design on youtube.com - looks great.
pixtur May 30, 2007	Antwort auf Reply to hmm... Sounds reasonable...
xl May 30, 2007	Solution 3 would be fine.
pixtur May 31, 2007	ok... somebody wants to give it a try?
tino May 31, 2007	Okay... so no prefix at all? I'll check this.
pixtur May 31, 2007	yep... Just removing the prefix should be straight forward.
jd May 31, 2007	Number 2 but if you wont to change the subj to your own you'll have this ability.
tino Jun 2, 2007	In order to get the best scalability... I set up a config variable in the config file. Now you can set the prefix of the title of your comment easily. /** * Reply on comment prefix * 0 - no prefix at all (default) * 1 - Re: (international standard) * 2 - default "Reply to" text from language file */ 'REPLY_ON_COMMENT_PREFIX' => '0',
pixtur Jun 21, 2007	no options please Hi Tino, please try to avoid options. Since anybody stated, that the reply prefix isn't needed, we should should remove it. Adding an option will only increase complexity.
xl Jun 21, 2007	agree no prefix needed.

Comments summary / #4738

Summary

Attached files

Attached files

11 Comments

11 Comments

Comment / Update