github/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2026-10-28 21:40:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

♻️ Use the DS stroke width in the migrated DS icons (#12673)

Browse Source 
* ♻️ Use the DS stroke width in the migrated DS icons

The DS icons in the areas already moved to the DS set their own
stroke width (1px, 1.33px or 2.25px) to match the legacy icons. They
now keep the DS stroke width, so some icon lines get slightly thinner
or thicker: copy button, context menu, layer row, comments filter,
assets section, share link, dashboard sidebar, team pages, templates,
team switcher and subscription icons.

AI-assisted-by: claude-opus-5-5

* ♻️ Give the share link page rows a class

The page rows of the share link permissions get their own class
instead of a bare li selector. No visual change.

AI-assisted-by: claude-opus-5-5
This commit is contained in:
Eva Marco 2026-10-28 14:27:38 +01:00 committed by GitHub
parent da36e36836
commit e308c2eca6
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
13 changed files with 5 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
frontend/src/app/main/ui/components/copy_button.scss
View File

@ -28,7 +28,6 @@
.icon {
flex-shrink: 0;
stroke-width: 1.33px;
color: var(--color-foreground-secondary);
}
}
@ -81,7 +80,6 @@
.icon {
flex-shrink: 0;
stroke-width: 1.33px;
color: var(--color-foreground-primary);
display: none;
}

1
frontend/src/app/main/ui/components/editable_select.scss
View File

@ -123,7 +123,6 @@
justify-content: center;
align-items: center;
fill: none;
stroke-width: 1.33px;
visibility: hidden;
color: var(--dropdown-element-icon-color);
}

3
frontend/src/app/main/ui/dashboard/organization_team_switch.scss
View File

@ -80,7 +80,6 @@
width: $sz-16;
color: transparent;
fill: none;
stroke-width: 1px;
stroke: var(--icon-foreground);
}
@ -127,7 +126,6 @@
.chevron-icon svg {
color: transparent;
fill: none;
stroke-width: 1px;
stroke: var(--icon-foreground);
}
@ -416,7 +414,6 @@
width: $sz-16;
color: transparent;
fill: none;
stroke-width: 1px;
stroke: var(--sidebar-action-icon-color);
}

3
frontend/src/app/main/ui/dashboard/sidebar.scss
View File

@ -147,7 +147,6 @@
// side keeps the drawing and the text where they were.
.pin-icon {
flex-shrink: 0;
stroke-width: 1.33px;
color: var(--color-foreground-secondary);
margin: 0 px2rem(10);
}
@ -213,7 +212,6 @@
.search-icon {
flex-shrink: 0;
stroke-width: 1px;
}
.clear-search-btn:hover {
@ -374,6 +372,5 @@
}
.exit-icon {
stroke-width: 1px;
color: var(--color-foreground-secondary);
}

1
frontend/src/app/main/ui/dashboard/subscription.scss
View File

@ -167,7 +167,6 @@
.subscription-icon {
stroke: var(--color-foreground-secondary);
stroke-width: 2.25px;
}
.menu-item {

6
frontend/src/app/main/ui/dashboard/team.scss
View File

@ -78,7 +78,6 @@
align-items: center;
block-size: $sz-16;
inline-size: $sz-16;
stroke-width: $b-1;
margin: 0 auto;
color: var(--color-foreground-secondary);
}
@ -125,7 +124,6 @@
}
.image-icon {
stroke-width: $b-1;
color: var(--color-foreground-primary);
}
@ -571,7 +569,6 @@
align-items: center;
block-size: $sz-16;
inline-size: $sz-16;
stroke-width: $b-1;
color: var(--color-accent-success);
}
@ -581,7 +578,6 @@
align-items: center;
block-size: $sz-16;
inline-size: $sz-16;
stroke-width: $b-1;
color: var(--color-accent-warning);
}
@ -641,7 +637,6 @@
align-items: center;
block-size: $sz-16;
inline-size: $sz-16;
stroke-width: $b-1;
color: var(--color-foreground-secondary);
}
@ -976,7 +971,6 @@
align-items: center;
block-size: $sz-16;
inline-size: $sz-16;
stroke-width: $b-1;
color: var(--stroke-color);
}

2
frontend/src/app/main/ui/dashboard/templates.scss
View File

@ -105,7 +105,6 @@
align-items: center;
height: $sz-16;
width: $sz-16;
stroke-width: $b-1;
color: var(--color-foreground-secondary);
}
@ -241,7 +240,6 @@
align-items: center;
height: $sz-16;
width: $sz-16;
stroke-width: $b-1;
color: var(--color-foreground-secondary);
}

2
frontend/src/app/main/ui/viewer/share_link.cljs
View File

@ -283,7 +283,7 @@
[:ul {:class (stl/css :pages-selection)}
(for [{:keys [id name]} pages]
[:li {:class (stl/css :checkbox-wrapper)
[:li {:class (stl/css :checkbox-wrapper :page-row)
:key (dm/str id)}
[:label {:for (dm/str "page-" id)}
[:span {:class (stl/css-case :global/checked (contains? selected id))}

10
frontend/src/app/main/ui/viewer/share_link.scss
View File

@ -136,7 +136,6 @@
.arrow-icon {
flex-shrink: 0;
stroke-width: 1px;
color: var(--color-foreground-secondary);
}
@ -187,13 +186,12 @@
.pages-selection {
margin: 0;
}
// The page rows have no class of their own.
li {
border-block-end: $b-1 solid var(--color-background-quaternary);
}
.page-row {
border-block-end: $b-1 solid var(--color-background-quaternary);
li:last-child {
&:last-child {
border-block-end: none;
}
}

2
frontend/src/app/main/ui/workspace/comments.scss
View File

@ -68,7 +68,6 @@
.arrow-svg {
flex-shrink: 0;
stroke-width: 1.33px;
color: var(--color-foreground-secondary);
}
}
@ -105,7 +104,6 @@
inline-size: $sz-16;
block-size: $sz-16;
stroke: currentcolor;
stroke-width: 1.33px;
color: transparent;
}
}

1
frontend/src/app/main/ui/workspace/context_menu.scss
View File

@ -127,7 +127,6 @@
align-items: center;
inline-size: $sz-16;
color: var(--color-foreground-primary);
stroke-width: 1.33px;
}
.separator {

1
frontend/src/app/main/ui/workspace/sidebar/assets.scss
View File

@ -41,7 +41,6 @@
.section-icon {
flex-shrink: 0;
stroke-width: 1px;
color: var(--color-foreground-secondary);
}

5
frontend/src/app/main/ui/workspace/sidebar/layer_item.scss
View File

@ -211,8 +211,6 @@
}
.toggle-content-icon {
// Same stroke weight as the old 12px icons.
stroke-width: 1.33px;
color: var(--color-foreground-secondary);
.layer-row.hidden & {
@ -284,9 +282,6 @@
// The DS icon box is 16px and the button leaves 12px: keep the box
// whole, centered, instead of letting it shrink and cut the icon.
flex-shrink: 0;
// Same stroke weight as the old 12px icons.
stroke-width: 1.33px;
color: var(--color-foreground-secondary);
.layer-row.hidden & {