468560 – improve inline comment dialog

Bug 468560 - improve inline comment dialog

Summary: improve inline comment dialog

Status:	RESOLVED FIXED

Alias:	None

Product:	z_Archived
Classification:	Eclipse Foundation
Component:	Mylyn (show other bugs)
Version:	unspecified
Hardware:	PC Windows 7

Importance:	P3 enhancement (vote)
Target Milestone:	2.8
Assignee:	Chris Poon
QA Contact:

URL:
Whiteboard:
Keywords:	contributed, noteworthy

Depends on:
Blocks:

Reported:	2015-05-27 19:43 EDT by Sam Davis
Modified:	2015-08-26 12:24 EDT (History)
CC List:	0 users

See Also:	Gerrit Change Gerrit Change Git Commit Git Commit Gerrit Change Gerrit Change Git Commit Git Commit

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

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.