Author Topic: Version 1 review feedback...  (Read 3731 times)

Offline guessed

  • Master Member
  • *******
  • Posts: 5300
  • Karma: +92/-22
  • Release compat is not a bolted-on afterthought
Version 1 review feedback...
« on: October 06, 2011, 11:21:51 am »
Overall this is a great Android app, very impressed with what you've done here.

A few comments based upon a liitle usage on my Nook:
a) update the version in the About box for each release.
b) each device Jumps due to the "Transmit was ok" positioning in the layout, make it more like the transient "Command sent" messages if possible to avoid the relayout issues. Consider inlining this into the right side of the device title.
c) add more Categories as "Other" covers the Weather plugin and Power measurement devices.  I'd like not to filter those out (in my quest to eliminate the "?" devices)
d) TStat adjustments are a little clunky.
e) my TStat fan mode always shows "-"
f) the green slider thumb, on a turned off Dimmer, is sometimes grey, sometimes green.
g) love the 'eye' icon on the HSM and it's light sensor icon
h) the thermometer icons are flat compared to the other 3D icons
i) the dimmer bulb icons stick out in the all devices list, due to the bars, not sure if it's needed since there's already a slider indicating it's dim able
j) made the bulb icons clickable, like the on/off action, and have them execute the Ha1 Toggle action
k) doing a Device Info on some devices can cause the app to Crash

Offline garrettwp

  • Moderator
  • Master Member
  • *****
  • Posts: 6371
  • Karma: +227/-128
  • Vera 3, Lite, ISY994
Re: Version 1 review feedback...
« Reply #1 on: October 06, 2011, 03:30:50 pm »
Thanks for the comments / suggestions. Here are some follow up answers:

a) I keep meaning to correct this. An oversite that I need to push myself to do.

b) I will correct this. I originally had it at the upper right in the name of the device area. But with the limited space on the phones, I had to put it where it is now. The Honeycomb version has this though.

c) I will add additional categories. As for the question mark, I have that in place for devices or plugins that I have not put support in yet.

d) With the limited space, It is hard to try and get the control into a small area. Do you have any suggestions? Mind you I can only have two layouts landscape and portrait and will need to work with phones. I wish I can utilize the extra space on the nook, but with Android 2.3 and older this is not possible.

e) The reason for the "-" is due to the lack of support of the fan mode in the lu_sdata. I put a bug in mantis back in august and contacted MCV the other day to make sure this is fixed in UI5. I got word back from Mcvflorin that this has been resolved and should show up in UI5.

f) This is a glitch with using the honeycomb sliders and the compatibility package. I will see if I can try and fix this.

g) Thanks, I thought Hal would be suitable for the hsm :)

h) I need to have those replaced. Trying to find good suitable free icons is hard and even harder for thermometer. But it is on my list to change out.

i) I can remove the bars and just have the light icon. To me it made it easier to know at quick glance that it was a dimmer.

j) This can be easily done. What do you mean by HA1 action?

k) Can you tell me what devices cause the crash? This is something I added real quick last night and has not gotten much testing.

Thanks again for the info. I will try and  get this in asap. Keep the feedback coming.

- Garrett

Offline garrettwp

  • Moderator
  • Master Member
  • *****
  • Posts: 6371
  • Karma: +227/-128
  • Vera 3, Lite, ISY994
Re: Version 1 review feedback...
« Reply #2 on: October 06, 2011, 03:48:29 pm »
One more thing, can you provide me the data for the devices that it does cause the crash when doing a device info? You can use this link to retrieve the data for that device and either pm me the info or email me it.

http://ip:3480/data_request?id=status&output_format=json&DeviceNum=deviceid

- Garrett

Offline guessed

  • Master Member
  • *******
  • Posts: 5300
  • Karma: +92/-22
  • Release compat is not a bolted-on afterthought
Re: Version 1 review feedback...
« Reply #3 on: October 13, 2011, 10:06:50 pm »
Thanks for the comments / suggestions. Here are some follow up answers:

j) This can be easily done. What do you mean by HA1 action?

When you click on the Device icon on the HTML Dashboard, it runs the following action behind the scenes:

urn:micasaverde-com:serviceId:HaDevice1
ToggleState


If there's no implementation, there's some sort of error returned, and the HTML UI simply ignores it.  I imagine something similar against each of your Device icons, and a corresponding call to the underlying "device" to do whatever it's ToggleState action does.

Quote
k) Can you tell me what devices cause the crash? This is something I added real quick last night and has not gotten much testing.

I've attached the JSON response that you were looking for.  This also crashes it under r1.1.  It's not the only device that does it, but it should provide enough data to sort it out.


I'd like to add item:

l) Requesting Autoconfigure returns "Invalid JSON Response" (or similar)
So I've manually configured my IP Address.  If you let me know what URL you're invoking for this, I can provide a PM version of the JSON response it's returning.  I have 2x Vera's registered so it's possible that's causing it grief.



Offline S-F

  • Hero Member
  • *****
  • Posts: 1248
  • Karma: +62/-12
  • Clueless N00b
Re: Version 1 review feedback...
« Reply #4 on: January 23, 2012, 11:53:44 pm »
It works fantastically. Thanks. Just what I was looking for.

One thing though. I selected the option for making devices instead of scenes the default and I'm not sure that it stuck.
My forum account was apparently hacked by a leprechaun. And he's drunk all of the time. If a post of mine seems a little off kilter, it's probably because he made it.

Offline garrettwp

  • Moderator
  • Master Member
  • *****
  • Posts: 6371
  • Karma: +227/-128
  • Vera 3, Lite, ISY994
Re: Version 1 review feedback...
« Reply #5 on: January 24, 2012, 04:45:53 am »
It works fantastically. Thanks. Just what I was looking for.

One thing though. I selected the option for making devices instead of scenes the default and I'm not sure that it stuck.

Is this running on a phone or tablet? The default tab option is only used for when running on a tablet or when force tablet layout for devices like the Nook Color/Tablet and Kindle Fire.

- Garrett

Offline S-F

  • Hero Member
  • *****
  • Posts: 1248
  • Karma: +62/-12
  • Clueless N00b
Re: Version 1 review feedback...
« Reply #6 on: January 24, 2012, 06:50:01 am »
It's a tablet. Asus Transformer Prime to be exact.
My forum account was apparently hacked by a leprechaun. And he's drunk all of the time. If a post of mine seems a little off kilter, it's probably because he made it.

Offline garrettwp

  • Moderator
  • Master Member
  • *****
  • Posts: 6371
  • Karma: +227/-128
  • Vera 3, Lite, ISY994
Re: Version 1 review feedback...
« Reply #7 on: January 24, 2012, 07:11:42 am »
Is this running the Ice Cream update? I'll take a look and see what is going on.

- Garrett

Offline S-F

  • Hero Member
  • *****
  • Posts: 1248
  • Karma: +62/-12
  • Clueless N00b
Re: Version 1 review feedback...
« Reply #8 on: January 24, 2012, 09:36:36 am »
It is running ICS. But remember I said "I'm not sure". I do need to test it more. I just got the thing yesterday and this is my first Android device so it's taking me some time to get used to.
Aside from that though I'm really liking this UI. I find it to be more stable than UI 4.
« Last Edit: January 24, 2012, 09:38:37 am by S-F »
My forum account was apparently hacked by a leprechaun. And he's drunk all of the time. If a post of mine seems a little off kilter, it's probably because he made it.