Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 170259 - [MacOS X] Text widget contains focus ring.
Summary: [MacOS X] Text widget contains focus ring.
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P1 normal (vote)
Target Milestone: ---   Edit
Assignee: Paul Slauenwhite CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 28842
Blocks:
  Show dependency tree
 
Reported: 2007-01-11 16:47 EST by Paul Slauenwhite CLA
Modified: 2016-05-05 10:28 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Slauenwhite CLA 2007-01-11 16:47:22 EST
[MacOS X] Text widget contains focus ring.

As described in the SWT dependant defect https://bugs.eclipse.org/bugs/show_bug.cgi?id=28842, MacOS X applications use a focus ring only for dialogs, not for top-level windows.  As such, the Rename Action in the Test Navigator (see org.eclipse.hyades.test.ui.internal.navigator.action.RenameAction.java) sometimes contains a focus ring (e.g. double border) and there is currently no API for determining if the focus ring is rendered to determine the require space for the text widget.  SWT dependant defect https://bugs.eclipse.org/bugs/show_bug.cgi?id=28842 will provide this configuration setting to be queried.

Note, the temporary work around provided by SWT (https://bugs.eclipse.org/bugs/show_bug.cgi?id=28842#c15) was removed under TPTP defect https://bugs.eclipse.org/bugs/show_bug.cgi?id=126575 since the work-around require using an internal system property which is effect an API.
Comment 1 Paul Slauenwhite CLA 2007-01-11 16:48:54 EST
Decreasing priority since MacOS X is not a TPTP supported platform.  Targeting to future since this defect is dependent on SWT defect 28842.
Comment 2 Paul Slauenwhite CLA 2007-01-11 17:11:00 EST
The fix is required in org.eclipse.hyades.test.ui.internal.navigator.action.RenameAction.getCellEditorInset(Control).
Comment 3 Paul Slauenwhite CLA 2007-01-26 08:25:32 EST
Targeting to future since not containable in 4.4.
Comment 4 Paul Slauenwhite CLA 2007-01-26 09:01:36 EST
As discussed on this week's Test Project call (January 22, 2007), the Test Project will focus on existing P1 - P2/Blocker - Major and P1/Normal and test creation/automation defects (omitting defects dependant on outstanding features) in TPTP 4.4.  All other Test Project defects have been targeted to future.

If this defect has been targeted to future and you/originator feel it should be completed in 4.4, please provide the necessary reason as a reply to this comment or a post to the Test Project mailing list (tptp-test-tooling-dev@eclipse.org).  We will collectively triage and assess our resources to determine a case-by-case decision. 
Comment 5 Paul Slauenwhite CLA 2007-01-26 09:04:09 EST
Correction:  The Test Project mailing list is tptp-testing-tools-dev@eclipse.org.
Comment 6 Paul Slauenwhite CLA 2007-01-26 10:19:07 EST
Changing the priority to P3 since TPTP only uses P1 - P3 priorities.
Comment 7 Paul Slauenwhite CLA 2007-02-22 11:35:06 EST
Jerome, lets get this defect fixed in 4.4 now that the dependent defect is fixed and fresh in our minds.  Since you and Patrick are over committed in 4.4, I can do the fix. 
Comment 8 Jerome Gout CLA 2007-02-22 11:40:21 EST
Ok, Thanks Paul.
Comment 9 Paul Slauenwhite CLA 2007-03-01 13:51:33 EST
org.eclipse.hyades.test.ui.internal.navigator.action.RenameAction.java checked-in to CVS (HEAD).
Comment 10 Paul Slauenwhite CLA 2007-03-12 14:06:31 EDT
Updating sizings.
Comment 11 Paul Slauenwhite CLA 2007-03-12 14:09:47 EDT
Updated/corrected sizings.
Comment 12 Paul Slauenwhite CLA 2007-06-02 14:51:48 EDT
Reporter: Please verify and close in preparation for shutting down the TPTP 4.4 release.  Thanks.
Comment 13 Paul Slauenwhite CLA 2007-06-27 15:21:12 EDT
Verified in TPTP-4.4.0-200706140100C.