<div class="gmail_quote">On Sun, Sep 5, 2010 at 5:17 PM, David Nolden <span dir="ltr"><<a href="mailto:zwabel@googlemail.com">zwabel@googlemail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
About the Git menu entries again:<br>
- I find the "Update" menu entry confusing. Seems like it does the<br>
same as "Pull", but there is also a "Pull" entry, so the "Update"<br>
entry should disappear.<br></blockquote><div>I agree, but I'm not sure if we should go deeper, maybe we could consider pull a type of update or something like that.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

- Also the "Revision History" entry does nothing at all here, and<br>
since there's "History", "Revision History" should also be removed.<br></blockquote><div>It doesn't do anything because you #if 0'd the slot, I'm not sure why. It does something and it looks ugly, as I said we should review that and probably refactor the code inside the git plguin instead of vcs/dvcs but i think it's still usefull to see the branch history.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
- Also the "Compare to Base" and "Compare to Head" entries are quite<br>
annoying, as I simply don't know what the difference between them is,<br>
and user also won't get it. "Compare to Head" should IMO go away, and<br>
maybe "Compare to Base" should be changed to "Local Changes..." or<br>
something like that. However since those changes can also be inspected<br>
when pushing "Commit...", maybe that entry could also go.<br></blockquote><div>I agree, Andreas any thought about that? (I'm asking because I think you added that)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

- I think "Branch Manager" should be renamed to "Branches..."<br></blockquote><div>I agree, changed it.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

- Since "Push..." and "Pull..." always operate on the whole<br>
repository, I think those should only be shown in the context-menu of<br>
the whole repository. The same is probably true for the stash- and<br>
branch-entries<br></blockquote><div>Meh... I thought about that too... but putting it there i think it would make these features a little harder to find.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

- "Push..." and "Pull..." are misleading, as their name suggests that<br>
there will be a menu (due to the "..."), but at least for "Pull" there<br>
is none.<br></blockquote><div>Agreed</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5"><br>
Greetings, David<br>
<br>
--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</div></div></blockquote></div><br>