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

chore(visualization): View encapsulation for global configuration dialog, file panel, ribbon bar #3712

Merged
Merged
Show file tree
Hide file tree
Changes from 52 commits
Commits
Show all changes
56 commits
Select commit Hold shift + click to select a range
35c3ba7
feat(visualization): enable ViewEncapsulation
polina-schoenfeld-mw Aug 1, 2024
3624f01
refactor(visualization): improve html structure and styling
polina-schoenfeld-mw Aug 1, 2024
8b659bd
feat(visualization): add own styling for the GlobalConfigurationDialo…
polina-schoenfeld-mw Aug 1, 2024
27c6b3e
refactor(visualization): only show space for error/hint if existing
polina-schoenfeld-mw Aug 1, 2024
baff713
chore(visualization): styling improvements
polina-schoenfeld-mw Aug 1, 2024
6fa4dd7
refactor(visualization): move search panel into ribbon bar module
alexanderroidl Aug 20, 2024
71702e0
fix(visualization): ribbon bar menu button display
alexanderroidl Aug 23, 2024
011ae94
fix(visualization): metric chooser display
alexanderroidl Aug 23, 2024
8e81f27
fix(visualization): metric chooser icons
alexanderroidl Aug 23, 2024
ae389cf
fix(visualization): metric chooser matselect dropdown
alexanderroidl Aug 23, 2024
714f84e
fix(visualization): edge metric chooser value display
alexanderroidl Aug 23, 2024
f8adfc7
refactor(visualization): ribbon bar modularization
alexanderroidl Aug 25, 2024
074eec1
refactor(visualization): ribbon bar panels
alexanderroidl Aug 25, 2024
a85346f
fix(visualization): ribbon bar panel settings
alexanderroidl Aug 26, 2024
182dc55
fix(visualization): ribbon bar tests
alexanderroidl Aug 26, 2024
8c5c488
test(visualization): ribbon bar panel tests
alexanderroidl Aug 26, 2024
7919921
test(visualization): ribbon bar panel tests
Aug 26, 2024
2503b8e
test(visualization): start tests for SearchPanelComponent
polina-schoenfeld-mw Aug 27, 2024
967b26d
test(visualization): search panel tests
alexanderroidl Aug 27, 2024
bcdf8e2
test(visualization): add tests for SearchPanelComponent
polina-schoenfeld-mw Aug 27, 2024
58a0036
fix(visualization): show color settings in delta mode and make Ribbon…
polina-schoenfeld-mw Aug 28, 2024
3e8208d
test(visualization): add tests for RibbonBarPanelComponent
polina-schoenfeld-mw Aug 29, 2024
d0e5e9f
test(visualization): add test for RibbonBarComponent
polina-schoenfeld-mw Aug 29, 2024
c604786
fix(visualization): remove failing component inputs
Aug 29, 2024
3ebb5ef
fix(visualization): fix formatting
Aug 29, 2024
ab58819
fix(visualization): remove deprecated test for icon color
Aug 29, 2024
a0fc6d8
fix(visualization): start fixing edgeMetricChooserComponent tests
Aug 29, 2024
a354d4c
test(visualization): edge metric chooser
alexanderroidl Aug 30, 2024
491cb6b
chore(visualization): formatting
polina-schoenfeld-mw Aug 30, 2024
b1f3a7d
fix(visualization): hide metric sum value when opened
polina-schoenfeld-mw Aug 30, 2024
880715d
refactor(visualization): introduce method for calculating the backgro…
polina-schoenfeld-mw Aug 30, 2024
81341c6
fix(visualization): metric color settings
alexanderroidl Aug 30, 2024
1b4d140
fix(visualization): fix range-slider-label position
Aug 30, 2024
094d331
fix(visualization): fix range-slider-container top margin
Aug 30, 2024
cf6ead1
fix(visualization): fix gradient-mode-selector width
Aug 30, 2024
70b1ddc
fix(visualization): toggle of search panel mode selector buttons
polina-schoenfeld-mw Sep 2, 2024
4fd6c68
feat(visualization): enable ViewEncapsulation
polina-schoenfeld-mw Sep 2, 2024
74f07af
chore(visualization): style improvements
polina-schoenfeld-mw Sep 2, 2024
fdc70ae
feat(visualization): enable ViewEncapsulation
polina-schoenfeld-mw Aug 21, 2024
9a111b4
chore(visualization): split styling
polina-schoenfeld-mw Aug 21, 2024
cd72448
chore(visualization): styling improvements
polina-schoenfeld-mw Aug 21, 2024
76e69dc
chore(visualization): styling improvements
polina-schoenfeld-mw Aug 21, 2024
6b31f9e
refactor(visualization): enable ViewEncapsulation in metric chooser c…
polina-schoenfeld-mw Aug 21, 2024
c120d03
feat(visualization): enable ViewEncapsulation and add styling
polina-schoenfeld-mw Aug 22, 2024
5fd3dcb
chore(visualization): style improvements
polina-schoenfeld-mw Aug 22, 2024
d8f84b0
chore(visualization): renaming
polina-schoenfeld-mw Sep 3, 2024
5020d30
chore(visualization): style improvements in work
polina-schoenfeld-mw Sep 3, 2024
ea0a711
fix(visualization): file panel overlay
alexanderroidl Sep 4, 2024
dcf72fa
fix(visualization): file search panel collapsing
alexanderroidl Sep 4, 2024
d950992
style(visualization): material select overlay padding
alexanderroidl Sep 4, 2024
be68441
chore(visualization): changelog
alexanderroidl Sep 4, 2024
c1fa56c
chore(visualization): prettier formatting
alexanderroidl Sep 4, 2024
79d1584
fix(visualization): e2e paths
alexanderroidl Sep 4, 2024
535510e
fix(visualization): e2e paths
alexanderroidl Sep 4, 2024
c86742e
chore(visualization): e2e
alexanderroidl Sep 4, 2024
2ce80aa
fix(visualization): fix tests
Sep 4, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions visualization/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/)
### Chore 👨‍💻 👩‍💻

- Bump node version from 18 to 20 [#3690](https://github.com/MaibornWolff/codecharta/pull/3690)
- Enabled view encapsulation for Global Configuration dialog, Ribbon Bar and File Panel [#3712](https://github.com/MaibornWolff/codecharta/pull/3712)

## [1.127.1] - 2024-08-12

Expand Down
2 changes: 1 addition & 1 deletion visualization/app/app.module.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import { ResetColorRangeEffect } from "./codeCharta/state/store/dynamicSettings/
import { appReducers, setStateMiddleware } from "./codeCharta/state/store/state.manager"
import { ChangelogDialogModule } from "./codeCharta/ui/dialogs/changelogDialog/changelogDialog.module"
import { dialogs } from "./codeCharta/ui/dialogs/dialogs"
import { BlacklistSearchPatternEffect } from "./codeCharta/ui/searchPanel/searchBar/blacklistSearchPattern.effect"
import { BlacklistSearchPatternEffect } from "./codeCharta/ui/ribbonBar/searchPanel/searchBar/blacklistSearchPattern.effect"
import { MaterialModule } from "./material/material.module"

@NgModule({
Expand Down
45 changes: 0 additions & 45 deletions visualization/app/codeCharta/codeCharta.component.scss
Original file line number Diff line number Diff line change
Expand Up @@ -53,49 +53,4 @@ cc-code-charta {
width: 24px;
vertical-align: top;
}

.state-selector-button {
height: 22px;
vertical-align: bottom;
width: 30px;
min-height: 0;
min-width: 0;
margin: 0;
padding: 0;
font-size: 9pt;
text-transform: none;
background-color: rgba(230, 230, 230, 0.96);

&:hover {
background-color: rgb(200, 200, 200) !important;
}

i {
vertical-align: middle;
height: 26px;
}

&.left {
border-radius: 5px 0 0 5px;
}

&.middle {
border-radius: 0;
border-left: 1px solid rgba(255, 255, 255, 1);
border-right: 1px solid rgba(255, 255, 255, 1);
}

&.right {
border-radius: 0 5px 5px 0;
}

&.current {
background-color: rgb(0, 150, 136);
color: #ffffff;
}

&.current:hover {
background-color: variables.$cc-emphasized-color !important;
}
}
}
Original file line number Diff line number Diff line change
@@ -1,3 +1,2 @@
<button class="cc-ribbon-bar-menu-button" title="Create new Custom View" (click)="showAddCustomConfigDialog()">
<i class="fa fa-plus"></i>
</button>
<cc-ribbon-bar-menu-button icon="fa-plus" [title]="'Create new Custom View'" (click)="showAddCustomConfigDialog()">
</cc-ribbon-bar-menu-button>
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,17 @@ import { FormsModule, ReactiveFormsModule } from "@angular/forms"
import { CommonModule } from "@angular/common"
import { DownloadAndPurgeConfigsComponent } from "./addCustomConfigDialog/downloadAndPurgeConfigs/downloadAndPurgeConfigs.component"
import { CustomConfigNoteDialogButtonModule } from "../customConfigNoteDialogButton/customConfigNoteDialogButton.module"
import { RibbonBarMenuButtonModule } from "../../ribbonBar/ribbonBarMenuButton/ribbonBarMenuButton.module"

@NgModule({
imports: [MaterialModule, ReactiveFormsModule, FormsModule, CommonModule, CustomConfigNoteDialogButtonModule],
imports: [
MaterialModule,
ReactiveFormsModule,
FormsModule,
CommonModule,
CustomConfigNoteDialogButtonModule,
RibbonBarMenuButtonModule
],
declarations: [AddCustomConfigButtonComponent, AddCustomConfigDialogComponent, DownloadAndPurgeConfigsComponent],
exports: [AddCustomConfigButtonComponent]
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,11 @@
display: block;
text-align: left;
padding: 0 16px;
color: var(--exceed-local-storage-message-text-color);
color: #b02929;
}

.exceed-local-storage-button {
margin-right: 89px;
color: var(--exceed-local-storage-button-text-color);
background-color: var(--exceed-local-storage-button-background-color);
color: #ffffff;
background-color: #b02929;
}
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<div class="custom-configs-panel-buttons">
<button class="cc-ribbon-bar-menu-button" title="Open your saved Custom Views" (click)="openCustomConfigDialog()">
<i class="fa fa-cogs"></i>
</button>
<cc-ribbon-bar-menu-button icon="fa-cogs" [title]="'Open your saved Custom Views'" (click)="openCustomConfigDialog()">
</cc-ribbon-bar-menu-button>

<cc-add-custom-config-button></cc-add-custom-config-button>
</div>
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import { FormsModule } from "@angular/forms"
import { CustomConfigNoteDialogButtonModule } from "./customConfigNoteDialogButton/customConfigNoteDialogButton.module"
import { TruncateTextPipeModule } from "../../util/pipes/TruncateTextPipeModule"
import { FilterCustomConfigDataBySearchTermPipe } from "./customConfigList/customConfigItemGroup/customConfigDescription/filterCustomConfigDataBySearchTerm.pipe"
import { RibbonBarMenuButtonModule } from "../ribbonBar/ribbonBarMenuButton/ribbonBarMenuButton.module"

@NgModule({
imports: [
Expand All @@ -25,7 +26,8 @@ import { FilterCustomConfigDataBySearchTermPipe } from "./customConfigList/custo
AddCustomConfigButtonModule,
DownloadCustomConfigButtonModule,
FormsModule,
TruncateTextPipeModule
TruncateTextPipeModule,
RibbonBarMenuButtonModule
],
declarations: [
CustomConfigsComponent,
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
:host {
width: 210px;
display: flex;
align-items: center;
font-size: 12px;
gap: 6px;
margin-left: 5px;
margin-right: 5px;
}
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { Component, ViewEncapsulation } from "@angular/core"
import { Component } from "@angular/core"
import { Store } from "@ngrx/store"
import { CcState } from "../../../codeCharta.model"
import { setDistributionMetric } from "../../../state/store/dynamicSettings/distributionMetric/distributionMetric.actions"
Expand All @@ -7,7 +7,7 @@ import { distributionMetricSelector } from "../../../state/store/dynamicSettings
@Component({
selector: "cc-distribution-metric-chooser",
templateUrl: "./distributionMetricChooser.component.html",
encapsulation: ViewEncapsulation.None
styleUrls: ["./distributionMetricChooser.component.scss"]
})
export class DistributionMetricChooserComponent {
distributionMetric$ = this.store.select(distributionMetricSelector)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,6 @@ cc-file-extension-bar {
display: flex;
height: 17px;

cc-distribution-metric-chooser {
width: 210px;
display: flex;
align-items: center;
font-size: 12px;
gap: 6px;
padding-left: 6px;
}

.cc-bar {
flex: 1;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,73 +1,10 @@
@use "variables";

cc-file-panel {
:host {
display: flex;
gap: inherit;

&.hidden {
display: none;
}

.state-selector-button {
&.mode-selector {
height: 24px;
width: 66px;
background-color: #ffffff;

&.middle {
border-left: 1px solid rgba(230, 230, 230, 0.96);
border-right: 1px solid rgba(230, 230, 230, 0.96);
}

&.current {
background-color: variables.$cc-primary-color;
color: #ffffff;
}

&.current:hover {
background-color: variables.$cc-primary-color;
}
}
}

.reference-comparison-switch {
background-color: variables.$cc-primary-color;
margin: 0;
height: 24px;
width: 36px;
border-radius: 5px;

&[disabled] {
background-color: variables.$cc-disabled-color;
}

&:hover {
background-color: variables.$cc-emphasized-color;

&[disabled] {
background-color: variables.$cc-disabled-color;
}
}

.fa {
vertical-align: middle;
color: rgb(255, 255, 255);
}
}

.pictogram {
display: inline-flex;
width: 10px;
margin-right: 5px;
margin-left: 5px;
height: 22px;
border-radius: 2px;
align-self: center;
vertical-align: bottom;
}

.standard-select,
.delta-select {
width: 200px;
}
}
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
import { Component, ViewEncapsulation } from "@angular/core"
import { Component } from "@angular/core"
import { Store } from "@ngrx/store"
import { CcState } from "../../codeCharta.model"
import { isDeltaStateSelector } from "../../state/selectors/isDeltaState.selector"

@Component({
selector: "cc-file-panel",
templateUrl: "./filePanel.component.html",
styleUrls: ["./filePanel.component.scss"],
encapsulation: ViewEncapsulation.None
styleUrls: ["./filePanel.component.scss"]
})
export class FilePanelComponent {
isDeltaState$ = this.store.select(isDeltaStateSelector)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
<div class="delta-select-wrapper">
<mat-select class="delta-select" [value]="referenceFile$ | async" (selectionChange)="handleDeltaReferenceFileChange($event.value)">
<mat-select
class="delta-select"
[value]="referenceFile$ | async"
(selectionChange)="handleDeltaReferenceFileChange($event.value)"
panelClass="cc-file-select"
>
@for (file of files$ | async; track file) {
<mat-option [value]="file.file">
{{ file.file.fileMeta.fileName | removeExtension }}
Expand All @@ -19,6 +24,7 @@
class="delta-select"
placeholder="select comparison file"
[value]="comparisonFile$ | async"
panelClass="cc-file-select"
(selectionChange)="handleDeltaComparisonFileChange($event.value)"
>
@for (file of possibleComparisonFiles$ | async; track file) {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,46 @@
cc-file-panel-delta-selector {
.delta-select-wrapper {
display: flex;
gap: 12px;
@use "variables";

.delta-select-wrapper {
display: flex;
gap: 12px;
}

.reference-comparison-switch {
background-color: variables.$cc-primary-color;
margin: 0;
height: 24px;
width: 36px;
border-radius: 5px;

&[disabled] {
background-color: variables.$cc-disabled-color;
}

&:hover {
background-color: variables.$cc-emphasized-color;

&[disabled] {
background-color: variables.$cc-disabled-color;
}
}

.fa {
vertical-align: middle;
color: rgb(255, 255, 255);
}
}

.pictogram {
display: inline-flex;
width: 10px;
margin-right: 5px;
margin-left: 5px;
height: 22px;
border-radius: 2px;
align-self: center;
vertical-align: bottom;
}

.delta-select {
width: 200px;
}
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { Component, ViewEncapsulation } from "@angular/core"
import { Component } from "@angular/core"
import { Store } from "@ngrx/store"
import { map } from "rxjs"
import { CCFile, CcState } from "../../../codeCharta.model"
Expand All @@ -11,8 +11,7 @@ import { pictogramBackgroundSelector } from "./pictogramBackground.selector"
@Component({
selector: "cc-file-panel-delta-selector",
templateUrl: "./filePanelDeltaSelector.component.html",
styleUrls: ["./filePanelDeltaSelector.component.scss"],
encapsulation: ViewEncapsulation.None
styleUrls: ["./filePanelDeltaSelector.component.scss"]
})
export class FilePanelDeltaSelectorComponent {
files$ = this.store.select(filesSelector)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,18 +1,18 @@
<mat-select
panelClass="file-select"
panelClass="cc-file-select"
[(value)]="selectedFilesInUI"
(selectionChange)="handleSelectedFilesChanged($event.value)"
(openedChange)="handleOpenedChanged($event)"
class="standard-select"
class="cc-standard-select"
multiple
>
<div class="file-selection-actions">
<button class="file-select-action-button" (click)="handleSelectAllFiles()">All</button>
<button class="file-select-action-button" (click)="handleSelectZeroFiles()">None</button>
<button class="file-select-action-button" (click)="handleInvertSelectedFiles()">Invert</button>
<div class="cc-file-selection-actions">
<button class="cc-file-select-action-button" (click)="handleSelectAllFiles()">All</button>
<button class="cc-file-select-action-button" (click)="handleSelectZeroFiles()">None</button>
<button class="cc-file-select-action-button" (click)="handleInvertSelectedFiles()">Invert</button>
</div>
@for (file of fileStates; track file) {
<mat-option class="filenames" [value]="file.file">
<mat-option class="cc-file-name" [value]="file.file">
{{ file.file.fileMeta.fileName | removeExtension }}
@if (fileStates.length > 1) {
<cc-remove-file-button [filename]="file.file.fileMeta.fileName"></cc-remove-file-button>
Expand Down
Loading
Loading