You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by "mcgilman (via GitHub)" <gi...@apache.org> on 2025/09/15 14:26:43 UTC
[PR] NIFI-14965: Adding the property value tooltip that helps see embedded… [nifi]
mcgilman opened a new pull request, #10304:
URL: https://github.com/apache/nifi/pull/10304
… parameter values.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] NIFI-14965: Adding the property value tooltip that helps see embedded… [nifi]
Posted by "rfellows (via GitHub)" <gi...@apache.org>.
rfellows merged PR #10304:
URL: https://github.com/apache/nifi/pull/10304
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] NIFI-14965: Adding the property value tooltip that helps see embedded… [nifi]
Posted by "rfellows (via GitHub)" <gi...@apache.org>.
rfellows commented on code in PR #10304:
URL: https://github.com/apache/nifi/pull/10304#discussion_r2349454456
##########
nifi-frontend/src/main/frontend/libs/shared/src/components/codemirror/autocomplete/parameter-tip/parameter-tip.component.html:
##########
@@ -19,6 +19,15 @@
@if (data?.parameter; as parameter) {
<div class="flex flex-col gap-y-3">
<div class="parameter-name text-lg font-bold">{{ parameter.name }}</div>
+ @if (!parameter.sensitive) {
+ @if (parameter.value === null) {
+ <div class="unset neutral-color">No value set</div>
+ } @else if (parameter.value === '') {
+ <div class="unset neutral-color">Empty string set</div>
+ } @else {
+ <div class="line-clamp-[20] leading-4 whitespace-pre font-mono text-sm">{{ parameter.value }}</div>
Review Comment:
should add `truncate` class here to be consistent.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org