Bug 532934 - Theme workspace selection dialog based on currently selected workspace
Summary: Theme workspace selection dialog based on currently selected workspace
Status: REOPENED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.8   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-27 04:44 EDT by Lars Vogel CLA
Modified: 2020-06-19 05:23 EDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2018-03-27 04:44:40 EDT
Now that I'm a full time dark theme user, I find it super annoying that I have to switch to the dark theme every time I open a new workspace with the same Eclipse. 

I suggest to persist the dark theme in the configuration area and use if for a workspace, unless the workspace has a different preference setting.
Comment 1 Andrey Loskutov CLA 2018-03-27 09:37:48 EDT
Why should we so so for theme but not for any other preference? You can either use Oomph or plugin customization argument.
Comment 2 Lars Vogel CLA 2018-03-27 09:47:19 EDT
(In reply to Andrey Loskutov from comment #1)
> Why should we so so for theme but not for any other preference? You can
> either use Oomph or plugin customization argument.

IMHO the theming selection is really a general user thing and re-using would help users. So the proposal is to re-use the user preference unless he has a specific workspace setting. A more general solution for new workspace is handled by Bug 511548.
Comment 3 Matthias Becker CLA 2018-03-28 06:06:29 EDT
(In reply to Lars Vogel from comment #2)
> A more general solution for new workspace is
> handled by Bug 511548.

Wouldn't it them make sense to drive Bug 511548 instead of providing a very specific fix for this?
Comment 4 Dani Megert CLA 2018-04-05 10:59:41 EDT
(In reply to Andrey Loskutov from comment #1)
> Why should we so so for theme but not for any other preference? You can
> either use Oomph or plugin customization argument.

+100!
Comment 5 Fabian Pfaff CLA 2018-04-05 11:23:31 EDT
Implementing this would allow us to style the workspace selector dialog with the dark theme.
It just feels very off to use the dark theme everywhere and then be greeted by a classic/light themed dialog on every startup.
Comment 6 Dani Megert CLA 2018-04-05 11:40:25 EDT
(In reply to Fabian Pfaff from comment #5)
> Implementing this would allow us to style the workspace selector dialog with
> the dark theme.
> It just feels very off to use the dark theme everywhere and then be greeted
> by a classic/light themed dialog on every startup.

That is true for every preference as already mentioned by Andrey in comment 1.
Comment 7 Lars Vogel CLA 2018-04-05 12:05:45 EDT
(In reply to Dani Megert from comment #6)
> (In reply to Fabian Pfaff from comment #5)
> > Implementing this would allow us to style the workspace selector dialog with
> > the dark theme.
> > It just feels very off to use the dark theme everywhere and then be greeted
> > by a classic/light themed dialog on every startup.
> 
> That is true for every preference as already mentioned by Andrey in comment
> 1.

I'm not aware of a preference which should affect the workspace selection dialog. But a dark theme user wants this selection dialog to be styled.
Comment 8 Dani Megert CLA 2018-04-06 09:52:07 EDT
(In reply to Lars Vogel from comment #7)
> (In reply to Dani Megert from comment #6)
> > (In reply to Fabian Pfaff from comment #5)
> > > Implementing this would allow us to style the workspace selector dialog with
> > > the dark theme.
> > > It just feels very off to use the dark theme everywhere and then be greeted
> > > by a classic/light themed dialog on every startup.
> > 
> > That is true for every preference as already mentioned by Andrey in comment
> > 1.
> 
> I'm not aware of a preference which should affect the workspace selection
> dialog. But a dark theme user wants this selection dialog to be styled.

What about the default compliance for Java when creating a new Java project in the new workspace? Or any other preference that you've set in your previous workspace? Adding special handling for the theme is just plain wrong.
Comment 9 Lars Vogel CLA 2018-04-06 10:12:26 EDT
We should allow users to have also the ws selection dialog in dark style. 

I'm ok with Dani's concern that we should not use this setting for new workspaces but if the user used the dark theme in his last ws we should also adjust the first dialog to this theme.
Comment 10 Dani Megert CLA 2018-04-06 10:41:09 EDT
(In reply to Lars Vogel from comment #9)
> We should allow users to have also the ws selection dialog in dark style. 
> 
> I'm ok with Dani's concern that we should not use this setting for new
> workspaces but if the user used the dark theme in his last ws we should also
> adjust the first dialog to this theme.

This is not a new argument. Again, the same can be said for any other preference like whether you want to see the heap status in the status line.

Please don't reopen again. A better solution is to somehow give users the possibility to share their preferences among workspace (not sure if we already have a bug for that). For example allow to export certain preferences to the configuration area. That's something I'd support.
Comment 11 Lars Vogel CLA 2018-04-06 10:53:51 EDT
Dani, you seem to fail to understand comment 9. Please read it again and let me know what is unclear. It is not (anymore) about using the setting in a new workspace.
Comment 12 Dani Megert CLA 2018-04-06 10:57:21 EDT
> if the user used the dark theme in his last ws we should also adjust the first > dialog to this theme.

So, how would you know what the last workspace was?
Comment 13 Dani Megert CLA 2018-04-06 10:58:18 EDT
(In reply to Dani Megert from comment #12)
> > if the user used the dark theme in his last ws we should also adjust the first > dialog to this theme.
> 
> So, how would you know what the last workspace was?

Please open a new bug report if you see a solution that is not along the lines of the provided summary.
Comment 14 Dani Megert CLA 2018-04-06 11:01:22 EDT
> Persist last theme in the JFace preference configuration area and re-use for ws selection dialog

That does not make sense.

Let's assume I have workspaces with "light" and "dark". Why should I get/want the workspace selection dialog in "dark", then choose a workspace with "light" and be in the light theme?

This is just wrong.
Comment 15 Dani Megert CLA 2018-04-07 03:46:09 EDT
Thought about this again. Let's try it.


(In reply to Dani Megert from comment #12)
> > if the user used the dark theme in his last ws we should also adjust the first > dialog to this theme.
> 
> So, how would you know what the last workspace was?

The dialog has the workspaces in LRU order and the last one used is selected by default in the dropdown.

NOTE: That workspace might not exist anymore, hence we need to fallback to the default theme in that case.


(In reply to Dani Megert from comment #14)
> Let's assume I have workspaces with "light" and "dark". Why should I
> get/want the workspace selection dialog in "dark", then choose a workspace
> with "light" and be in the light theme?

We could (re-)theme the dialog whenever the user selects another workspace in  the dropdown.