Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Help Button: Add information #127

Open
andrewclancy opened this issue May 28, 2015 · 10 comments
Open

Help Button: Add information #127

andrewclancy opened this issue May 28, 2015 · 10 comments

Comments

@andrewclancy
Copy link

The help button located at the bottom of the "Choose refactorings" box does not work for me. I tested on two different machines and resulted in the same problem.

OS: Windows 7

Eclipse Version: Luna Service Release 1 (4.4.1)

@JnRouvignac
Copy link
Owner

Hi Andrew,

Thanks for the bug report.

 

This help button is created by the Eclipse UI framework. I have two choices:

  1. hide it
  2. do something with it

What do you prefer? If you want to do something with it, what would be useful to you? As a user, what do you expect to see? What information would be useful to you after clicking that button?

 

If you do not have any preference I will simply hide it.

Note to self: add the following line after creating the dialog:

dialog.setHelpAvailable(false);

@andrewclancy
Copy link
Author

Hi,

Ohh I see what you mean. No problem I thought I would just raise it just incase it became a problem in the future. Although I would recommend a little information about what each refactoring actually does in the help button? This would be nice for first time users but is not a big issue.

@JnRouvignac
Copy link
Owner

Could you be more specific than "a little information", and come up with concrete examples?
Apart from stating "Strings" refactors string usage, I currently do not have a good idea on what to do.

Do you expect to see code samples demonstrating the changes?
Or would you want to see something like what currently is in the javadocs?
For example:
https://github.com/JnRouvignac/AutoRefactor/blob/master/plugin/src/main/java/org/autorefactor/refactoring/rules/StringRefactoring.java#L39

@andrewclancy
Copy link
Author

Have you used Sonar before? I like the way each violation is explained. For Auto Refactor I dont think code examples would be needed but an explanation about each code refactor would be more user friendly. This is just my opinion though :)

@JnRouvignac
Copy link
Owner

Yes I know very well Sonar :)
I introduced it in your company :)

Sonar is developed professionally. I am developing AutoRefactor on my free time. Coming up with an excellent level of documentation takes up a lot of time. I don't have this time.

You did nit answer: what do you think of this Javadoc? Would it suit your needs?

@andrewclancy
Copy link
Author

Good stuff :) yes that would be perfect, thanks for your response on this

@JnRouvignac
Copy link
Owner

OK I will try to see if I can do it.

@andrewclancy
Copy link
Author

Cool, if there is anything I can do to help let me know :-)

@JnRouvignac
Copy link
Owner

Resolving this is definitely more involved than what I had imagined:

I was hoping I could have some Java code triggered by the help button, that would generate the help information on the fly. However it does not seem possible.

I am reluctant for now to spend time generating the XMLs and HTML required to make the help system work.
Since such work could also be interesting to generate documentation for the web site, I might revisit this decision later.

@JnRouvignac
Copy link
Owner

A first step would be to have a tooltip in the "Choose refactorings" dialog, as reported by @xpoinsard.

Implemented in #129.

@Fabrice-TIERCELIN Fabrice-TIERCELIN changed the title Help Button: Does not respond Help Button: Addinformation Jul 9, 2017
@Fabrice-TIERCELIN Fabrice-TIERCELIN changed the title Help Button: Addinformation Help Button: Add information Jul 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants