Issue #2800Opened May 25, 2020by meyerco0 reactions

[Bug] when removing a tooltip its css is not removed

Question

Description

  • The bug is reproduced on the current demo

Expected behavior

  • When a tooltip is removed its css is removed

What happens instead ?

  • When a tooltip is removed its css is not removed

Steps to reproduce on the demo

  • Clear the canvas
  • Drag a tooltip
  • Remove the tooltip
  • Click on Import button on top toolbar , the tooltip css is not removed .

image

The conclusions of my research

Trying to understand this bug i notified that the style-signature feature is not working properly .

My supposition is that the bug occurs inside the run method of the command core:component-style-clear .

On Components model the core:component-style-clear command run before the component is removed . So when the core:component-style-clear command check if an element of a specific type exists yet on the canvas the response is true . Then this condition is false and the css is not removed .

Answers (3)

artfJune 12, 20200 reactions

Yeah, I think you're right, probably replacing

// relies on the view
const len = dc.getWrapper().find(`[data-gjs-type="${type}"]`).length;

with

// relies on the model
const len = dc.getWrapper().findType('tooltip').length;

might fix the issue, would you like to check and create a PR? :)

meyercoJune 14, 20200 reactions

@artf
Maybe this code to keep the code generic ?

// relies on the model
const len = dc.getWrapper().findType(type).length;
artfJune 18, 20200 reactions

@meyerco 😅 yeah sure, sorry, it was a copy-paste from my try

Related Questions and Answers

Continue research with similar issue discussions.

Paid Plugins That Match This Issue

Curated by issue keywords and label relevance to help you ship faster.

View all plugins

Loading paid plugin recommendations...

Browse Plugin Categories

Jump directly to plugin category pages on the marketplace.