**** BEGIN LOGGING AT Thu May 08 02:59:58 2014 May 08 08:13:28 morning May 08 08:14:19 Tofe|Away: Yes, when screenshot work work I could make one :P Might take a pic with my Veer later today so you can see what I mean ;) May 08 13:19:14 Herrie|away: yScale: textVisible?0.8:1 looks like the culprit: why not 0.5 instead of 0.8 ? May 08 13:19:47 Herrie: ^ May 08 13:22:01 I think I tried that, let me look again :P May 08 13:23:01 The height of the battery is 1/4 of the total height instead of 1/2 then :P May 08 13:23:26 well, that's because the original image is square, IIRC May 08 13:23:31 let me check, I'm not that sure May 08 13:23:56 See email for picture :P May 08 13:24:28 mmmh May 08 13:25:45 yScaling of 1 is fine even, just the location needs to be at the bottom :P May 08 13:26:05 0.5 is the right thing to do, I think the remaining issue is about the image ratio May 08 13:27:11 we could disable the KeepRatio thingy when text is visible May 08 13:27:39 Image.PreserveAspectFit <-- May 08 13:28:42 -->textVisible ? Image.Stretch : Image.PreserveAspectFit May 08 13:30:00 yes, that could work actually May 08 13:34:55 It doesn't :P May 08 13:37:19 Ah wait it does... May 08 13:37:30 Just alignment is off now :P May 08 13:37:57 text is left aligned and image center it seems May 08 15:28:11 Herrie: sorry for the delayed answered, I was out for a while May 08 15:29:17 No worries, we all have other things to do unfortunately :P Otherwise we'd have a fully working port running on every possible device long time ago :P May 08 15:29:46 Herrie: with Image.Stretch, one other issue is that the width of the image is unknown to Qt May 08 15:31:32 But we define item width above? May 08 15:31:55 Doesn't it automatically use that for elements within the item? May 08 15:32:23 no, the Items within can be bigger without problem in Qt May 08 15:32:32 it doesn't even clip by default May 08 15:33:03 Ah ok :) May 08 15:33:23 Added width: indicatorRoot.width and now it looks a lot better! May 08 15:33:29 Need to try for other options May 08 15:33:49 we have to be careful, we are very close to a property binding loop with width May 08 15:35:49 I think the right way to do it is a conditional anchors.right: textVisible ? indicatorText.contentWidth : undefined May 08 15:35:58 Works for percentage and icon + percentage. When just icon we have a problem when using indicatorRoot.width. It's simply gone, along with telephony bars :s May 08 15:37:00 if you look at getIndicatorWidth, you'll see why we can't just put "width = indicatorRoot.width" for the image :) May 08 15:37:14 if I put textVisible ? indicatorRoot.width : "100%" seems ok :P May 08 15:37:57 Yeah LOL :P Circling to itself doesn't work :P May 08 15:38:22 mmmh okay... this "100%" is a bit magic to me when we only have the icon May 08 15:39:13 if it works well, let's do it like that for the moment May 08 15:39:32 Seems like it does :P May 08 15:39:40 ok, good May 08 15:39:45 Tried all 3 options and look fine :), May 08 15:40:30 It's a PR then :) Can you merge a bit your commits ? a "git rebase -i HEAD^^^" could help you a lot there May 08 15:41:12 (don't know if you have used interactive rebase before, or the "^" syntax) May 08 15:51:41 Tofe: only have GitHub web interface here, so deleted branch and recreated the changed files :P PR is done. May 08 16:04:50 Herrie: merged May 08 16:05:15 Anyway heading to hairdresser & home :P Will login when home :P PC is connected again :) May 08 16:05:49 I merged the two similar commits during the merge May 08 17:31:07 hey, you guys cross compile... i have a question. :) May 08 17:31:30 i'm trying to cross compile, and my code requires a library. i've downloaded the library, compiled it, and now i'm trying to compile the code. May 08 17:31:49 how do i tell autoconf to use the library i just compiled? setting CFLAGS didn't seem to work, unless i did it wrong. **** ENDING LOGGING AT Fri May 09 03:00:01 2014