Re: Issue 3160 in reviewboard: "Add comment" should be "New comment" in diff view
Comment #3 on issue 3160 by trowb...@gmail.com: "Add comment" should be "New comment" in diff view http://code.google.com/p/reviewboard/issues/detail?id=3160 I'm not sure there's really a tremendous difference between the language, but it may make some improvement. I'll push a change to do this on the master branch (2.0+) but I'm not sure about making a UI change like this in the 1.7.x releases. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 3160 in reviewboard: "Add comment" should be "New comment" in diff view
Comment #2 on issue 3160 by goo...@keithmoyer.com: "Add comment" should be "New comment" in diff view http://code.google.com/p/reviewboard/issues/detail?id=3160 Thank you for your attention. I understand your desire to merge the issues and am not objecting to that action. However, I want to reiterate my reasoning for purposely separating it: As you mention, the solution to issue 1426 would be a major undertaking. _This_ issue, however, would be a simple text substitution and would be beneficial with or without any further work on issue 1426. I only pointed out issue 1426 at all, as I thought separately making this change would actually be a quick _improvement_ (not complete fix) toward addressing the confusion mentioned in issue 1426. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 3160 in reviewboard: "Add comment" should be "New comment" in diff view
Updates: Status: Duplicate Mergedinto: 1426 Comment #1 on issue 3160 by trowb...@gmail.com: "Add comment" should be "New comment" in diff view http://code.google.com/p/reviewboard/issues/detail?id=3160 I'm going to merge this, because I think any solution to 1426 will need to involve a redesign of this whole UI. I'll make sure that that bug report has your observations as well. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Issue 3160 in reviewboard: "Add comment" should be "New comment" in diff view
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 3160 by goo...@keithmoyer.com: "Add comment" should be "New comment" in diff view http://code.google.com/p/reviewboard/issues/detail?id=3160 What version are you running? 1.7.12 What's the URL of the page this enhancement relates to, if any? http://site/r/1/diff/ Describe the enhancement and the motivation for it. 1. Person A posts a comment/review 2. Person B clicks on the blue bubble or on the same line number 3. Person B sees the previous comment in a blue box and a green box ready for input that says "Add comment". 3. Person B is confused on whether the green box is a reply or a new comment. I would expect that the title of the green box is "New comment", as "Add comment" can be read as adding a comment/reply to the review comment seen in the blue box. What operating system are you using? What browser? Windows 7, Firefox Please provide any additional information below. This is related to issue 1426, but I feel it is a separate issue (issue 1426 is about the confusion when clicking the reply button, this about the confusion on whether you need to click the reply button or whether the nice big text box is already a reply). I plan to address this in a translation file for now (along with translating "Submitted" to "Completed", but I can understand that that may not be universally better), but having it more clear in the actual application would be preferred. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
