Skip to content
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

UX: change 'Delete' button to grey instead of red #4290

Closed
2 tasks done
abctaylor opened this issue Dec 27, 2023 · 7 comments
Closed
2 tasks done

UX: change 'Delete' button to grey instead of red #4290

abctaylor opened this issue Dec 27, 2023 · 7 comments
Labels
area:dashboard The main dashboard page where monitors' status are shown area:ui/ux Interface and User Experience issues feature-request Request for new features to be added

Comments

@abctaylor
Copy link

⚠️ Please verify that this bug has NOT been raised before.

  • I checked and didn't find similar issue

🛡️ Security Policy

Description

Hi - on a healthy setup, you'd want to see all green/neutral colours on your screen. The 'delete' button remains red. To demonstrate, consider this perspective. The red grabs your attention for no real reason. I'd suggest it be changed to grey.

2023-12-27 15_33_51-Uptime Kuma

👟 Reproduction steps

N/A

👀 Expected behavior

Grey button to look better

😓 Actual Behavior

Red button...

🐻 Uptime-Kuma Version

1.23.10

💻 Operating System and Arch

Almalinux kernel 6.6.8

🌐 Browser

Chrome

🐋 Docker Version

Kubernetes

🟩 NodeJS Version

No response

📝 Relevant log output

No response

@abctaylor abctaylor added the bug Something isn't working label Dec 27, 2023
@chakflying chakflying added feature-request Request for new features to be added area:ui/ux Interface and User Experience issues and removed bug Something isn't working labels Dec 27, 2023
@CommanderStorm CommanderStorm added the area:dashboard The main dashboard page where monitors' status are shown label Dec 30, 2023
@iamabhshk
Copy link
Contributor

Is this issue resolved? If not can I take this up?

@chakflying
Copy link
Collaborator

I don't think the proposed solution is a good idea, since it should still be red to signify a "danger action". But maybe we can think about hiding it in an overflow menu.

@abctaylor
Copy link
Author

abctaylor commented Jan 2, 2024 via email

@CommanderStorm
Copy link
Collaborator

my design textbook has this to say about this topic:
image

@iamabhshk
Copy link
Contributor

iamabhshk commented Jan 2, 2024

I made the changes, here is the Pull request
#4307

@CommanderStorm
Copy link
Collaborator

Resolved by #4307

@iamabhshk FYI: next time please add Resolves #4290 to your PRs => said issue will get closed automatically ^^

@iamabhshk
Copy link
Contributor

Sure, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dashboard The main dashboard page where monitors' status are shown area:ui/ux Interface and User Experience issues feature-request Request for new features to be added
Projects
None yet
Development

No branches or pull requests

4 participants