Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 313862

Summary: PushWizard appears unnecessary complex
Product: [Technology] EGit Reporter: Lars Vogel <Lars.Vogel>
Component: UIAssignee: Project Inbox <egit.ui-inbox>
Status: CLOSED DUPLICATE QA Contact:
Severity: enhancement    
Priority: P3 CC: b.muskalla, ekke, Lars.Vogel, matthias.sohn, matthiasheinrich, milesparker, remy.suen, robin, stefan.lay
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on:    
Bug Blocks: 312586    
Attachments:
Description Flags
Push Dialog none

Description Lars Vogel CLA 2010-05-21 02:58:35 EDT
The EGit push dialog seems a bit to complex. Would it be possible to use labels which are more descriptive as they are now.

For example the following labels bear no meaning for me:

Source ref 
Destination ref
Add Spec - What is a spec?
Add delete ref specfication
Remote ref to delete

There *must* be better names then these... ;-)

These fields I get I I suggest to rename them:

Add all branches spec -> All all branches
Add all tags spec -> All all tags
Comment 1 Matthias Sohn CLA 2010-05-21 03:32:56 EDT
- first this needs some documentation, I will care for that soon since we will update the documentation for the upcoming Helios release
- for the simplification discussion have a look at bug 312586, hence linking it to this one
- as a first step to simplify push and fetch Mathias recently implemented http://egit.eclipse.org/r/#change,715. Maybe you want to give this a try using a recent nightly ? Go to the repositories view as described here http://wiki.eclipse.org/EGit/User_Guide#Configuration_of_remote_Repositories, select a remote and from the context menu create a push or fetch and configure it (ok, here you still see the offending labels). Then you can push and fetch from the context menu of these new entries. This also needs proper documentation.
Comment 2 Lars Vogel CLA 2010-05-21 03:59:02 EDT
I'm using the latest nightly build. Perhaps this change is not yet in the build?
Comment 3 Matthias Sohn CLA 2010-05-31 20:06:14 EDT
I added some documentation for the Push Wizard here :
http://wiki.eclipse.org/EGit/User_Guide#Pushing_to_other_Repositories 

Direct Push is described here :
http://wiki.eclipse.org/EGit/User_Guide#Direct_Fetch_and_Push_Support
Comment 4 Lars Vogel CLA 2010-06-07 08:44:08 EDT
Created attachment 171257 [details]
Push Dialog 

Hi Matthias,

in the latest nightly build the push dialog does not yet contain the configured remote repository (see screenshot). 

Best regards, Lars
Comment 5 Miles Parker CLA 2010-12-30 16:34:53 EST
I look forward to taking a look as there are a lot of changes and I'm unfamiliar w/ Gerrit it's hard for me to understand exactly what's happening, but it does sound like a great deal of progress. One suggestion re:
"What is still missing is some smart way of editing the Push/FetchRefSpecs, perhaps some buttons like "Suggest Gerrit pattern" or such should be available..." That doesn't sound to me like a simplification. ;D One thing that P2 does that I think works quite well is to use the hyper-link style for "alternative things you can do that are related to the current task". For example, when you're n the feature install dialog you can click on "what is already installed".
Comment 6 Matthias Sohn CLA 2011-01-13 05:29:47 EST
(In reply to comment #4)
> Created attachment 171257 [details]
> Push Dialog 
> 
> Hi Matthias,
> 
> in the latest nightly build the push dialog does not yet contain the configured
> remote repository (see screenshot). 

yep, since this change is still in review you have to build that locally, our attempt to get hudson builds also for changes pending in review using gerrit-hudson plugin did not yet succeed.
Comment 7 Robin Stocker CLA 2013-09-27 11:10:36 EDT
Please see bug 312586 comment 16 for pointers for further improvements.

*** This bug has been marked as a duplicate of bug 312586 ***