From b9fbf4209bc11fd7e032fff6e89c6a0816c75fab Mon Sep 17 00:00:00 2001 From: Sean Date: Fri, 12 Jun 2020 12:37:54 +0200 Subject: [PATCH] Cleanup theming related code Enhance color usage, add more default themes, add solarized theme --- .../src/app/core/ui-services/theme.service.ts | 4 +- .../components/jitsi/jitsi.component.html | 4 +- .../media-upload-content.component.html | 2 +- .../projection-dialog.component.html | 2 +- .../user-menu/user-menu.component.html | 2 +- .../assignment-list.component.html | 2 +- .../reset-password-confirm.component.html | 2 +- .../reset-password.component.html | 2 +- .../mediafile-list.component.html | 6 +- .../category-detail.component.html | 2 +- .../motion-block-detail.component.html | 2 +- .../motion-detail.component.html | 6 +- .../motion-list/motion-list.component.html | 2 +- .../workflow-detail.component.html | 2 +- .../workflow-list.component.html | 2 +- .../motion-export-dialog.component.html | 2 +- .../countdown-dialog.component.html | 2 +- .../message-dialog.component.html | 2 +- .../projector-edit-dialog.component.html | 2 +- .../projector-list.component.html | 2 +- .../app/site/site.component.scss-theme.scss | 5 +- .../tag-list/tag-list.component.html | 2 +- .../group-list/group-list.component.html | 2 +- .../user-list/user-list.component.html | 2 +- .../assets/styles/color-palettes/os-blue.scss | 32 +++ .../assets/styles/color-palettes/os-cyan.scss | 32 +++ .../assets/styles/color-palettes/os-gray.scss | 32 +++ .../os-green.scss} | 6 - .../assets/styles/color-palettes/os-red.scss | 32 +++ .../src/assets/styles/component-themes.scss | 152 +++++++++++++ .../assets/styles/openslides-dark-theme.scss | 45 ---- .../styles/openslides-developer-theme.scss | 75 ------- .../styles/openslides-general-theme.scss | 40 ---- .../src/assets/styles/openslides-theme.scss | 75 ------- .../assets/styles/themes/default-dark.scss | 8 + .../assets/styles/themes/default-light.scss | 38 ++++ .../src/assets/styles/themes/green-dark.scss | 7 + .../src/assets/styles/themes/green-light.scss | 7 + client/src/assets/styles/themes/red-dark.scss | 7 + .../src/assets/styles/themes/red-light.scss | 7 + .../assets/styles/themes/solarized-dark.scss | 8 + client/src/styles.scss | 212 ++---------------- openslides/core/config_variables.py | 30 ++- 43 files changed, 442 insertions(+), 466 deletions(-) create mode 100644 client/src/assets/styles/color-palettes/os-blue.scss create mode 100644 client/src/assets/styles/color-palettes/os-cyan.scss create mode 100644 client/src/assets/styles/color-palettes/os-gray.scss rename client/src/assets/styles/{openslides-green-theme.scss => color-palettes/os-green.scss} (71%) create mode 100644 client/src/assets/styles/color-palettes/os-red.scss create mode 100644 client/src/assets/styles/component-themes.scss delete mode 100644 client/src/assets/styles/openslides-dark-theme.scss delete mode 100644 client/src/assets/styles/openslides-developer-theme.scss delete mode 100644 client/src/assets/styles/openslides-general-theme.scss delete mode 100644 client/src/assets/styles/openslides-theme.scss create mode 100644 client/src/assets/styles/themes/default-dark.scss create mode 100644 client/src/assets/styles/themes/default-light.scss create mode 100644 client/src/assets/styles/themes/green-dark.scss create mode 100644 client/src/assets/styles/themes/green-light.scss create mode 100644 client/src/assets/styles/themes/red-dark.scss create mode 100644 client/src/assets/styles/themes/red-light.scss create mode 100644 client/src/assets/styles/themes/solarized-dark.scss diff --git a/client/src/app/core/ui-services/theme.service.ts b/client/src/app/core/ui-services/theme.service.ts index fabf3b125..5f48621ca 100644 --- a/client/src/app/core/ui-services/theme.service.ts +++ b/client/src/app/core/ui-services/theme.service.ts @@ -12,7 +12,7 @@ export class ThemeService { /** * Constant, that describes the default theme class. */ - public static DEFAULT_THEME = 'openslides-theme'; + public static DEFAULT_THEME = 'openslides-default-light-theme'; /** * Constant path of the logo with dark colors for bright themes. @@ -54,7 +54,7 @@ export class ThemeService { this.currentTheme = theme; const classList = document.getElementsByTagName('body')[0].classList; // Get the classlist of the body. - const toRemove = Array.from(classList).filter((item: string) => item.includes('theme')); + const toRemove = Array.from(classList).filter((item: string) => item.includes('-theme')); if (toRemove.length) { classList.remove(...toRemove); // Remove all old themes. } diff --git a/client/src/app/shared/components/jitsi/jitsi.component.html b/client/src/app/shared/components/jitsi/jitsi.component.html index 50fb2af0b..d15c86d86 100644 --- a/client/src/app/shared/components/jitsi/jitsi.component.html +++ b/client/src/app/shared/components/jitsi/jitsi.component.html @@ -5,7 +5,7 @@ no_meeting_room diff --git a/client/src/app/shared/components/media-upload-content/media-upload-content.component.html b/client/src/app/shared/components/media-upload-content/media-upload-content.component.html index 63bb23f0f..9780162be 100644 --- a/client/src/app/shared/components/media-upload-content/media-upload-content.component.html +++ b/client/src/app/shared/components/media-upload-content/media-upload-content.component.html @@ -115,7 +115,7 @@ type="button" mat-raised-button (click)="onUploadButton()" - color="primary" + color="accent" [disabled]="uploadList.data.length === 0" > {{ 'Upload' | translate }} diff --git a/client/src/app/shared/components/projection-dialog/projection-dialog.component.html b/client/src/app/shared/components/projection-dialog/projection-dialog.component.html index f57c08522..98c1eb9e6 100644 --- a/client/src/app/shared/components/projection-dialog/projection-dialog.component.html +++ b/client/src/app/shared/components/projection-dialog/projection-dialog.component.html @@ -45,7 +45,7 @@ - + diff --git a/client/src/app/shared/components/user-menu/user-menu.component.html b/client/src/app/shared/components/user-menu/user-menu.component.html index e728905b4..6d630beef 100644 --- a/client/src/app/shared/components/user-menu/user-menu.component.html +++ b/client/src/app/shared/components/user-menu/user-menu.component.html @@ -17,7 +17,7 @@ (click)="toggleUserIsPresent()" *ngIf="allowSelfSetPresent" > - + {{ user.is_present ? 'check_box' : 'check_box_outline_blank' }} {{ 'Present' | translate }} diff --git a/client/src/app/site/assignments/components/assignment-list/assignment-list.component.html b/client/src/app/site/assignments/components/assignment-list/assignment-list.component.html index 08191b5fc..0ffd91b35 100644 --- a/client/src/app/site/assignments/components/assignment-list/assignment-list.component.html +++ b/client/src/app/site/assignments/components/assignment-list/assignment-list.component.html @@ -41,7 +41,7 @@ {{ assignment.getListTitle() }} - + {{ assignment.phaseString | translate }} diff --git a/client/src/app/site/login/components/reset-password-confirm/reset-password-confirm.component.html b/client/src/app/site/login/components/reset-password-confirm/reset-password-confirm.component.html index 8dea176ff..5732bc431 100644 --- a/client/src/app/site/login/components/reset-password-confirm/reset-password-confirm.component.html +++ b/client/src/app/site/login/components/reset-password-confirm/reset-password-confirm.component.html @@ -18,7 +18,7 @@