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

Enable theming preference check box is not working as expected and forces to restart twice #2522

Open
2 tasks done
opcoach opened this issue Nov 18, 2024 · 0 comments
Open
2 tasks done
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@opcoach
Copy link
Member

opcoach commented Nov 18, 2024

Let's make sure issue is not already fixed in latest builds first.

Steps to reproduce

From a fresh installation and clean workspace:

Install the setup of 'platform ui' by dragging the setup link in the banner of the installer : see Setting your environment and projects guide

Open the preference dialog and filter on 'theme'. Disable theming if there is one selected by unchecking the 'Enable Theming' checkbox and restart. Eclipse restarts without any theme.

Then reopen the preference dialog and filter on 'theme' then check the 'Enable theming' check box again and click Apply.

Eclipse will ask to restart, while no Theme has been yet selected.. and no theme has been proposed. The possible themes will appear after the restart and will be applicable, but we will have to restart again... this is not straightforward and totally counterintuitive.

Actually, when the 'Enable theming' is selected, it must display the themes and then propose to select one and restart when it is applied (even if the theme was not changed in the combo).

This is the current situation when we switch from 'Enable theming' unchecked to checked:

What we have (unchecked to checked) What we expect
image image

On the other hand, if we unchecked the theme while it was checked, the list of themes should be hidden:

What we have (checked to unchecked) What we expect
image image

|

The Plugin Spy (Alt Shift F1), launched when the preference page is displayed says that this preference page is managed by the ViewsPreferencePage in the org.eclipse.ui.workbench plugin :

image

A change event listener could be added on the preference checkbox editor so as to react on the change and display or not the bloc of theme list.

Tested under this environment:

  • OS & version: macOS X 15.1 (sequoia).
  • Eclipse IDE/Platform version (as shown in Help > About): Version: 2024-12 (4.34) Build id: I20241117-1800 (got from the platform.ui current setup)

Community

  • I understand reporting an issue to this OSS project does not mandate anyone to fix it. Other contributors may consider the issue, or not, at their own convenience. The most efficient way to get it fixed is that I fix it myself and contribute it back as a good quality patch to the project.
@opcoach opcoach added bug Something isn't working good first issue Good for newcomers labels Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant