starred/documenso

Fork 0

mirror of https://github.com/documenso/documenso.git synced 2026-04-24 21:36:01 +03:00

[PR #2358] [MERGED] fix: replace hardcoded #7AC455 with text-documenso-700 token #2244

New issue

Closed

opened 2026-02-26 20:33:03 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 20:33:03 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/2358
Author: @dylantarre
Created: 1/5/2026
Status: ✅ Merged
Merged: 1/6/2026
Merged by: @Mythie

Base: main ← Head: fix/design-consistency-navigation-links

📝 Commits (1)

cd1f573 fix: replace hardcoded #7AC455 with text-documenso-700 token

📊 Changes

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

View changed files

📝 apps/remix/app/components/general/skeletons/document-edit-skeleton.tsx (+1 -1)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id._index.tsx (+1 -1)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.legacy_editor.tsx (+1 -1)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.logs.tsx (+1 -1)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id._index.tsx (+1 -1)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id.legacy_editor.tsx (+1 -1)

📄 Description

Summary

Standardizes navigation link colors by replacing hardcoded #7AC455 hex values with the existing text-documenso-700 design token.

Changes

Replaced text-[#7AC455] with text-documenso-700 in 6 files:

apps/remix/app/components/general/skeletons/document-edit-skeleton.tsx
apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.logs.tsx
apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id._index.tsx
apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id.legacy_editor.tsx
apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.legacy_editor.tsx
apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id._index.tsx

Why `text-documenso-700`?

After testing in both light and dark mode, documenso-700 (#66C622) best preserves the intended visual hierarchy for back navigation links while using the design system.

Color	Hex	Notes
Original (hardcoded)	#7AC455	Deliberately darker than brand default
documenso-700	#66C622	✓ Best match after visual testing
documenso-600	#83DF41	Too bright
documenso (default)	#A2E771	Brand green, too light for nav

Closes #2357

Found with Buoy — design system drift detection

_{🔄 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/documenso/documenso/pull/2358 **Author:** [@dylantarre](https://github.com/dylantarre) **Created:** 1/5/2026 **Status:** ✅ Merged **Merged:** 1/6/2026 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/design-consistency-navigation-links` --- ### 📝 Commits (1) - [`cd1f573`](https://github.com/documenso/documenso/commit/cd1f57366dd6b32ad68e6b36e0ab6434294bff50) fix: replace hardcoded #7AC455 with text-documenso-700 token ### 📊 Changes **6 files changed** (+6 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `apps/remix/app/components/general/skeletons/document-edit-skeleton.tsx` (+1 -1) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id._index.tsx` (+1 -1) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.legacy_editor.tsx` (+1 -1) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.logs.tsx` (+1 -1) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id._index.tsx` (+1 -1) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id.legacy_editor.tsx` (+1 -1) </details> ### 📄 Description ## Summary Standardizes navigation link colors by replacing hardcoded `#7AC455` hex values with the existing `text-documenso-700` design token. ### Changes Replaced `text-[#7AC455]` with `text-documenso-700` in 6 files: - `apps/remix/app/components/general/skeletons/document-edit-skeleton.tsx` - `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.logs.tsx` - `apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id._index.tsx` - `apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.$id.legacy_editor.tsx` - `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id.legacy_editor.tsx` - `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id._index.tsx` ### Why `text-documenso-700`? After testing in both light and dark mode, `documenso-700` (#66C622) best preserves the intended visual hierarchy for back navigation links while using the design system. | Color | Hex | Notes | |-------|-----|-------| | Original (hardcoded) | #7AC455 | Deliberately darker than brand default | | **documenso-700** | #66C622 | ✓ Best match after visual testing | | documenso-600 | #83DF41 | Too bright | | documenso (default) | #A2E771 | Brand green, too light for nav | Closes #2357 --- *Found with [Buoy](https://github.com/buoy-design/buoy) — design system drift detection* --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 20:33:03 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/documenso#2244

No description provided.

Rows
Columns