Problem with rect.contains mouse x and y

[Lionlev]

Hi, something is wrong with my code..
Basically I want my boolean "mouseRolled" turn to true

if(MenuUI.loginBt.getBounds().contains(e.getX(), e.getY())){
			
			MenuUI.mouseRolled = true;
		} else {
			MenuUI.mouseRolled = false;
		}

(MenuUI.loginBt is the rectangle that repaints every 100 milliseconds)

When I roll my mouse on to that square nothing happens HOWEVER when I move the mouse little bit upward by about 25 pixels, the mouseRolled turns true
Which is quite weird...

(Go down for the short source code)

[DarrylBurke]

To get better help sooner, post a [_SSCCE_|http://mindprod.com/jgloss/sscce.html] that clearly demonstrates your problem.

This advice applies to any future questions you may have.

db

[Lionlev]

ok, I'll do it

[Lionlev]

Ok here is the SSCCE, I've made it short as I could:
3 classes: BuildUI, PaintUI and ListenMouse
BuildUI:

import javax.swing.JFrame;
public class BuildUI implements Runnable{
	public static PaintUI paintIt = new PaintUI();
	public void start(){
		new Thread(this).start();
	}
	public static void main(String[] args){
		BuildUI window = new BuildUI();
		JFrame w = new JFrame();
		w.setSize(600,600);
		w.setVisible(true);
		w.add(paintIt);
		w.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		w.setResizable(false);
		window.start();
		
		w.addMouseMotionListener(new ListenMouse());
	}
	public void run() {
		while(true){
			paintIt.repaint();
			try {
				Thread.sleep(1000/60);
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
		}
	}
}

PaintUI

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Rectangle;
import java.awt.image.BufferedImage;
import javax.swing.JPanel;
public class PaintUI extends JPanel{
	private static final long serialVersionUID = 1L;
	public static boolean mouseRolled = false;
	public static Rectangle loginBt = new Rectangle(100,100,50,50);
	public BufferedImage loginBtn;
	
	public void paintComponent(Graphics g){
		if(mouseRolled){
			g.setColor(Color.GREEN);
		} else {
			g.setColor(Color.CYAN);
		}
		g.fillRect(loginBt.x,loginBt.y, loginBt.width, loginBt.height);
	}
}

ListenMouse

import java.awt.event.MouseEvent;
import java.awt.event.MouseMotionListener;
public class ListenMouse implements MouseMotionListener{
	public void mouseDragged(MouseEvent e) {
	}
	public void mouseMoved(MouseEvent e) {
		if(PaintUI.loginBt.getBounds().contains(e.getX(), e.getY())){
			
			PaintUI.mouseRolled = true;
		} else {
			PaintUI.mouseRolled = false;
		}
	}
}

:DDD

[eRaaaa]

You have added the MouseListener to the JFrame, so you have an offset because of the size of title bar!
Try to add the MouseListener to the contentpane of your jframe!

w.addMouseMotionListener(new ListenMouse());
->
w.getContentPane().addMouseMotionListener(new ListenMouse());

[Lionlev]

You have added the MouseListener to the JFrame, so you have an offset because of the size of title bar!
Try to add the MouseListener to the contentpane of your jframe!

w.addMouseMotionListener(new ListenMouse());
->
w.getContentPane().addMouseMotionListener(new ListenMouse());

Oh wow, thanks! That fixes everything rep++ to you!