-
-
Notifications
You must be signed in to change notification settings - Fork 117
improvements #43
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
improvements #43
Conversation
Thanks very much for your contribution @frangametv Can you describe what your PR does and add some screenshots of the text editor after your changes? |
I moved the "icons" folder inside the .jar and used class resource loading; then i forced the JavaAWT UIManager Style to Windows Modern Look, and this gives a completely new modern look to the text editor. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Just two minor changes needed to be confirmed/changed.
@frangametv With what Java version did you test it? @frangametv Could you also update the |
But my branch is updated, i'll use two commits and try to fix this
Java version: From 1.8 to the newest, it worked just fine |
Sorry for the deleted comment, my bad |
Beautiful! This sounds brilliant @frangametv! I will check this out this evening. |
fixed |
This is brilliant! Amazing job! |
I’ve now merged your changes. Thanks again for your contribution to this project. I really appreciate the improvements you have done to it 👍 |
No description provided.