Bug 548056 - [GTK][Preferences] Add space between "Show whitespace characters" and "(configure visiblity)"
Summary: [GTK][Preferences] Add space between "Show whitespace characters" and "(confi...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.13   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.15 M1   Edit
Assignee: Jens Lideström CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
: 550937 (view as bug list)
Depends on:
Blocks: 548042
  Show dependency tree
 
Reported: 2019-06-07 11:19 EDT by Andrew Obuchowicz CLA
Modified: 2020-01-07 05:13 EST (History)
7 users (show)

See Also:


Attachments
Image of missing spacing (110.65 KB, image/png)
2019-06-07 11:22 EDT, Andrew Obuchowicz CLA
no flags Details
No issue on Windows 7 (7.08 KB, image/png)
2019-06-11 09:46 EDT, Dani Megert CLA
no flags Details
Space before and after change on Windows 10 (22.71 KB, image/png)
2019-12-18 14:57 EST, Paul Pazderski CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Obuchowicz CLA 2019-06-07 11:19:26 EDT
Where it occurs:
Preferences → Editors → Text Editors (near the middle of the window)

There is a missing space character between  "Show whitespace characters" and "(configure visiblity)".


Should be changed to:
"Show whitespace characters (configure visibility)"

instead of:

"Show whitespace characters(configure visibility)"
Comment 1 Andrew Obuchowicz CLA 2019-06-07 11:22:45 EDT
Created attachment 278855 [details]
Image of missing spacing
Comment 2 Dani Megert CLA 2019-06-11 09:44:30 EDT
This is on Linux? Which font?

On Windows 7 this looks OK. Screenshot to follow.
Comment 3 Dani Megert CLA 2019-06-11 09:46:13 EDT
Created attachment 278897 [details]
No issue on Windows 7
Comment 4 Andrew Obuchowicz CLA 2019-06-11 14:42:46 EDT
(In reply to Dani Megert from comment #2)
> This is on Linux? Which font?
> 
> On Windows 7 this looks OK. Screenshot to follow.

Thank you for confirming - this must be a Linux problem (unless it is also occurring on OSX which I am unable to confirm, personally).
Comment 5 Lakshmi P Shanmugam CLA 2019-06-12 04:01:04 EDT
(In reply to Andrew Obuchowicz from comment #4)
> (In reply to Dani Megert from comment #2)
> > This is on Linux? Which font?
> > 
> > On Windows 7 this looks OK. Screenshot to follow.
> 
> Thank you for confirming - this must be a Linux problem (unless it is also
> occurring on OSX which I am unable to confirm, personally).

Looks fine on Mac too. Which version of Eclipse are you using?
Comment 6 Sravan Kumar Lakkimsetti CLA 2019-06-12 04:06:11 EDT
I can reproduce this on Ubuntu 18.04. Looks like linux specific problem
Comment 7 Eric Williams CLA 2019-06-12 09:19:57 EDT
I can reproduce the issue on Fedora 30, GTK 3.24, 4.12.
Comment 8 Eric Williams CLA 2019-09-11 16:14:35 EDT
*** Bug 550937 has been marked as a duplicate of this bug. ***
Comment 9 Jens Lideström CLA 2019-12-18 14:41:59 EST
I don't get any space in this place. It's driving me mad!

I'll push a change which adds some spacing between the checkbox control and the link control.

This makes the GUI looks like good for me, but maybe there is a little too much space on Windows and Mac. But I figure that the resulting situation will be better: No space on Linux probably looks worse than a little too much space on Windows and Mac.

I guess that the cause is that these controls have different amount of padding on GTK, Windows and Cocoa.

I use Lubuntu 18.03 with GTK 3.22.30.
Comment 10 Eclipse Genie CLA 2019-12-18 14:50:54 EST
New Gerrit change created: https://git.eclipse.org/r/154753
Comment 11 Paul Pazderski CLA 2019-12-18 14:57:02 EST
Created attachment 281268 [details]
Space before and after change on Windows 10

IMO ok on Windows.
Comment 12 Jens Lideström CLA 2019-12-18 15:51:35 EST
(In reply to Paul Pazderski from comment #11)
> Space before and after change on Windows 10

Thanks Paul!

I can make the space a little smaller. That would still look good on Linux and look a little better on Windows.
Comment 13 Dani Megert CLA 2019-12-19 03:48:33 EST
(In reply to Jens Lideström from comment #12)
> (In reply to Paul Pazderski from comment #11)
> > Space before and after change on Windows 10
> 
> Thanks Paul!
> 
> I can make the space a little smaller. That would still look good on Linux
> and look a little better on Windows.
Mac would also look worse. So, only make the change when running on GTK.
Comment 14 Jens Lideström CLA 2019-12-19 12:13:36 EST
(In reply to Dani Megert from comment #13)
> Mac would also look worse. So, only make the change when running on GTK.

Hm... I see your point. But I think the small layout inperfection (a tiny bit to small on Linux, and tiny bit to long on Windows and Mac) warrants any kind of special casing.

Also, if this SWT platform difference changes in the future, it is more likeley that that layout will be weird if we add special casing.

I vote for simply adding space on all platform.
Comment 15 Dani Megert CLA 2019-12-20 02:31:18 EST
(In reply to Jens Lideström from comment #14)
> I vote for simply adding space on all platform.
-1 for that. The amount of users that get hurt is bigger than those who would profit.
Comment 16 Jens Lideström CLA 2019-12-20 12:29:05 EST
(In reply to Dani Megert from comment #15)
> -1 for that. The amount of users that get hurt is bigger than those who
> would profit.

Alright. I'll update the change to that effect.
Comment 17 Jens Lideström CLA 2019-12-20 13:46:45 EST
Could this difference in layout with checkboxes and links be an SWT bug?

I will not investigate, but maybe somebody else feels the inspiration to do that...
Comment 19 Andrew Obuchowicz CLA 2019-12-23 10:13:47 EST
Thanks for fixing this Jens :)
Comment 20 Sravan Kumar Lakkimsetti CLA 2020-01-07 05:13:20 EST
verified on 
Eclipse SDK
Version: 2020-03 (4.15)
Build id: I20200106-1805
OS: Linux, v.4.15.0-72-generic, x86_64 / gtk 3.22.30
Java version: 14-ea