Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TitledSeparator shows no separator on linux #606

Closed
laeubi opened this issue Jul 22, 2024 · 1 comment · Fixed by #618
Closed

TitledSeparator shows no separator on linux #606

laeubi opened this issue Jul 22, 2024 · 1 comment · Fixed by #618
Milestone

Comments

@laeubi
Copy link
Contributor

laeubi commented Jul 22, 2024

This is how the example snippet looks on linux:

grafik

the examples page say it should look like this:

grafik

@wimjongman
Copy link
Contributor

This is a different behavior of SWT.

I think we can add SWT.SHADOW_OUT to get the desired effect on all platforms.

	/**
	 * Create a separator
	 */
	private void createSeparator() {
		final Label separator = new Label(this, SWT.SEPARATOR | SWT.HORIZONTAL | SWT.SHADOW_OUT);
		separator.setLayoutData(new GridData(GridData.FILL, GridData.CENTER, true, false));
		separator.setBackground(getBackground());
	}

image

@wimjongman wimjongman added this to the 3.2.0 milestone Nov 26, 2024
wimjongman added a commit to wimjongman/nebula that referenced this issue Nov 26, 2024
Added explicit shadow parameter to get the same behavior on linux.
wimjongman added a commit to wimjongman/nebula that referenced this issue Nov 26, 2024
Added explicit shadow parameter to get the same behavior on linux.
wimjongman added a commit that referenced this issue Nov 26, 2024
Added explicit shadow parameter to get the same behavior on linux.
@wimjongman wimjongman linked a pull request Nov 26, 2024 that will close this issue
wimjongman added a commit that referenced this issue Nov 27, 2024
made snippet standalone by inlining the image
removed background setting which did nothing on windows and hid the separator on linux
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants