19:01 #startmeeting 19:01 Meeting started Thu Feb 20 19:01:50 2014 UTC. The chair is dpm. Information about MeetBot at http://wiki.ubuntu.com/meetingology. 19:01 19:01 Available commands: action commands idea info link nick 19:02 So the main thing this week is convergence! 19:02 indeed! :) 19:02 I've got a couple of updates since the conversation this morning 19:03 the first one is that we changed the number of pixels per grid unit on the N7 19:03 so now, instead of 21, the Nexus 7 has 18 px/gu 19:03 that makes things look a lot better 19:04 have a look at weather running on a Nexus 7 with this setting now: 19:04 does it fix the issues? 19:04 http://imgur.com/QRohlUz,NwazDKg#0 19:04 it kind of does 19:04 but the footer is still broken 19:04 ah 19:04 any idea why? 19:04 I'm really happy about the second image on that link, where we don't have an overlap of the 4 boxes anymore 19:04 not yet 19:05 I've only had time to do a small code change to adapt to the new GU (changing the wideModeThreshold) 19:05 so I've not spent that much time on it 19:06 dpm: when has it to be ready? 19:06 but some good news is that I could kind of reproduce the issue on the desktop 19:06 m-b-o, so here's the plan: 19:06 - We are uploading the twc-version-with-only-phone-layout to the store. This way we are on the safe side 19:07 - We will do a best effort try to see if we can fix the footer issue between today and tomorrow 19:08 - If we can fix it, we'll upload it and we'll _try_ to get it in the final images. As it will be a bit late, it's not sure if it'll be included 19:08 okay 19:09 so that's the plan. I'll try to debug a bit between tonight and tomorrow 19:09 but I don't have a device, so I might be a bit slow 19:09 how have you reproduced it on desktop? 19:10 changing the height and the width of the window, there is a point where the footer gets stuck at a size smaller than the main window 19:11 any ideo at which gu? 19:13 I'm trying to reproduce now 19:15 592px width on the desktop 19:16 which at 8px per GU on my laptop, that's 74 GU 19:17 hwo can I calculate px per GU? 19:17 m-b-o, I just did a print (units.gu(1)) 19:18 m-b-o, http://i.imgur.com/8lrlSDZ.png 19:18 you can see the pixels output on the settings button there, and the footer being at a strange position and stuck on a fixed length 19:19 ah, saw that once today 19:19 if I make the window wider, the footer stays that way 19:19 dpm: if you need testing done tonight on your hacking, ping me 19:19 awesome, thanks popey 19:21 m-b-o, but I really don't understand why it's happening, because the width should be the same as the main view. I'm wondering if anchors come into play somehow... 19:22 can't say. 19:24 does thi shappen in the phone onyl version too? 19:25 I see a footer not at 100% on desktop! 19:25 damn, I know hwo to reprduce, perhaps 19:25 I can try on an older revision 19:26 I have it on desktop 19:28 ok, it's difficult to reproduce on an older, phone-only revision 19:28 http://imgur.com/N1ui046 19:28 look 19:28 aha! 19:28 ooh 19:29 stucks at ~460 19:29 460/8 are 19:29 57.5 19:29 lets try 19:30 hmm 19:34 i can reproduce 19:35 it's not dependent on a fixed size 19:38 you start it with width 91 gu 19:38 then you are going to reduce the width in small steps, until it switches to phone view 19:39 then sometimes the twc logo get cut, like this http://imgur.com/fS5AGGG 19:39 yep 19:39 so it seems to do with the resize, which doesn't happen on a tablet, but the effect is the same, which makes me think it's the same issue 19:39 when this happens, there's a chance the width gets stucked 19:40 yeah 19:40 pehrpas becasue the size gets decreased, perhaps we shoudl give a smaller width at startup 19:41 but we already have quite a small size on startup 19:42 i can now fully reproduce 19:42 start the app: make the app narrow in a fast move 19:42 than make it wider 19:42 again 19:43 make it narrow to phone size fast, look at it. then make it wide 19:44 yes, fast resizing is it 19:46 yes 19:47 so could it be that it happens when the app is maximized to full screen when running on the N7? 19:47 what is the intital width on n7? 19:47 then a workaround would be to give it an initial size of N7 widht 19:47 we had the same thought :) 19:47 let me calculate... 19:49 Nexus 7 is 1920/18 = 106.67 GU width and 1200/18 = 66.67 GU height 19:49 what about n10? 19:50 it seems not to happen there, I'm not sure about why. But in any case, the resizing is also happening on the N10 19:50 since we have quite a small initial size, which gets maximized to the screen size on all devices 19:51 it's not exclusive to the footer it seems 19:56 popey, could you try to run lp:~dpm/ubuntu-weather-app/n7-tweaks on the N7? If it makes things easier, I've created a .click too: http://people.canonical.com/~dpm/click/com.ubuntu.weather_1.0.208_all.click 19:57 it's a bit of a shot in the dark, but worth a try 19:57 yes 19:57 does that click have the key? 19:57 yes 19:57 k 19:58 I know how to detect this in the code: 19:58 in Location Tab: 19:58 onWidthChanged: { 19:58 print("width changed") 19:58 print(mainView.width) 19:58 print(tabFooter.width) 19:58 } 19:59 m-b-o, ah, good idea 19:59 nope 20:00 argh :( 20:02 http://i.imgur.com/I9idfNc.jpg 20:03 popey, can you try to install http://people.canonical.com/~dpm/click/com.ubuntu.weather_1.0.209_all.click , run it from the command line and paste the output? 20:03 ok 20:08 it seems not to happen without Layouts 20:13 dpm: sorry, had a unity crash ⍨ 20:14 np 20:14 same 20:14 you want logs? 20:14 yeah, that should not make any difference, I just wanted to see the output 20:15 http://paste.ubuntu.com/6967343/ 20:18 dpm: you can add print("start: "+mainView.width) to Component.onCompleted in mainView to getthe size at startup 20:20 * popey reboots to get rid of dead chromiums.. brb 20:24 popey, when you're back, here's a new .click to run on the command line to get more output: http://people.canonical.com/~dpm/click/com.ubuntu.weather_1.0.210_all.click 20:24 k 20:24 m-b-o, done, pushed it on https://code.launchpad.net/~dpm/ubuntu-weather-app/n7-tweaks and on that .click ^ 20:27 http://paste.ubuntu.com/6967398/ 20:27 although the app just gives white screen now 20:27 what did you do!? 20:27 oh, okay when i restart 20:27 odd 20:28 http://paste.ubuntu.com/6967405/ 20:30 m-b-o, it seems there is some confusion between width and height? ^ 20:31 I wonder if the app thinks the height is the width because of landscape mode. But then, it should not work on the N10, either 20:32 why is the size changed so often in the logs? 20:32 popey, is that the log after having launched the app twice? 20:32 yes 20:36 m-b-o, the log, after filtering out the non-relevant output would be: http://pastebin.ubuntu.com/6967433/ 20:38 to reproduce: when one in the wide layout and then going to narrow wide, the move after that it's get stucked 20:43 I'm still puzzled by the swapping between width and height on the device, though 20:44 can you implement a wait until the size settles? 20:44 no 20:44 like leave the loading thing up until the size stops flapping about? 20:45 it's related to Layouts somehow 20:45 parts are getting not rerendered, or something 20:53 I would say it's perhaps a bug in the SDK 21:09 http://paste.ubuntu.com/6967584/ 21:09 fixes it on my desktop 21:10 ok, thanks, sending it over for testing to cwayne, it seems popey has got some computer breakage 21:10 http://people.canonical.com/~dpm/click/com.ubuntu.weather_1.0.212_all.click 21:11 oh, cwayne is away, we're N7-less 21:13 hmm 21:13 i ninja'ed that edit on my device 21:13 and it didnt work 21:13 maybe I messed up 21:13 lemme try your click 21:13 I fixed my computer by replacing lightdm with gdm! 21:14 dpm: installing now.. 21:14 * dpm is excited 21:15 no, that kinda broke it more oddly 21:15 the size thing, i know what it is I think 21:15 its when the app starts, it slides in from the right 21:16 ah 21:17 now the whole app is offset to the right 21:17 very odd 21:17 maybe not 21:17 i think i just started the app as unity was starting 21:17 on desktop it's fixed 21:17 now I let it calm down, its okay 21:17 but the bar is still not wide enough 21:17 i wonder if this is running an old build or something 21:17 the log? 21:17 phablet@ubuntu-phablet:~$ click list | grep weather 21:17 com.ubuntu.weather 1.0.212 21:19 that's the right onw 21:19 *one 21:19 http://paste.ubuntu.com/6967638/ 21:19 argh, I removed the print lines on resize, so the log won't be much use, sorry :( 21:19 ok, creating the last .click for tonight 21:19 one mo 21:19 k 21:19 gimmie the print lines? 21:21 popey, basically you can readd the lines marked as minus in here, but you should add them after line 14: http://pastebin.ubuntu.com/6967589/ 21:21 k