[GitHub] [netbeans] DevCharly commented on a change in pull request #1918: [NETBEANS-3785] Implement simplified HeapView widget

[GitBox <gi...@apache.org>]

DevCharly commented on a change in pull request #1918: [NETBEANS-3785] Implement simplified HeapView widget
URL: https://github.com/apache/netbeans/pull/1918#discussion_r374915679
 
 

 ##########
 File path: platform/openide.actions/src/org/openide/actions/HeapView.java
 ##########
 @@ -350,60 +148,39 @@ public HeapView() {
         setToolTipText(NbBundle.getMessage(GarbageCollectAction.class, "CTL_GC"));
         updateUI();
     }
-    
+
     /**
-     * Overriden to return true, GCComponent paints in its entire bounds in
-     * an opaque manner.
+     * Overridden to return true, GCComponent paints in its entire bounds in an
+     * opaque manner.
      */
-    @Override public boolean isOpaque() {
+    @Override
+    public boolean isOpaque() {
         return true;
     }
-    
+
     /**
      * Updates the look and feel for this component.
      */
-    @Override public void updateUI() {
-        Font f = new JLabel().getFont();
-        f = new Font(f.getName(), Font.BOLD, f.getSize());
+    @Override
+    public void updateUI() {
+        Font f = new JLabel().getFont().deriveFont(Font.BOLD);
 
 Review comment:
   Probably better use `UIManager.getFont("Label.font")` instead of `new JLabel().getFont()`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists