Skip to content

Switch to outlined icon variant - #4277

Merged
dartcafe merged 9 commits into
mainfrom
enh/icons-outline
Sep 4, 2025
Merged

dartcafe merged 9 commits into
mainfrom
enh/icons-outline

Conversation

@dartcafe

@dartcafe dartcafe commented Sep 4, 2025 •

Copy link
Copy Markdown
Collaborator

resolves #4269

Reverted some individual icons, because outlined looks a little bit off IMHO:

outlined filled
grafik grafik

Used the chance to change the "Maybe" icon from Checkmark with braces to a tilde:

before after
grafik grafik

Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe dartcafe added this to the 8.4.4 milestone Sep 4, 2025
@dartcafe dartcafe added the design label Sep 4, 2025
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Comment thread src/components/Configuration/ConfigDangerArea.vue Outdated
Comment thread src/components/Navigation/PollNavigationItems.vue Outdated
Comment thread src/components/Navigation/PollNavigationItems.vue Outdated
Comment thread src/components/Configuration/ConfigDangerArea.vue Outdated
Comment thread src/components/Actions/modules/ActionDelete.vue Outdated
Comment thread src/components/Actions/modules/ActionDelete.vue Outdated
Comment thread src/components/Options/OptionMenu.vue Outdated
Comment thread src/components/Options/OptionMenu.vue Outdated
Comment thread src/components/Options/OptionsText.vue Outdated
Signed-off-by: dartcafe <github@dartcafe.de>
Comment thread src/components/Poll/PollInfoLine.vue Outdated
Comment thread src/components/Poll/PollInformation.vue Outdated
Comment thread src/components/PollList/PollItem.vue Outdated
Comment thread src/components/PollList/PollItem.vue Outdated
@AndyScherzinger

Copy link
Copy Markdown
Member

Love the chance for using tilde πŸ‘

Comment thread src/components/User/UserItem.vue Outdated
@AndyScherzinger

Copy link
Copy Markdown
Member

Done with the review @dartcafe πŸ‘

I can have another final look if you want and ping me πŸ‘

Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe

dartcafe commented Sep 4, 2025

Copy link
Copy Markdown
Collaborator Author

Done with the review @dartcafe πŸ‘

Too early, I already added some missing ones.

I can have another final look if you want and ping me πŸ‘

Await the rerequest.

Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe

dartcafe commented Sep 4, 2025 •

Copy link
Copy Markdown
Collaborator Author

I can have another final look if you want and ping me πŸ‘

@AndyScherzinger Ping.

Signed-off-by: dartcafe <github@dartcafe.de>

@AndyScherzinger AndyScherzinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good and complete πŸ‘

@AndyScherzinger

Copy link
Copy Markdown
Member

Nice and blazing fast work @dartcafe - checked all changes, looks good πŸ‘

@dartcafe
dartcafe merged commit ba93552 into main Sep 4, 2025
40 checks passed
@AndyScherzinger
AndyScherzinger deleted the enh/icons-outline branch September 4, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Switch to outlined icon variant

2 participants