[PR #16845] [MERGED] 16650 button contrast issues #14927

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

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/16845
Author: @andrewgormley
Created: 7/8/2024
Status: Merged
Merged: 7/16/2024
Merged by: @jeremystretch

Base: feature-uiHead: 16650-button-contrast-issues


📝 Commits (7)

  • 4a96238 fixed #16650: button contrast issues
  • 5f24cbc fixed #16650: green bg text contrast issue
  • 576c554 Merge branch 'feature-ui' into 16650-button-contrast-issues
  • 2800bca Revert errant JS resource updates
  • ea51bda Merge branch 'feature-ui' into 16650-button-contrast-issues
  • d61387a Revert custom button colors
  • a9d396f Fix indentation

📊 Changes

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

View changed files

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

📄 Description

Fixes: #16650

  • (List page) Low contrast on buttons [both, multiple]
  • (Racks) Low contrast download svg button [dark, multiple]
  • (Buttons) Focus state on buttons is missing focus ring causing contrast and CVD issue [both, global]
  • (Misc) Low contrast on cancel button [dark, multiple]
  • (Misc) Low contrast on create combo button [dark, multiple]

🔄 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/16845 **Author:** [@andrewgormley](https://github.com/andrewgormley) **Created:** 7/8/2024 **Status:** ✅ Merged **Merged:** 7/16/2024 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `feature-ui` ← **Head:** `16650-button-contrast-issues` --- ### 📝 Commits (7) - [`4a96238`](https://github.com/netbox-community/netbox/commit/4a962386c06478ed8df5356e70fd10123c36565c) fixed #16650: button contrast issues - [`5f24cbc`](https://github.com/netbox-community/netbox/commit/5f24cbc5ecbaeb505db30ea4ccff405b6624ad58) fixed #16650: green bg text contrast issue - [`576c554`](https://github.com/netbox-community/netbox/commit/576c554976c6726c6a87420739f91c8e2a00b6c9) Merge branch 'feature-ui' into 16650-button-contrast-issues - [`2800bca`](https://github.com/netbox-community/netbox/commit/2800bca465c0114ad5c9708a018793a8e6b49560) Revert errant JS resource updates - [`ea51bda`](https://github.com/netbox-community/netbox/commit/ea51bdabdaff9890eb3b2474d10a02fdcf375874) Merge branch 'feature-ui' into 16650-button-contrast-issues - [`d61387a`](https://github.com/netbox-community/netbox/commit/d61387a7ccf333f0af1b95a6cff2644774b2b06b) Revert custom button colors - [`a9d396f`](https://github.com/netbox-community/netbox/commit/a9d396f6eb3142a6c19b5b02eca48cbd80c89b76) Fix indentation ### 📊 Changes **2 files changed** (+29 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `netbox/project-static/dist/netbox.css` (+1 -1) 📝 `netbox/project-static/styles/overrides/_tabler.scss` (+28 -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: #16650 <!-- Please include a summary of the proposed changes below. --> - [x] (List page) Low contrast on buttons [both, multiple] - [x] (Racks) Low contrast download svg button [dark, multiple] - [x] (Buttons) Focus state on buttons is missing focus ring causing contrast and CVD issue [both, global] - [x] (Misc) Low contrast on cancel button [dark, multiple] - [x] (Misc) Low contrast on create combo button [dark, multiple] --- <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:31 +01:00
adam closed this issue 2025-12-29 23:27:31 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#14927