-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Text not displaying properly #2
Comments
Thanks. Probably just a CSS issue. I'll look into it. |
Hi @flatsiedatsie, After more checking I can see that this is only happening with the native WebThings connected via the things-url-adapter. All the WebThings are written with the Arduino libraries but of various versions. Maybe there's a problem with the adapter, the library or my code? Cheers 🙂 |
If it only happens with one type of device, then that's strange. Other things do have a normally populated list? It could be that the thing or property ID's of those devices are leading to a problem. They may have stange characters in them, for example. Could you give an example of what the URL bar looks like when you look at one of these devices? And what are the properties supposed to be? |
Hi @flatsiedatsie, This is an example of one: /things/http---192.168.1.43-things-bme280wbattery Looking at that, the problem is probably the full stops "." or the dashes "-" as your extension might not be expecting to see code like that. Cheers 🙂 |
Yes that might be it. |
I did a test by creating a device with dots in the thing ID, property ID and titles. And it worked just as normal. |
Hi @flatsiedatsie, |
Perhaps the addon still sends 'name' instead of 'title'. |
Try clearing your browser's cache too. |
Older versions of the library used |
A new version has been released that, in theory, should also check for 'label' if it can't find a 'title'. |
Hi @flatsiedatsie, The new version brings improvement! There is now text for these native WebThings' properties but it all just says "unknown". My latest WebThings are from about two months ago which i think is after the change from title to label isn't it @mrstegeman? My code is here: https://github.com/madb1lly/ESP8266-BME280-WebThing/blob/master/bme280wbattery.ino Cheers 🙂 |
Hi @flatsiedatsie, For info, the Macros extension doesn't show the thing properties as Unknown, so maybe you'll find the answer in that code. Cheers 🙂 |
Macros uses javascript instead of python |
Hello @flatsiedatsie, okay, but you can ignore me anyway, I got mixed up - Macros also shows "unknown" for these properties on these native webthings. |
If two addons do this, then it's a pattern. Perhaps @mrstegeman should be informed. |
You should log what you're passing into each |
Hi @flatsiedatsie,
On Fenix and Fennec Firefox on Android the text isn't displaying properly for me:
Nothing obvious in the internal logs and I've got debug logs on.
Cheers 🙂
The text was updated successfully, but these errors were encountered: