Bug 567138 - [GTK4] Get ToolBar/ToolItem snippets to run
Summary: [GTK4] Get ToolBar/ToolItem snippets to run
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.23   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-SWT-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 577370
Blocks: 545942
  Show dependency tree
 
Reported: 2020-09-18 16:36 EDT by Soraphol (Paul) Damrongpiriyapong CLA
Modified: 2021-11-26 15:42 EST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Soraphol (Paul) Damrongpiriyapong CLA 2020-09-18 16:36:53 EDT

    
Comment 1 Eclipse Genie CLA 2020-09-18 16:37:59 EDT
New Gerrit change created: https://git.eclipse.org/r/c/platform/eclipse.platform.swt/+/169602
Comment 2 Eclipse Genie CLA 2020-09-24 11:49:34 EDT
New Gerrit change created: https://git.eclipse.org/r/c/platform/eclipse.platform.swt/+/169846
Comment 5 Phil Beauvoir CLA 2020-11-03 06:37:10 EST
Hi, I'm trying to trace the cause of Bug 568407.

Is it possible that changes in this one caused icons to be drawn too big on Linux high DPI?
Comment 6 Andrey Loskutov CLA 2020-11-03 07:00:24 EST
(In reply to Phil Beauvoir from comment #5)
> Hi, I'm trying to trace the cause of Bug 568407.
> 
> Is it possible that changes in this one caused icons to be drawn too big on
> Linux high DPI?

If, then likely this one: https://git.eclipse.org/r/c/platform/eclipse.platform.swt/+/169846

Paul, can you reproduce the problem, and if yes, could you revert this commit and check if that fixes the problem?
Comment 7 Andrey Loskutov CLA 2020-11-05 05:37:11 EST
Paul, could you please check if changes here caused regression in bug 568407? If yes, we should either fix it or revert *before* 4.18 M3.
Comment 8 Soraphol (Paul) Damrongpiriyapong CLA 2020-11-05 12:23:03 EST
Sorry for the late reply, I missed the email from Bugzilla. I'm looking into this now.
Comment 9 Soraphol (Paul) Damrongpiriyapong CLA 2020-11-05 13:06:22 EST
Firstly, the incorrect scaling doesn't happen on Fedora, however the icons are not scaling properly. I'm assuming that the icons not scaling in Fedora is equivalent to the large icons on Ubuntu.

I can confirm the behavior between the I20200921-1800, and I20200928-1800.

I reverted the changes from https://git.eclipse.org/r/c/platform/eclipse.platform.swt/+/169846, however the behavior still remains. The way icons are scaled through the dpiChanged function in ToolItem, and that has not been change in the patch.
Comment 10 Phil Beauvoir CLA 2020-11-05 13:10:52 EST
(In reply to Soraphol (Paul) Damrongpiriyapong from comment #9)
> 
> 
> I can confirm the behavior between the I20200921-1800, and I20200928-1800.
> 

To narrow it down, the icons are the right size in I20200922-1800 but large in I20200924-1800.
Comment 11 Eclipse Genie CLA 2020-12-03 14:21:06 EST
New Gerrit change created: https://git.eclipse.org/r/c/platform/eclipse.platform.swt/+/173331
Comment 13 Joel Majano CLA 2021-11-26 15:42:15 EST
Snippet 18: Each ToolItem is much larger than in GTK3, making the toolbar take up more space in GTK4.

Snippet 36: Each ToolItem is much larger than in GTK3, making the toolbar take up more space in GTK4.

Snippet 47: Each ToolItem image is the correct size, but the box its surrounded by is too big horizontally, and the image is not centered in the middle (slightly offset downwards). Furthermore, the "disabled" image has a box around it at all times, which is not correct behavior. 

Snippet 49: Larger items, but also does not wrap. (This appears to be broken on GTK3 too)

Snippet 58: Larger items, combo overlaps with the item to its right (Item 5).

Snippet 67: Larger items, and the drop down menu does not appear.

Snippet 153: Larger items, status line does not appear at all when mouse enters a ToolItem.

Snippet 288: File dialog says "The folder contents could not be displayed" and "Operation not supported". If this is ignored and one of the folders on the left side of the dialog are used, a GIF file can be selected. However, when "Open" is selected, a Segfault occurs at the g_slist_length frame.