Bug 484463 - [GTK3] Missing line under the Quick Access box in the toolbar
Summary: [GTK3] Missing line under the Quick Access box in the toolbar
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.6   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.6 M7   Edit
Assignee: Eric Williams CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 497635
  Show dependency tree
 
Reported: 2015-12-16 03:48 EST by Lars Vogel CLA
Modified: 2016-11-17 10:06 EST (History)
4 users (show)

See Also:


Attachments
Eclipse 4.5.1 (3.20 KB, image/png)
2015-12-16 03:48 EST, Lars Vogel CLA
no flags Details
No line in Eclipse 4.6 (6.60 KB, image/png)
2015-12-16 03:49 EST, Lars Vogel CLA
no flags Details
Quick Access line visible on I20151215-0800 (43.16 KB, image/png)
2015-12-16 09:16 EST, Eric Williams CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2015-12-16 03:48:39 EST
Created attachment 258719 [details]
Eclipse 4.5.1

The line under the QuickAccess box is not visible anymore in 4.6. See screenshots for 4.5.1
Comment 1 Lars Vogel CLA 2015-12-16 03:49:02 EST
Created attachment 258720 [details]
No line in Eclipse 4.6
Comment 2 Eric Williams CLA 2015-12-16 09:16:55 EST
Created attachment 258728 [details]
Quick Access line visible on I20151215-0800

What build is the line disappearing in?

Attached is a screenshot of the latest I-build (20151215-0800) running on my system. A child Eclipse running on the latest master looks the same.
Comment 3 Lars Vogel CLA 2015-12-16 09:39:10 EST
(In reply to Eric Williams from comment #2)
> Created attachment 258728 [details]
> Quick Access line visible on I20151215-0800
> 
> What build is the line disappearing in?
> 
> Attached is a screenshot of the latest I-build (20151215-0800) running on my
> system. A child Eclipse running on the latest master looks the same.

I use 4.6.0.I20151215-0800 on Ubuntu 15.10 (plain).
Comment 4 Eric Williams CLA 2015-12-16 11:13:59 EST
(In reply to Lars Vogel from comment #3)
> I use 4.6.0.I20151215-0800 on Ubuntu 15.10 (plain).

I can see it now, you have to right click one of the perspective buttons, and check "Show Text". Once the text disappear the line does as well.
Comment 5 Eclipse Genie CLA 2016-03-24 16:03:00 EDT
New Gerrit change created: https://git.eclipse.org/r/69275
Comment 7 Eric Williams CLA 2016-03-24 16:13:21 EDT
(In reply to Eclipse Genie from comment #6)
> Gerrit change https://git.eclipse.org/r/69275 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/
> ?id=de95e2ca9a623c4260b4f48866099a052f6848a4

Sorry it took me so long to get around to this fix. The solution is to adjust the padding for buttons that are children of toolbars. In total this will add 3px to the overall height of the toolbar. This is a marginal increase in my opinion as the lack of a line is more annoying.

Pre-patch:
Toolbar height (perspective buttons with text): 148px
Toolbar height (perspective buttons without text): 146px

Post patch:
Toolbar height (perspective buttons with text): 151px
Toolbar height (perspective buttons without text): 149px
Comment 8 Arun Thondapu CLA 2016-04-28 16:29:04 EDT
Verified in 4.6.0.I20160428-0800.

Thanks for the fix Eric!
Comment 9 Lars Vogel CLA 2016-07-08 14:49:04 EDT
I'm planning to remove the underline, as this reduces visual clutter and locks better. Once we have done this, I suggest to revert this change. 

Can we use this bug or shall I open a new one?
Comment 10 Arun Thondapu CLA 2016-07-08 16:45:50 EDT
(In reply to Lars Vogel from comment #9)
> I'm planning to remove the underline, as this reduces visual clutter and
> locks better. Once we have done this, I suggest to revert this change. 
> 
> Can we use this bug or shall I open a new one?

I would suggest opening a new bug instead of reopening this one as this fix continues to remain valid in Neon.
Comment 11 Andrey Loskutov CLA 2016-11-17 08:53:00 EST
Lars, do you know which GTK3 version Ubuntu 15.10 has? Or can you validate that the patch https://git.eclipse.org/r/85143 on bug 507604 do not cause regression (on Ubuntu) to this one?
Comment 12 Lars Vogel CLA 2016-11-17 10:06:00 EST
Andrey, sorry I updated my machines to 16.04.