Bug 468560

Summary:	improve inline comment dialog
Product:	z_Archived	Reporter:	Sam Davis <sam.davis>
Component:	Mylyn	Assignee:	Chris Poon <chris.poon>
Status:	RESOLVED FIXED	QA Contact:
Severity:	enhancement
Priority:	P3	Keywords:	contributed, noteworthy
Version:	unspecified
Target Milestone:	2.8
Hardware:	PC
OS:	Windows 7
See Also:	https://git.eclipse.org/r/49943 https://git.eclipse.org/r/49874 https://git.eclipse.org/c/mylyn/org.eclipse.mylyn.reviews.git/commit/?id=5a7cf080ebd4fbd915ef4881d814c9777ffae73c https://git.eclipse.org/c/mylyn/org.eclipse.mylyn.reviews.git/commit/?id=cbc486fc22891bd2b5c8ab27bd2c9e3874f69e1b https://git.eclipse.org/r/54348 https://git.eclipse.org/r/54349 https://git.eclipse.org/c/mylyn/org.eclipse.mylyn.commons.git/commit/?id=5564fc7fe4268fa9ecd92f831577728a2203da0a https://git.eclipse.org/c/mylyn/org.eclipse.mylyn.reviews.git/commit/?id=2f29dc193ed772c9f2b038bc76b1488ba031ecc3
Whiteboard:

Description Sam Davis

2015-05-27 19:43:48 EDT

The inline comment dialog is awkward to use because there are so many clicks needed to perform any action. We should simplify the workflow. It would also be nice to be able to see the comment you're replying to as you type a reply.

Comment 1 Eclipse Genie

2015-07-14 17:41:49 EDT

WARNING: this patchset contains 1231 new lines of code and may require a Contribution Questionnaire (CQ) if the author is not a committer on the project. Please see:https://wiki.eclipse.org/Project_Management_Infrastructure/Creating_A_Contribution_Questionnaire

Comment 2 Eclipse Genie

2015-08-21 14:31:26 EDT

Gerrit change https://git.eclipse.org/r/49943 was merged to [master].
Commit: http://git.eclipse.org/c/mylyn/org.eclipse.mylyn.reviews.git/commit/?id=5a7cf080ebd4fbd915ef4881d814c9777ffae73c

Comment 3 Eclipse Genie

2015-08-21 14:32:47 EDT

Gerrit change https://git.eclipse.org/r/49874 was merged to [master].
Commit: http://git.eclipse.org/c/mylyn/org.eclipse.mylyn.reviews.git/commit/?id=cbc486fc22891bd2b5c8ab27bd2c9e3874f69e1b

Comment 4 Eclipse Genie

2015-08-21 17:48:23 EDT

New Gerrit change created: https://git.eclipse.org/r/54348

Comment 5 Eclipse Genie

2015-08-21 17:49:05 EDT

New Gerrit change created: https://git.eclipse.org/r/54349

Comment 6 Sam Davis

2015-08-21 18:00:02 EDT

Chris, can you take a look at the 2 reviews I pushed?

Comment 7 Eclipse Genie

2015-08-25 17:19:24 EDT

Gerrit change https://git.eclipse.org/r/54349 was merged to [master].
Commit: http://git.eclipse.org/c/mylyn/org.eclipse.mylyn.commons.git/commit/?id=5564fc7fe4268fa9ecd92f831577728a2203da0a

Comment 8 Eclipse Genie

2015-08-26 12:23:28 EDT

Gerrit change https://git.eclipse.org/r/54348 was merged to [master].
Commit: http://git.eclipse.org/c/mylyn/org.eclipse.mylyn.reviews.git/commit/?id=2f29dc193ed772c9f2b038bc76b1488ba031ecc3

Comment 9 Sam Davis

2015-08-26 12:24:10 EDT

All changes have been merged.