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

Replace icon for JOptionPane with nicer icons #46

Closed
markiewb opened this issue Jan 27, 2016 · 5 comments
Closed

Replace icon for JOptionPane with nicer icons #46

markiewb opened this issue Jan 27, 2016 · 5 comments
Assignees
Milestone

Comments

@markiewb
Copy link
Contributor

2016-01-27_21h55_16

I haven't seen such icons in Idea yet, so I cannot provide you an expected state.
Keep this issue for the record

granella added a commit to granella/nb-darcula that referenced this issue Jan 28, 2016
copied icons from Nimbus LAF for JOptionPane Revivius#46
@granella
Copy link
Contributor

Github was unstable in this early hours :)
closed PR for my branch and adding correct PR

@AlexFalappa
Copy link
Contributor

@granella: do you think to open a PR for improving the JOptionPane icons also against the original Darcula project?

I would be interested in that. It seems to me that the original project is stalling, I would be in favour maintaining an improved fork of the LAF alone where contributions like yours could be integrated.

Since a lot of improvements seem to come from this project why not let @Revivius create it?

@Revivius
Copy link
Owner

@AlexFalappa modifications and fixes we are performing takes effect only when loaded by NetBeans Platform. For example, try placing a JToggleButton in Form Designer and notice that how different it is from the 'Source' button in editor multiview. Form Designer is using unmodified Darcula LAF and hence has different borders for buttons for instance. Long story short, I think contributing to or forking original LAF should be the route we take eventually.

Revivius added a commit that referenced this issue Jan 28, 2016
Tooltip border and JOptionPane icons

Thanks to @granella. Fixes #45 and #46 .
@Revivius
Copy link
Owner

Fixed with merging of PR #50. Thank you @granella !

@Revivius Revivius added this to the v1.4 milestone Jan 28, 2016
@Revivius Revivius self-assigned this Jan 28, 2016
@AlexFalappa
Copy link
Contributor

@Revivius I played a bit with the Darcula LAF using it in some test Swing apps and found some lacks indeed.

An evident lack is missing JToggleButton styling as you highlighted. Other lacks I found are bad JOptionPane icons, less than optimal TitledBorder appearance, no JButton prelighting on mouse hover and no pressed state, no styling of JSplitPane (there is a pull request for this in the Darcula project however).

I plan on tackling the JToggleButton support, I'll keep you informed

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

No branches or pull requests

4 participants