Skip to content

fix(theming): replace standard generic package icon by material design - #45480

Merged
skjnldsv merged 1 commit into
masterfrom
fix/archive-grey-color
May 30, 2024
Merged

fix(theming): replace standard generic package icon by material design#45480
skjnldsv merged 1 commit into
masterfrom
fix/archive-grey-color

Conversation

@skjnldsv

Copy link
Copy Markdown
Member

From #42517

Considering we have replaced the office icons and other filetypes are the standard grey color, ket's replace that old #0082c9 Nextcloud blue by the same #969696 grey.
I also used that opportunity to update the icon from the material design, like it was done with others in the last months.

BeforeAfter
2024-05-23_15-122024-05-23_15-11

cc @nextcloud/designers

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsvskjnldsv added design Design, UI, UX, etc. 3. to review Waiting for reviews feature: files papercut Annoying recurring UX issue with possibly simple fix. labels May 23, 2024
@skjnldsvskjnldsv added this to the Nextcloud 30 milestone May 23, 2024
@skjnldsv
skjnldsv requested review from a team, marcoambrosini, susnux and szaimenMay 23, 2024 13:16
@skjnldsvskjnldsv self-assigned this May 23, 2024
@skjnldsv
skjnldsv requested review from sorbaugh and removed request for a teamMay 23, 2024 13:16
@skjnldsv
skjnldsv merged commit b8856e2 into masterMay 30, 2024
@skjnldsv
skjnldsv deleted the fix/archive-grey-color branch May 30, 2024 06:45
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable29

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable28

@susnux

Copy link
Copy Markdown
Contributor

BTW if the icon has #0082c9 as color then it will be automatically themed using the current primary color (see IconController).
But for this I think package in grey is better

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

yeah, I think we need to discuss what we really want to do with the theming mime colour api. I'm not sure this is really relevant anymore 🤔

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

Discussion: #45579

@blizzzblizzz mentioned this pull request Jul 24, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsdesignDesign, UI, UX, etc.feature: filespapercutAnnoying recurring UX issue with possibly simple fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@skjnldsv@susnux@marcoambrosini