domoticz / domoticz-android

Domoticz Client Application for Android Devices
106 stars 71 forks source link

Text disapears behind button #107

Closed RoadXY closed 8 years ago

RoadXY commented 8 years ago

http://www.domoticz.com/forum/viewtopic.php?f=37&t=8884&start=560#p70871

by RickDB » Tue Jan 19, 2016 3:31 pm

1) The green "SET" icon is misaligned in potrait mode as it overlaps the text label (name) 2) an empty white space below (like it expects more info in a new row)

432392be-bf6a-11e5-9b93-e621e05ae5f6

Jaap-Jan commented 8 years ago

Per-zone temperature control is actually more of an enhancement. I think it's better to separate the two issues. I've got permanent override more or less working, I'll see if I can push it soon. Temporary override (where you can set a temperature and a date and time, after which the normal program is followed) is more of a problem, as we don't have a way to select both a date/time AND a temperature.

RoadXY commented 8 years ago

Extracted the feature request and made an new issue #112

Jaap-Jan commented 8 years ago

First problem (evohome device too large) is fixed. Second problem is, I think, essentially the same as #110.

galadril commented 8 years ago

So the text issue is fixed? then we can close this issue :+1:

Jaap-Jan commented 8 years ago

The text issue (text appearing behind the button) still exists. Isn't that also a problem on other layouts?

RoadXY commented 8 years ago

@Jaap-jan what do you mean with "evohome device too large is fixed"? You don't mend text then?

Jaap-Jan commented 8 years ago

No, I meant the problem that there was too much whitespace. The issue the user explained as "adds a an empty white space below (like it expects more info in a new row)". I see that I referred to the wrong one by using 'first' and 'second', sorry for the confusion.

device-2016-01-21-005616 device-2016-01-21-005616

galadril commented 8 years ago

I think i've fixed the text overlay issues.. please validate

Jaap-Jan commented 8 years ago

Confirmed. But the height issue is back. Made a pull request.

RoadXY commented 8 years ago

Updated the issue in post one to clarify