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

Bug 320761

Summary: Tabs should not have blue focus ring on Mac
Product: [Eclipse Project] e4 Reporter: Boris Bokowski <bokowski>
Component: UIAssignee: Bogdan Gheorghe <gheorghe>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: Mike_Wilson, pwebster, remy.suen, susan
Version: unspecifiedFlags: bokowski: review+
Target Milestone: 1.0 RC3   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
Patch for CSS
none
GTK focus line none

Description Boris Bokowski CLA 2010-07-23 14:42:17 EDT
we need a simple: false in the syle sheet.
Comment 1 Bogdan Gheorghe CLA 2010-07-23 18:19:05 EDT
Created attachment 175112 [details]
Patch for CSS
Comment 2 Boris Bokowski CLA 2010-07-23 18:22:12 EDT
Paul, can you please commit this (org.eclipse.platform)? Thanks.
Comment 3 Remy Suen CLA 2010-07-23 19:26:01 EDT
Instead of the focus rectangle (on Windows), I get an underline now. I'm not sure which is uglier. In any case, this is more of a renderer/part activation problem than a CSS issue.
Comment 4 Boris Bokowski CLA 2010-07-23 22:19:22 EDT
Agreed. We talked about this today and said that the activation issue will have to be solved in 4.1. The main goal was to make this not look ugly on the Mac. We might just go back to what we had for Windows, so that this change only applies to the Mac. Bogdan and I put "simple: false" in all the stylesheets for consistency, but we should look at this one more time on Monday.
Comment 5 Susan McCourt CLA 2010-07-26 17:45:48 EDT
(In reply to comment #3)
> Instead of the focus rectangle (on Windows), I get an underline now. I'm not
> sure which is uglier. In any case, this is more of a renderer/part activation
> problem than a CSS issue.

I think the focus ring is well-known on windows and esp. in light of the fact that it flashes on/off as the editor grabs focus, I find that the flashing underline looks like cheese, whereas I understand that focus ring.

I think we should set simple back to true for windows at least.
The question is what do GTK folks think?
Comment 6 Remy Suen CLA 2010-07-26 17:56:59 EDT
(In reply to comment #5)
> I think we should set simple back to true for windows at least.
> The question is what do GTK folks think?

I think GTK+ needs to go back to simple too. The line doesn't really tell me anything.
Comment 7 Boris Bokowski CLA 2010-07-26 18:03:22 EDT
Susan, are you going to make the changes?
Comment 8 Bogdan Gheorghe CLA 2010-07-26 18:05:25 EDT
Created attachment 175277 [details]
GTK focus line

The focus line has always been around in 3.x (as all CTabFolders are not "simple" if they have the swoosh appearance). The difference is when you click on a tab in 3.x, the editor takes focus right away. If you hit Ctrl+Shift+Tab you will get the line in GTK as shown in the pic.
Comment 9 Susan McCourt CLA 2010-07-26 18:16:18 EDT
(In reply to comment #8)
> Created an attachment (id=175277) [details]
> GTK focus line
> 
> The focus line has always been around in 3.x (as all CTabFolders are not
> "simple" if they have the swoosh appearance). The difference is when you click
> on a tab in 3.x, the editor takes focus right away. If you hit Ctrl+Shift+Tab
> you will get the line in GTK as shown in the pic.

sure enough.  And on windows too.
So the fact that you get the momentary flash is probably bringing it more to our attention...

I'm inclined to leave as is on GTK and Mac, change on Windows?
I don't feel strongly.
Comment 10 Susan McCourt CLA 2010-07-26 18:50:13 EDT
committed changes to HEAD, consensus (or more honestly, no resistance, or perhaps apathy)... on IRC to change on windows, but not on GTK.

I think we'll get questions about the flash irrelevant to the focus emphasis used.