Skip to content

Conversation

@Sobyt483
Copy link
Collaborator

@Sobyt483 Sobyt483 commented Nov 7, 2025

image image image

@Sobyt483 Sobyt483 requested review from a team as code owners November 7, 2025 14:22
@Sobyt483 Sobyt483 requested a review from gkrajniak November 7, 2025 14:22
@Sobyt483 Sobyt483 self-assigned this Nov 7, 2025
@Sobyt483 Sobyt483 added the enhancement New feature or request label Nov 7, 2025
@Sobyt483 Sobyt483 linked an issue Nov 7, 2025 that may be closed by this pull request
})
export class SecretValueComponent {
value = input.required<string>();
maskedValue = computed(() => '*'.repeat(this.value().length || 8));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or we can always have 8

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought about this. But then if the value would be toolong when you hover to secret field it will show actual value and cell would shift

{{ value() }}
} @else if (displayAsSecret()) {
<wc-secret-value [value]="value()"></wc-secret-value>
} @else {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here we can have logic all the way down wiht the if else withouth nested if

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True. Will fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Ability to show and copy hidden properties

3 participants