Github user mcgilman commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/1310#discussion_r92043862
  
    --- Diff: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-policy-management.js
 ---
    @@ -759,18 +759,23 @@ nf.PolicyManagement = (function () {
                 });
     
                 // build the mark up
    -            return $('<span>Showing effective policy inherited from 
Process Group </span>').append($('<span 
class="link"></span>').text(processGroupName).on('click', function () {
    -                // close the shell
    -                $('#shell-close-button').click();
    -
    -                // load the correct group and unselect everything if 
necessary
    -                nf.CanvasUtils.enterGroup(processGroupId).done(function () 
{
    -                    nf.CanvasUtils.getSelection().classed('selected', 
false);
    -
    -                    // inform Angular app that values have changed
    -                    nf.ng.Bridge.digest();
    -                });
    -            })).append('<span>.</span>');
    +            return $('<span style="position: relative;top: -2px;">Showing 
effective policy inherited from Process Group </span>').append(
    +                $('<span class="link ellipsis" style="max-width: 200px; 
position: relative; top: 3px;"></span>')
    --- End diff --
    
    I'm pretty sure that the div's (`policy-message, new-policy-message, 
override-policy-message, and add-local-admin-message`) already align. The only 
issue was the content of `policy-message`. Specifically, the styles in the 
ellipsis class causes the surrounding text to shift downwards. Setting the link 
to `vertical-align: middle` should address this.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to