johnbhurst / concordion-input-style-extension Goto Github PK

Hi,

I like the extension to see which parts are used as input parameters!

i would like to propose a little enhancement, where also text parts which are used in the execute command with the special variable #TEXT are considered as input.

<span concordion:execute="myMethod(#TEXT)">this sould be considered as input</span>

When trying to hack around I came up with an ExecuteListener like:

public void executeCompleted(ExecuteEvent e) {
    if(e.getElement().getAttributeValue("execute", "http://www.concordion.org/2007/concordion").contains("#TEXT")) {
            e.getElement().addStyleClass("inputvalue");
    }
}

But the namespace handling would definitely need enhancements - I'm not sure if concordion offers a nice way to deal with it. Maybe there is also a way for a generally cleaner way to do this. But what if have seen the #TEXT variable is always set to the element value regardless if it's used or not.

What do you think of this Idea?

Kind regards,
Carlo