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

Minor change in the wording in the comment that starts an empty applications.css file #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gitt0h0
Copy link

@gitt0h0 gitt0h0 commented Dec 24, 2019

I found the wording of the empty applications.css file created by the JavaFXProjectWizard.java slightly odd. So I propose this minor change to use better words in this comment.

Proposing better wording for the empty .css file.
Minor update JavaFXProjectWizard.java
@tomsontom
Copy link
Contributor

Looks good but to allow me to merge this trivial change you need to sign the eca and update your commit with the right e-mail address

@gitt0h0
Copy link
Author

gitt0h0 commented Dec 26, 2019

Tom,
OK, thanks, I was not aware that my pull request would be noticed.
I’m a frequent user of Eclipse and I’m glad I could make a minor contribution this way.
Happy holidays !
Tom

@tomsontom
Copy link
Contributor

Every small change counts and is important!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants