**** BEGIN LOGGING AT Tue Aug 25 02:59:59 2015
Aug 25 06:10:20 <Herrie|Veer> morning
Aug 25 07:09:18 <Herrie|Veer> Tofe I'm debugging the metrics issue. Seems that somehow 20 gets converted to 200 and we end up with 3600 px instead of 360px for the N4 so I added some additional logging to luna-webappmanager to see where it goes wrong. Could be one of the casts is not
Aug 25 07:09:19 <Herrie|Veer> working properly.
Aug 25 07:09:49 <nizovn> morning
Aug 25 07:10:15 <Herrie|Veer> nizovn: morning
Aug 25 07:17:32 <Tofe> Herrie|Veer: on N4, Units.gu(20) = 200, isn't it?
Aug 25 07:17:58 <Herrie|Veer> No 1 unit = 18 on N4, 10 on TP
Aug 25 07:18:14 <Herrie|Veer> I can see it fetches 18 as gridunit value for N4 in the log
Aug 25 07:18:35 <Tofe> ah yes it's on TP
Aug 25 07:19:10 <Herrie|Veer> Just it does a cast in some cases for the height that's passed in. I can see I passed in 20 and metrics:units in my local luna-systemui and it ends up as 3600 in the logs ;)
Aug 25 07:19:27 <Herrie|Veer> So I suspect the cast. Added some logging and running a new build now ;)
Aug 25 07:19:52 <Herrie|Veer> Just to see if one of the casts messes it up somehow :%
Aug 25 07:19:54 <Herrie|Veer> m)
Aug 25 07:19:58 <Herrie|Veer> ;)
Aug 25 07:28:09 <Tofe> well, the cast was introduced so avoid this kind of issue
Aug 25 07:29:03 <Herrie|Veer> Tofe: Yeah :P I added some logging now to see if that goes wrong somehow. But that at least explains why it doesn't work properly :P
Aug 25 07:29:17 <Tofe> yup
Aug 25 07:30:28 <Herrie|Veer> Not sure I can fix it but I could probably at least point out where the problem is, fixing should then be easy for someone who knows c++. But seeing it's probably a 1 or 2 liner I might be able to do that too :P
Aug 25 07:31:05 <Herrie|Veer> I did notice a bug in rotation sometimes when the power menu is visible. I'll try to see if I can find steps to replicate it :)
Aug 25 19:10:40 <Tofe> I think I've fixed the browser crash, yeppeee
Aug 25 19:11:07 <Tofe> Herrie|2: can I send you the qtwayland ipk for some stress-testing?
Aug 25 19:12:52 <Tofe> anyway, want it or not, here it is: https://dl.dropboxusercontent.com/u/4679068/luna-next/qtwayland_5.4.2%2Bgit0%2B8d4ea4eb60-r0.3_armv7a-vfp-neon.ipk
Aug 25 19:13:10 <Tofe> I'm preparing a PR in the meantime
Aug 25 19:30:47 <Tofe> morphis, Herrie|2: https://github.com/webOS-ports/qtwayland/pull/5 corresponds to the fix of multi-window apps crashs. Some little more tests are welcome though, and I've asked giucam to have a look at it too, in case I wrote non-sense :)
Aug 25 20:06:16 <nizovn> Tofe: wow!
Aug 25 20:19:12 <Tofe> nizovn: yes, quite happy right now :)
Aug 25 20:22:40 <Herrie|2> Tofe: Great work!
Aug 25 20:22:46 <Herrie|2> I'll test it in a bit
Aug 25 20:23:05 <Herrie|2> The units stuff is puzzling... It always ends up with 200 and double value no matter what I pass :S
Aug 25 20:30:57 <Tofe> mh
Aug 25 20:32:17 <Herrie|2> Tofe: https://bpaste.net/show/d46282db49da
Aug 25 20:32:23 <Herrie|2> I'm really puzzled where the 200 comes from
Aug 25 20:32:30 <Herrie|2> I searched for it in whole systemui :S
Aug 25 20:32:35 <Herrie|2> It's not in CSS, not in HTML
Aug 25 20:32:47 <Herrie|2> Tried luna-webappmanager, luna-next, luna-next-cardshell
Aug 25 20:32:52 <Herrie|2> I didn't see the 200 anywhere :S
Aug 25 20:32:58 <Herrie|2> It's only when you specify the metrics
Aug 25 20:33:06 <Herrie|2> If you don't do metrics and just enter the PX it's fine :S
Aug 25 20:34:07 <Tofe> let me have a quick look
Aug 25 20:34:16 <Herrie|2> It must be something really silly somewhere :S
Aug 25 20:34:24 <Herrie|2> I'm just testing with hard px again now
Aug 25 20:34:27 <Herrie|2> To see if I'm right
Aug 25 20:34:30 <Herrie|2> But quite sure I am
Aug 25 20:34:46 <Herrie|2> Yeah just hard pix works :S
Aug 25 20:36:29 <Herrie|2> So enyo.windows.openPopup("app/PowerdAlerts/powerdalerts.html", "PowerOffAlert", {}, undefined, 450); works
Aug 25 20:36:57 <Herrie|2> But enyo.windows.openPopup("app/PowerdAlerts/powerdalerts.html", "PowerOffAlert", {}, {"metrics":"units"}, undefined, 25); gives me 200 as value all the time which it then multiplies with 18 so 3600 px
Aug 25 20:37:26 <Herrie|2> Ah wait I see what I might be doing wrong :S
Aug 25 20:38:34 <Tofe> one parameter too many
Aug 25 20:39:14 <Tofe> Javascript 1 - 0 Herrie; next round !
Aug 25 20:39:48 <Herrie|2> WEll that was the code morphis mentioned :P So maybe that's how I got on the wrong side of things :S
Aug 25 20:39:54 <Herrie|2> And my C++ stuff sucks :P
Aug 25 20:40:55 <Tofe> ok, so the typo has always been there, we were just lucky on the N4 ?
Aug 25 20:41:50 <Herrie|2> No N4 still used hard px
Aug 25 20:42:05 <Herrie|2> But that's what causes the black bottom area on TP on power stuff
Aug 25 20:42:07 <Herrie|2> Apr 30 17:56:28 <morphis> try with enyo.windows.openPopup("app/PowerdAlerts/powerdalerts.html", "PowerOffAlert", {"metrics":"units"}, undefined, 30);
Aug 25 20:42:14 <Herrie|2> So that might be the wrong syntax actually?
Aug 25 20:43:30 <Tofe> morphis' syntax looks ok to me -- but it's not 'enyo.windows.openPopup("app/PowerdAlerts/powerdalerts.html", "PowerOffAlert", {}, {"metrics":"units"}, undefined, 25); '
Aug 25 20:44:31 <Tofe> (I'm not sure we are talking about the same thing; I'll sync with your next sentence :) )
Aug 25 20:56:41 <Herrie|2> OK found it... It should be enyo.windows.openPopup("app/PowerdAlerts/powerdalerts.html", "PowerOffAlert", {}, {"metrics":"units"}, 25);
Aug 25 20:57:05 <Herrie|2> So morphis sample was WRONG :P
Aug 25 20:57:38 <Tofe> hehe
Aug 25 20:57:59 <Herrie|2> https://bpaste.net/show/dbc8e65700e8
Aug 25 20:58:09 <Herrie|2> I just assumed the sample code was OK
Aug 25 20:58:20 <Herrie|2> Now moved things around after your comments and found out it wasn't
Aug 25 20:58:27 <Tofe> hopefully fixing the issue will now be a piece of cake
Aug 25 20:58:28 <Herrie|2> Still puzzling where the 200 comes from though :P
Aug 25 20:58:29 <Herrie|2> But well
Aug 25 20:58:32 <Herrie|2> Yeah it is ;)
Aug 25 20:58:56 <Tofe> 200 must be a default javascript value for openPopup ?
Aug 25 20:59:07 <Herrie|2> I couldn't find it anywhere :P
Aug 25 20:59:14 <Herrie|2> And I was searching many places LOL
Aug 25 20:59:36 <Herrie|2> Well this solves things... I always found morphis' sample code odd to start with, but I don't doubt morphis :P
Aug 25 20:59:42 <Herrie|2> So took it as being correct
Aug 25 20:59:53 <Herrie|2> Seems that he probably wrote it on a Monday morning LOL ;)
Aug 25 21:00:17 <Tofe> :)
Aug 25 21:00:37 <Tofe> ok, I'll go to sleep now, reassured and at peace with two bugs :)
Aug 25 21:04:17 <Herrie|2> Yeah :D
Aug 25 21:06:12 <Herrie|2> Good night! Good work on the fix, will test it now and merge when OK and trigger new nightlyy
Aug 25 21:06:22 <Herrie|2> nizovn: Where you able to test already?
Aug 25 21:08:02 <Herrie|2> Tofe: It seems to fix the Open Settings, open Maps, close Maps, settings blank issue.
Aug 25 21:08:12 <Herrie|2> But I still seem to get lnc restart with teh 2 browser cards
Aug 25 21:08:27 <nizovn> Herrie|2: on emulator
Aug 25 21:11:08 <Herrie|2> 2 browser cards still cause LNC reboot here
Aug 25 21:11:11 <Herrie|2> When closing them both
Aug 25 21:11:21 <Herrie|2> doesn't matter if I close 1st or 2nd first
Aug 25 21:11:25 <Herrie|2> When both are closed lnc reboot
Aug 25 21:12:11 <nizovn> Herrie|2: did you try to reboot?
Aug 25 21:12:55 <Herrie|2> Yeah let me do again
Aug 25 21:13:04 <Herrie|2> And reinstall IPK with force as well
Aug 25 21:13:10 <Herrie|2> THough it seems to have gone OL
Aug 25 21:13:11 <Herrie|2> OK
Aug 25 21:23:54 <Herrie|2> Ah now it seems OK
Aug 25 21:24:09 <Herrie|2> THe Settings->Maps->Settings still seems to be white screen though
Aug 25 21:24:13 <Herrie|2> But browser one fixed
Aug 25 21:42:45 <Herrie|2> Tofe: I'm OK to merge it as is since it improves the situation. In case giucam has improvements we can take it from there. Seems I don't have access to qtwayland branch though to merge. But you can if you want and I'll trigger a new nightly
**** ENDING LOGGING AT Wed Aug 26 02:59:59 2015