Move decision responsibility entirely to Tooltip view (#448 #494)

Author: jgonggrijp

frontend/src/label/label-view.ts

@@ -37,11 +37,7 @@ export default class LabelView extends View<FlatItem> {
     }
 
     addTooltip(): void {
-        const cls = this.model.get('class');
-        if (
-            typeof this.toolTipSetting === 'string' &&
-            (cls.has(skos.definition) || cls.has(rdfs.comment))
-        ) {
+        if (typeof this.toolTipSetting === 'string') {
             attachTooltip(this, {
                 direction: this.toolTipSetting,
                 model: this.model,

frontend/src/tooltip/tooltip-view.ts

@@ -47,18 +47,20 @@ export class Tooltip extends View<FlatItem> {
     constructor(options?: ViewOptions) {
         super(options);
         this.preferredDirection = options && options.direction || 'right';
-        this.model.when('class', this.render, this);
+        this.model.when('classLabel', this.render, this);
     }
 
     render(): this {
         const cls = this.model.get('class');
         const languageOption = { '@language': i18next.language };
         const definition = cls.get(skos.definition, languageOption);
         const comment = definition || cls.get(rdfs.comment, languageOption);
-        this.$el.attr(
-            'data-tooltip',
-            definition && definition[0] || comment && comment[0]
-        );
+        const text = definition && definition[0] || comment && comment[0];
+        if (text) {
+            this.$el.attr('data-tooltip', text);
+        } else {
+            this.$el.removeClass('tooltip');
+        }
         return this;
     }