| Summary: | Add "Attach screenshot" to task editor context menu | ||||||
|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Willian Mitsuda <wmitsuda> | ||||
| Component: | Mylyn | Assignee: | Mik Kersten <mik.kersten> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | minor | ||||||
| Priority: | P5 | ||||||
| Version: | dev | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Windows Vista | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Willian Mitsuda
I'm a bit torn on this one. "Attach File" we know to be a common action, whereas "Attach Screenshot" is a more specialized action and mostly used for bug reporting. I'd like to leave this open for now, and consider further when we next iterate on the editor and its actions. I missed that because I actually only use the context menu to attach things. Right-click -> Attach file. It is the quickest way to invoke attach. The another way involves having to scroll the task editor to the attachment section, expand it and then press the button. I rarely use the last one. +1 for adding screenshot action to context menu Personally I am either using drag and drop when attaching files or attaching from the clipboard, but "attach file" is really weird action name for creating attachments from the clipboard... (In reply to comment #3) > Personally I am either using drag and drop when attaching files or attaching > from the clipboard, but "attach file" is really weird action name for creating > attachments from the clipboard... > Good point. I think "attach file" sounds strange to me because most times I use it to attach patches from clipboard. Attach from...? "Add Attachment..." Note that in the table the action is no longer called "Attach File" but "Add File...", since it is adding a file to the attachments of the bug. How about we add the following to the editor's popup menu? Attachments (group) Add File... Add Screenshot... This way it will be consistent with the editor's actions. (In reply to comment #7) > Note that in the table the action is no longer called "Attach File" but "Add > File...", since it is adding a file to the attachments of the bug. It is adding an attachment, not a file. Though attachment can be created from a file. Note that Bugzilla web UI don't refer to a file in the corresponding action - "Add an attachment (proposed patch, testcase, etc.)". Sigh, yes, the wording is a bit confusing in that sense. Anyone have better suggestions? What about simply "Attach..." and "Attach Screenshot..."? (In reply to comment #10) > What about simply "Attach..." and "Attach Screenshot..."? Agreed that this is the simplest. Done. Actions are are now in the Task Editor's context menu as well. Created attachment 81227 [details]
mylyn/context/zip
I think that the image here is too dark and is distracting in the context menu full of muted icons. Could you either create a different icon or mute the current one a bit so that it doesn't stand out so much? (In reply to comment #13) > I think that the image here is too dark and is distracting in the context menu > full of muted icons. Could you either create a different icon or mute the > current one a bit so that it doesn't stand out so much? +1 I've been noticing this lately myself. Agreed. I lightened the icon. |