Bug 28057 - JRadioButton: focus indicator is not painted completely
Summary: JRadioButton: focus indicator is not painted completely
Status: RESOLVED FIXED
Alias: None
Product: classpath
Classification: Unclassified
Component: swing (show other bugs)
Version: 0.92
: P3 normal
Target Milestone: 0.93
Assignee: Roman Kennke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-06-16 12:39 UTC by Robert Schuster
Modified: 2006-10-12 12:25 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Schuster 2006-06-16 12:39:29 UTC
When a JRadioButton has the focus a box should painted around the text. However in classpath the upper and the lower line of that box is not drawn (it is possibly clipped).
Comment 1 cvs-commit@developer.classpath.org 2006-10-12 12:25:23 UTC
Subject: Bug 28057

CVSROOT:	/cvsroot/classpath
Module name:	classpath
Changes by:	Roman Kennke <rabbit78>	06/10/12 12:24:46

Modified files:
	javax/swing/plaf/basic: BasicRadioButtonUI.java 
	                        BasicLookAndFeel.java 
	javax/swing/plaf/metal: MetalRadioButtonUI.java 
	.              : ChangeLog 

Log message:
	2006-10-12  Roman Kennke  <kennke@aicas.com>
	
		PR 28057
		* javax/swing/plaf/basic/BasicRadioButtonUI.java
		(paint): Determine correct icon. Added support for HTML label.
		Added small optimizations.
		(getPreferredSize): Only consider the buttons iconTextGap, and
		only when the text is not null.
		* javax/swing/plaf/basic/BasicLookAndFeel.java
		(initComponentDefaults): Fetch border for RadioButton from
		BasicButtons.getRadioButtonBorder().
		* javax/swing/plaf/metal/MetalRadioButtonUI.java
		(paintFocus): Paint focus rectangle one pixel smaller.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/classpath/javax/swing/plaf/basic/BasicRadioButtonUI.java?cvsroot=classpath&r1=1.21&r2=1.22
http://cvs.savannah.gnu.org/viewcvs/classpath/javax/swing/plaf/basic/BasicLookAndFeel.java?cvsroot=classpath&r1=1.99&r2=1.100
http://cvs.savannah.gnu.org/viewcvs/classpath/javax/swing/plaf/metal/MetalRadioButtonUI.java?cvsroot=classpath&r1=1.10&r2=1.11
http://cvs.savannah.gnu.org/viewcvs/classpath/ChangeLog?cvsroot=classpath&r1=1.8668&r2=1.8669



Comment 2 Roman Kennke 2006-10-12 12:25:56 UTC
Fixed in CVS HEAD.