Skip to content

Commit

Permalink
fix missing icons in wallpaper settings (#4308)
Browse files Browse the repository at this point in the history
* remove references to non-existing icons

* fix missing icons in wallpaper settings

* add pr number to changelog
  • Loading branch information
Simon-Laux authored Nov 2, 2024
1 parent f83ce46 commit e2cc34e
Show file tree
Hide file tree
Showing 7 changed files with 12 additions and 5 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
- improve keyboard and screen-reader accessibility #4210
- "Page Up" / "Page Down" not working on scrollable elements except for messages list #4269
- make name more readable in sticker reply #3291
- fix missing icons in wallpaper settings #4308

<a id="1_47_0"></a>

Expand Down
1 change: 1 addition & 0 deletions images/icons/image_outline.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
1 change: 1 addition & 0 deletions images/icons/palette.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion packages/frontend/scss/dialogs/_settings.scss
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
align-content: center;
justify-content: center;
background-color: #d6d6d6 !important;
color: #545454;
--icon-color: #545454;
}

.background-options > div,
Expand Down
4 changes: 2 additions & 2 deletions packages/frontend/src/components/Icon/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,15 @@ export type IconName =
| 'favorite'
| 'forum'
| 'image'
| 'image_outline'
| 'info'
| 'lead-pencil'
| 'list'
| 'map'
| 'media'
| 'minus'
| 'more'
| 'open_in_new'
| 'palette'
| 'paperclip'
| 'person'
| 'person-filled'
Expand All @@ -41,7 +42,6 @@ export type IconName =
| 'settings'
| 'swap_vert'
| 'swap_hor'
| 'tint'
| 'undo'
| 'upload-file'

Expand Down
4 changes: 4 additions & 0 deletions packages/frontend/src/components/Icon/styles.module.scss
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@
background-color: var(--fullScreenMediaButtons);
}

&.iconColorCSSVar {
background-color: var(--icon-color);
}

&.remove {
&:hover {
background-color: #bf0000;
Expand Down
4 changes: 2 additions & 2 deletions packages/frontend/src/components/Settings/Appearance.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -279,14 +279,14 @@ function BackgroundSelector({
className='custom-image'
aria-label={tx('pref_background_custom_image')}
>
<Icon icon='media' size={30} />
<Icon icon='image_outline' size={36} coloring='iconColorCSSVar' />
</div>
<div
onClick={onButton.bind(null, SetBackgroundAction.customColor)}
className='custom-color'
aria-label={tx('pref_background_custom_color')}
>
<Icon icon='tint' size={30} />
<Icon icon='palette' size={36} coloring='iconColorCSSVar' />
</div>
</div>
</div>
Expand Down

0 comments on commit e2cc34e

Please sign in to comment.