Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364933 - 'Delete Tag...' should be 'Delete Tag'
Summary: 'Delete Tag...' should be 'Delete Tag'
Status: VERIFIED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 1.2   Edit
Hardware: All All
: P3 trivial (vote)
Target Milestone: 1.2   Edit
Assignee: Kevin Sawicki CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-28 05:15 EST by Dani Megert CLA
Modified: 2011-12-06 06:22 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2011-11-28 05:15:49 EST

    
Comment 1 Dani Megert CLA 2011-11-28 05:18:50 EST
1.2.0.201111201913.

The new 'Delete Tag...' action should not have "...". This is only added when additional user input/selection is required to complete the action.
Comment 2 Kevin Sawicki CLA 2011-11-28 12:40:17 EST
So '...' shouldn't be used when showing a confirmation dialog?
Comment 3 Dani Megert CLA 2011-11-29 01:59:32 EST
(In reply to comment #2)
> So '...' shouldn't be used when showing a confirmation dialog?

Exactly. Only when the user is requested for additional input to actually run the action one uses the "...".
Comment 4 Kevin Sawicki CLA 2011-11-29 12:12:24 EST
Thanks Dani for the clarification
Comment 5 Kevin Sawicki CLA 2011-11-29 18:35:19 EST
Proposed fix pushed to: http://egit.eclipse.org/r/4697
Comment 6 Matthias Sohn CLA 2011-11-29 18:55:40 EST
merged as 1da78ce7306b4d2f1ffcfc79b3cc15eabd277b60
Comment 7 Dani Megert CLA 2011-12-06 06:22:04 EST
Verified in 1.2.0.201112051848.