Skip to content

Commit c26014b

Browse files
authored
fix: update crud base styles (#10294)
* fix: update crud base styles * Update vaadin-crud-base-styles.js * test: update screenshots
1 parent 9af9299 commit c26014b

26 files changed

+26
-20
lines changed

packages/component-base/src/styles/style-props.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/crud/src/styles/vaadin-crud-base-styles.js

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ export const crudStyles = css`
1919
--vaadin-crud-editor-max-width: 40%;
2020
border: var(--vaadin-crud-border-width, 1px) solid
2121
var(--vaadin-crud-border-color, var(--vaadin-border-color-secondary));
22-
border-radius: var(--vaadin-crud-border-radius, var(--vaadin-radius-l));
22+
border-radius: var(--vaadin-crud-border-radius, var(--vaadin-radius-m));
2323
height: 400px;
2424
width: 100%;
2525
background: var(--vaadin-crud-background, var(--vaadin-background-color));
@@ -92,7 +92,7 @@ export const crudStyles = css`
9292
display: flex;
9393
flex-shrink: 0;
9494
justify-content: flex-end;
95-
padding: var(--vaadin-crud-toolbar-padding, var(--vaadin-padding-m));
95+
padding: var(--vaadin-crud-toolbar-padding, var(--vaadin-dialog-padding, var(--vaadin-padding-l)));
9696
}
9797
9898
:host([no-toolbar]) [part='toolbar'] {
@@ -131,11 +131,11 @@ export const crudStyles = css`
131131
}
132132
133133
[part='header'] {
134-
color: var(--vaadin-crud-header-color, var(--vaadin-text-color));
135-
font-size: var(--vaadin-crud-header-font-size, 1em);
136-
font-weight: var(--vaadin-crud-header-font-weight, 600);
137-
line-height: var(--vaadin-crud-header-line-height, inherit);
138-
padding: var(--vaadin-crud-header-padding, var(--vaadin-padding-m));
134+
color: var(--vaadin-crud-header-color, var(--vaadin-dialog-title-color, var(--vaadin-text-color)));
135+
font-size: var(--vaadin-crud-header-font-size, var(--vaadin-dialog-title-font-size, 1em));
136+
font-weight: var(--vaadin-crud-header-font-weight, var(--vaadin-dialog-title-font-weight, 600));
137+
line-height: var(--vaadin-crud-header-line-height, var(--vaadin-dialog-title-line-height, inherit));
138+
padding: var(--vaadin-crud-header-padding, var(--vaadin-dialog-padding, var(--vaadin-padding-l)));
139139
}
140140
141141
::slotted([slot='header']) {
@@ -146,7 +146,8 @@ export const crudStyles = css`
146146
}
147147
148148
::slotted([slot='form']) {
149-
padding: var(--vaadin-crud-form-padding, var(--vaadin-padding-m));
149+
padding: var(--vaadin-crud-form-padding, var(--vaadin-dialog-padding, var(--vaadin-padding-l)));
150+
padding-top: 0;
150151
}
151152
152153
[part='footer'] {
@@ -156,10 +157,15 @@ export const crudStyles = css`
156157
display: flex;
157158
flex: none;
158159
gap: var(--vaadin-crud-footer-gap, var(--vaadin-gap-s));
159-
padding: var(--vaadin-crud-footer-padding, var(--vaadin-padding-m));
160+
padding: var(--vaadin-crud-footer-padding, var(--vaadin-dialog-padding, var(--vaadin-padding-l)));
160161
}
161162
162163
::slotted([slot='delete-button']) {
163164
margin-inline-start: auto;
164165
}
166+
167+
:host([theme~='no-border']) {
168+
border: 0;
169+
border-radius: 0;
170+
}
165171
`;

packages/crud/src/styles/vaadin-crud-dialog-overlay-base-styles.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,11 @@ import { dialogOverlayStyles } from '@vaadin/dialog/src/styles/vaadin-dialog-ove
1414

1515
const crudDialogOverlay = css`
1616
[part='header'] {
17-
color: var(--vaadin-crud-dialog-header-color, var(--vaadin-text-color));
18-
font-size: var(--vaadin-crud-dialog-header-font-size, 1em);
19-
font-weight: var(--vaadin-crud-dialog-header-font-weight, 600);
20-
line-height: var(--vaadin-crud-dialog-header-line-height, inherit);
21-
padding: var(--vaadin-crud-header-padding, var(--vaadin-padding-l));
17+
color: var(--vaadin-dialog-title-color, var(--vaadin-text-color));
18+
font-size: var(--vaadin-dialog-title-font-size, 1em);
19+
font-weight: var(--vaadin-dialog-title-font-weight, 600);
20+
line-height: var(--vaadin-dialog-title-line-height, inherit);
21+
padding: var(--vaadin-crud-header-padding, var(--vaadin-dialog-padding, var(--vaadin-padding-l)));
2222
}
2323
2424
::slotted([slot='header']) {

packages/crud/src/styles/vaadin-crud-edit-base-styles.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export const crudEditStyles = css`
1616
background: currentColor;
1717
display: block;
1818
height: var(--vaadin-icon-size, 1lh);
19-
mask: var(--_vaadin-icon-edit) 50% / var(--vaadin-icon-visual-size, 100%) no-repeat;
19+
mask: var(--_vaadin-icon-pen) 50% / var(--vaadin-icon-visual-size, 100%) no-repeat;
2020
width: var(--vaadin-icon-size, 1lh);
2121
}
2222

packages/crud/src/vaadin-crud-edit-column.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,12 @@ class CrudEditColumn extends GridColumn {
3939
static get properties() {
4040
return {
4141
/**
42-
* Width of the cells for this column.
42+
* Automatically sets the width of the column based on the column contents when this is set to `true`.
4343
* @private
4444
*/
45-
width: {
46-
type: String,
47-
value: '4rem',
45+
autoWidth: {
46+
type: Boolean,
47+
value: true,
4848
},
4949

5050
/**
-776 Bytes
Loading
-753 Bytes
Loading
-303 Bytes
Loading
-80 Bytes
Loading
-319 Bytes
Loading

0 commit comments

Comments
 (0)