Clean up

Author: IsabelParedes

ipynao/nao_robot.py

@@ -9,7 +9,7 @@
 """
 
 from ipywidgets import DOMWidget
-from traitlets import Unicode, Bool, Integer
+from traitlets import Unicode, Integer
 from ._frontend import module_name, module_version
 from time import sleep
 import asyncio
@@ -62,9 +62,8 @@ class NaoRobotWidget(DOMWidget):
     value = Unicode('Hello World').tag(sync=True)
     connected = Unicode("Disconnected").tag(sync=True)
     status = Unicode("Not busy").tag(sync=True)
-    synco = Unicode("test message").tag(sync=True)
     response = Unicode("").tag(sync=True)
-    counter = Integer(0).tag(sync=True)
+    counter = Integer(0).tag(sync=True, read_only=True)
     js_response = None
 
     def __init__(self, **kwargs):

src/widget.ts

@@ -25,7 +25,6 @@ export class NaoRobotModel extends DOMWidgetModel {
   connected = 'Disconnected';
   status = 'Not busy';
   _services: serviceDict = {};
-  synco: string;
 
   defaults() {
     return {
@@ -37,7 +36,6 @@ export class NaoRobotModel extends DOMWidgetModel {
       _view_module: NaoRobotModel.view_module,
       _view_module_version: NaoRobotModel.view_module_version,
       value: 'Hello World',
-      synco: 'something silly',
       connected: 'Disconnected',
       status: 'Not busy',
       response: '',
@@ -241,7 +239,6 @@ export class NaoRobotModel extends DOMWidgetModel {
 }
 
 export class NaoRobotView extends DOMWidgetView {
-  synco: HTMLDivElement;
   txt_connected: HTMLDivElement;
   txt_status: HTMLDivElement;
 
@@ -258,23 +255,13 @@ export class NaoRobotView extends DOMWidgetView {
     this.txt_status.textContent = 'Not busy';
     this.el.appendChild(this.txt_status);
 
-    // Testing element
-    this.synco = document.createElement('div');
-    this.synco.textContent = 'it should be here';
-    this.el.appendChild(this.synco);
-
     this.value_changed();
     this.model.on('change:connected', this.value_changed, this);
     this.model.on('change:status', this.value_changed, this);
-    this.model.on('change:synco', this.value_changed, this);
   }
 
   value_changed() {
-    // this.el.textContent = this.model.get('value');
-    // this.synco = this.model.get('synco');
-    console.log('THE VALUE CHANGED');
     this.txt_connected.textContent = this.model.get('connected');
     this.txt_status.textContent = this.model.get('status');
-    this.synco.textContent = this.model.get('synco');
   }
 }