Skip to content

feat(ui): switch to outline icons where possible #11322

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat(ui): switch to outline icons where possible #11322

wants to merge 2 commits into from

Conversation

ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst commented Jun 26, 2025 •
edited
Loading

Closes #11321

There is no outline variant for these:

Airplane
Alarm
ArrowCollapse
ArrowDown
ArrowExpand
ArrowLeft
ArrowRight
ArrowUp
cancel (mdi)
ChevronDown
ChevronLeft
ChevronRight
ChevronUp
CloseThick
ContentCopy
Domain
DotsHorizontal
DotsHorizontal
EraserVariant
FilterVariant
Fire
FormatColumns
FormatSize
ImageSizeSelectActual
Link
magnify (outline)
Minus
OpenInNew
OpenInNew
Paperclip
Refresh
ReorderHorizontal
Train
Translate

Examples

Bildschirmfoto vom 2025-06-26 17-55-12
Bildschirmfoto vom 2025-06-26 17-54-36
Bildschirmfoto vom 2025-06-26 17-54-06
Bildschirmfoto vom 2025-06-26 17-53-58

@ChristophWurst
feat(ui): switch to outline icons where possible
a7fef87 
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst
fixup! feat(ui): switch to outline icons where possible
17ed6a2 
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst self-assigned this Jun 26, 2025
@ChristophWurst ChristophWurst requested review from jancborchardt and kra-mo and removed request for kesselb and GretaD June 26, 2025 16:01
@ChristophWurst
Copy link
Member Author

@jancborchardt @kra-mo so, shall we proceed and get this merged? It would be released with the Mail v5.2.0 release in the next few weeks

@kra-mo
Copy link
Member

kra-mo commented Jun 30, 2025

@jancborchardt @kra-mo so, shall we proceed and get this merged? It would be released with the Mail v5.2.0 release in the next few weeks

Has the outline plus button been changed? In that case, sure :)

jancborchardt
jancborchardt reviewed Jun 30, 2025
View reviewed changes
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me too except the outlinded "+" icon as @kra-mo also noted. :)

Also FYI @nimishavijay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jancborchardt jancborchardt jancborchardt left review comments

@kra-mo kra-mo Awaiting requested review from kra-mo

@nimishavijay nimishavijay Awaiting requested review from nimishavijay

At least 1 approving review is required to merge this pull request.

Assignees

@ChristophWurst ChristophWurst

Labels
3. to review design enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use outline icons
3 participants
@ChristophWurst @kra-mo @jancborchardt