[PR #16355] [MERGED] Fixes #16274: Dark mode highlight color #14837

Closed
opened 2025-12-29 23:27:02 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/16355
Author: @ljarasius
Created: 5/30/2024
Status: Merged
Merged: 6/6/2024
Merged by: @jeremystretch

Base: developHead: 16274-dark-mode-highlight


📝 Commits (6)

  • c9c5bdb Increase ::selection background-color aplha
  • 4228d80 Improve comment for override
  • c5e157c Merge branch 'netbox-community:develop' into 16274-dark-mode-highlight
  • 12df38b Add compiled CSS
  • f4dc84f Merge branch 'develop' into 16274-dark-mode-highlight
  • baf469d Only override on dark theme

📊 Changes

2 files changed (+6 additions, -1 deletions)

View changed files

📝 netbox/project-static/dist/netbox.css (+1 -1)
📝 netbox/project-static/styles/overrides/_tabler.scss (+5 -0)

📄 Description

Fixes: #16274

Tabler has an alpha value for .16 by default for the selection background colour. Increasing this makes the distinction much clearer, .48 chosen as it is "3 times" more and seems to be a good place to start. Light mode is still clear with this change.

Before:
Screenshot 2024-05-30 at 11 03 54

After:
Screenshot 2024-05-30 at 11 04 47


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbox-community/netbox/pull/16355 **Author:** [@ljarasius](https://github.com/ljarasius) **Created:** 5/30/2024 **Status:** ✅ Merged **Merged:** 6/6/2024 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `16274-dark-mode-highlight` --- ### 📝 Commits (6) - [`c9c5bdb`](https://github.com/netbox-community/netbox/commit/c9c5bdbcd245fcd35d238d77b67f14d091ae4737) Increase ::selection background-color aplha - [`4228d80`](https://github.com/netbox-community/netbox/commit/4228d809bdd2fd53453c9c09e24ab6f889664fd6) Improve comment for override - [`c5e157c`](https://github.com/netbox-community/netbox/commit/c5e157c607399b51d2811532393648dd34fa73aa) Merge branch 'netbox-community:develop' into 16274-dark-mode-highlight - [`12df38b`](https://github.com/netbox-community/netbox/commit/12df38bd22d1685474ae04a584c2b8d346836f77) Add compiled CSS - [`f4dc84f`](https://github.com/netbox-community/netbox/commit/f4dc84f2340303f526b18cba0a330f551720fae3) Merge branch 'develop' into 16274-dark-mode-highlight - [`baf469d`](https://github.com/netbox-community/netbox/commit/baf469d545afff702b742a7eb455489cbb1c7b65) Only override on dark theme ### 📊 Changes **2 files changed** (+6 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `netbox/project-static/dist/netbox.css` (+1 -1) 📝 `netbox/project-static/styles/overrides/_tabler.scss` (+5 -0) </details> ### 📄 Description <!-- Thank you for your interest in contributing to NetBox! Please note that our contribution policy requires that a feature request or bug report be approved and assigned prior to opening a pull request. This helps avoid waste time and effort on a proposed change that we might not be able to accept. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ISSUE WHICH HAS BEEN ASSIGNED TO YOU, IT WILL BE CLOSED AUTOMATICALLY. Please specify your assigned issue number on the line below. --> ### Fixes: #16274 <!-- Please include a summary of the proposed changes below. --> Tabler has an alpha value for .16 by default for the selection background colour. Increasing this makes the distinction much clearer, .48 chosen as it is "3 times" more and seems to be a good place to start. Light mode is still clear with this change. Before: ![Screenshot 2024-05-30 at 11 03 54](https://github.com/netbox-community/netbox/assets/19970910/636eae23-3488-4dca-99aa-8079a4f22a2b) After: ![Screenshot 2024-05-30 at 11 04 47](https://github.com/netbox-community/netbox/assets/19970910/9a0ed385-592e-4d72-beeb-bbad4c954744) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 23:27:02 +01:00
adam closed this issue 2025-12-29 23:27:02 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#14837