Own definition of Icon for JMenuItem

Question

Every time I try to add ColorIcon to JMenuItem it eats name space. How can I separate name from icon?
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.Color;
import java.awt.Component;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Toolkit;
import javax.swing.AbstractAction;
import javax.swing.Icon;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
public class Test extends JFrame {
     public Test() {
          JMenuBar menu = new JMenuBar();
          JMenu mFile = new JMenu("File");
          menu.add(mFile);
          JMenu mColor = new JMenu("Color");
          mFile.add(mColor);
          mColor.add(new TakeAction(Color.RED, "This is red"));
          setJMenuBar(menu);
          Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();
          setPreferredSize(new Dimension(screenSize.width/2, screenSize.height/2));
          setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
          pack();
          setLocationRelativeTo(null);
          setVisible(true);
     public static void main(String[] args) {
          new Test();
     public class TakeAction extends AbstractAction implements ActionListener{
          public TakeAction (Color color, String name) {
               super(name);
               putValue(SMALL_ICON, new ColorIcon(color));
          @Override
          public void actionPerformed(ActionEvent e) {
     public class ColorIcon implements Icon {
         private int width = 15;
         private int height = 15;
         private Color color;
         public ColorIcon(Color color) {
              this.color = color;
          @Override
          public int getIconHeight() {
               return 0;
          @Override
          public int getIconWidth() {
               return 0;
          @Override
          public void paintIcon(Component c, Graphics g, int x, int y) {
               Graphics2D g2d = (Graphics2D) g.create();
                  g2d.setColor(color);
                  g2d.fillOval(3, 3, width-3, height-3);
                      g2d.dispose();
}Edited by: Giraphant on Dec 30, 2010 3:23 AM
Edited by: Giraphant on Dec 30, 2010 3:24 AM

Answer 1

private int width = 15;
private int height = 15;
@Override
public int getIconHeight() {
  return 0;
@Override
public int getIconWidth() {
  return 0;
}Does this make sense to you?