[PR #809] [CLOSED] Added Password with Reveal Eye option #2332

Closed
opened 2026-01-29 18:18:19 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/radzenhq/radzen-blazor/pull/809
Author: @codepend
Created: 2/12/2023
Status: Closed

Base: masterHead: master


📝 Commits (5)

  • 0bf0092 Added Password with Reveal Eye option
  • 473b896 Updated Pasword component to handle style changes
  • 7d05746 Changed test name to match parameter name change
  • 7ae6832 added test for allowing reveal when disabled
  • 6f41620 rebase

📊 Changes

4 files changed (+122 additions, -3 deletions)

View changed files

📝 Radzen.Blazor.Tests/PasswordTests.cs (+35 -1)
📝 Radzen.Blazor/RadzenPassword.razor (+25 -2)
📝 Radzen.Blazor/RadzenPassword.razor.cs (+44 -0)
📝 RadzenBlazorDemos/Pages/PasswordConfig.razor (+18 -0)

📄 Description

Hi,

I've added a Reveal Password option to the RadzenPassword class.

A couple things to note:
I could not figure out the CSS structure so I re-used some classes from rz-calendar. I would be happy to fix/update this if I knew how.
I tried to follow Radzen's naming conventions.
I added a test as well as updated the components demo page.

Also, if you don't like this or the way it was done, I will not take it personally. Just trying to contribute.
Thanks for any consideration!


🔄 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/radzenhq/radzen-blazor/pull/809 **Author:** [@codepend](https://github.com/codepend) **Created:** 2/12/2023 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (5) - [`0bf0092`](https://github.com/radzenhq/radzen-blazor/commit/0bf0092290072c03f386e1c66ed574aec5b17f5f) Added Password with Reveal Eye option - [`473b896`](https://github.com/radzenhq/radzen-blazor/commit/473b896b690265682291374e668836b6a2428c39) Updated Pasword component to handle style changes - [`7d05746`](https://github.com/radzenhq/radzen-blazor/commit/7d05746577ed7febcb881ae29343133c405a0079) Changed test name to match parameter name change - [`7ae6832`](https://github.com/radzenhq/radzen-blazor/commit/7ae683237fdabc9fe55126867ec63363d30f6bdb) added test for allowing reveal when disabled - [`6f41620`](https://github.com/radzenhq/radzen-blazor/commit/6f41620a2bc7003ee22cbb9d94268cdd0a883e8b) rebase ### 📊 Changes **4 files changed** (+122 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `Radzen.Blazor.Tests/PasswordTests.cs` (+35 -1) 📝 `Radzen.Blazor/RadzenPassword.razor` (+25 -2) 📝 `Radzen.Blazor/RadzenPassword.razor.cs` (+44 -0) 📝 `RadzenBlazorDemos/Pages/PasswordConfig.razor` (+18 -0) </details> ### 📄 Description Hi, I've added a Reveal Password option to the RadzenPassword class. A couple things to note: I could not figure out the CSS structure so I re-used some classes from rz-calendar. I would be happy to fix/update this if I knew how. I tried to follow Radzen's naming conventions. I added a test as well as updated the components demo page. Also, if you don't like this or the way it was done, I will not take it personally. Just trying to contribute. Thanks for any consideration! --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 18:18:19 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/radzen-blazor#2332