Refactor: move actions/tap_action from card section to tool section about swiss-army-knife-card HOT 1 CLOSED

Implementation notes

Tool

The current tool implementation looks like this. The event handler uses the card entities configuration to process and forward the click event

    return svg`
      <g id="line-${this.toolId}" class="${classMap(this.classes.tool)}" style="${styleMap(this.styles.tool)}"
        @click=${e => this._card.handleEvent(e, this._card.config.entities[this.config.entity_index])}>
        ${this._renderLine()}
      </g>
    `;

Currently, there is ALWAYS a click handler, as this one is added by default to process the "more-info" popup. This won't be the case anymore with the new tool-based click handlers.

So a click handler should be registered only if

• a this.config.user_actions.tap_action is defined
• the actions.action is either more-info or call-service AND an entity_index is defined
• the actions.action is navigate

The slider-tool is a special one: that one should register its own handlers for click and drag!

EventHandler (Card)

The current eventhandler (card::handleEvent and card::_handleClick are both part of the card class, and should be moved to the baseTool class.

from swiss-army-knife-card.