Skip to content

Revert "Merge pull request #62947 from nextcloud/fix/noid/filled-core… - #63012

Merged
pringelmann merged 1 commit into
masterfrom
revert/filled-core-action-icons
Aug 10, 2026
Merged

Revert "Merge pull request #62947 from nextcloud/fix/noid/filled-core…#63012
pringelmann merged 1 commit into
masterfrom
revert/filled-core-action-icons

Conversation

@pringelmann

Copy link
Copy Markdown
Contributor

Reverts #62947

@pringelmann
pringelmann requested review from a team as code owners August 7, 2026 10:07
@pringelmann
pringelmann requested review from Altahrim, come-nc, icewind1991, kristian-zendato, skjnldsv and sorbaugh and removed request for a team August 7, 2026 10:07
@pringelmann pringelmann self-assigned this Aug 7, 2026
@pringelmann pringelmann added 3. to review Waiting for reviews design Design, UI, UX, etc. labels Aug 7, 2026
@pringelmann pringelmann added this to the Nextcloud 35 milestone Aug 7, 2026
…-action-icons"

This reverts commit e9b5cf8, reversing
changes made to 9ee4fa8.

Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann
pringelmann force-pushed the revert/filled-core-action-icons branch from a78ecc6 to 08cbc4e Compare August 7, 2026 10:11

@artonge artonge left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not dramatic to keep as at least it make the view consistent with itself. But approved anyway

@pringelmann
pringelmann enabled auto-merge August 7, 2026 14:36
@come-nc
come-nc removed their request for review August 10, 2026 08:10

@jancborchardt jancborchardt 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.

Nice! :)

Regarding your Deck icon question from #62947 (comment)
Which lib are we using there? tbh I thought the app icon was from Material Design, cc @juliusknorr :D

It’s ok to have the Deck icon there for now. 2 possible alternatives are these, but would be more confusing than beneficial:

@pringelmann
pringelmann merged commit 32eaeaa into master Aug 10, 2026
132 checks passed
@pringelmann
pringelmann deleted the revert/filled-core-action-icons branch August 10, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews design Design, UI, UX, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants