Bug 510945 - Provide option to copy file names from staging view to clipboard
Summary: Provide option to copy file names from staging view to clipboard
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.11   Edit
Assignee: Fabian Pfaff CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, helpwanted, usability
Depends on:
Blocks:
 
Reported: 2017-01-24 07:13 EST by Philippe Brossel CLA
Modified: 2018-02-03 04:54 EST (History)
5 users (show)

See Also:


Attachments
Where to make it available (152.07 KB, image/jpeg)
2017-01-24 07:13 EST, Philippe Brossel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Brossel CLA 2017-01-24 07:13:07 EST
Created attachment 266432 [details]
Where to make it available

It would help me a lot in injecting the list of changed files into our internal documentation system.

'Select all' is already there. Would be nice to be able to copy.
Comment 1 Matthias Sohn CLA 2017-01-24 08:19:13 EST
+1, maybe you want to contribute this ?

Find the contributor guide here
https://wiki.eclipse.org/EGit/Contributor_Guide
Comment 2 Philippe Brossel CLA 2017-01-24 13:40:43 EST
(In reply to Matthias Sohn from comment #1)
> +1, maybe you want to contribute this ?
> 
> Find the contributor guide here
> https://wiki.eclipse.org/EGit/Contributor_Guide

yes, I'll give it a try.
Comment 3 Lars Vogel CLA 2017-04-11 12:04:51 EDT
(In reply to Philippe Brossel from comment #2)
> yes, I'll give it a try.

Any update here Philippe?
Comment 4 Philippe Brossel CLA 2017-04-12 03:29:58 EDT
Sorry, didn't have the time up to now.
I'll dive into it during next summer holiday.
Comment 5 Lars Vogel CLA 2017-04-12 06:26:41 EDT
(In reply to Philippe Brossel from comment #4)
> Sorry, didn't have the time up to now.
> I'll dive into it during next summer holiday.

Fabian, please have a look if you can fix that. This is really annoying.
Comment 6 Philippe Brossel CLA 2017-04-12 10:04:39 EDT
Lars, can you explain what you mean by 'annoying' ?

Remember I opened the bug to learn how to contribute to projects not to lose anyone's time.

The expected function will probably not be of any interest to anyone but me.

So either close the bug without losing more time if it is hindering
or let it open for just me to get started with contributing.

In this case I will 'fix it' in august this year.
Comment 7 Lars Vogel CLA 2017-04-12 11:05:03 EDT
(In reply to Philippe Brossel from comment #6)
> Lars, can you explain what you mean by 'annoying' ?

Sorry, this was indeed unclear. 

Let me try to clarify: I find it annoying that copy and paste does not work in Git Staging view. The last weeks I had to copy the file names which I committed and this missing feature was super annoying.

This was not at all a statement about you, again sorry in case that was unclear.

And I hope that Fabian has time to contribute this within the next days, hence I asked him to have a look at this issue.

Thanks for reporting and I'm sure if Fabian solves this, you will find another issue to contribute to.
Comment 8 Philippe Brossel CLA 2017-04-12 11:16:02 EDT
OK, understood.

So now we're at least 2 guys interested in this new feature. 
Just kidding ...
Comment 9 Eclipse Genie CLA 2017-06-03 16:43:42 EDT
New Gerrit change created: https://git.eclipse.org/r/98589
Comment 10 Fabian Pfaff CLA 2017-06-03 16:49:20 EDT
I've added a gerrit but it's still WIP. I'm not sure I'm taking the right approach in the implementation but it works.

Do you think the full file path or only the file name should be copied?
I think there can be made a case for both options.
Comment 11 Lars Vogel CLA 2017-06-28 17:18:33 EDT
(In reply to Fabian Pfaff from comment #10)
> Do you think the full file path or only the file name should be copied?

I suggest to copy only the file name.
Comment 12 Philippe Brossel CLA 2017-06-29 09:52:45 EDT
Would this not be confusing when having the same filename at different hierachy levels ? (e.g 'plugin.xml').

I would personaly prefer something like the eclipse context menu option 'Copy qualified name', yielding e.g. '/com.eupen.test.pluginThree/src/com/eupen/test/pluginThree/export/EPluginThreePublicClassOne.java' for the class 'EPluginThreePublicClassOne'.
Comment 13 Lars Vogel CLA 2017-06-29 10:31:30 EDT
(In reply to Philippe Brossel from comment #12)
> Would this not be confusing when having the same filename at different
> hierachy levels ? (e.g 'plugin.xml').
> 
> I would personaly prefer something like the eclipse context menu option
> 'Copy qualified name', yielding e.g.
> '/com.eupen.test.pluginThree/src/com/eupen/test/pluginThree/export/
> EPluginThreePublicClassOne.java' for the class 'EPluginThreePublicClassOne'.


Also fine for me.
Comment 14 Lars Vogel CLA 2017-07-13 13:23:45 EDT
(In reply to Lars Vogel from comment #13)
> Also fine for me.

I frequently need only the file name as displayed (not the full qualified name), e.g., for the commit message. @Fabian, if you provide a patch, please use this as the first version. We or Philippe may extend this later, e.g., with a shift +copy+c copy.
Comment 15 Matthias Sohn CLA 2017-07-13 17:34:13 EDT
(In reply to Lars Vogel from comment #14)
> (In reply to Lars Vogel from comment #13)
> > Also fine for me.
> 
> I frequently need only the file name as displayed (not the full qualified
> name), e.g., for the commit message. @Fabian, if you provide a patch, please
> use this as the first version. We or Philippe may extend this later, e.g.,
> with a shift +copy+c copy.

Try content assist for filenames of files which are staged for the commit. This was implemented a long time back. Type the first few characters of the file name of a file which is staged for the commit and then click CTRL-spacebar to experience the magic of content assist :-)
Comment 16 Lars Vogel CLA 2017-07-13 17:45:01 EDT
(In reply to Matthias Sohn from comment #15)
> Try content assist for filenames of files which are staged for the commit.
> This was implemented a long time back. 

Cool feature, thanks for notifying. I primarily use the tree representation. Would be nice if the tree nodes could be added to the content assists. I open a separate bug for that.
Comment 17 Lars Vogel CLA 2017-07-13 17:49:08 EDT
(In reply to Lars Vogel from comment #16)
> I open a separate bug for that.

Bug 519660.
Comment 18 Philippe Brossel CLA 2017-10-13 14:19:25 EDT
(In reply to Lars Vogel from comment #14)
> (In reply to Lars Vogel from comment #13)
> > Also fine for me.
> 
> I frequently need only the file name as displayed (not the full qualified
> name), e.g., for the commit message. @Fabian, if you provide a patch, please
> use this as the first version. We or Philippe may extend this later, e.g.,
> with a shift +copy+c copy.

Any update ? 

Can I help to finalize Fabian's work ?
Comment 19 Matthias Sohn CLA 2017-10-15 05:20:40 EDT
(In reply to Philippe Brossel from comment #18)
> (In reply to Lars Vogel from comment #14)
> > (In reply to Lars Vogel from comment #13)
> > > Also fine for me.
> > 
> > I frequently need only the file name as displayed (not the full qualified
> > name), e.g., for the commit message. @Fabian, if you provide a patch, please
> > use this as the first version. We or Philippe may extend this later, e.g.,
> > with a shift +copy+c copy.
> 
> Any update ? 

there was no update on https://git.eclipse.org/r/#/c/98589/ since June 3 and there are a number of comments on this patch which need to be fixed

> Can I help to finalize Fabian's work ?

sure
Comment 20 Eclipse Genie CLA 2018-02-03 04:53:10 EST
Gerrit change https://git.eclipse.org/r/98589 was merged to [master].
Commit: http://git.eclipse.org/c/egit/egit.git/commit/?id=14652472b2aa7901279a01d76cc1b6dff3fa8207