/srv/irclogs.ubuntu.com/2014/02/21/#ubuntu-unity.txt

=== ajmitch_ is now known as ajmitch
=== Daviey_ is now known as Daviey
=== duflu_ is now known as duflu
=== zequence_ is now known as zequence
Saviqsil2100, hey, so apparently silo 16 is unhappy about the package ogra uploaded directly... what did we do wrong?08:45
sil2100Saviq: hello, let me see08:46
sil2100Saviq: interesting08:50
sil2100Saviq: hmmm, I wonder if anyone was reconfiguring the silo for you guys yesterday and forgot adding that source08:50
sil2100Let me retry08:50
Saviqsil2100, robru reconfigured indeed08:50
Saviqsil2100, so that might be it08:51
tsdgeospeople08:53
tsdgeosi'm still getting the assertion08:53
tsdgeosthis is not fun08:53
tsdgeosGLib:ERROR:/build/buildd/glib2.0-2.39.90/./glib/gvarianttypeinfo.c:184:g_variant_type_info_check: assertion failed: (0 <= index && index < 24)08:53
tsdgeosAborted (core dumped)08:53
tsdgeosnoone else is getting this?08:53
mzanettinot here. when should this happen?08:54
didrockstsdgeos: there has been a new glib recently (and you are on latest of latest)08:54
tsdgeosmzanetti: ./run_on_device on new-scopes branch08:55
mzanettiI haven't seen that with the merged new-scopes and right-edge one yet08:55
mzanettibut I updated/merged last time yesterday lunch time08:56
karnio/08:56
tsdgeosof course if i run it with gdb it doesn't segfault ^_^08:57
karni:D09:00
sil2100Saviq: try building now?09:01
Saviqsil2100, thanks09:04
sil2100Saviq: it should be ok I think, at least the backend says it's ok ;p09:06
Saviqkarni, font summary: we just need the title weight fixed, but you need to refactor it slightly, as components are not available in CardHeader, so either we need to pass it there or as a different prop09:07
Saviqsil2100, yeah, thanks!09:07
karniSaviq: gotcha, on it09:07
mhr3Saviq, thomas would like to see the default renderer changed, he does have a point that doing searches the medium cards look pretty bad09:10
mhr3when doing*09:10
Saviqmhr3, so medium > small by default?09:11
mhr3that's one option09:11
mhr3remapping things from art to mascot could be another09:12
mhr3but that seems to big of a change09:12
tsdgeosSaviq: i get a "spinning wheel" in the middle of the dash when swiping from scope to scope, think it may be the preview loading indicator that is escaping the list ?09:12
Saviqtsdgeos, yeah, probably09:13
* Saviq tries to protect09:13
mhr3tsdgeos, i noticed that the text preview widget sometimes does html markup, can we disable that?09:15
tsdgeosmhr3: don't send html? :D09:15
Saviqtsdgeos, http://paste.ubuntu.com/6969862/09:15
tsdgeosmhr3: what do you want to disable?09:15
mhr3tsdgeos, don't parse it! :P09:15
tsdgeosmhr3: i mean you want us to show <i>cool</i> to the user?09:15
Saviqmhr3, we can make it dumb text, but yeah ↑ will show09:16
mhr3tsdgeos, yes09:16
mhr3then the scope authors will see that it doesn't do markup09:16
mhr3and it shouldn't, that's what the spec says09:16
tsdgeosSaviq: should work09:16
tsdgeosmhr3: we should be able to do that09:16
davidcallemhr3, tsdgeos, could you please not do that before MWC?09:17
tsdgeosSaviq: i'll apply it and try09:17
Saviqmhr3, http://paste.ubuntu.com/6969873/09:17
mhr3davidcalle, why not?09:17
mhr3davidcalle, it looks crap with it anyway09:17
davidcallemhr3, hmm, in which scope?09:18
mhr3seen it amazon a couple of times09:18
tsdgeosanother crash09:19
tsdgeoshttp://paste.ubuntu.com/6969885/09:19
davidcallemhr3, I'm afraid they will look much worse if they are missing their html formatting09:19
davidcallemhr3, and we don't have time right now to make them look right before monday09:19
mhr3davidcalle, isn't there a switch for the api where you can disable marked-up text?09:20
Saviqtsdgeos, whoa your device is unhappy :/09:21
mhr3Saviq, tsdgeos, also, missing max height? http://imgur.com/ogJ4KO609:21
mhr3eh, width09:21
davidcallemhr3, in the Amazon API? I don't know, but if that exists, I'm trying asap09:21
Saviqmhr3, I think that's gallery, so you can swipe?09:21
mhr3Saviq, swiping takes me to next preview09:22
Saviqmhr3, on the image? /me tries09:22
tsdgeosmhr3: is that image or gallery?09:22
mhr3no idea09:22
mhr3probably image09:23
mhr3davidcalle, ^ do you know?09:23
davidcallemhr3, image09:23
mhr3tsdgeos, ^09:23
Saviqmhr3, yeah, bug09:23
Saviqnot sure what do we do, scale the image down? :/09:24
Saviqit'll end up quite small09:24
tsdgeosSaviq: i think we don't have any other option than scaling it down09:25
tsdgeosand in the futue add the clicking brings you to the magical world of single image on screen where you can zoom/pan/behappy09:25
Saviqtsdgeos, probably :/09:26
=== mpt_ is now known as mpt
tsdgeosmore detailed crash http://paste.ubuntu.com/6969920/09:29
tsdgeosi can't work09:29
Saviqmhr3, http://paste.ubuntu.com/6969922/09:29
Saviqtsdgeos, I think you need to start from scratch (flash and all)09:29
* mhr3 tries09:29
tsdgeosSaviq: which i did 5 minutes ago09:29
Saviqtsdgeos, crap09:30
tsdgeoswell not 5 minutes, 1 hour when i woke up09:30
tsdgeosrebooting doesn't help09:30
tsdgeosshuting down and waiting for a bit doesn't help09:30
tsdgeosi'd like our glib friends not to decide to take down the process so easily09:30
tsdgeosConnection reset by peer (g-io-error-quark, 0). Exiting.09:31
tsdgeoscome on, what about trying to survive?09:31
Saviqtsdgeos, so that's mwc-flashed manta and run_on_device?09:31
mhr3tsdgeos, that means it couldn't connect to dbus, it's not like things would work09:31
tsdgeosSaviq: not sure what you mean with mwc-flashed09:31
Saviqtsdgeos, using the mwc-flash tool09:32
tsdgeosit's devel-propsed + demo-stuff + run_on_device09:32
tsdgeosnoone told me of that tool, so no, not using it09:32
Saviqtsdgeos, ok nvm, same difference really09:32
mhr3Saviq, the images are vertically centered now, shouldn't they be top-attached?09:33
Saviqmhr3, you want too much, don't you... but anyway TBH it looked nice this way09:33
Saviqmhr3, there'd be quite some empty space between image and other widgets otherwise09:34
davidcallemhr3, no luck with the Amazon API, but removing the markup is not *that* bad in Amazon descriptions. Please tell me when this land if I have to change other scopes. +109:34
Saviqmhr3, unless we move stuff around once we load the image09:34
tsdgeos(process:2394): GLib-GIO-WARNING **: Error releasing name com.canonical.hud.query2.appstack: The connection is closed09:34
tsdgeosg_dbus_connection_real_closed: Remote peer vanished with error: Error receiving message: Connection reset by peer (g-io-error-quark, 0). Exiting.09:34
mhr3Saviq, yea, it actually does look nice09:34
mhr3Saviq, so meh.. +1 :)09:35
tsdgeosno really, i can't believe i'm the only one with this problem :-/09:35
Saviqtsdgeos, your dbus is fooked09:35
Saviqtsdgeos, for whatever reason :|09:35
tsdgeosok, i'll waste anohter hour in flashing everything09:35
mhr3davidcalle, let's do it this way - we'll keep the markup enabled for now, but bear in mind that it will be disabled soonish, so start updating scopes that use it09:37
mhr3Saviq, tsdgeos, so let's keep the markup for mwc09:37
Saviqmhr3, k, shelving09:37
davidcallemhr3, ok09:38
Saviqmhr3, the other change pushed09:38
mhr3thx09:38
Saviqmhr3, so, about the default card...09:40
mhr3Saviq, ok to change the default to small?09:41
Saviqmhr3, yeah, we have it defined in -scopes-shell, right? although I apply a default myself09:42
Saviqbut that won't need changing then09:42
Saviqfor now09:42
mhr3Saviq, yea, mping shell branch09:43
mhr3i want to remove support for Unity 0.109:43
mhr3cause the plugin will compile twice as fast!09:44
=== Guest79143 is now known as Debolaz
Saviq;)09:45
Saviqtsdgeos, so one thing you could try https://wiki.ubuntu.com/Touch/Install_UT_on_android4.4.209:46
tsdgeosSaviq: but i don't have android 4.4.2, no?09:46
tsdgeosi mean this thing is old old old09:46
Saviqtsdgeos, Nexus10?09:47
tsdgeosSaviq: yes09:47
Saviqtsdgeos, so you *can* have 4.4.209:47
tsdgeosah09:47
Saviqhttp://people.canonical.com/~rsalveti/aosp/manta/09:47
tsdgeosyou mean install android 4.4.2 and then intall Ubuntu again?09:47
Saviqtsdgeos, no not even09:48
Saviqtsdgeos, the files ↑ have 4.4.2 in them09:48
tsdgeosi see09:48
tsdgeosi'll try if current reflash doesn't help09:48
karniCimi: from last Kevin's e-mail sounds like you took care of the Carousel fix, thank you for that.09:50
Saviqmzanetti, hey, I sometimes have a thick dark vertical bar between apps in right-edge, that's an app that failed launching?09:52
mzanettiSaviq: could that be the drop shadow?09:52
Saviqmzanetti, probably, yeah, I had an app that didn't launch properly and was a ghost09:53
mzanettiSaviq: yeah... so in that case upstart tells the appmanager: "yep, the app has started" but it never starts to paint on any surface09:53
Saviqmzanetti, yup09:54
mzanettiI'm thinking of generating a white opaque app screenshot in that case... not sure if that really makes it any better tho09:54
mzanettistill feels broken... so probably we should just get the root problem fixed09:55
Saviqmzanetti, we'll have a splash screen that shell will draw09:55
Saviqmzanetti, but app crashing needs to be reported anyway09:56
mhr3Saviq, https://code.launchpad.net/~mhr3/unity-scopes-shell/small-cards-by-default/+merge/20760609:56
Saviqmzanetti, you're done with the crash, btw?09:56
mzanettijust flashing a mwc image to run final tests09:56
Saviqmzanetti, next thing would be https://bugs.launchpad.net/unity8/+bug/1282126 if you can09:56
ubot5Launchpad bug 1282126 in Unity 8 "Main stage running app screenshots in dash are incorrectly proportioned" [High,Triaged]09:56
mzanettimeh... another one that would be fixed in my branch :D09:57
Saviqmhr3, NEWLINES, ANYONE?09:57
mzanettimaybe I can just pull over my code into trunk in this case.09:57
mzanettiwill look at it09:57
mhr3Saviq, i'm saving 2ns by not having them :P09:57
mhr3Saviq, not to mention all the battery i save!09:58
Saviqmzanetti, yeah ;)09:58
Saviqmzanetti, that's why it's one for you :)09:58
Saviqmy unity is dying today...09:59
SaviqI might need to reboot!09:59
mzanettiask Cimi for a solution :P09:59
Saviq"solution"10:00
Saviqif I wanted to slash my wrists, I would!10:00
Saviqdon't need my DE to do it for me10:00
Saviqas I'm sure that's a "feature" somewhere in KDE ;P10:01
mzanettiprobably10:01
CimiSaviq, which issue you have with unity?10:03
SaviqCimi, black windows and some flashing10:03
Cimiah no, different to mine10:03
=== rachelliu_ is now known as rachelliu
CimiSaviq, currently unity is not usable with qtcreator10:03
SaviqCimi, works fine here10:03
CimiSaviq, qtcreator?10:03
SaviqCimi, yes10:04
SaviqCimi, latest trusty10:04
SaviqCimi, and I imagine it works fine for most of the people on the team, too10:04
mhr3tsdgeos, when doing gotoScope, the pageheader doesn't often show up10:04
mhr3tsdgeos, known?10:04
mhr3wanna bug?10:04
CimiSaviq, indeed, fix got merged last evening10:04
Cimi:10:04
Cimitime to upgrade10:04
Saviqmhr3, sdk bug10:04
Saviqmhr3, wanna10:05
Saviq   ↑↑↑↑↑ is bathtub in PL10:05
tsdgeosmhr3: sure open a bug, but since i can't run anything in a device i don't think i'll be able to fix it tbh unless some magic happens10:05
karni:D10:05
Saviqhuh ↑ isn't monospace?10:05
Saviqbooo10:05
tsdgeos↓↓↓↓↑↑↑↑↑10:05
tsdgeosit is or me10:05
tsdgeosfor me10:05
Saviq--------10:05
Saviq↑↓↑↓↑↑↑10:06
Saviqtwice the width10:06
tsdgeosnot here :D10:06
* Saviq checks the font10:06
Cimimzanetti, tsdgeos bye bye kde10:06
tsdgeosCimi: why do you think i care what you use?10:07
Saviqhmm Ubuntu Mono broken :|10:07
Cimitsdgeos, you seemed to care yesterday :D10:07
Saviqtsdgeos, easy there :D10:07
karniScopes scope labels with current font sizes = meh. "Grooveshar/nk", "The\nWeathe...", "Canonical..." ;_;10:07
Saviqtsdgeos, breathe in, breath out, in, out10:07
Saviqkarni, yeah :/10:07
mhr3can we do anything to fix?10:08
mhr3demo fix at least :)10:08
karnidecrase Row "spacing" prop in CardHeader from 1gu to 0.5gu10:08
Saviqkarni, won't have a difference10:08
karnibut then we'd need to decrease Summary margins in Card as well10:08
Saviqkarni, cards are preset width10:08
Saviqkarni, they won't fit on phone10:09
karniSaviq: at least "Canonical..." wouldn't be trimmed, I think I tried it the other day10:09
karnimeh :(10:09
* karni might give it a shot later today10:09
mhr3Saviq, karni, lower the font size for the scopes card type only?10:09
karnimhr3: How would we do that?10:10
mhr3karni, magic? :)10:10
Saviqmhr3!!!!10:10
karni:)10:10
karnimhr3: they should fix this in the SDK10:10
karnimhr3: Saviq: JIC Jouni hasn't seen it, I'll send him a screenshot in a moment, and I'll expect "uuuugh we need to fix this!" ;)10:11
karniHe must have seen it, though. They want through current mwc image with Rosie already.10:11
karniSo, meh.10:11
mhr3karni, tell me what to screenshot, my phablet-screenshot actually works :)10:11
mhr3actually, i'll go to the office in a bit10:12
karnimhr3: current mwc image after dist upgrade. Scopes scope contains cards that have heavily elipsized labels.10:12
karninotably, TWC and Grooveshark cards10:12
karniTWC more, but Grooveshar\nk just looks wrong.10:13
mhr3karni, http://imgur.com/BhEOSlj10:13
karnimhr3: thank you, sir. I'll ping Jouni10:13
Saviqmhr3, http://imgur.com/fqjoXJz10:14
Saviqoh look! you don't have any calendar events ;D10:14
Saviq(and better WiFi reception) :P10:14
CimiTrevinho_, LIM10:17
CimiTrevinho_, when you resize a window, don't switch LIM on10:17
CimiTrevinho_, otherwise you see the title flashing10:17
mhr3Saviq, and you live in the future! :)10:18
karniSaviq: Can I put header font weight tests in tst_Card.qml instead tst_CardHeader.qml? Because Card.qml has concept of components, while tst_CardHeader does not relate to component mapping.10:22
Saviqkarni, of course, otherwise it'd be just a binding test10:23
karnigood10:23
Saviqkarni, but shouldn't it be in CardTool anyway?10:23
Saviqkarni, since it's category-wide?10:23
karniSaviq: well, then I'd have to pass in each CardCarousel, CardFitlerGrid, etc etc, pass down from CardTool, through CardCarousel/etc, through Card, through CardHeader. I can do that in Card instead, with10:24
karnititleWeight: components && components["subtitle"] !== undefined ? Font.DemiBold : Font.Normal10:24
karniI know it's category-wide, but it seems there's lot of hops just to get it from CardTool. You'd still want that, tho?10:25
Saviqkarni, I just wonder which is more intensive10:25
Saviqkarni, calculating it in every card and one binding, or calculating it once and bind twice10:25
karniquite subtle10:25
karniyeah :D10:25
Saviqkarni, so probably not a huge difference10:26
Saviqkarni, leave it be, then10:26
karniSaviq: basically, each renderer would need to pass that value down from CardTool instance, right?10:26
karnibecause it's the renderers that have the CardTool instance10:26
Saviqkarni, yes10:26
Saviqkarni, but pass, not calculate10:27
karniok10:27
Saviqkarni, OTOH that is a very simple calculation10:27
Saviqkarni, so leave it be10:27
karni^_^10:27
mzanettianyone else seeing the issue that the MWC issue has troubles connecting to 5GHz wifis?10:37
mzanettiseems to work fine with my 2.4GHz wifi. But for 5GHz I need to select it, enter the key and then reboot for it to connect10:37
Saviqmzanetti, /me tries10:38
mzanettiSaviq: only happens if network settings are completely wiped (e.g. fresh mwc-flash)10:38
Saviqmzanetti, oh ok, just flashing my flo10:38
mhr3Saviq, something's off - http://imgur.com/YFKPTL510:39
mhr3Saviq, the widths of the two match fyi10:40
Saviqmhr3, we never said it's 100% as designed10:41
Saviqmhr3, there's still things like that to fix10:41
mhr3ok10:41
mhr3just wanted to see it side by side10:42
mhr3wanted *you* to see it ;)10:42
Saviqmhr3, http://paste.ubuntu.com/6970181/10:42
mzanettiseems the spacing between the tiles is smaller too10:43
mhr3Saviq, +110:43
Saviqmzanetti, that's because the visual designs are not in sync with the spec10:45
mzanettiok10:45
* Saviq wonders where the HR comes from...10:45
* mzanetti is confirming that the MWC image + apt-get upgrade does not crash unity on app crashes any more10:46
Saviqmzanetti, cool beanz10:47
Saviqmzanetti, connected fine to 5GHz hefe10:48
Saviqhere10:48
Saviqjefe10:48
mzanettihmm... didn't work a single time here10:48
mzanettibut we know how it is with wifi... really depends on the weather and whatnot10:48
karniSaviq: I can push to trunk, but got this MP so you could give sanity check https://code.launchpad.net/~unity-team/unity8/unity8-fix-title-weight/+merge/20761910:52
Saviqkarni, push to new-scopes you mean ;)10:52
tsdgeosSaviq: so i got the thing to work somehow reliably, seems i need to unlock the greeter before run_on_device10:52
tsdgeosand then things are acceptably stable10:52
Saviqtsdgeos, interesting10:52
tsdgeosSaviq: back to playing with sidestage and dash10:53
Saviqkarni, I'll take care of it, thanks10:53
tsdgeosSaviq: you can hide and show the sidestage-only app by the handle, that works10:53
karniSaviq: aye.. only landing to trunk. so umm.. this should have been against new-scopes? or I'd just push last commit to new-scopes? oh, ok. thanks :D10:53
tsdgeosSaviq: but you can also hide sidestage-dash by dragging from left edge all the way, but then you can't bring it back from the right edge, i guess that has to be fixed too, right?10:53
Saviqkarni, no, it's fine against trunk, but I'll merge into new-scopes straight away10:54
karniack10:54
Saviqtsdgeos, you should always be able to bring it back from right edge, yes10:54
karniSaviq: no immediate work on the other team, got somtn for me? :D10:56
karniif not, I could try finishing the vertical journal that's been laying around for a while.10:57
Saviqkarni, you should've based off of card-background, there's a big conflict there10:57
Saviqkarni, can you merge card-background into your branch and resubmit, please?10:57
karnifudge. I can fix that10:57
karnisure10:57
CimiI cannot find anymore mockups for previews11:03
CimiSaviq, tsdgeos ^11:03
CimiI need the visuals for the review display11:04
CimiI finished the widget with tests11:04
SaviqCimi, star this https://drive.google.com/a/canonical.com/?tab=co#folders/0B-a_7E3tDxOgWDlCSWxkUGZlcnM11:04
CimiSaviq, dziekuje11:05
SaviqCimi, zaniemaco11:05
CimiSaviq, zaniemaco or prosze?11:06
SaviqCimi, ask your GF ;)11:06
Cimihah ok11:06
CimiSaviq, she'll probably learn italian way before I learn polish11:07
Cimihah11:07
karniSaviq: there, I fixzed it! https://code.launchpad.net/~unity-team/unity8/unity8-fix-title-weight/+merge/20762111:07
karni:)11:07
CimiSaviq, there's no visual for reviews, idneed11:09
Saviqkarni, huh :/ still conflicts with new-scopes11:10
Saviqkarni, I'll manage11:10
karniSaviq: I can do it11:10
Saviqkarni, nah, it's fine11:11
karnilet me merge new-scopes into it, and push to MP?11:11
karniok..11:11
CimiSaviq, tsdgeos I added seemore widget to rating display, what sort of signal shall I send when I press see more/ less?11:11
tsdgeosno idea sorry11:11
Cimiroot.triggered(widgetId, "more", widgetData) ?11:11
karniSaviq: I think it's because it's been a while new-scopes has been mered to trunk. I'm getting a tiny bit > < confused with branches and conflicts, which I've seen myself. new-scopes has diverged a bit, and we land some stuff in trunk, then some in new-scopes. then we get conflicts. should we perhaps merge trunk to new-scopes (?)11:13
Cimiactually, we don't need a see more / less11:13
Cimiwe need probably just a "more"11:13
karniCimi: no.11:13
karniCimi: I talked to John about this, we need see more / less11:13
Cimikarni, ok but what is "more"?11:13
karniCimi: I dislike that, if you ask me, but that doesn't matter.11:13
karniCimi: "< Cimi> we need probably just a "more"" - you tell me :)11:14
Saviqkarni, well yeah, there's a lot on top of trunk in new-scopes indeed11:14
Saviqkarni, sanity check pleae http://bazaar.launchpad.net/~unity-team/unity8/new-scopes/revision/67711:14
karniSaviq: and there's some changes in trunk that would be happy to be in new-scopes, as we try to do as much in trunk as we can11:15
karniack11:15
Saviqkarni, there's nothing in trunk that isn't in new-scopes11:15
Saviqkarni, checklist, please11:15
karniSaviq: +1, exactly that. ok, adding checklist11:15
Saviqmzanetti, top-ack https://code.launchpad.net/~saviq/unity8/sidestage-threshold/+merge/207260 ?11:16
mzanettiSaviq: done11:16
Saviqmzanetti, thanks11:17
karniSaviq: added11:19
Saviqkarni, ah now I see why it still conflicted ;)11:19
karniSaviq: why did it?11:19
Saviqkarni, you didn't actually merge card-background into your branch, just resubmitted the MP ;D11:19
Saviqsneaky11:19
karniwhat? no.. Saviq, I actually branched card-background, and applied changes, and pushed11:19
karniwtf11:19
Saviqkarni, didn't push? https://code.launchpad.net/~unity-team/unity8/unity8-fix-title-weight11:20
Saviqkarni, anyway, nvm11:20
karniSaviq: sorry about that, man. I was sure I got it right :/11:20
Saviqkarni, no worries :)11:20
karni:)11:20
karniSaviq: unless you throw stuff at me, I'll wrap vertical journal integration11:20
=== lool- is now known as lool
karnisomehow I feel meh going back to it ;D took me a while back then haha11:21
Saviqkarni, you could try and find out why we have so big a margin under cards11:22
Saviqkarni, so between the last row of cards of one category and the next one's section header11:22
karniSaviq: under subtitle? which screen gives it away the best?11:22
karniaha11:22
Saviqkarni, hum oh wait11:23
karniSaviq: stuff looks real good here11:23
karniwas about to send you a picture.11:23
Saviqkarni, http://imgur.com/noU0Aeg11:27
CimiSaviq, tsdgeos https://code.launchpad.net/~cimi/unity8/previews.PreviewRatingDisplay/+merge/20762511:27
* Saviq wonders if my last changes did that11:27
karniSaviq: dist-upgrade? :D11:27
karniSaviq: I don't see that spacing11:27
Saviqkarni, pull new-scopes11:27
karniSaviq: want me to ./run-on-device?11:28
karninvm, I'll use unity-scope-tool11:28
Saviqkarni, nah, I'm seeing that on desktop11:28
karniack11:28
karniyeah I can see it11:31
mzanettigreyback: Saviq: https://code.launchpad.net/~mzanetti/unity-mir/expose-appimage-sourceSize/+merge/207626 and https://code.launchpad.net/~mzanetti/unity8/keep-appImage-aspect-ratio/+merge/20762711:32
* karni need to leave for a bit11:33
=== karni is now known as karni-afk
Saviqmzanetti, cheers11:36
Saviqmzanetti, would it make sense to include that change in the new screenshotting api MP instead?11:46
Saviqmzanetti, for unity-mir that is11:46
Saviqhmm probably not11:47
Saviqsince we don't want the whole thing for rotate-n711:47
mzanettiSaviq: huh?11:47
Saviqmzanetti, nvm11:48
mzanettiSaviq: you mean in the right edge stuff?11:48
Saviqmzanetti, no, the sourcesize stuff11:48
Saviqmzanetti, but no, I explained why not ;)11:48
Saviqto myself11:48
mzanettiSaviq: still not sure what you mean... the screenshotting API doesn't use ApplicationImage any more at all11:48
mzanettiits just Image { source: app.screenshot }11:49
Saviqmzanetti, right, ignore11:49
mzanettiack11:49
Saviqmzanetti, hmm it doesn't look like this worked...11:57
mzanettiSaviq: hmm... in which sense?11:57
Saviqmzanetti, I built n7-rotate with your patch11:57
Saviqunity-mir that is11:57
Saviqmzanetti, patched unity8 on-device11:57
mzanettihmm... I tried with trunk only11:57
Saviqmzanetti, and no screenshot at all11:57
mzanettiuh... let me check again11:57
Saviqmzanetti, can you try with new-scopes please11:57
mzanettiack11:58
mzanettiSaviq: what's the easiest way to flash that?11:58
mzanettiwithout the right edge11:58
Saviqmzanetti, just flash and upgrade from ppa:unity-team/demo-stuff11:58
greybackmzanetti: Saviq: I've just tried it on N7 with latest mwc image, I get screenshots but they're wrongly sized11:58
mzanettimeh /me failed11:59
Saviqgreyback, did you patch unity8 too?11:59
greybackSaviq: yes11:59
Saviqso yeah11:59
greybackI think it is just for the N7, need width<->height swap11:59
mzanettiah... that might be11:59
mzanettiI tried on N1011:59
Saviqgreyback, right, rotation12:00
greybackmzanetti: http://pastebin.ubuntu.com/6970407/12:00
greybackthat's on the N712:01
=== MacSlow is now known as MacSlow|lunch
Saviqgreyback, sure that's the right paste?12:01
mzanetti^12:01
greybackdamn, http://imgur.com/kS8JHyR12:01
Saviqgreyback, yeah, looks like it12:01
mzanettilol... that's the opposite of what it should be :D12:01
greybackmzanetti: yep :)12:01
Saviqmzanetti, probably because of the NATIVE_ORIENTATION12:01
mzanettiyeah, sounds reasonable12:02
mzanettibut doesn't this mean that the images have this aspect ratio when coming from mir?12:02
mzanettigreyback: ^12:04
greybackmzanetti: yeah, since the N7 is portrait12:05
mzanettiso I'd say my 2 branches are ok, but something's wrong in the layers below12:05
greybackso MS has aspect ratio like 3:5, but sidestage has 2:3 (guesses)12:05
Saviqgreyback, MS 16:912:06
greybackSaviq: but according to the hardware & Mir, N7 is 9:1612:07
Saviqgreyback, yeah ok, I meant not 3:5, but 9:16 then12:07
greybackSaviq: gotcha12:07
Saviqgreyback, we were cropping the shot to square before12:08
greybackmzanetti: Mir only knows portrait, so gives us screenshots relative to the portrait orientation of the device. It's only unity8 & unity-mir that know about rotating the view to landscape12:08
greybackSaviq: right12:08
tsdgeosSaviq: can you reproduce that clicking in the apps grid in the middle of the grid (i.e. in a place with no icons) makes the device bibrate as if some button was being pressed?12:08
Saviqtsdgeos, yes12:09
tsdgeosi guess that's a bug?12:09
mzanettigreyback: where is the rotation happening?12:10
greybackmzanetti: for screenshots, in the ApplicationImage.qml12:10
Saviqtsdgeos, dunno12:10
tsdgeosok12:10
mzanettigreyback: I'm blind... do I need some other branch?12:11
Saviqmzanetti, yes12:11
greybackmzanetti: yeah sorry, shoulda said: lp:~unity-team/unity-mir/rotate-n712:11
mzanettiah.. ok. np12:11
Saviqmzanetti, so sourceSize needs to be adapted to that probably12:12
mzanettiSaviq: yep.12:12
Saviqmzanetti, in unity8, though... we were cropping to square, I think that's still desired12:13
Saviqmzanetti, so we could limit aspect ratio to < 112:13
Saviq<=112:13
mzanettiSaviq: again not following12:13
mzanettiwhy cropping to square? isn't that what it was before?12:13
Saviqmzanetti, in the running apps list12:13
Saviqmzanetti, cropping to square we want12:14
Saviqmzanetti, what we don't want is stretching12:14
Saviqmzanetti, which was happening12:14
mzanettiok... need to leave for an hour and a half12:15
Saviqmzanetti, https://launchpadlibrarian.net/166769826/2014-02-19%2015.28.59.jpg12:15
Saviqk12:15
mzanettiuh... just finished flashing the N10 and upgraded to demo-stuff... seems sidestage is completely borked here12:16
mzanettithe side stage app shows up on the left hand side12:16
tsdgeos?¿12:16
tsdgeoslol12:16
tsdgeosnot here :D12:17
greybackmzanetti: not here either, but I did use the mwc script12:19
greybackshould be the same tho12:19
Saviqgreyback, mzanetti, only difference probably aosp 4.2 vs. 4.4.2, but it's not like that should matter12:20
* Saviq mwc-flashes12:20
greybackindeed12:20
cwayneSaviq, hey, confirming your fixes for backgrounds works :D12:42
=== alan_g is now known as alan_g|lunch
=== karni-afk is now known as karni
tsdgeosSaviq: ok, all the stuff for the sidestage-only+dash should be ready at https://code.launchpad.net/~aacid/unity8/new-scopes-show-dash-app-running/+merge/207446 been playing with it for a while and seems it does work as i expected, now you'll need to play with it and see if my expectations and yours are the same :D12:48
=== MacSlow|lunch is now known as MacSlow
Saviqcwayne, ok cool13:01
Saviqtsdgeos, doing :)13:02
=== gatox is now known as gatox_brb
karnialecu: Saviq: on the desktop, I see scope screenshots instead of icons in the Scopes scope. you guys know anything about that?13:07
Saviqkarni, scope screenshots?13:07
karnilike, instead of grooveshark icon there's like.. grooveshark scope screenshot in the icon place13:07
Saviqkarni, weeeird13:08
karniwill share a screenshot13:08
karniSaviq: http://imgur.com/6BibDcp13:09
karnithat's new-scopes ./run on the desktop13:09
karniwhich is not what we want.. and this looks like a "feature" to me13:10
Saviqkarni, indeed, I think this was meant to be in previews13:10
karnialecu: ↑13:10
Saviqkarni, confirmed here13:10
karniralsina_: ↑ see the screenshot13:11
Saviqthostr_, mhr3, can you comment on http://imgur.com/6BibDcp ?13:11
karniralsina_: scope screenshots were not supposed to replace the icons in scopes scope cards, were they?13:11
ralsina_karni: honestly, I don't know anymore13:11
karniralsina_: ack, we'll see that thostr_ says13:12
mhr3Saviq, definitely not13:12
mhr3marcustomlinson, ^^ any idea what's up with that?13:12
ralsina_karni: scopes scope is thostr_'s, yes. I have a branch for screenshots, and a branch for icon, so I can go either way ;-)13:12
mhr3Saviq, what is that anyway?13:12
marcustomlinsonmhr3, Saviq: mwc-build?13:12
mhr3Saviq, doesn't seem too recent13:12
Saviqmhr3, very recent13:12
Saviqmhr3, current13:12
karniThis looks bit better: http://imgur.com/BhEOSlj13:13
Saviqmhr3, on desktop13:13
karnimhr3: new-scopes on desktop13:13
mhr3Saviq, can't be current, current has grey icons for non-server scopes13:13
Saviqmhr3, 0.1+14.04.20140220-0ubuntu113:13
mhr3Saviq, libunity-scopes0?13:13
karni0.3.2+14.04.20140220.1-0ubuntu113:14
Saviq0.3.2+14.04.20140220.1-0ubuntu113:14
ralsina_karni: if it's worth anything, the visual design has icons, not screenshots https://drive.google.com/a/canonical.com/?tab=co#folders/0B-a_7E3tDxOgTTBybG1TZG9GWnM (00_scopes.png)13:14
karniralsina_: +1 yes, thank you13:14
karniralsina_: that's where my question came from, actually :) should have shared the spec as well13:14
ralsina_I got a request (from marcustomlinson?) and a branch from alecu to use screenshots and a branch from mhr3 to use icons. So... someone decide quick ;-)13:15
Saviqralsina_, icons13:16
ralsina_ok, cool13:16
marcustomlinsonralsina_: no, no. seperate concerns.13:16
mhr3marcustomlinson, where's the cache again?13:16
marcustomlinsonralsina_: icons AND screenschots needed13:17
Saviqmarcustomlinson, screenshots for preview, I assume?13:17
marcustomlinsonSaviq: yes13:17
marcustomlinsonralsina_: this branch adds the screenshot image we need: https://code.launchpad.net/~alecu/unity-scope-click/add-screenshot13:18
mhr3ralsina_, there's nothing to decide, the branches are fine, we'll deal with this13:18
marcustomlinsonralsina_: mhr3's request for icon is a seperate request13:18
mhr3ralsina_, not your issue13:18
ralsina_cool then13:19
ralsina_I'll land both then13:19
mhr3thx13:19
mhr3marcustomlinson, i can't find the cached json, where is it supposed to be?13:19
marcustomlinsonmarcustomlinson: the remote scopes cache?13:19
mhr3yes13:19
marcustomlinsonmhr3: ~/.cache/unity-scopes/remote-scopes.json13:20
mhr3Saviq, rm ^ && restart smart-scopes-proxy13:20
mhr3&& restart scope-registry13:20
mhr3and try again13:20
Saviqmhr3, oof!13:21
Saviqkarni, ↑↑13:21
karnihehe13:21
karniindeed, fixed :)13:21
mhr3good :)13:21
karniphew13:21
mhr3and chillax now13:22
karnihehe13:25
karniSaviq: still looking for that spacing13:25
karnibefore the category header13:25
* karni would LOVE boundingbox hint dev-feature right now13:25
karniI did raise this at the last sprint ;)13:25
Saviqkarni, enable qml debugging and connect from qtc13:26
karni:D13:26
Saviqcwayne, in case the 21→18px change does not land in trusty (there's a lock on landings due to some regression)13:32
Saviqcwayne, would you want to include it in mwc image?13:32
cwayneSaviq, yeah13:33
cwaynewe could just carry it in the ppa13:33
Saviqcwayne, let me bin-copy it already13:33
Saviqcwayne, no other changes matter for mwc13:34
=== karni is now known as karni-reboot
=== karni-reboot is now known as karni
=== alan_g|lunch is now known as alan_g
FunnyLookinHatAnyone else on Trusty notice that the HUD misses matching in some use cases?  i.e. with Sublime Text I can type "replac" and get "Replace... Find" - but if I type "replace" it won't find it.13:51
=== Pici` is now known as Pici
=== Trevinho_ is now known as Trevinho
Saviqcwayne, flo@18px is in demo-stuff ppa14:03
Saviqcwayne, and it's looking great, too :)14:03
cwayneSaviq, :)14:04
dandradergreyback, did you merge that application.visible branch?14:05
greybackdandrader: no, sorry, I forgot14:06
greybackdandrader: I can do it now14:07
dandradergreyback, ok, thanks!14:07
=== larsu_ is now known as larsu
Saviqtsdgeos, +1, merging into new-scopes14:08
tsdgeosSaviq: ok, will do in a bit14:10
Saviqtsdgeos, I did already14:10
tsdgeosah14:10
tsdgeoscool :D14:10
greybackdandrader: ok pushed14:11
dandradergreyback, ack14:11
=== gatox_brb is now known as gatox
karnivideo preview on N10, N7 - no title & button -- known issue?14:13
SaviqAAAARRGGH LP ;(14:14
cwaynekarni, scroll down14:14
karnicwayne: 1) I don't have a tablet with utouch xD 2) Joe reported that problem. is the title/button below the bottom edge of the screen?14:14
cwayneyes14:15
cwayneand i've already emailed them saying the same thing14:15
cwayneWHY DONT PEOPLE READ THEIR GODDAMN EMAILS14:15
karnicwayne: hrm.14:15
Saviqcwayne, karni, any particular video preview? the local videos one is in columns now14:15
Saviqand I can see both14:16
karninope. but I assume cwayne understands the "issue" I just proxied from Joe14:16
Saviqyup14:16
karni(feature, not a bug? I've no idea how that looks :/ )14:16
cwayneits fixed in later updates i think karni14:16
karnicwayne: oh great, that's one item off the TODO list14:17
karnithanks cwayne14:17
mzanetticwayne: hey, mwc-flash broke my flo14:17
mzanetticwayne: now it's not booting any more :)14:17
cwaynemzanetti, how so?14:17
mzanetticwayne: and because phablet-flash doesnt support -d flo I'm somewhat stuck14:17
cwaynemzanetti, press the power button for literally like a full minute14:17
mzanetticwayne: it just turns on and off and on and off...14:19
cwaynemzanetti, can you boot into fastboot?14:20
mzanettinow it doesn't even turn on any more :D14:21
cwayneeven when holding the power button for a minute?14:21
mzanetticwayne: "fastboot reboot recovery" just says "waiting for device"14:23
mzanettiwhile the screen shows the google text14:23
cwaynemzanetti, try powerinf it off, then power it on while holding volume up14:25
mzanettigot a broken andoid bot now saying "No command"14:26
cwaynemzanetti, oh , do the same thing but with holding both volume buttons14:27
mzanettithat's what I tried before, when it didn't turn on any more at all. let me try again14:27
mzanettisame again14:28
MacSlowSaviq, greyback, mzanetti: mumble is crapping out here... can't connect14:28
mzanettiMacSlow: works here14:29
cwaynemzanetti, even when powering on holding both volume buttons down?14:29
MacSlowmzanetti, it's on my side... the mumble-app just hangs (greyed out)14:29
mzanetticwayne: yeah... it doesn't do anything... just turns black and I need to press the power button for 20 secs to make it do anything at all again14:30
karniSaviq: still no news, but the spacing is within CardFilterGrid. it's outside of cards, but within inner DashFilterGrid14:30
Saviqkarni, yeah, I had a quick look and couldn't find anything obvious14:30
cwaynemzanetti, press and hold all of them for a full minute14:31
mzanettiit starts hurting my fingers :D14:31
karnithen throw it out of the window14:31
mzanettiI might14:32
karniYou guys cancelled SU? I was hoping to listen on, and everyone just left :D14:38
Saviqkarni, apparently you didn't hear us ;)14:39
Saviqmterry, so it seems fine on desktop, but on device, with:14:39
karnifancy ;D14:39
Saviqmterry, http://paste.ubuntu.com/6971181/14:39
Saviqmterry, can't drag launcher in at all14:39
Saviqmterry, if removed that file, it's fine14:39
* MacSlow would like to know why mumble completely failed for him today14:40
mterrySaviq, so even on desktop with that file it works?  But on device with the file it doesn't?  huh14:40
Saviqmterry, indeed14:42
Saviqmterry, HUH, and now it works14:43
mterry:-/14:43
Saviqmterry, ignore14:43
Saviqmterry, must've been some intermittent issue, or maybe just got fixed with your stuff yesterday somehow14:43
* Saviq mwc-flashes from scratch14:43
mterrySaviq, if it comes up again, let me know.  Maybe it's a sequence of flipping between users and the greeter gets confused about whether it is draggable or not?14:44
Saviqmterry, maybe, but I couldn't get it in at all before, and now it behaves correctlyt14:44
mhr3Saviq, could we vertically center the preview widgets for tablet?14:44
Saviqmterry, so yeah, will ping14:44
mhr3Saviq, or well, for columns != 114:44
mhr3Saviq, also, images tiny on tablet14:45
Saviqmhr3, not sure that'd work that great14:45
Saviqmhr3, what do I look at?14:45
mhr3Saviq, for example scopes previews14:45
tsdgeosSaviq: the tabs are buggy, they randomly fail to render, i've tried to workaround it, but every workaround has its defect, so never got it to work 100%14:45
Saviqtsdgeos, yeah14:45
tsdgeosSaviq: someone said we wanted to kill tabs in there?14:45
Saviqmhr3, right, they're height-bound, but I'm not sure we can do better for gallery14:46
tsdgeosmaybe i can quickly do that if that's a real "want"14:46
Saviqmhr3, for image we might just do width-bound instead of height-bound14:46
Saviqtsdgeos, we don't know what we want, so not yet14:46
tsdgeosok14:46
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/new-scopes-dash-preview-fullscreen/+merge/207655 is for the fullscreen thing14:46
Saviqtsdgeos, cool14:47
tsdgeosnice thing is it kills quite a bit of code14:47
mhr3Saviq, gallery is also tiny14:47
tsdgeoswhaaat, i lost the header :S14:48
tsdgeosok, fixed14:51
karniSaviq: it's not a fix, but I found the source of the problem. this fixes the wrong spacing within DashFilterGrid (below delegates, but still inside the component) http://paste.ubuntu.com/6971222/14:52
davidcalletsdgeos or Saviq, the preview Art as moved to the center in a recent commit, or Dash is acting up?14:52
Saviqmhr3, maaan, you're sending full-screen images for scopes, like how the heck am I supposed to know what size to make those ;/14:52
karniSaviq: like.. childrenRect.height was incorrect14:52
Saviqkarni, yeah, I've learned that's the case14:53
Saviqdavidcalle, yes, mhr3 +1'd14:53
mhr3Saviq, think marcustomlinson was doing something about that?14:53
tsdgeosdavidcalle: i think Saviq changed something regarding the preview image, yes14:53
karniSaviq: I guess I doubt I can dig anything up further :S14:53
karniWhat do we do about that?14:53
mhr3davidcalle, we found it prettier :)14:53
Saviqkarni, we need to replace childrenRect with some other, real data14:54
mhr3davidcalle, better than cutoff images anyway14:54
davidcallenp !14:54
* karni scratches on the head14:54
davidcalleBy the way, can I expect this to be fixed today, or should I work around it? https://ubuntuone.com/73xYf9S6c1yB1bKayql1Pf14:54
karniwow14:54
Saviqkarni, like iconTileGrid.height basically14:55
tsdgeosSaviq: is the image height wrongly calculated in davidcalle's screenshot?14:55
mhr3Saviq, is there some known mir issue with manta?14:56
mhr3Saviq, animations are weird in some cases there14:56
Saviqmhr3, you mean that it's real jerky?14:56
Saviqtsdgeos, davidcalle, yeah, that looks wrong14:56
mhr3Saviq, sometimes it's ok, other times really jerky14:56
karniSaviq: replaced childrenRect.height with iconTileGrid.height, no worky.14:56
tsdgeosimplicitHeight: units.gu(22) ¿?14:56
marcustomlinsonSaviq: full screen images for "scopes" scope preview images you mean?14:56
Saviqtsdgeos, yeah, but the LazyImage grows14:57
Saviqmarcustomlinson, yes14:57
tsdgeosSaviq: sure, and what does that mean? implicitHeight will keep being 22gu14:57
Saviqtsdgeos, yeah, and the LazyImage grows out of that for some reason14:57
marcustomlinsonSaviq: It seems to be working. whats the issue. I have updated the filesizes to smaller jpgs but resolution still the same14:58
tsdgeosSaviq: ah14:58
Saviqmarcustomlinson, yeah, I'm tweaking it and saw that it was quite big, but then it'd not be that big on devices14:58
marcustomlinsonSaviq: when I say "update" I mean for the next SSS rollout14:58
Saviqmarcustomlinson, k14:58
tsdgeosdavidcalle: Saviq: i'll have a look14:59
marcustomlinsonSaviq: file sizes are around 70k now14:59
Saviqtsdgeos, ok, if you can address mhr3's complaints about images being small on tablet14:59
Saviqtsdgeos, although I'm not sure what to do14:59
marcustomlinsonSaviq: should probably make them smaller still. what res is best?14:59
Saviqmarcustomlinson, problem is we don't have a "best", because there's a different best per device15:00
marcustomlinsonSaviq: yeah sure.15:00
mterryDoes anyone know a good way to fake an incoming text message?  I'm stuck with either actually texting it or some complicated thing with sending javascript to the phonesim daemon15:00
Saviqmarcustomlinson, so we need to deal with all of them15:00
Saviqmterry, try pitti15:00
karnimterry: Apps team has tools. ping tiagosh or alfonso maybe15:00
MacSlowSaviq, fyi... my branch just passed CI... https://code.launchpad.net/~macslow/unity8/snap-decisions-states/+merge/195394/comments/48736315:03
SaviqMacSlow, yay15:04
MacSlowthis is really a goo day... I also finally received my trashcans from the city... after applying for them 2.5 months ago15:07
Saviqkarni, 'fraid I don't have a ready-made answer15:07
Saviqgoo day15:07
karniSaviq: found something interesting15:07
Saviq;)15:07
MacSlowSaviq, still getting used to the new keyboard ;)15:08
Saviqtsdgeos, we're missing a clip: somewhere, the preview header shows over the scope header for a moment15:10
tsdgeosSaviq: does it? damn15:10
tsdgeoswhen?15:10
tsdgeoson opening?15:10
=== charles_ is now known as charles
karniSaviq: http://imgur.com/olJ1LcQ http://paste.ubuntu.com/6971325/15:12
karniSaviq: totalContentHeight was calculated with rows * cellHeight + verticalSpacing, while each of the cells already has verticalSpacing below its content15:13
karniSaviq: I don't know if this has been changed recently, but this change does get rid of that spacing you pointed out15:13
mhr3_Saviq, what's the quickest way to change unity8's qml and see it on the device?15:13
Saviqkarni, looks better15:13
Saviqmhr3_, patch -p1, restart unity815:14
mhr3_k, thx15:14
Saviqmhr3_ /usr/share/unity815:14
mhr3_yea, i was doing reboot always15:14
karniSaviq: shoudl I also vertical-center align cell content? (the Cards, basically)15:14
mhr3_restart should be better :)15:14
karniactually, I'll look at the visuals15:14
Saviqmhr3_ ;)15:14
Saviqkarni, or we should actually lose the space between last row and header15:15
mhr3_Saviq, i'll sit with jouni and try to do something about the font size thing15:15
Saviqmhr3_, good, thanks15:15
Saviqmhr3_ pull from new-scopes, though, I tweaked some things15:15
karniSaviq: yeah, v-align or loose last spacing15:15
Saviqmhr3_, or I can get you packages15:15
tsdgeosSaviq: hmmmm, maybe i can exclude the header from the open effect15:16
tsdgeosi think it makes more sense if it's out15:16
mhr3_Saviq, yea, trigger build pls15:16
tsdgeossince it's there anyway15:16
* karni imagins Saviq's brain must be frying right now15:16
Saviqtsdgeos, you mean that header would remain? and cross-fade?15:16
tsdgeosSaviq: something like that, what you think?15:16
Saviqtsdgeos, truth is I think the preview should come from the right, as temp scopes do...15:16
* Saviq checks the proto15:16
mhr3_karni, what did you change yesterday when you were trying the x-small?15:16
tsdgeosSaviq: true, probably the open effect does not make much sense now that we don't have bottom gap15:17
karnimhr3_: CardHeader.qml titleLabel and subtitleLabel fontSize, Card.qml summary fontSize15:17
Saviqtsdgeos, exactly15:17
karnimhr3_: qml/Dash/*15:17
Saviqkarni, that basically starts to suggest we should *actually* use the spacing from FilterGrid15:18
mhr3_karni, k, just wanted to make sure15:18
Saviqkarni, instead of including it in the tiles themselves15:18
tsdgeosSaviq: so come from left like the temp scopes? what does the proto say?15:18
karniSaviq: let me look into that15:18
tsdgeosthere's no previews in the proto?15:18
Saviqtsdgeos, no :/15:19
tsdgeosarg15:19
Saviqtsdgeos, actually there is15:19
karnimhr3_: you heard any news on the fonts? (i.e. Grooveshar\nk)15:19
Saviqtsdgeos, but just in-place replace15:19
Saviqkarni, he just went to sit with jounih15:19
Saviqkarni, about that15:19
karni\o/ !15:19
karnifantastic15:19
Saviqtsdgeos, but yeah, go for it15:20
tsdgeosSaviq: ok, let's see how hard it is ^_^15:20
Saviqtsdgeos, yeah, that...15:20
tsdgeosSaviq: at least i found a bug in lazyImage, there's a .scale that should be .scaleTo, but doesn't fix the problem15:20
Saviqtsdgeos, mhm15:20
Saviqmhr3_, build triggered15:21
mhr3_thx15:21
karniSaviq: GridView (used by ResponsiveGridView) has no concept of spacing. ResponsiveGridView (used by FilterGrid) passes verticalSpacing, that is then used to enlarge GridView cell height artificially. so, re: what you wrote above, there's no build in vertical cell spacing, that would not leave additional spacing at the end. we basically have to do it on our own anyway. so either center align Cards within cells, or make last grid raw smaller in ...15:26
karni... height.15:26
karniwe have no other way than to include spacing in tiles themselve.15:26
karnithe problem was that, for unknown reason to me, totalContentHeight was calculated as rowCount * cellHeight + veritcalSpacing (what? why add that here?)15:27
karnito me, that was a bug right there, to start with.15:27
paulliuhi.. http://paste.ubuntu.com/6971412/  I need some help on 50~66. Please see if I'm doing good on the touchEvent.15:27
paulliuI can see the press event in the zoomableImage object. But it doesn't trigger a pinch.15:28
Saviqah right, there's no internal spacing in GridView, forgot about that15:31
Saviqkarni, last row smaller height won't help15:31
Saviqkarni, 'cause GridView assumes... a grid15:31
SaviqIIRC15:32
karnieh15:32
Saviqkarni, yeah, static cell dimensions15:32
karniSaviq: if we v-center align, that'd at least make the last spacing smaller. you know, spread.15:32
Saviqkarni, yeah, probably best we can do right now15:32
Saviqkarni, or wait15:33
karniack15:33
karniyea?15:33
Saviqkarni, we can - verticalSpacing15:33
Saviqkarni, from the height15:33
karniyes15:34
karnigotcha15:34
Saviqkarni, that should give us what we need15:34
Saviqthere's never going to be anything in there anyway15:34
karniSaviq: wait, wasn't it what I pasted ;D?15:34
karnioh, like once more15:34
karniright15:34
tsdgeosSaviq: preview comes from right pushed15:35
karnihehe, sneaky.15:35
Saviqtsdgeos, yay15:35
Saviqtsdgeos, works with temp scopes, too?15:36
tsdgeosyep15:36
tsdgeosSaviq: tried on desktop wit15:36
tsdgeos UNITY_SCOPES_LIST=scopes ./run15:36
tsdgeosthen go to amazon scope15:36
tsdgeosand preview a book15:36
mhr3_Saviq, right edge is dead jim!15:36
* tsdgeos hides15:36
Saviqmhr3_, ?15:37
Saviqtsdgeos, no, that's good15:37
tsdgeosmzanetti: when/what/how?15:37
mhr3_ * Build Log: https://launchpadlibrarian.net/167187829/buildlog.txt.gz15:37
mhr3_Saviq, ^15:37
tsdgeosSaviq: i was hiding from mhr3_ :D15:37
mzanettitsdgeos: ?15:37
tsdgeosmzanetti: damn sorry that was for mhr3_15:37
mzanetti:)15:37
tsdgeosmhr3_: when/what/how?15:37
Saviqmhr3_, I'll fix15:37
tsdgeosgood15:37
Saviqtsdgeos, please try to break it15:38
Saviqtsdgeos, I got preview overlaid on dash page15:38
mhr3_tsdgeos, don't hide from me! :)15:38
Saviqtsdgeos, probably some race15:38
Saviqtsdgeos, i.e. preview came after I went back or something15:38
Saviqtsdgeos, yeah, not working well15:39
tsdgeosSaviq: hmmm know what you did? can't seem to break it here15:39
mhr3_tsdgeos, i gave you separate preview objects, so i'm gonna say shell issue :)15:39
Saviqaargh not again15:39
=== dandrader is now known as dandrader|afk
Saviqtsdgeos, preview in scopes, then clicked in scopes15:40
* Saviq tries again15:40
Saviqmzanetti, can you resolve new-scopes → right-edge-new-scopes-2 please?15:41
mzanettiak15:41
mzanetti+c15:41
Saviqtsdgeos, long-press for preview in scopes, go back, click on scope15:41
tsdgeosok15:41
* tsdgeos tries15:41
mhr3_does anyone else still get a crash with scope-tool when running it without specifying scopes_list?15:42
mhr3_cause i just did15:42
karninope15:42
Saviqmhr3_, nope, works fine15:43
mhr3_hm, can't repro15:44
mhr3_sucks15:44
Saviqtsdgeos, I think what's happening is that one of the other previews in PreviewListView gets in15:44
tsdgeosSaviq: still can't repro :_S is this phone or desktop?15:44
Saviqtsdgeos, desktop15:44
karniSaviq: what I don't understand is.. current mwc image looks fine with scopes scope. that change we were looking into with spacing - it looked real bad when I pushed the single file to the phone. so I'm attempting run_on_device now from new_scopes branch. But that means something must have changed like.. since yesterday? and I doubt anyone touched the Components/ResponsiveGridView.qml since yesterday.15:44
karniactually, I could have checked that15:45
Saviqkarni, well, maybe it's just unreliable15:45
Saviqkarni, I mean childrenRect15:45
Saviqkarni, on phone unity8 is slightly old15:46
Saviqkarni, mzanetti is resolving a conflict15:46
karniyeah, maybe. works on the phone, not so good on the desktop.15:46
mzanettitrying at least15:46
karniack15:46
mzanettinot really sure what happened15:46
Saviqmzanetti, try rev-by-rev15:46
Saviqmzanetti, helped me a lot yesterday15:46
tsdgeosmhr3_: any idea why i get the "amazon" scope in the dash? i should get only in the "Scopes scope", no?15:46
Saviqkarni, remember you won't be able to run_on_device new-scopes on phone, as it uses the right-edge branch15:46
karniI just run_on_device. even removing the + verticalSpacing makes it look worse (the card literally sits on top edge of next category header)15:46
Saviqtsdgeos, if you export UNITY_SCOPES_LIST15:47
karnioh15:47
karnithat actually restarted unity only heh15:47
* karni noticed Connection to 127.0.0.1 closed15:47
mzanettiSaviq: is there any bzr foo to do that or only manually?15:47
tsdgeosSaviq: no, i mean without exporting anything, i get all the scopes in the dash, not just 3 or 4 like in the tablet15:47
Saviqkarni, well, maybe you're right15:47
mhr3_tsdgeos, did you set the SCOPES_LIST?15:47
Saviqkarni, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgQ002UmQxbjJmMHc/edit has quite some distance15:47
tsdgeosmhr3_: Saviq: do i have to?15:48
Saviq(apart from see more / see less )15:48
tsdgeosi thought it wasn't needed15:48
Saviqtsdgeos, yes it is15:48
mhr3_tsdgeos, it is15:48
tsdgeosoh15:48
Saviqtsdgeos, for now it loads everything15:48
karnithat see more | less has beefy margins around it15:48
mhr3_tsdgeos, you get them all if you don't specify it15:48
Saviqkarni, so maybe we shouldn't fix that at all...15:48
karnior that, yeah15:48
Saviqkarni, you could get the shadow from jounih, though15:49
karniSaviq: all in all, it does look larger on the desktop, but looks just fine on the phone15:49
Saviqkarni, maybe that would helpvisually15:49
Saviqkarni, what I'm afraid of is that it doesn't look the same on phone and tablet...15:49
karniSaviq: for the end of the category? xD I'm blind15:49
tsdgeosSaviq: can you tell me the exact export you use to see if that helps reproducing the problem you see?15:49
Saviqkarni, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgQ002UmQxbjJmMHc/edit15:49
karniSaviq: uhm :S15:49
Saviqkarni, there's a slight shadow at category end15:49
tsdgeosSaviq: also which scope are you previeweing+clicking?15:50
karniSaviq: right, ack15:50
Saviqtsdgeos, GRID_UNIT_PX=11 UNITY_SCOPES_LIST="scopes;clickscope" ./run15:50
tsdgeosah15:51
tsdgeosgot it now15:51
mzanettiSaviq: but actually I can discard side stage fixes in here15:52
Saviqmzanetti, yes15:52
Saviqmzanetti, that's the whole conflict indeed15:53
karnimhr3_: please let me know if there's news on the font size/scopes scope labels front, appreciated15:53
mhr3_karni, waiting for arm packages in demo ppa15:53
mzanettiI guess we can keep the unfocusApplication in Component.onComplete15:53
Saviqtsdgeos, I think what we need is to unset the model on preview close15:53
Saviqtsdgeos, and/or clip: true15:53
karnimhr3_: you decided on changes with Jouni?15:54
mhr3_not yet15:54
karniack15:54
Saviqkgunn, you awfully quiet today? ;D15:55
* Saviq brb15:56
kgunnSaviq: hey man...that's cause you got it handled :)15:56
kgunnaltho..saw mhr3 saw right edge is dead jim15:56
kgunnmildly worrying15:56
karnimhr3_: If you're in the office, could you also ask Jouni for me, for the slight shade assert at the bottom of each category?15:56
karnimhr3_: as seen here https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgQ002UmQxbjJmMHc/edit15:56
karnimhr3_: if you're not near, I'll e-mail him instead15:56
mzanettikgunn: what's with the right edge?15:57
mhr3_karni, ah, asset :)15:57
karniyes :)15:57
mhr3_karni, had trouble parsing what you want :)15:57
karnimhr3_: the shadeeeez :D15:57
Saviqkgunn, he meant conflicting branch ;_15:58
kgunnmzanetti:  ^15:58
kgunncool...simple fix15:58
mzanettiah... yeah, on it. no need to worry kgunn15:58
Saviqkgunn, if you're free, can you find out whether landing is resumed?15:59
mzanettithe conflict is everything but simple, but I can discard all the side stage fixes in the branch, which eases it up15:59
kgunnkgunn: yeah...working on landing unity-mir stuff now...16:00
Saviqkgunn, talkin' to someone smart, too? ;)16:00
Saviqmzanetti, yeah, had the same yesterday, but once I looked at the actual diff16:00
mhr3_karni, you'll get it soon16:00
Saviqmzanetti, also went step-by-step (merge -r123)16:00
karnimhr3_: thanks!!16:01
ralsina_Saviq, kgunn: if landings don't unblock, we'll need to land the contents of silo landing-001 in the demo PPA.  Just a heads-up so it doesn't get dropped before you guys EOW16:03
ralsina_OTOH, sil2100 just greenlighted landings, so nevermind :-)16:03
Saviqralsina_, just landed ;)16:03
Saviqralsina_, or well, published16:03
ralsina_yeah16:03
Saviqmzanetti, please let me know when you push right-edge resolved (or kick https://code.launchpad.net/~unity-team/+recipe/u8-phone-right-edge yourself)16:05
mzanettiyep, I will16:05
=== dandrader|afk is now known as dandrader
tsdgeosSaviq: should work now, can you give a try?16:20
Saviqtsdgeos, yup16:20
mzanettidiverged branches???16:21
mzanettinoooooooo16:21
Saviquh oh16:21
mzanettieasy one16:22
mzanettiSaviq: merged, pushed and kicked recipe16:23
Saviqmzanetti, thanks16:24
mzanettiSaviq: what's the ultimate deadline btw?16:25
Saviqmzanetti, I don't know of such a thing ;)16:25
mzanettiwell, I guess at some point people will flash the image and leave for Barcelona16:25
mzanettiI hope there's such a thing. not going to merge 2 complete different Stages codebases much longer :D16:26
karniI heard they wanted folks continue on unity stability, so that they could even reflash on monday16:26
Saviqsil2100, silo 16 tested → true16:43
sil2100\o/16:44
sil2100Thanks16:44
Saviqsil2100, thanks to you16:49
mzanettiSaviq: greyback: so this should fix the aspect ratio on the n7 too: https://code.launchpad.net/~mzanetti/unity-mir/appimage-sourcesize-rotate-n7/+merge/20770516:51
mzanettiSaviq: still need to clip it to a square in unity816:51
Saviqmzanetti, great, thanks16:51
greybackmzanetti: nice, thanks16:52
Saviqmzanetti, that's bonus16:52
Saviqmzanetti, but in theory should be easily doable16:52
mzanettiok. eating something and then trying to play the bonus level16:52
Saviqwidth: Math.min(width * aspect, width) ?16:52
mzanettiyeah. something like this16:52
mhr3_Saviq, ok, got the desired sizes of things16:55
mhr3_Saviq, but one problem16:55
Saviqmhr3_, just one?16:55
mzanettibtw. I know this above patch would break on 180° rotation. But again, this is subject to go away when the right edge merges16:55
* karni listens to mhr3_ 16:55
mhr3_Saviq, title size in preview header is different than in card header16:55
Saviqmhr3_, not only that16:55
karniand preview uses CardHeader, right?16:55
Saviqmhr3_, mascot size is different16:55
mhr3_karni, you assumed the current default size is "small", but it's actually "medium"16:55
karnimhr3_: I assumed nothing16:55
mhr3_Saviq, yea, mascot isn't going to be according to designs16:56
mhr3_but that's fine16:56
mhr3_for now16:56
mhr3_karni, should have tried "small" then ;)16:56
Saviqaaand I quote "Header uses the same widget in the preview as in the cards"16:56
Saviqhttps://docs.google.com/a/canonical.com/document/d/1n880Fih5KyGPcoP5chidnHDG_8TxXUgSuij7f4rHpuk/edit?pli=1#16:56
mhr3_Saviq, spec is wrong16:57
Saviqmhr3_, they should read it16:57
karnimhr3_: did my best, sorry :(16:57
mhr3_Saviq, it's *almost* the same16:57
mhr3_:)16:57
Saviqmhr3_, know what time it is, btw? ;)16:57
mhr3_Saviq, ah, yea a bit late for you16:59
Saviqmhr3_, a bit late for MWC, I'd say..16:59
mhr3_Saviq, quick hangout?16:59
Saviqmhr3_, _now_ you want me to put on pants17:00
Saviqpfft!17:00
mhr3_Saviq, no need, just point the camera af your face17:00
mhr3_:)17:00
mhr3_at*17:00
Saviqmhr3_, set it up17:00
mhr3_Saviq, https://plus.google.com/hangouts/_/calendar/bWljaGFsLmhydWJ5QGNhbm9uaWNhbC5jb20.kbnj1tr5gsi5q0ju23rlb0nm6g?authuser=117:01
=== jhodapp_ is now known as jhodapp
mhr3_Saviq, sent17:14
mhr3_Saviq, feel free to use me more for this, it's much more effective when the designers see stuff "live" then exchanging emails back and forth17:15
mhr3_and i might learn more qml along the way :)17:15
karniSaviq: would you allow fancy "shadow" at bottom of last category (at the bottom)? it does sit on the bottom of the screen anyway when you scroll to the bottom.17:16
Saviqkarni, I think it's supposed to be there, yeah17:17
karniSaviq: it's not here, but I guess we're in a bit of rush, and it looks nice anyway. https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgQ002UmQxbjJmMHc/edit17:18
karnihttp://imgur.com/5DaM8UG17:18
Saviqmhr3_, the change to font.weight looks wrong (useless), as there's an alias to that prop higher up17:19
Saviqmhr3_,     property alias titleWeight: titleLabel.font.weight17:19
karniif I showed you it scrolled to the bottom, you wouldn't even notice the shadow there at the bottom. so, I think it's okay for now17:19
Saviqmhr3_, which then is17:19
SaviqtitleWeight: components && components["subtitle"] ? Font.DemiBold : Font.Normal17:19
Saviqin Card.qml17:20
Saviqmhr3_, http://paste.ubuntu.com/6971946/17:21
Saviqmhr3_, although I think that only makes sense when over background and no art17:22
mhr3_Saviq, it should never be demibold17:22
Saviqmhr3_, WHAT17:23
mhr3_oh wait17:23
Saviqhttps://docs.google.com/a/canonical.com/document/d/1n880Fih5KyGPcoP5chidnHDG_8TxXUgSuij7f4rHpuk/edit?pli=1#heading=h.5uwk9y3j0hso17:23
mhr3_Saviq, ok, nevermind17:23
karniit should be bold when subtitle is present17:24
karnic'mon :)17:24
mhr3_Saviq, somehow it does work with just the change i made though17:24
Saviqmhr3_, sure it does, 'cause it's overwritten anyway ;)17:24
mhr3_probably :)17:24
Saviqmhr3_, so basically you didn't change anything :)17:24
Saviqmhr3_, ok my diff for centering seems to actually work fine17:24
mhr3_oh, is that what it does?17:24
mhr3_:)17:25
mhr3_let me try to apply17:25
karniSaviq: I have the shade dividing categories ready. which branch should I base this on? I have changes against trunk.17:25
Saviqkarni, MP against trunk, merge into new-scopes17:25
karniack17:25
mzanettiSaviq: ok. got cropping. now, where should those 2 branches be merged to?17:25
Saviqmzanetti, new-scopes and rotate-n717:26
Saviqunity8 and unity-mir, respectively17:26
Saviqmzanetti, as I expect this to not apply to unity8 right edge (and that's fine)17:26
karnif, again I got something wrong with MP. /me retries17:26
mzanettiSaviq: yeah, need to do redo the crop to square there too, but that's no biggie17:27
mzanettiSaviq: ok. here they are: https://code.launchpad.net/~mzanetti/unity8/keep-appImage-aspect-ratio/+merge/207714 and https://code.launchpad.net/~mzanetti/unity-mir/appimage-sourcesize-rotate-n7/+merge/20770517:28
Saviqmhr3_, title → medium in preview?17:28
Saviqmzanetti, thanks, will test soon and merge17:28
mhr3_Saviq, large17:29
Saviqmhr3_, http://paste.ubuntu.com/6972008/17:29
mhr3_Saviq, design +1 on the centering17:31
mhr3_Saviq, still doable to lower the margin between the two?17:34
Saviqmhr3_, you mean title and subtitle? http://paste.ubuntu.com/6972023/17:34
mhr3_Saviq, yep17:35
karniSaviq: https://code.launchpad.net/~unity-team/unity8/unity8-category-gradients/+merge/20771817:35
Saviqkarni, make it 1px wide and stretch instead of tile17:36
Saviqkarni, cheaper on GPU17:36
karniack17:36
Saviqkarni, there's actually a top gradient, too? ;D17:37
karniSaviq: don't ask me. I can't see sh!t ;)17:37
karnineither will you hahaha17:37
Saviqkarni, make sure to add @ to the file names at the GU those were prepared for17:37
karniyes, they're there already17:37
Saviqkarni, 'qml/Dash/graphics/dash_divider_top_darkgrad.png'17:37
Saviqkarni, doesn't look like it17:38
karniwuuut17:38
Saviqkarni, no need to use Qt.resolvedUrl()17:38
karniSaviq: sorry, I'll add it. must have removed it :/17:38
karniack17:38
Saviqkarni, it only matters when you pass it outside of the current file17:38
Saviqkarni, as they're by default resolved against current file17:38
Saviqmhr3_, I'm afraid we need to take back the "fit" for image in previews, it gets confused with some images17:40
Saviqmhr3_, will need to fix LazyImage17:40
mhr3_owww :/17:41
mhr3_but yea, do what you have to17:41
mhr3_Saviq, btw the spinner is too eager, it's displayed before the result view is split17:42
Saviqmhr3_, not any more17:42
Saviqmhr3_, pull17:42
Saviqmhr3_, previews come from the right now17:42
mhr3_oh, ok that's on the device17:42
Saviqmhr3_, made more sense to be the same as openScope17:42
Saviqmhr3_, the split view really made sense for when we had the bottom part see-through17:43
=== dbarth__ is now known as dbarth
mhr3_Saviq, iirc last discussions with design were saying to make it go from bottom :)17:43
SaviqLOL17:43
Saviqfrom space17:43
mhr3_although maybe that was just the hierarchy of things...17:43
Saviqmhr3_, so you push the tweaks to your branch still?17:51
Saviq(that I sent the diffs up)17:51
mhr3_Saviq, i can17:51
mhr3_Saviq, got the margin too?17:52
mhr3_don't see that there17:52
Saviq-            spacing: units.gu(0.5)17:52
Saviq+            spacing: units.gu(0.25)17:52
mhr3_ah, didn't click on the last one :)17:53
mhr3_Saviq, pushed17:57
Saviqmhr3_, so looks like all the issues from your email are done?17:57
mhr3_Saviq, indeed \o/17:57
Saviqmhr3_, cool17:57
karniSaviq: sourceSize.width: 1 makes stuff no worky17:58
karniotherwise, I'm set17:58
Saviqkarni, no need to set that17:58
Saviqkarni, if you know the asset17:59
karniSaviq: updated https://code.launchpad.net/~unity-team/unity8/unity8-category-gradients/+merge/20771817:59
Saviqkarni, o/17:59
mhr3_Saviq, should i mp against new-scopes?17:59
karniSaviq: yeah, I just didn't feel like making the asset 1px, I'm that bad at touching graphics ;P17:59
Saviqmhr3_, nah18:00
Saviqkarni, how wide is it?18:00
=== alan_g is now known as alan_g|EOW
karniSaviq: dash_divider_top_darkgrad@27.png: PNG image data, 27 x 134, 8-bit/color RGBA, non-interlaced18:00
Saviqmhr3_, remember making sourceSize 1px wide would squash it on CPU just to stretch it on GPU18:00
karni27px18:00
Saviqkarni, is fine18:01
karnigood :)18:01
Saviqmhr3_, "run" script a design tweak? ;)18:01
mhr3_eh18:01
mhr3_Saviq, re-pushed18:02
karniSaviq: want me to merge my gradient to new-scopes, or leave it with you?18:02
Saviqkarni, I'm on it18:03
karnishweet18:03
Saviqkarni, you're pulling my leg, that thing is completely transparent ;D18:04
karnihahhaha18:05
karniSaviq: yeah, I felt like playing a joke. but seriously tho, don't ask me dawg :D18:05
* greyback EOW, good weekend all18:07
Saviqkarni, http://imgur.com/s6MO1kx18:08
Saviqkarni, that's the gradient on white18:08
karniSaviq: I got that from Jouni hahaaha18:08
* karni bursts in laughter18:08
karniSaviq: shall we leave it and let Jouni tweak the asset then :D?18:09
Saviqkarni, yeah, I can't probe anything other than #ffffff off of that image ;)18:09
Saviqand they're gone18:09
karni;D18:09
karnid'oh, indeed18:09
karniSaviq: I'd suggest we leave it. it's easy to drop in a new one, since we already have those couple lines of code. No?18:10
Saviqkarni, pushed18:10
karnisince you're probing white, it's all transparent18:10
karnithanks :)18:10
Saviqkarni, yeah18:10
karnidemo-stuff rebuilds each time we push to new-scopes?18:11
mhr3_once a day18:11
karniack18:11
mhr3_or manual button press18:11
karniso, tomorrow stuff will be neat18:11
karnimhr3_: looking forward to font fixes \o/18:12
Saviqkarni, already there18:12
karni\\o//18:12
mhr3_karni, yea, we actually didn't look at it when running the mwc scopes18:12
* karni fat hands right there18:12
mhr3_karni, so if you could check it, it'd be great18:12
mhr3_obviously things will be smaller :)18:13
* karni looks18:13
* karni in love18:14
karnihttp://imgur.com/BkaVlAl18:14
karnimhr3_: Saviq: good stuff, gentlemen. good stuff :)18:15
Saviqgetting there18:16
mhr3_uuuh, so pretty18:16
mhr3_karni, hmm, in the design the last category doesn't have the bottom gradient18:17
karnimhr3_: shhhh already. I did talk to Saviq about that ;D18:17
mhr3_heh, ok18:17
mhr3_no biggie18:17
Saviqkarni, easy18:17
karniSaviq: I'm sure it's like 2 seconds for you ;D18:18
Saviqkarni, visible: index == list.count18:18
Saviqor -118:18
karnihaha18:18
Saviqor so18:18
karniattached property. gotta love it.18:18
Saviqkarni, no no18:18
karniindex, no?18:18
Saviqkarni, index is just a prop18:18
Saviqah well, maybe it's attached in that sense ;)18:18
Saviqkarni, not sure if it's just put in the context or actually on the delegate18:19
karnidelegate property basically18:19
karniI see18:19
karniI've no idea  ;D18:19
karniSaviq: want me to commit that line?18:19
Saviqkarni, after you've checked that it works ;)18:19
karniyeah18:19
Saviqkarni, it was pseudo-code18:19
=== dandrader is now known as dandrader|lunch
karniSaviq: visible: index !== categoryView.model.count - 118:29
karnicommit to new-scopes without MP? yes, it does work18:30
Saviqkarni, noo18:30
Saviqkarni, categoryView.model is the actual results18:30
Saviqkarni, or well, maybe no18:30
karnithat's the model of the category list18:30
karni3 scopes = count === 318:30
Saviqkarni, doesn't it have count property directly?18:30
* mhr3_ was eager and pushed the button for demo ppa18:31
karniit's not .count, I tried to look it up and got stuck at ListviewWithHeadersomething that I couldn't find on my system18:31
karni:|18:31
Saviqkarni, hmm no it doesn't seem to have18:31
Saviqkarni, listviewwithpageheader.cpp18:31
Saviqkarni, but it doesn't have a count18:31
karniheh.. there we go18:31
karniack18:31
Saviqkarni, problem is .count isn't common on models18:32
Saviqkarni, but maybe it's fine in that case18:32
Saviqkarni, add a FIXME there, though18:32
karniok18:32
Saviqkarni, because if we change the model, it will stop working18:32
Saviqor might18:32
Saviqkarni, FWIW it only works because we're putting it through SortFilterProxyModel18:33
Saviqkarni, which does have .count18:33
karniI see18:33
Saviqbut any other QAbstractListModel won't have a .count, but a .rowCount18:33
karnihuh18:33
Saviqso the view itself should have a .count property that you rely on18:33
karnihow unintuitive18:33
* karni nod18:33
Saviqkarni, well, it's inconsistent indeed, but that's because QALMs were there before QML18:34
karnipushed this to new-scopes:18:35
karni+                // FIXME Should not rely on count field of model, SortfilterProxyModel in this case.18:35
karni+                visible: index !== categoryView.model.count - 118:35
karniSaviq: how are things looking for MWC?18:35
Saviqkarni, looking good I think18:35
karniSaviq: anything large on your plate?18:35
Saviqkarni, nah18:35
Saviqkarni, just checking one last thing and have to do some screenshots, that it18:36
karnimhr3_: glad you pushed the button18:36
mhr3_note that rowCount is not Q_INVOKABLE18:36
Saviqmhr3_, Q_PROPERTY you mean?18:36
mhr3_or that18:36
Saviqbut yeah18:36
karniSaviq: cool, good work :)18:36
mhr3_so should i push the button for rightedge too?18:37
Saviqmhr3_, sure18:37
Saviqmhr3_, I only have a fix for tablet recent apps brewing18:37
mhr3_or Saviq you want the privilege?18:37
mhr3_ah, ok, so demo will need one more rebuild18:38
Saviqmhr3_, ah, and need to revert the fit18:38
mhr3_ah18:38
mhr3_it was so pretty with it :/18:38
karniSaviq: is this number of failures expected :/ ? https://code.launchpad.net/~unity-team/unity8/unity8-category-gradients/+merge/20771818:39
Saviqkarni,  9/20 Test  #7: whitespace ...........................***Failed    0.27 sec18:40
Saviq/tmp/buildd/unity8-7.84+14.04.20140218bzr731pkg0trusty900/qml/Dash/GenericScopeView.qml: bad whitespace in lines 146, 15418:40
karnigod18:40
Saviqkarni, you really need to use a proper editor18:40
* karni facepalms ;)18:40
karniSaviq: either that, or actually learn to use vim.18:40
Saviqkarni, or that18:41
Saviqkarni, there's a fakevim mode in QtC18:41
karniack18:41
Saviqkarni, although it wasn't working for me for some time :/18:41
SaviqQtC crashed, waiting for QtC 3.018:41
karniSaviq: how come it tells you where it failed with whitespace, and tells me not :( http://paste.ubuntu.com/6972339/18:42
Saviqkarni, you need CTEST_SOMETHING_OUTPUT_ON_FAILURE_STUPID_CAUSE_WHY_WOULD_YOU_WANT_ERROR_OUTPUT_FROM_TESTS_RIGHT=118:43
karniof course.. le'sigh18:43
Saviqkarni, CTEST_OUTPUT_ON_FAILURE=118:44
* karni notes down18:44
karnithanks :)18:44
karnithat should really be on by default18:44
karnibtw.. lines 146 and 154 look legit to me. it's all spaces, no tabs.18:45
* karni cries a ltitle18:45
CimiSaviq, I installed mwc image on nexus 418:45
mhr3_karni, there's also ./builddir/Temporary/Testing/LastFailed.log18:45
karniah18:45
CimiSaviq, I saw bold font in app dash18:46
Cimiis that a mistake?18:46
SaviqCimi, upgrade18:46
CimiSaviq, ok18:46
mhr3_didn't build yet :)18:46
CimiSaviq, from the phone?18:46
SaviqCimi, apt-get18:46
karni./builddir/Testing/Temporary/LastTestsFailed.log18:46
karniclose enough :)18:46
mhr3_karni, right.. L<tab>18:46
Saviqkarni, trailing space18:47
Saviqkarni, on line 14618:47
karnihahahahah. mhr3_, without that switch from Saviq, contains "7:whitespace" - how informative ;D18:47
Saviqlol18:47
CimiSaviq, how are we supposed to do this at mwc?18:47
karniSaviq: fuuuuu :D18:47
CimiSaviq, we won't have wifi18:47
SaviqCimi, you're not18:47
Cimiok18:47
SaviqCimi, cwayne will build new images18:47
Cimigreat18:47
mhr3_you won't have wifi?18:48
Ciminope18:48
mhr3_half the scopes query internet18:48
mhr3_yey...18:48
karniSaviq: how about line 154 :D last thing, tell me what's wrog with taht one :D18:48
Cimihalf the scopes won't work18:48
Saviqmhr3_, you know how wifi works at such events18:48
Cimimhr3_, unless we will have sim cards18:48
Saviqkarni, you must've fixed it with a later push18:48
mhr3_Saviq, no i don't, i'm usually at canonical events :)18:48
Saviqmhr3_, you mean... CPT?18:48
mhr3_cpt?18:49
Saviqmhr3_, cape town18:49
Saviqkarni, doesn't complain about that line any more18:49
Cimino idea what we will do with the tablets if we won't have wifi18:49
mhr3_ehm.. right... exception for the rule ;)18:49
karniSaviq: heheheh thanks :) pushed that whitespace fix to MP18:49
Saviqmhr3_, and ehm... hangouts from BlueFin, right?18:49
karnioh boy :)18:49
mhr3_Saviq, oh you mean hangouts do work more than half the time? :)18:50
Saviqkarni, http://vim.wikia.com/wiki/Highlight_unwanted_spaces18:50
Saviqmhr3_, naah, j\k ;)18:50
* karni hugs Saviq 18:50
karnibookmarked. will make good use of it.18:51
karniit's been at least twice this week I failed with whitespace18:51
mhr3_eh, demo builds failed becuase of whitespace18:54
mhr3_anyway, time for me to head home... ttyl18:55
karnimhr3_: sorry about that :/18:56
karniSaviq: ouch :( http://paste.ubuntu.com/6972424/18:57
karnithat, I believe, happened after dist-upgrade on the phone18:57
Saviqkarni, mismatch between unity-mir and unity818:57
Saviqkarni, i.e. you got unity8 from demo-stuff instead of from right-edge18:57
Saviqor so18:57
Saviqkarni, on phone you need https://launchpad.net/~unity-team/+archive/phone-right-edge and only that18:57
mhr3_there's new unity8 in distro, overrides rightedge18:57
karniIf I knew what that even means ;D (I only suspect)18:57
karniaha18:58
* mhr3_ really out18:58
mhr3_cyas18:58
CimiSaviq, maybe you know, how do you workaround the usual issue with lxc-android-config19:00
Cimi?19:00
Cimiwhen upgrading it19:01
SaviqCimi, sudo umount /lib//// the file that was mentioned19:03
SaviqCimi, dpkg --configure -a19:03
Saviqdone19:03
CimiSaviq, (apart that upgrading broke the phone, black screen, I'm reflashing)19:21
CimiSaviq, I'll be at the booth sunday at 13:0019:21
SaviqCimi, you got unity8 from trunk19:21
SaviqCimi, instead of from ppa19:21
CimiSaviq, so if there is some late modification, let me know19:21
SaviqCimi, bad timing19:21
SaviqCimi, no need to reflash19:21
CimiSaviq, ah, do I need a ppa?19:22
Cimisame demo ppa?19:22
SaviqCimi, if you used mwc-flash, you have it already19:23
SaviqCimi, but a newer version built in trusty19:23
SaviqCimi, and not yet in the ppa19:23
CimiSaviq, nope19:23
Cimidon't worry19:23
SaviqCimi, same ppa19:23
Saviqppa:unity-team/demo-stuff19:23
CimiSaviq, I used mwc-flash19:23
CimiSaviq, I enabled rw19:23
SaviqCimi, it's rw by default19:24
CimiSaviq, run update / upgrade19:24
SaviqCimi, yes19:24
CimiSaviq, it wasn't19:24
SaviqCimi, newer unity8 in trunk than in ppa19:24
SaviqCimi, you can force-downgrade19:24
Cimianyway that's what I did19:24
Saviqor wait 20 mins19:24
CimiI have to start packing19:24
Saviqor do that19:25
CimiI just wanted to learn in case other people have issues sunday while flashing/upgrading19:25
CimiSaviq, but if we won't have wifi, how do we use all the webapps?19:25
Cimi3G?19:26
Cimiand tablet?19:26
SaviqCimi, and how am I supposed to answer that? :D19:26
SaviqCimi, I'm not even there19:26
CimiSaviq, hope they have an idea and won't turn into an epic fail :D19:29
CimiSaviq, but I'm sure we won't have wifi19:29
SaviqJEEZ FUCK19:30
SaviqCimi, not at you19:30
=== dandrader|lunch is now known as dandrader
karniCimi: see #ferrets, I had same problem19:39
karniSaviq: you good, pal?19:40
Saviqkarni, bzr19:40
karnile'sigh19:40
Saviqkarni, sorted19:40
karniphew :)19:40
Cimikarni, can you paste?19:41
karniCimi: mwc-flash updated -- make sure youre on mwcinstall 0.19 and rootstock-touch 0.1519:41
karniCimi: then mwc-flash, update, dist-upgrade19:41
Cimikarni, ok... I installed mwc-flash 4 hours ago...19:41
Cimireinstalled19:42
karniCimi: that's not good enough :) update your mwc-flash19:42
karnimwcinstall that is. just update your machine first :)19:42
karniCimi: I confirm stuff works.19:49
Cimikarni, indeed19:50
karniSaviq: Can I ask for favor? I need to leave, and my branch stopped the build because of whitespace once today. I pushed the fix (2 commits) to same MP, but Jenkins clearly didn't enjoy it either. Can I pretty please you push this through? If not, I'll do that tomorrow. https://code.launchpad.net/~unity-team/unity8/unity8-category-gradients/+merge/20771819:51
karni-> eod19:52
Saviqkarni, that's against trunk19:52
Saviqkarni, we're not touching that19:52
Saviqkarni, new-scopes is fixed already19:52
karniSaviq: right. sorry, I meant merging to new-scopes19:53
karnioh.. good19:53
karniI thought I blocked the build mhr3 kicked off19:53
Saviqo/19:53
Cimiok does not work for me19:54
karni\o19:54
Cimidist-upgrade from mwc flashed image19:54
Cimiresults in black screen19:54
Cimi->eod too19:54
karniCimi: apt-cache policy mwcinstall19:54
karni&& rootstock-touch19:54
Cimi0.1719:54
karni20:41 < karni> Cimi: mwc-flash updated -- make sure youre on mwcinstall 0.19 and rootstock-touch 0.1519:55
Cimipuch19:55
Cimiok19:55
karniCimi: clearly, you did not update19:55
karni:)19:55
karnienjoy your evening guys19:55
Cimilike 30 mins ago!19:55
Cimiok19:55
Cimiyou too19:55
Cimithx19:55
karni:)19:55
karniyw!19:55
karniSaviq: you think we could kick off new-scopes build?19:55
Saviqkarni, just checking why it's not going19:55
karniThank you so much19:56
Saviqkarni, it's going19:56
karni\o/ !!19:56
Saviqstarted 4 mins ago19:56
karnithanks, Michal :)19:56
=== karni is now known as karni-away
CimiSaviq, enjoy weekend, catch you next week20:05
SaviqCimi, good travels20:05
Cimithx20:05
kgunnmzanetti: dude you're on late...21:03
mzanettikgunn: ?21:03
mzanettianything broken?21:04
kgunnjust noticed you doing stuff on launchpad...go to your weekend already21:04
mzanettihmm... not that I know of21:04
mzanettiwhat happened? I didn't touch launchpad in 3 hours21:04
kgunnoh maybe it was auto mail21:05
mzanettiI triggered the build recipe before I "left", maybe that is done now21:05
kgunnnvmd...return to your silence :)21:05
mzanetti:)21:05
kgunnor beer drinking21:05
mzanettihave a nice weekend!21:05
kgunnlater21:05
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== shiznix_ is now known as shiznix
=== _salem` is now known as _salem

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!