[ 
https://issues.apache.org/jira/browse/NIFI-2629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15450942#comment-15450942
 ] 

ASF GitHub Bot commented on NIFI-2629:
--------------------------------------

Github user patricker commented on the issue:

    https://github.com/apache/nifi/pull/969
  
    I would love to just show/hide it, but I can see where people are coming 
from.
    What if for v1.1 an outright show/hide was replaced with an alternative 
color, maybe make the button look like it's disabled while leaving it enabled 
by applying a CSS class.  This would at least give a visual indication to users 
without breaking anything for custom processors.
    
    Try to let users know what's going to happen, get other processors updated, 
then in v1.x you can start actually showing/hiding?


> Hide add property "+" in Processors, Controller Services and Reporting Tasks 
> that don't allow dynamic properties
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: NIFI-2629
>                 URL: https://issues.apache.org/jira/browse/NIFI-2629
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Core Framework, Core UI
>    Affects Versions: 0.7.0
>            Reporter: Kirk Tarou
>            Priority: Minor
>
> A few Processors, Controller Services and Reporting Tasks expect or allow 
> extra properties to be added on the 'Properties' tab in the Detail view. The 
> rest will show a "not a supported property" error.
> To avoid confusion, the new property "+" icon should be hidden if dynamic 
> properties are not allowed.
> The existence of a "DynamicProperty" annotation could be used as an indicator.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to