[PR #461] Fix: Secret input field texts were bleeding under obscure toggle button #1963

Open
opened 2026-05-13 22:34:09 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/461
Author: @zPush
Created: 5/10/2026
Status: 🔄 Open

Base: mainHead: main


📝 Commits (1)

  • b282aad Fix: Secret input field texts were bleeding under obscure toggle button

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 apps/yaak-client/components/core/Input.tsx (+1 -1)

📄 Description

Summary

In secret input fields (e.g. Bearer token, API key), the masked text would visually bleed into the obscure/reveal toggle button due to insufficient right-side padding on the editor. Added pr-1.5 to the editor's className in BaseInput to push text content away from the button, matching the padding already applied in EncryptionInput (used by JWT Bearer, where the issue was not present).

Submission

  • This PR is a bug fix or small-scope improvement.
  • If this PR is not a bug fix or small-scope improvement, I linked an approved feedback item below.
  • I have read and followed CONTRIBUTING.md.
  • I tested this change locally.
  • I added or updated tests when reasonable.

Approved feedback item (required if not a bug fix or small-scope improvement):

Screenshot 2026-05-10 at 23 58 35 Screenshot 2026-05-10 at 23 58 30

🔄 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/mountain-loop/yaak/pull/461 **Author:** [@zPush](https://github.com/zPush) **Created:** 5/10/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (1) - [`b282aad`](https://github.com/mountain-loop/yaak/commit/b282aad5ca2a31985c741a92e3ba36cef7e29be8) Fix: Secret input field texts were bleeding under obscure toggle button ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `apps/yaak-client/components/core/Input.tsx` (+1 -1) </details> ### 📄 Description ## Summary <!-- Describe the bug and the fix in 1-3 sentences. --> In secret input fields (e.g. Bearer token, API key), the masked text would visually bleed into the obscure/reveal toggle button due to insufficient right-side padding on the editor. Added pr-1.5 to the editor's className in BaseInput to push text content away from the button, matching the padding already applied in EncryptionInput (used by JWT Bearer, where the issue was not present). ## Submission - [x] This PR is a bug fix or small-scope improvement. - [ ] If this PR is not a bug fix or small-scope improvement, I linked an approved feedback item below. - [x] I have read and followed [`CONTRIBUTING.md`](CONTRIBUTING.md). - [x] I tested this change locally. - [ ] I added or updated tests when reasonable. Approved feedback item (required if not a bug fix or small-scope improvement): <!-- https://yaak.app/feedback/... --> ## Related <!-- Link related issues, discussions, or feedback items. --> <img width="108" height="107" alt="Screenshot 2026-05-10 at 23 58 35" src="https://github.com/user-attachments/assets/02683f19-bfdf-4aec-9e6f-2dcd1e40fb8b" /> <img width="108" height="107" alt="Screenshot 2026-05-10 at 23 58 30" src="https://github.com/user-attachments/assets/7c8c934c-2e52-42fc-8ee4-22c978d2c9f3" /> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-13 22:34:09 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/yaak#1963