Skip to content

Use Material CSS to make the example look nicer#1552

Merged
neilcsmith-net merged 2 commits intoapache:masterfrom
dukescript:MaterialCss
Oct 9, 2019
Merged

Use Material CSS to make the example look nicer#1552
neilcsmith-net merged 2 commits intoapache:masterfrom
dukescript:MaterialCss

Conversation

@dukescript
Copy link
Contributor

@dukescript dukescript commented Oct 6, 2019

In order to make the Java Frontend Application Gradle project type more attractive, it would be better to make it less HTML-like, and more component like. Using Material CSS is a simple way to do it.

Hopefully @geertjanw finds the new UI more attractive to shoot a video about the excellent support NetBeans provide when it comes to JavaFX light developement. @neilcsmith-net, can we get it into 11.2? It is just a simple HTML change, no code is affected. Thank you.

CCing @lkishalmi

@dukescript
Copy link
Contributor Author

FYI: The application generated from the new project template is available here online.

@neilcsmith-net neilcsmith-net added this to the 11.2 milestone Oct 7, 2019
<ul id="nav-mobile" class="right hide-on-med-and-down">
<li><a target="_blank" href="http://netbeans.org">NetBeans</a></li>
<li><a target="_blank" href="https://github.com/dukescript/kt-mvvm-demo/tree/FXBeanInfo#readme">JavaFX</a></li>
<li><a target="_blank" href="https://materializecss.com/">Material CSS</a></li>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't that say Materialize CSS ? Otherwise happy to merge for 11.2.

Copy link
Member

@neilcsmith-net neilcsmith-net left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@neilcsmith-net neilcsmith-net self-requested a review October 9, 2019 10:31
@neilcsmith-net neilcsmith-net added the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Oct 9, 2019
@neilcsmith-net neilcsmith-net changed the base branch from release112 to master October 9, 2019 10:32
Copy link
Member

@neilcsmith-net neilcsmith-net left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the noise! Missed that this was opened against release112. Needs to be merged to master. However, realised I can change the base branch myself with this one - last one this applied to I couldn't.

@neilcsmith-net neilcsmith-net removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Oct 9, 2019
@neilcsmith-net neilcsmith-net merged commit 1b56603 into apache:master Oct 9, 2019
Akshay-Gupta-19 pushed a commit to Akshay-Gupta-19/netbeans that referenced this pull request Oct 30, 2019
* Use Material CSS to make the example look nicer

* It's Materialize CSS
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.

3 participants