-
Notifications
You must be signed in to change notification settings - Fork 898
feat: More modern status icons #8691
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.
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
Conversation
82fe2ff to
26174a3
Compare
Co-authored-by: Rello <[email protected]>
Co-authored-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
Signed-off-by: Rello <[email protected]>
a7013e7 to
cf94314
Compare
|
Artifact containing the AppImage: nextcloud-appimage-pr-8691.zip Digest: To test this change/fix you can download the above artifact file, unzip it, and run it. Please make sure to quit your existing Nextcloud app and backup your data. |
|
I'm guessing this is now resolved? In case it isn't, I can provide assets as needed (it would be nice to update the ones without the logo with the same colors, etc.) I've said before that the badge is kept on the colored icon just so that it doesn't look off-center as it would without it. It's not ideal, but I couldn't figure out a better option :) |
Hi, |






Use themed or neutral icons depending on branding
This PR succeeds #8554 for technical reasons. there, the history of discussions can be found