| Summary: | allow to star a review | ||
|---|---|---|---|
| Product: | z_Archived | Reporter: | Tomasz Zarna <tomasz.zarna> |
| Component: | Mylyn | Assignee: | Project Inbox <mylyn-triaged> |
| Status: | CLOSED MOVED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | P3 | Keywords: | helpwanted |
| Version: | 2.0 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows 7 | ||
| Whiteboard: | |||
| Bug Depends on: | 426505 | ||
| Bug Blocks: | |||
|
Description
Tomasz Zarna
It's not clear to me how we would show this or whether it even makes sense in the Mylyn UI. (In reply to comment #1) > It's not clear to me how we would show this A Star/Unstar button in the header seems to be the most obvious approach. But even a button next to [Publish comments], [Fetch], etc would work for me. > or whether it even makes sense in the Mylyn UI. Well, I have a query in Task List with starred reviews. But in order to star a review I need to leave Eclipse. Interesting. I wonder how/whether we should support starring tasks in general. Perhaps there should be some mechanism that works for all connectors and that we could tie to the Gerrit notion of starred reviews. But given we already have categories and scheduling in the task list, maybe the only reason to support it is for consistency with the Gerrit web ui. Maybe the least invasive approach would be having an attribute for the option, e.g. even a combo box with "Starred"/"Unstarred" would do. (In reply to comment #3) > Interesting. I wonder how/whether we should support starring tasks in general. I understand your concern, the idea here is to add the option for Gerrit reviews only. btw, querying for starred reviews is covered in bug 414253 Thank you for your request. The bug has been marked helpwanted though to indicate that we would be happy to support a community contribution to resolve it. Mylyn has been restructured, and our issue tracking has moved to GitHub [1]. We are closing ~14K Bugzilla issues to give the new team a fresh start. If you feel that this issue is still relevant, please create a new one on GitHub. [1] https://github.com/orgs/eclipse-mylyn |