angelozerr / jdt-codemining

Eclipse plugin for JDT Java CodeMining
Eclipse Public License 1.0
298 stars 15 forks source link

Add support for enabling references only on classes #75

Closed gayanper closed 6 years ago

gayanper commented 6 years ago

In Some instances showing references on each method might not be that usefull. So it will be good if we can configure where to show the references on method level as well. Under in settings:

gayanper commented 6 years ago

If this is a good feature i can look into implementing it 😄

angelozerr commented 6 years ago

I think its a Good idea but imho We should have 3 préférences like enable for class méthod and fields in the future

gayanper commented 6 years ago

you mean like Show reference for:

?

angelozerr commented 6 years ago

Yes but for fields is not supported yet

gayanper commented 6 years ago

Yes so for now we can have only

I will try to send a screenshot after i do some changes so you could see if it done correctly.

angelozerr commented 6 years ago

Thank a lot @gayanper !

For your information, I'm trying to move JDT CodeMining to JDT at https://git.eclipse.org/r/#/c/126847/ for only references and implementation. References for fields is required for JDT, I will try to include too your work if you are OK in this gerrit patch.

gayanper commented 6 years ago

How does this looks like ? image Please let me know if you want different wordings or different presentation.

angelozerr commented 6 years ago

Perhaps remove the "Show" and have "On classes", no?

gayanper commented 6 years ago

image How is that ?

angelozerr commented 6 years ago

Yes it was my idea, do you like it too?

gayanper commented 6 years ago

Yes i think that's go idea

gayanper commented 5 years ago

@angelozerr when will this issue be released ?

angelozerr commented 5 years ago

There were a buld problem. Now it should be fixed. The idea is that on each commit, update site is done. You can follow jobs at https://opensagres.ci.cloudbees.com/job/jdt-codemining/

gayanper commented 5 years ago

Thanks got the changes with he latest build.