Skip to content

Commit

Permalink
refactor(review): changes from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
owilliams320 committed Feb 8, 2024
1 parent 2184f6d commit b6f7241
Show file tree
Hide file tree
Showing 10 changed files with 1,324 additions and 45,643 deletions.
2 changes: 1 addition & 1 deletion .storybook/theme/covalent.dark.theme.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import {
CvDarkOnBackground,
CvLightOnBackground,
CvDarkTextIconOnBackground,
} from '../../libs/tokens/_generated/index';
} from '@covalent/tokens';

export default create({
base: 'dark',
Expand Down
2 changes: 1 addition & 1 deletion .storybook/theme/covalent.light.theme.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import {
CvDarkOnBackground,
CvLightOnBackground,
CvLightTextIconOnBackground,
} from '../../libs/tokens/_generated/index';
} from '@covalent/tokens';

export default create({
base: 'light',
Expand Down
7 changes: 7 additions & 0 deletions libs/components/.storybook/manager-head.html
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,13 @@
box-shadow: none;
}

.light #root > div > div[role='main'] > div {
border: 1px solid rgba(0, 0, 0, 0.12);
}
.dark #root > div > div[role='main'] > div {
border: 1px solid rgba(255, 255, 255, 0.12);
}

.light #storybook-explorer-tree .sidebar-item[data-selected='true'],
.light #storybook-explorer-tree .sidebar-item[data-selected='true'] svg {
color: var(--cv-light-on-primary);
Expand Down
5 changes: 4 additions & 1 deletion libs/components/src/app-shell/app-shell.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,9 @@ export class CovalentAppShell extends DrawerBase {
}

private _handleMenuClick() {
this.mdcRoot.dispatchEvent(
new Event('transitionend', { bubbles: true, composed: true })
);
// Forcefully toggle the open/close state
this._toggleOpen(!this.forcedOpen);

Expand Down Expand Up @@ -152,7 +155,7 @@ export class CovalentAppShell extends DrawerBase {
const modal = this.type === 'modal';
const classes = {
'cov-drawer--forced-open': this.forcedOpen,
'cov-drawer--open': this.drawerOpen,
'cov-drawer--open': this.drawerOpen || this.forcedOpen,
'cov-drawer--hovered': this.hovered,
};
const drawerClasses = {
Expand Down
2,397 changes: 132 additions & 2,265 deletions libs/tokens/_generated/_index.scss

Large diffs are not rendered by default.

1,242 changes: 114 additions & 1,128 deletions libs/tokens/_generated/index.css

Large diffs are not rendered by default.

1,052 changes: 19 additions & 1,033 deletions libs/tokens/_generated/index.d.ts

Large diffs are not rendered by default.

1,242 changes: 114 additions & 1,128 deletions libs/tokens/_generated/index.js

Large diffs are not rendered by default.

Loading

0 comments on commit b6f7241

Please sign in to comment.