Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 223327

Summary: [navigation] MultiHyperlink rendered with gray borders on right and bottom
Product: [Eclipse Project] Platform Reporter: Markus Keller <markus.kell.r>
Component: TextAssignee: Benno Baumgartner <benno.baumgartner>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: daniel_megert
Version: 3.4   
Target Milestone: 3.4 M6   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Bug Depends on: 199441, 224071    
Bug Blocks:    
Attachments:
Description Flags
Screenshot
none
proposed fix
none
Fix
none
Fix 3 - no margins, looks good on the Mac
none
vista
none
vista none

Description Markus Keller CLA 2008-03-20 05:29:48 EDT
Created attachment 93011 [details]
Screenshot

HEAD

MultiHyperlinks are rendered with gray borders on right and bottom on the Mac. The problem seems to be that the Table widget always shows a scroll bar on carbon, which is partly shown with the current layout.
Comment 1 Markus Keller CLA 2008-03-20 06:24:41 EDT
> The problem seems to be that the Table widget always shows a scroll bar on
> carbon, which is partly shown with the current layout.

This is bug 199441.
Comment 2 Benno Baumgartner CLA 2008-03-26 03:59:45 EDT
Created attachment 93553 [details]
proposed fix
Comment 3 Dani Megert CLA 2008-03-26 06:41:44 EDT
Created attachment 93565 [details]
Fix

The previous fix brings back unbalanced borders on WinXP.
Comment 4 Markus Keller CLA 2008-03-26 07:56:15 EDT
Created attachment 93570 [details]
Fix 3 - no margins, looks good on the Mac
Comment 5 Dani Megert CLA 2008-03-26 09:02:58 EDT
Applied previous patch with two modifications:
1. removed the -1 which is yet another platform bug/specialty
2. added platform dependent code for WindowsXP

Benno, please test the code from HEAD under Vista. Thanks.
Comment 6 Benno Baumgartner CLA 2008-03-26 10:47:26 EDT
Created attachment 93607 [details]
vista

(In reply to comment #5)
> Applied previous patch with two modifications:
> 1. removed the -1 which is yet another platform bug/specialty
> 2. added platform dependent code for WindowsXP
> 
> Benno, please test the code from HEAD under Vista. Thanks.
> 

No comment
Comment 7 Benno Baumgartner CLA 2008-03-26 10:48:44 EDT
Created attachment 93608 [details]
vista

(In reply to comment #5)
> Applied previous patch with two modifications:
> 1. removed the -1 which is yet another platform bug/specialty
> 2. added platform dependent code for WindowsXP
> 
> Benno, please test the code from HEAD under Vista. Thanks.
> 

No comment
Comment 8 Markus Keller CLA 2008-03-27 10:25:52 EDT
Verified in I20080326-1950 on WinXP, Mac OS X, Linux GTK.