Page 16 of 28
Re: Dashticz - Bug report
Posted: Thursday 23 November 2017 12:22
by aiolos
I think some default styling for images instead of icons is missing then. There have been changes in those parts and I don't have images as icons, so I didn't really notice
Re: Dashticz - Bug report
Posted: Thursday 23 November 2017 12:23
by blacksn0w
EdwinK wrote: ↑Thursday 23 November 2017 12:16
Screen Shot 2017-11-23 at 12.15.02.pngNow I've got this little problem. Not to serious, just need to resize them myself
Got the same, since im using the .svg resizing is not easy possible.
Re: Dashticz - Bug report
Posted: Thursday 23 November 2017 17:25
by aiolos
EdwinK wrote: ↑Thursday 23 November 2017 12:16
Screen Shot 2017-11-23 at 12.15.02.png
Now I've got this little problem. Not to serious, just need to resize them myself
I created the fix for this, I expect it to be merged soon.
Re: Dashticz - Bug report
Posted: Thursday 23 November 2017 17:26
by robgeerts
Merged
Re: Dashticz - Bug report
Posted: Thursday 23 November 2017 22:21
by EdwinK
Thanks Rob and aiolos
you rock
Re: Dashticz - Bug report
Posted: Friday 01 December 2017 12:07
by htilburgs
In the latest beta, Dashticz won't load correct / complete in IE11.
Missing a lot of switches.
Error (in dutch):
SCRIPT5008: Ongeldige linkerkant van toewijzing (incorrect assigning on the left side)
main.js (1050,33)
line 1049 - 1051:
Code: Select all
catch (err) {
[html, addHTML] = handleDevice(device, idx);
}
For as far as I can find:
this is a Illegal assignment (JavaScript)
You attempted to assign a value to a read-only identifier. You cannot assign a value to a read-only identifier. For example, host defined objects and external COM objects. To correct this error --> Do not assign values to read-only identifiers.
Re: Dashticz - Bug report
Posted: Friday 01 December 2017 16:16
by htilburgs
htilburgs wrote: ↑Friday 01 December 2017 12:07
In the latest beta, Dashticz won't load correct / complete in IE11.
Missing a lot of switches.
Error (in dutch):
SCRIPT5008: Ongeldige linkerkant van toewijzing (incorrect assigning on the left side)
main.js (1050,33)
line 1049 - 1051:
Code: Select all
catch (err) {
[html, addHTML] = handleDevice(device, idx);
}
For as far as I can find:
this is a Illegal assignment (JavaScript)
You attempted to assign a value to a read-only identifier. You cannot assign a value to a read-only identifier. For example, host defined objects and external COM objects. To correct this error --> Do not assign values to read-only identifiers.
Tried it also on Microsoft Edge, same problem.
In my case my dashboard is on a Microsoft Surface RT (windows 8.1), that only supports Edge and IE11
Re: Dashticz - Bug report
Posted: Friday 01 December 2017 16:28
by aiolos
htilburgs wrote: ↑Friday 01 December 2017 16:16
htilburgs wrote: ↑Friday 01 December 2017 12:07
In the latest beta, Dashticz won't load correct / complete in IE11.
Missing a lot of switches.
Error (in dutch):
SCRIPT5008: Ongeldige linkerkant van toewijzing (incorrect assigning on the left side)
main.js (1050,33)
line 1049 - 1051:
Code: Select all
catch (err) {
[html, addHTML] = handleDevice(device, idx);
}
For as far as I can find:
this is a Illegal assignment (JavaScript)
You attempted to assign a value to a read-only identifier. You cannot assign a value to a read-only identifier. For example, host defined objects and external COM objects. To correct this error --> Do not assign values to read-only identifiers.
Tried it also on Microsoft Edge, same problem.
In my case my dashboard is on a Microsoft Surface RT (windows 8.1), that only supports Edge and IE11
On IE11 I can reproduce it, on Edge it works (but that's windows 10, so that might be different.) What version of Edge do you use? (I testen on Edge 38)
It is easy to fix, but problems like this are more likely to happen if we want to use the functions of the newer Javascript versions. IE11 is a bit getting the new IE6...
Re: Dashticz - Bug report
Posted: Friday 01 December 2017 16:41
by htilburgs
I can't find the version of Edge on the Windows Surface RT. Even don't know if its the real Edge or just IE11 in another look.
For IE:
11.0.9600.18838
Update: 11.0.48 (KB4047206)
It would be disappointing if we could'nt use IE11 as browser for Dashticz.
Re: Dashticz - Bug report
Posted: Friday 01 December 2017 16:56
by aiolos
You can find the version on
https://www.whatismybrowser.com/
And yes, it would be disappointing if it doesn't work in IE anymore, but on the other hand: it is an old browser and edge is now the one to use (if you are required to use a Microsoft browser). Nevertheless this needs to be fixed, but there might be a moment where it won't be, since the new javascript functions can make the code much more readable and faster. (it's a pity Microsoft still supports IE, it would be better if they drop it completely and fully focus on Edge).
Re: Dashticz - Bug report
Posted: Saturday 02 December 2017 11:39
by htilburgs
But for now, can you fix it? So I can update my dashboard with te latest settings.
Re: Dashticz - Bug report
Posted: Saturday 02 December 2017 19:11
by aiolos
Yes, a PR is waiting to be merged. But I'm also still wondering what Edge version also causes this problem.
Re: Dashticz - Bug report
Posted: Saturday 02 December 2017 21:50
by robgeerts
Merged, check latest beta!
Re: Dashticz - Bug report
Posted: Sunday 03 December 2017 15:48
by MariH
Latest Beta is not working correctly with groups/scene's
Activating this code in CONFIG.js:
Code: Select all
blocks['s1'] = {}
blocks['s1']['width'] = 4;
blocks['s1']['title'] = 'Zitkamer';
blocks['s1']['hide_data'] = true;
will not send an 'ON' or 'OFF' command to 'group 1' but to 'switch 1'
this it out the domoticz logging:
Code: Select all
2017-12-03 15:39:24.286 User: Admin initiated a switch command (1/Weer: Luchtdruk/Off)
status is reflected OK
this use to work in earlier versions
Re: Dashticz - Bug report
Posted: Sunday 03 December 2017 20:08
by aiolos
MariH wrote: ↑Sunday 03 December 2017 15:48
Latest Beta is not working correctly with groups/scene's
...
this use to work in earlier versions
Good find! The big cleanup a few weeks ago caused this. I pushed a fix for it, which Rob will merge very soon I think.
Re: Dashticz - Bug report
Posted: Sunday 03 December 2017 20:11
by robgeerts
Merged!
Check out latest beta!
Re: Dashticz - Bug report
Posted: Sunday 03 December 2017 20:12
by aiolos
robgeerts wrote: ↑Sunday 03 December 2017 20:11
Merged!
Check out latest beta!
You're just waiting for my pull requests all day, aren't you?
Re: Dashticz - Bug report
Posted: Sunday 03 December 2017 20:40
by MariH
Guys,
thx for the quick fix, works as a charm!
Re: Dashticz - Bug report
Posted: Sunday 03 December 2017 21:47
by robgeerts
aiolos wrote: ↑Sunday 03 December 2017 20:12
robgeerts wrote: ↑Sunday 03 December 2017 20:11
Merged!
Check out latest beta!
You're just waiting for my pull requests all day, aren't you?
Yes, I've nothing better to do
No, I got notifications about new mail on my devices, as my laptop is almost always on I merge your request immediately
Re: Dashticz - Bug report
Posted: Monday 04 December 2017 9:37
by htilburgs
aiolos wrote: ↑Saturday 02 December 2017 19:11
Yes, a PR is waiting to be merged. But I'm also still wondering what Edge version also causes this problem.
Aiolos, first thanx for the fix. It works like a charm again.
I'm using a Microsoft Surface RT Windows 8.1. I'm not sure it's EDGE that is installed. Maybe it's just an other way to show IE11 on the Windows 8.1 way.
I've been searching, but cannot find any version information.