/srv/irclogs.ubuntu.com/2013/12/12/#ubuntu-unity.txt

=== thomi_ is now known as thomi
=== duflu_ is now known as duflu
=== tvoss|food is now known as tvoss
veebersSaviq: the qmlui tests seems to have a different failure, looking at this job from earlier: https://code.launchpad.net/~saviq/unity8/helper-retry-unlock/+merge/19864808:07
veebersI've fired off the job again  as the failing test is different each time :-\08:07
Saviqveebers, right :/ - thanks, I'll take over now08:18
=== iahmad is now known as iahmad|afk
Saviqmornin'08:24
Saviqtsdgeos, o/08:24
Saviqtsdgeos, can you make anything of http://paste.ubuntu.com/6557929/ ?08:24
* tsdgeos reads08:25
tsdgeoswow :D08:26
tsdgeosis that a crash08:26
tsdgeos?08:26
tsdgeosbasically seems to me that something completed it's building and a lot of properties are being evaluated/set08:26
tsdgeosSaviq: ↑↑↑↑08:34
Saviqtsdgeos, yeah, it's a unity-api qmltest crashing under 5.208:34
Saviqtsdgeos, unfortunately whatever I touch in that test, the crash is different straight away :/08:35
Saviqtsdgeos, and well, somewhere else, too08:35
tsdgeosouch08:35
tsdgeosnothing really obvious tbh08:36
tsdgeosargggg damn test dashbar tests08:57
tsdgeostabbar i mean08:57
=== iahmad|afk is now known as iahmad
Saviqdednick, hey, review done09:47
mhr3could use reviews for the testing stuff in scopes-shell09:51
dednickSaviq: thanks09:51
dednickSaviq: text: menuData && menuData.label || ""    - really? thats pretty weird09:51
mhr3Saviq, any progress on the dev-tool?09:51
* tsdgeos cries09:52
tsdgeoswhy is the tabbar not creating children on CI :-/09:53
tsdgeosoh wait09:53
* tsdgeos <--- stupid09:55
tsdgeosthe sdk code needed still not been released09:55
tsdgeosand here i was trying to make it work  ^_^09:55
tsdgeoschildren you shall not make install to the system and then don't remember you had done it09:56
tsdgeosgood thing at least i found one of the bugs i was having in the verticalJournal implementation meanwhile :D09:58
Saviqmhr3, didn't get there yet, but plan to start real soon09:59
Saviqmzanetti, can you do review for mhr3?10:00
mzanettiSaviq: yip yip10:00
Saviqmzanetti, https://code.launchpad.net/unity-scopes-shell/+activereviews10:00
mzanetticheers10:00
Saviqmzanetti, I'll do the json-defaults10:00
nic-doffaySaviq, hey. Got a second to chat about the component to dismiss the keyboard? As you mentioned it's a bit more complicated differentiating between taps and swipes.10:06
Saviqnic-doffay, well, yeah, only I don't think we can do it right now, maybe it's fine to tap-to-launch straight away10:08
Saviqdednick, yeah it's weird, more concise, but as I said, not really readable10:09
nic-doffaySaviq, I'm dubious about that. But if you'd like me to commit it I'll go ahead. It's a one line change since it looks like someone already added an inverse mouse area for this behaviour.10:09
dednickSaviq: is it a javascript operator?10:09
dednick*expression, or whatever10:10
dednickthingy10:10
Saviqdednick, false || undefined === undefined10:15
Saviqdednick, false || "foo" === "foo"10:15
Saviqnic-doffay, while having moved to dandrader's TappedOutsideNotifier?10:16
dednickSaviq: true || "foo" ?10:17
Saviqdednick, true10:17
Saviqdednick, true && "foo" === "foo"10:17
Saviqdednick, it's kind of like python's `True and "foo" or "bar" == "foo"`10:18
dednickSaviq: i c10:18
tsdgeoswops, my vertical journal does not get reconstucted up correctly :-D10:18
Saviqdednick, when using logical operators on non-bools, they're not converted into bools, but the expression evaluates to the last encountered value, more or less10:19
Saviqdednick, so when you do foo && foo.bar ? foo.bar : ""10:19
Saviqdednick, it does not evaluate to bool(foo) && bool(foo.bar) ? foo.bar : ""10:19
Saviqdednick, but rather to bool(foo && foo.bar) ? foo.bar : ""10:20
Saviqdednick, so if foo.bar is "baz"10:20
Saviqbool("baz") ? "baz" : ""10:20
Saviqso true ? "baz" : ""10:20
Saviqso "baz"10:20
Saviqand foo && foo.bar || ""10:21
Saviqgoes, more or less, to "baz" || "", so "baz"10:21
dednickhm10:22
Saviqtvoss, there?10:27
tvossSaviq, yup10:27
nic-doffaySaviq, sorry not sure what you meant: '<Saviq> nic-doffay, while having moved to dandrader's TappedOutsideNotifier?'10:29
Saviqnic-doffay, just one line change for switching from InverseMouseArea to TappedOutsideNotifier?10:31
Saviqnic-doffay, that'd be great ;)10:31
Saviqnic-doffay, we need to switch to TON, 'cause IMA didn't work well on devices10:31
nic-doffaySaviq, haha yeah it's just one line.10:32
nic-doffaySaviq, or two I think :P10:32
Saviqnic-doffay, sure, please test it works on devices then10:32
Saviqnic-doffay, and write a test, too!10:32
nic-doffaySaviq, sure10:34
mzanettimhr3: https://code.launchpad.net/~mhr3/unity-scopes-shell/scopes-ng-tests/+merge/198437 line 437. wanted?10:34
mhr3meh, it's copied over from the lib10:35
mhr3which means there's lgpl licenced test... guess i should change that10:36
mzanettitrue10:37
tsdgeos5.2.0 official out10:37
mzanettihuh? that was fast10:37
mzanettididn't they just discuss when the next rc should come?10:37
mhr3mzanetti, pushed10:38
tsdgeosdamnit, reconstructing the verticalJournal up is not easy at all :/10:41
Saviqmzanetti, tsdgeos old news, it was out yesterday ;)10:49
Saviqtsdgeos, less easy and doing top-down?10:50
tsdgeosSaviq: not officially ;-)10:50
Saviqtsdgeos, pfft10:50
Saviqtsdgeos, I SAW IT THERE IN official-releases!10:50
greybackthe tarballs were out10:50
Saviqhere http://download.qt-project.org/official_releases/qt/ !10:50
greybackit was on hacker news too :)10:50
Saviqsee? 11-Dec-2013 16:1210:50
tsdgeosSaviq: there was no email ;-)10:50
Saviqtsdgeos, bah!10:50
tsdgeostarball may had been changed :D10:50
Saviq;)10:51
tsdgeosSaviq: anyway, thing is that when you go up in the verticalJournal you can't really know in which of the columns you have to position the item10:51
tsdgeosbecause the next item you position up may mean you have to rearrange the things10:51
tsdgeosso i'll have to do that i guess, which is not cool10:51
tsdgeosthat or remember the columns of every item10:51
tsdgeoswhich is worse :D10:51
Saviqtsdgeos, hmm ah in case one of the previous items is like *long*?10:53
tsdgeosyep10:53
Saviqtsdgeos, so that it covers two rows when going top-down, but you don't know that yet when going up10:53
Saviqtsdgeos, you think it's really worse to store them?10:53
tsdgeosif i order them the same way going up that when going down10:53
tsdgeoswhen i reach the top10:53
tsdgeosstuff is jaggy and not 0 aligned10:53
Saviqtsdgeos, yup, understood10:53
tsdgeosideally i should be able to reorder them before you see them on screen10:54
tsdgeossince i have a buffer zone too10:54
Saviqtsdgeos, I just wonder if it's really that bad to store the positions...10:54
Saviqtsdgeos, more memory-hungry, sure (how much?)10:54
tsdgeoswell it's not bad, just a bit more memory intensive10:54
Saviqtsdgeos, but the other one is much more computationally heavy10:54
tsdgeosnot much, i just need to store the integer of the column10:54
Saviqyeah exactly10:55
Saviqtsdgeos, your call, I can see pros and cons of both10:55
tsdgeosi mean storing the column is so much easier10:56
Saviqmzanetti, https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1237923 → Confirmed10:56
ubot5Ubuntu bug 1237923 in Ubuntu UI Toolkit "SSH keys should not be generated automatically - or at least kept for QtCreator use only" [Undecided,New]10:56
Saviqtsdgeos, yeah, and what... a kB memory per 1000 entries?10:56
tsdgeosgetting the algo right to reconstruct + move the stuff as needed is much harder10:56
Saviq(probably more, but not much)10:56
mzanettiSaviq: done10:56
Saviqmzanetti, cheers10:56
tsdgeosSaviq: ok, let's go storing the column for now and we can always make it better if we find it's taking to much memory10:57
Saviqtsdgeos, I really doubt we will :)10:57
tsdgeosSaviq: storing the column doesn't work if we want to support resizing11:11
Saviqtsdgeos, right, but I'm fine with redoing it in that case11:12
Saviqtsdgeos, like I can actually agree to do the whole model top-down when you resize the column11:13
tsdgeossure, but means i still need to write the algorithm :D11:13
tsdgeosah11:13
Saviqtsdgeos, it's not like resizing is going to happen often11:13
tsdgeossure11:13
Saviqtsdgeos, iteration 2, basically11:14
=== _salem is now known as salem_
nic-doffaydandrader, did you write any tests for PressedOutsideNotifier?11:53
dandradernic-doffay, yes11:54
dandraderin tests/plugins/Ubuntu/Gestures11:54
dandradernic-doffay, ↑11:54
nic-doffaydandrader, cool11:55
dandradernic-doffay, adding stuff to it?11:56
nic-doffaydandrader, nah not yet at least.11:57
=== MacSlow is now known as MacSlow|lunch
mhr3mzanetti, regarding the QVariant returns, i find it odd when a method is returning QVariant and you see "return QString(foo);"12:04
mzanettimhr3: fair enough... it's personal preference. So, your code, your rules. feel free to ignore12:07
=== _salem is now known as salem_
mhr3mzanetti, otoh i do like less typing :) if it's a common thing with variants i can change it, and will get used to it12:08
mzanettimhr3: yeah. it's quite common in Qt to let it convert to QVariants implicitly12:08
mhr3fine then12:09
dednickanyone noticed that if you exit the unity8 shell app on desktop, if stops the desktop indicators? :)12:14
dednickoops12:14
dednicki think it's sending upstart a command12:14
greybackdednick: yep, here too. I usually restart them with "start indicator-session"12:15
nic-doffaydandrader, do I need to change any CMakeLists to use Ubuntu gestures from qmluitests?12:22
dandradernic-doffay, you mean you want a test listed in tests/qmltests/CMakeLists.txt to "import Ubuntu.Gestures"?12:25
Saviqdednick, one for you https://code.launchpad.net/~saviq/unity8/clean-paths-cmakelists/+merge/19872512:26
nic-doffaydandrader, that's right.12:26
dandradernic-doffay, you will have to add "${CMAKE_BINARY_DIR}/plugins" to the IMPORT_PATHS parameter of his add_qml_test entry12:27
Saviqisn't that default?12:28
Saviqnope12:28
dandraderSaviq, no. some tests want to use mock versions of some of the plugins there12:28
Saviqdandrader, yup12:28
Saviqnic-doffay, so something like:12:28
Saviqadd_qml_test(Panel Indicators IMPORT_PATHS ${CMAKE_BINARY_DIR}/plugins ${qmltest_DEFAULT_IMPORT_PATHS})12:28
dandradernic-doffay, so bear in mind the order of the IMPORT_PATHS12:28
dandradermatter12:29
nic-doffaydandrader, cool I see it's already been added for dash etc.12:30
dednickSaviq: ok, will take a look after lunch.12:34
Saviqdednick, cheers12:34
=== dednick is now known as dednick|lunch
tsdgeosSaviq: So  lp:~aacid/+junk/verticalJournal  should be pretty good12:39
tsdgeosSaviq: needs someone to go over the functional and test review12:39
tsdgeosSaviq: and a place to merge it :D12:39
Saviqtsdgeos, new-scopes12:39
Saviqtsdgeos, or well...12:39
tsdgeosSaviq: unity-scopes-s12:39
Saviqtsdgeos, it can go into lp:unity8 directly12:39
tsdgeosright12:40
tsdgeosok12:40
Saviqtsdgeos, it doesn't have to be used, as long as it's tested12:40
tsdgeossure12:40
tsdgeosi need to work a bit more on the tests12:40
tsdgeosbut it's almost there12:40
Saviqmhr3, hmm scopes.get(scopeSelector.selectedIndex) → ASSERT failure in QList<T>::at: "index out of range"12:44
mhr3who was telling me that those index checks in data are useless?12:45
mhr3mzanetti, tsdgeos ^? :P12:45
mzanettimhr3: I did12:45
mhr3oh but wait, that's get(), not data()12:45
mzanettimhr3: well... otherwise noone would have noticed and there's just some data missing12:45
mzanettiah wait... this is a manual get()12:46
mzanettinot the data() method12:46
mzanettiin that case it's different12:46
mhr3fixing in the tests branch12:47
mhr3and adding test for it :)12:47
dandraderAnyone up for a quick & east review? https://code.launchpad.net/~dandrader/unity-mir/lp1248795/+merge/19873112:48
Cimimzanetti,12:48
dandraders/east/easy12:48
Cimihttps://bugs.launchpad.net/unity8/+bug/125812612:48
ubot5Ubuntu bug 1258126 in Unity 8 "Arrow in tabs in indicators positioned badly" [High,New]12:48
Cimithis is weird12:48
Cimiin ambiance theme12:49
greybackdandrader: I'll take it12:49
Cimithe indicator is placed with x: button.width - width12:49
dandradergreyback, thanks!12:49
greybackdandrader: I always get it wrong and thing 'var' is the deprecated one12:49
Saviqmhr3, lp:~unity-team/unity8/scope-dev-tool12:49
Cimiwith button.width being text.paintedWidth + text.anchors.leftMargin + text.anchors.rightMargin12:49
greybacks/thing/think/12:49
Cimitext is the label in indicators12:50
dandradergreyback, http://qt-project.org/doc/qt-5.0/qtqml/qml-variant.html12:50
dandrader"The variant type is a generic property type. It is obsolete and exists only to support old applications; new applications should use var type properties instead."12:50
greybackdandrader: yep I know. I just get that mixed up in my head12:50
dandradergreyback, a ok, got what you meant now :)12:51
mzanettiCimi: hmm... maybe paintedWidth borked?12:52
greybackdandrader: approved12:52
Cimimzanetti, it's my guess too12:52
Saviqmhr3, re: auto → sure, easier to write, not easier to read ;)12:53
Saviqmhr3, but yeah, maybe var names should just carry what they are, so that you don't have to go and look at its type anyway12:53
mzanettimhr3: complaining about me nitpicking on "auto" :P ?12:54
mhr3yes :)12:54
mhr3Saviq, pushed the index thing12:55
Saviqmhr3, you mean fixed it?12:55
mhr3yes12:55
Saviqmzanetti, does QtC resolve auto?12:55
Saviqmhr3, ok12:55
mzanettidunno..12:55
Saviqmzanetti, I mean it should ideally resolve it in a tooltip, for example12:55
greyback2.9 wasn't able to, I remember that anywy12:56
Saviqmzanetti, or well, you can look at return type of the function anyway12:56
Saviqmzanetti, and I agree with mhr3 that unless you put type info into the var name itself,  you still need to hunt for the type12:56
mzanettiSaviq: sure... but it requires me to move the mouse and wait a second for the tooltip to show up :D12:56
mhr3pfff :P12:56
mzanetti*the mouse*12:57
mzanetti:P12:57
mhr3the horror!12:57
Saviqmhr3, http://people.canonical.com/~msawicz/scope-tool/12:57
mhr3Saviq, also, why do you call scopes.get() with incorrect index? :)12:58
Saviqmhr3, I wasn't12:58
Saviqmhr3, well, was trying not to12:58
mhr3Saviq, yey, lovely :)12:58
mhr3i mean the tool12:59
=== dandrader is now known as dandrader|afk
Saviqmhr3, right, it's a race probably12:59
Cimimzanetti, confirmed...12:59
Saviqmhr3, I get assert with 012:59
mhr3Saviq, calling too early?13:00
mzanettiCimi: hmm.. can you reproduce that in a simplified testcase?13:00
Saviqmhr3, binding13:00
Saviqmhr3, the option selector already created and selected its 0 index13:00
Cimimzanetti, nope13:01
Saviqmhr3, but somehow the model isn't ready for it yet13:01
mhr3Saviq, yea, it takes a while to init the scopes model, anyway, with the patch you'll get null back13:01
mhr3Saviq, you could wait for the loaded signal though13:02
mhr3Saviq, also ekran is screen?13:03
mhr3sounds like a wild animal :D13:03
Saviqmhr3, yes :)13:03
Saviqmhr3, how about "zrzut"? ;)13:03
Saviqmhr3, not sure where covers went, though :/13:04
mhr3yea, that one is *just* odd, no associations :)13:04
mhr3but ekran totally has wings13:04
mhr3and is blue13:04
mhr3https://www.google.co.uk/search?q=ikran&source=lnms&tbm=isch&sa=X&ei=d7SpUta_OoTnywPFmYCACA&ved=0CAcQ_AUoAQ&biw=1301&bih=65613:05
Saviq;D13:09
Saviqmhr3, look again http://people.canonical.com/~msawicz/scope-tool/13:10
mhr3why the double edit?13:10
Saviqmhr3, 'cause you can't access it in phone layout13:11
Saviqmhr3, there's no "SEARCH" panel13:11
mhr3oh13:11
mhr3ok13:11
Saviqmhr3, we could go for button, but thought that would be good enough13:11
mhr3sure, doesn't need to be perfect13:11
Saviqmhr3, any idea where covers went?13:12
Saviqmhr3, don't get them in plain unity8 either13:12
mhr3hmm, server issue?13:12
=== alan_g is now known as alan_g|lunch
mhr3Saviq, oh do you have the icon -> art branch installed?13:13
Saviqmhr3, good question13:13
Saviqmhr3, prolly not13:13
mhr3that would explain it13:13
=== dandrader|afk is now known as dandrader
=== MacSlow|lunch is now known as MacSlow
dednick|lunchSaviq: what is shell_app_HDRS?13:23
=== dednick|lunch is now known as dednick
Saviqdednick, yeah, exactly - nothing13:24
Saviqdednick, it's not defined anywhere13:24
dednickok13:24
=== iahmad_ is now known as iahmad
Saviqok... I was trying to get food for two hours now...13:25
Saviq/food13:25
Saviqmhr3, right - icon vs. art indeed - where *is* that branch, btw?13:29
* Saviq got lost13:29
mhr3Saviq,  lp:~mhr3/unity-scopes-shell/scopes-ng-tests13:32
mhr3so also in the json-defaults one13:32
Saviqmhr3, right!13:33
Saviqmhr3, http://people.canonical.com/~msawicz/scope-tool/ ;)13:37
Saviqmhr3, will package it and it basically get in into new-scopes13:37
Saviq+can13:37
mhr3much better :)13:38
Saviqmhr3, tests will only come later when we do something the dash itself doesn't do (like putting JSON through the validator)13:38
Saviqreally... */food*13:39
mhr3i'll also play with it so it's able to talk to an uninstalled scope13:39
dednickSaviq: hm, that getProperty doesnt work yet. changes to roles doesnt update the values :(13:42
dednickSaviq: because it's a string property we're checking for.13:42
dednickSaviq: ie getProperty(menuData, "label", "") will not respond to changes to menuData.label13:43
Saviqdednick, "obviously that assumes that extAttrib only changes13:43
Saviqatomically - which is not the case for menuData, for example - so it makes13:43
Saviqsense to keep menuData explicit, while moving extAttrib and maybe some others.13:43
SaviqOr well, menuData could be handled as well, by means of Qt.binding(), I think."13:43
dednickwould have nice to have the menuData like that it as well.13:44
dednickSaviq: can you return a binding from a function?13:45
Saviqdednick, Qt.binding(function() { return menuData[property] }) should work13:46
Saviqdednick, hmm or not13:46
Saviqdednick, you'd have to make sure that menuData.property is named somewhere there13:47
Saviqdednick, so that the binding attaches to dataChanged13:47
Saviqdednick, not sure [] will be good enough13:47
dednickeh. would need to do it in onCompleted as well. can't use binding in declaration13:50
dednicki knew there was a reason for all those && ?, i tried this before...13:52
Saviqdednick, why?13:52
Saviqdednick, as long as the changing thing is called explicitly in the binding13:53
Saviqdednick, it should update fine13:53
Saviqdednick, that's why I mentioned a custom get(object, property, default) method13:53
Saviqdednick, as .getProperty assumes the object is !undefined13:53
Saviqdednick, the get() would not assume that13:54
dednickSaviq: yeah, but need get(object, property, propertyName, default13:54
Saviqdednick, and should be evaluated whenever any of its arguments change13:54
Saviqdednick, property vs. propertyName?13:54
dednickhasOwnProperty.13:54
dednicki guess can just to !== undefined13:54
Saviqdednick, yeah, first check if object is defined13:55
Saviqdednick, then if it has property13:55
Saviqdednick, if either is false, return default13:55
Saviqdednick, and that should cause the thing to reevaluate as needed - *assuming* that object changes atomically13:55
dednickSaviq: how to you check if it has a property? you mean just "menuData && menuData.label != undefined"13:55
Saviqdednick, which menuData doesn't - but extendedAttr has13:56
Saviqdednick, right, menuData is different13:56
Saviqdednick, that's why I said that a binding is needed13:56
Saviqdednick, and I doubt menuData[property] will be good enough13:56
Saviqdednick, you need to call menuData.label explicitly for dataChanged to be connected13:57
Saviqdednick, so for menuData indeed I'm not sure it makes sense13:57
Saviqdednick, you *could* hack it around with something along the lines of: menuData.onDataChanged: menuData.changed()13:58
Saviqdednick, but that's probably not the best thing to do13:58
dednickSaviq: http://pastebin.ubuntu.com/6561497/14:01
dednickcan really use either for extended data14:01
dednickdont need a binding if we give the property, as it will be reevaluated when the parameter changes14:02
Saviqdednick, only extendedData.xCanonicalTime will complain about extendedData undefined14:03
Saviqdednick, as will menuItem.label14:03
dednickoh, you mean when you call into the function? hmm.14:04
=== alan_g|lunch is now known as alan_g
Saviqdednick, yeah, at that point you'd have to go menuData && menuData.property14:08
Saviqdednick, as the argument - which is bleh.14:08
dednickSaviq: but as you said, doesnt that statement evaluate to the value of menuData.property if it's true? or is that only if you use || ?14:10
dednickbut i guess that defeats the purpose. sigh....14:10
Saviqdednick, yeah, it will, but yes - it defeats the purpose exactly14:10
dednicki might just change the menuData.label back to good old && || . dont think there's really any other neat way around it.14:11
Saviqdednick, only thing I could think of was menuData[propertyName] in a Qt.binding()14:13
Saviqdednick, but I'm worried that doesn't actually work14:13
dednickSaviq: it doesnt.14:13
dednickplus you need to use inside onCompleted, which sucks14:13
Saviqdednick, yeah, I'm ok with && for menuData14:14
Saviqdednick, since there's no clean way around it14:14
Saviqdednick, standup14:33
CimiSaviq, you should have another automated bot for automated standup calls14:34
mzanettiMacSlow: hey. as you'll be gone soon, what's the status of the SIM pinlock?14:44
mzanettiMacSlow: did you fix those two issues I found?14:44
=== dandrader is now known as dandrader|lunch
MacSlowmzanetti, branch is ready since London with your requested fixes14:45
mzanettiMacSlow: oh... damn... ok... I'll review it and fix any comments myself then while you're away14:45
MacSlowmzanetti, I confidnet you'll be happy with the current state14:45
mzanettiMacSlow: so far I only did a testing review. not a code review. but yeah... I don't think its really far off any more14:46
mzanettiMacSlow: I'll try to have it merged by the time you come back.14:46
MacSlowmzanetti, even made a screencast for you -> https://www.youtube.com/watch?v=raJpJx6DkQU :)14:47
mzanettihaha14:47
mzanettialways fun to see some motor bikes14:47
Saviq;D14:47
MacSlowmzanetti, no bikes in that screencast this time :)14:48
SaviqMacSlow, there's some in the "you should also watch" screen ;)14:48
mzanettiand in your avatar14:48
MacSlowSaviq, those are "bugs" in Google's recommendations-algorithms ;)14:48
SaviqCimi, re: bug #1258571 - yeah, that's what we discussed in London14:49
ubot5bug 1258571 in unity8 (Ubuntu) "Select backgrounds based on aspect ratio and read them from dconf" [High,Triaged] https://launchpad.net/bugs/125857114:49
SaviqCimi, stretching is the worst thing you can do to any image ;)14:49
CimiSaviq, stretch/crop14:49
CimiSaviq, all OSes do that14:49
SaviqCimi, so fit, not stretch14:49
Cimiwhatever14:49
Cimiscale14:49
SaviqCimi, yes14:49
SaviqCimi, yeah, but having a single one that works well on the phone in portrait and on tablet in landscape isn't really easy14:50
SaviqCimi, we decided to go landscape + portrait for the default ones14:50
CimiI think it will be weird14:50
SaviqCimi, when you select a different background - that's when it will be scaled+croped14:51
Saviqcropped14:51
Cimirotating the tablet and seeing wallpaper changing14:51
SaviqCimi, no no14:51
SaviqCimi, it's only about native aspect14:51
CimiSaviq, the bug report is about orientation14:51
SaviqCimi, so on tablet it will always be the tablet one14:51
SaviqCimi, ok maybe it's worded wrong indeed14:51
SaviqCimi, so write down in the bug please that the selection is only supposed to be made based on native device orientation14:52
Cimiok14:52
Saviq/aspect ratio14:52
Cimiok14:52
Cimidone14:53
Cimiseb128, ^^14:53
Cimiseb128, that bug, we need extra gestating schema14:53
Cimigsetting14:53
seb128Cimi, why?14:56
seb128what schemas/setting14:56
Cimiseb128, look at the bug14:56
seb128I did14:56
Cimihttps://launchpad.net/bugs/125857114:56
ubot5Ubuntu bug 1258571 in unity8 (Ubuntu) "Select backgrounds based on aspect ratio and read them from dconf" [High,Triaged]14:56
seb128that doesn't make sense14:56
seb128we discussed that via email with Saviq some time ago14:56
seb128having per form factor keys doesn't make any sense14:56
Saviqseb128, we talked with cwayne and design folk in London14:57
Saviqseb128, it's not per-form-factor14:57
Saviqseb128, it's for-portrait and for-landscape14:57
Saviqseb128, and well, I don't really care whether it comes from gsettings or somewhere14:57
seb128are those actually different backgrounds, or just one rotated?14:57
Saviqseb128, different ones14:57
cwayneapologies if i didn't word the bug clearly enough14:58
seb128shrug14:58
seb128Saviq, cwayne: and which one of those 2 keys to plan to use in desktop mode?14:58
Saviqseb128, the idea is that OEMs can provide two backgrounds, one working well on portrait, one on landscape - only taking *native* aspect into account14:58
Saviqseb128, desktop is native landscape - so that one14:59
Saviqseb128, so we might just add background_portrait - or even take it from somewhere else than gsettings14:59
cwaynewe just suggested gsettings as that's where the current background info lives and would be the easiest in terms of customization15:00
seb128Saviq, I was going to say, maybe just read the gsettings key and append -portrait to the filename and try to load that first15:00
seb128e.g gsettings would be UbuntuRock.png15:01
seb128you would try loading UbuntuRock-portrait.png if it's there15:01
seb128cwayne, Saviq: the current design of the background config doesn't allow for portrait/landscape difference, what happen if you pick a background in portrait?15:02
seb128does it change only that one? or landscape as well?15:02
seb128we need design guidance on how the config is going to work to not be confusing15:02
Saviqseb128, that's a problem in the settings app - it should reflect the native resolution of the device, I think15:02
Saviqseb128, but, again, I probably have even more questions for that than you do!15:03
seb128Saviq, well, if there are 2 images for landscape/portrait, do we need to be able to select both?15:03
seb128cwayne^^15:03
Saviqseb128, again, I have more questions!15:03
seb128Saviq, well, let's get those resolved before adding gsettings keys15:03
Saviqseb128, like that's gonna happen!15:04
Saviqseb128, I'm fine with appending -portrait, tbh15:04
seb128Saviq, well, the issue with all those solutions, is that15:05
seb128- you end up with the setting not reflecting what you have on screen15:05
seb128- if you pick a custom background, how do you got back to the vendor ones?15:05
Saviqseb128, right back at you :)15:05
seb128well, if you ask me I'm staying that those different images seems like a stupid idea and we shouldn't do it..15:06
cwaynewell if the device is portrait, only show/care about the -portrait gsettings key15:06
Saviqcwayne yeah, which ends up being just one gsettings key ultimately15:06
cwayne right15:07
Saviqcwayne, which begs the question... should the default not be per-device?15:07
cwayneSaviq: what if we have one key that points to a dir that can contain a -landsape.png and a -portrait.png15:07
seb128cwayne: then the configuration behaves differently when you rotate the device without telling you?15:07
cwayneseb128: its about the native aspect ratio, not the current one15:08
cwayneAIUI its not supposed to switch when you rotate15:08
Saviqcwayne, I'm starting to think it should just use the image override15:08
Saviqcwayne as in the system image - per-device15:08
Saviqcwayne so you'd put a different background on manta, different on mako15:08
Saviqeven under the same name15:08
seb128cwayne: portrait is not supposed to be used in portrait mode?15:09
Saviqseb128, only native aspect ratio15:09
Saviqseb128, not when you change orientation15:09
seb128so it's 1 image by device?15:10
cwayneSaviq: right, but then we'd need different custom tarballs for different devices, which we were trying to avoid15:10
Saviqseb128, effectively yes15:10
seb128so why just not shipping that as the default wallpaper?15:10
Saviqseb128, ↑↑15:10
Saviqbrb15:11
cwaynebecause then that image wouldn't look good on a mako for example15:11
cwaynedavidcalle: heya, where would be the best place to file a wishlist bug against a server-side scope?15:12
davidcallecwayne, if you find it in the list of unity-scopes sub-projects (https://launchpad.net/unity-scopes), on the scope itself. Or more generally, against unity-scope. There is no public place for server specific bugs (yet?).15:14
cwaynedavidcalle: awesome, thanks15:15
cwayneseb128: so the whole point is to be able to have 1 custom tarball that will dynamically choose the right background so that we don't have to make a -tablet image15:16
davidcallecwayne, np :) *now waits for bugmail*15:16
seb128cwayne: there is 0 difference in image/setup between a phone and a tablet image/for different devices?15:18
seb128cwayne, Saviq: do we have some sort of code running on first device boot? could we just set the background key to the right value then?15:19
cwayneseb128: correct. there's 1 image that works on both15:19
seb128it would make things easy/compatible with what we have now and not at a runtime cost15:20
seb128better to do a check/write a key once that check at every unity start15:20
=== alan_g is now known as alan_g|tea
davidcallecwayne, oh wait, you can't actually report a bug against unity-scopes... Ugh. Is it the weather not on top bug?15:31
tsdgeosthis is veeeeeeery confusing15:31
cwaynedavidcalle: no it was jcastro asking me, i think he had a wishlist bug for the askunity scope15:31
tsdgeos        QTRY_COMPARE(item.y(), y);15:32
tsdgeos        qDebug() << item.y() << y;15:32
cwaynedavidcalle: i just guessed and did the weather one to the home scope15:32
tsdgeosFAIL!  : VerticalJournalTest::testVerticalSpacing() Compared doubles are not the same (fuzzy compare)15:32
tsdgeos   Actual   (item.y()): 35515:32
tsdgeos   Expected (y): 1115:32
tsdgeos   Loc: [verticaljournaltest.cpp(33)]15:32
tsdgeosQDEBUG : VerticalJournalTest::testVerticalSpacing() 11 1115:32
cwayneseb128: that seems reasonable to me, i don't think anyone wanted us to do the check every time unity started, just once :)15:32
tsdgeosso it's 355 vs 11 but then i print them and it's 11 and 11 ?¿?¿?¿15:32
davidcallecwayne, it's actually on the scope too... I'll make a place for these bugs. And for askubuntu, it's on unity-scope-askubuntu, I'm pretty sure I already know the bug :)15:33
cwaynedavidcalle: :)15:34
=== alan_g|tea is now known as alan_g
mhr3_tsdgeos, TRY_COMPARE is too much magic, not too surprised :P15:41
tsdgeosmhr3_: is not magic at all15:41
tsdgeosit's just a loop15:41
tsdgeosahh, know what's going on15:42
mhr3_tsdgeos, hm?15:43
tsdgeosfound my problem15:43
tsdgeosor the "hm?" is about try compare being just a loop?15:43
mhr3_tsdgeos, "hm" == so what's the actual problem?15:44
tsdgeosi was working on deleted memory :D15:44
mhr3_nice :)15:44
tsdgeosgood guy valgrind at rescue15:45
mhr3_mzanetti, happy enough with the tests branch now?15:48
mzanettimhr3_: will have a look15:48
mhr3_got two more branches built on top of it, so would like it in asap15:49
Saviqcwayne, seb128, right, so there's a few jobs that run on system / session startup15:49
Saviqcwayne, seb128, that do something only once (and write a .file somewhere to not do it again)15:50
cwayneright, we even have some of those for customization already as well15:50
Saviqcwayne, seb128, so we could add a job that writes an .override file if it doesn't exist15:50
seb128right15:50
seb128I like that approach better tbh15:51
Saviqseb128, +115:53
seb128;-)15:53
Saviqcwayne, could you mark affefcts bug #1258571 for the customization project? sssssomething?15:54
ubot5bug 1258571 in Ubuntu UX "Select backgrounds based on aspect ratio and read them from dconf" [Undecided,New] https://launchpad.net/bugs/125857115:54
mzanettimhr3_: hmm... not sure if its my setup (probably it is) but tests are failing here15:55
cwayneSaviq: done, sorry i thought it had already been attached15:55
mhr3_mzanetti, ci likes them15:55
mhr3_mzanetti, so it's you :)15:55
kgunndednick: just to be sure...this branch https://code.launchpad.net/~nick-dedekind/unity8/indicator-menuitems-lp1253810/+merge/19859716:01
kgunndoesn't really fix that bug right ?16:01
kgunne.g. its just an artifact in the name of the branch16:01
dednickkgunn: it's part of the fix.16:02
dednickkgunn: along with a qmenumodel fix which has already gone int16:02
dednick*in16:02
Saviqdidrocks, bug #126037916:02
ubot5bug 1260379 in Ubuntu CI Services "A unity8.override file should be shipped to allow apport completion during test runs" [Undecided,New] https://launchpad.net/bugs/126037916:02
=== dandrader|lunch is now known as dandrader
didrocksSaviq: I think we can make that part of the -autopilot package16:03
didrockswdyt?16:03
didrocksunity8-autopilot that is16:03
dednickkgunn: i'm just in the process of checking the tests that i've made against the branch, and we should be able to merge.16:03
Saviqdidrocks, could16:03
didrocksSaviq: I would start with that TBH ;)16:03
Saviqdidrocks, please comment/mark-affect16:03
didrocksok16:03
kgunndednick: thanks...sorry to be a nag, was that bug consistent enough to know it fixes it for sure16:04
dednickkgunn: i haven't been able to reproduce, but Saviq has.16:04
dednickalthough i havent tried it on my phone now that i have it with me.16:04
kgunndednick: mmmm....polish texts :)16:04
kgunn...or excuse me Polish16:05
dednickhaha16:05
Saviqkgunn, excuse me, there were no texts involved - missed calls are enough ;P16:05
Saviqdednick, btw, that branch ready yet?16:05
Saviqdednick, or are you tweaking still?16:05
Saviqah 8 minutes ago :)16:07
dednickSaviq: it's ready.16:07
Saviqdednick, ok16:07
mhr3_Saviq, seems like you didn't push the search edit16:09
Saviqmhr3_, possible16:10
Saviqmhr3_, pushed16:10
mhr3_lovely16:10
Saviqdidrocks, but bug #126038416:17
ubot5bug 1260384 in Ubuntu CI Services "Should preprocess .crash files on test devices" [Undecided,New] https://launchpad.net/bugs/126038416:17
didrocksSaviq: can you address those to the CI team? :)16:17
Saviqdidrocks, isn't the -itself enough?16:17
didrocksSaviq: sinon, ack on that one, that would make our life easier :)16:17
Saviqdidrocks, not sure I can, either16:18
didrocksSaviq: should be, but not sure why you really ping me :)16:18
Saviqdidrocks, for → Confirmed ;)16:18
didrocksah16:18
didrocksok, yeah, can do16:18
Saviqdidrocks, as we discussed this before16:18
didrocksdone16:19
didrockswith a comment ;)16:19
Saviqnic-doffay, a simple one when you have a bit of time https://bugs.launchpad.net/ubuntu-ci-services-itself/+bug/126037916:19
ubot5Ubuntu bug 1260379 in Unity 8 "A unity8.override file should be shipped to allow apport completion during test runs" [High,Triaged]16:19
nic-doffaySaviq, cool16:20
Saviqdidrocks, see you want for false || value in the end :)16:20
tsdgeosSaviq: do you want me to propose the verticaljournal as a new plugin or rename the LVWPH plugin into something like "DashViews" and add it there?16:20
Saviqtsdgeos, same plugin is fine I think16:20
didrocksSaviq: right ;)16:20
tsdgeosSaviq: oki, will do some renaming then16:20
Saviqdidrocks, erm16:21
Saviqdidrocks, that was at dednick ;)16:21
Saviqsorry16:21
Saviqdednick, see you want for false || value in the end :)16:21
Saviqdednick, there seem to be some unrelated fixes in there now?16:22
didrocksSaviq: I was reacting on "a simple one" (telling, right, it's a simple one)16:22
Saviqdidrocks, ok :)16:23
dednickSaviq: um, related to the Factory, but not necessarily to that bug.16:24
Saviqdednick, yeah, it's ok, though - assume there's no other bugs for those16:24
Saviqdednick, but we'll get tests for them ;)16:25
dednickSaviq: yeah, that's what i'm doing now. I found those bugs with the tests.16:25
Saviqdednick, ok awesome16:25
Saviqdednick, changes look good, testing again16:26
* Saviq goes `rm -R shell/builddir`...16:26
Cimimzanetti, ping16:30
mzanettiCimi: o/16:30
Cimimzanetti, so, the binding seems broken16:30
mzanettifix it :)16:30
Cimimzanetti, well, I dunno why doesn't work16:31
Cimimzanetti, on the item16:31
mzanettimhm... can you point me to the line of code?16:31
Cimix: button.width - width16:31
Cimimzanetti, /usr/lib/x86_64-linux-gnu/qt5/qml/Ubuntu/Components/Themes/Ambiance/TabBarStyle.qml16:31
mzanettiCimi: oh, inside the theme?16:31
Cimiyou can sudo qtcreator that :)16:31
Cimiyes16:31
Cimiit's useful to debug16:32
mzanettiand why don't the apps suffer from that?16:32
Cimidon't ask me yet16:32
CimiI need more brain power16:32
Cimi:P16:32
Cimiso I need your head as well16:32
Cimimzanetti, for indicatorImage16:33
Cimimzanetti, there's x: button.width - width16:33
dednickSaviq: how do i get whitespace test failure output?16:33
Cimibut in the case of the indicator button.width changes AFTER x, and x is not update :-\16:33
Cimimzanetti, you can do that if you do onXChanged and onWidthChanged16:34
Cimion indicatorImage and button respectively16:34
mzanettiCimi: probably a binding loop detection that kicks in16:34
Saviqdednick, ./tests/whitespace/check_whitespace.py . | grep -v builddir16:34
dednickSaviq: ta16:35
Saviqdednick, or well, make -C builddir works, too16:35
Saviqtest16:35
Saviqmake -C builddir test16:35
didrocksSaviq: kgunn: for the unity8 AP failures, as it's due to screenlocking, I don't remember but the fix will be in the unity8 AP tests, right?16:35
=== dandrader is now known as dandrader|afk
Cimimzanetti, I can't see any print16:36
Cimimzanetti, on the terminal16:36
Cimiabout binding loops16:36
Saviqdidrocks, https://code.launchpad.net/~saviq/unity8/helper-retry-unlock/+merge/19864816:36
Cimimzanetti, but on the gallery app it updates, not here16:36
mhr3_tsdgeos, is there a way to know that a plugin is running under qmlplugindump?16:37
tsdgeosmhr3_: no idea, sorry16:37
didrocksSaviq: that should fix all the AP issues we saw, right?16:38
Saviqdidrocks, as far as I can tell, yes16:38
Saviqdidrocks, bear in mind that's not something I could reproduce locally16:39
didrocksSaviq: yeah, still worth a try16:39
Saviqdidrocks, worse case it will give us more data16:39
Saviq*worst16:39
didrocksSaviq: we'll release unity8 once the 2 other bugs are fixed as well16:39
nic-doffaySaviq, this test_pressed_outside_notifier causes the top test to fail. It doesn't set the search as active even though triggerSearch is being called in the top most test. Any insights as to why?16:39
didrocksSaviq: ok, I'm writing that :)16:39
didrocksthanks!16:39
Saviqdidrocks, yup16:39
Saviqdidrocks, next step - we'll need cameras onto the devices16:40
didrocksSaviq: complete +1, that was so helpful in otto16:40
Saviqdidrocks, if it doesn't help, that is16:40
mhr3_tsdgeos, cause that thing is highly broken, it manages to unload the module but doesn't destroy objects it creates from there16:40
didrocksSaviq: well, generally, way better to see visually at least what happened16:40
tsdgeosmhr3_: and?16:40
Saviqdidrocks, yup16:40
Saviqnic-doffay, nope, sorry16:40
tsdgeosmhr3_: does that crash or something?16:40
mhr3_tsdgeos, yes, cause i spawn a thread16:41
tsdgeosboo16:41
tsdgeosmhr3_: on module load?16:41
mhr3_after a timeout16:41
nic-doffaySaviq, clicking it manually with the mouse causes the tests to pass.16:41
Saviqdednick, http://pastebin.ubuntu.com/6562236/ no worky :/16:41
nic-doffayBut that's just the same as doing triggerSearch?16:41
tsdgeosmhr3_: can you delay the spawning until it's really needed? would that help16:42
tsdgeos?16:42
Saviqnic-doffay, would need a deeper look, sorry no can do right now16:42
mhr3_tsdgeos, it's needed when you instantiate the class :P16:42
Saviqdednick, and it does end up broken still16:42
tsdgeosmhr3_: then i think someone should fix qmlpluginloader thingie16:42
tsdgeosmhr3_: or maybe it even got fixed in 5.2?16:44
Saviqtsdgeos, mhr3_, we probably should only be running qmlplugindump manually16:44
Saviqtsdgeos, mhr3_, same as with qdbusxml2cpp16:44
Saviqstuff like that16:44
mhr3_Saviq, you mean to put the generated stuff in bzr?16:44
Saviqmhr3_, yes16:44
mhr3_yea, that'd work16:44
Saviqmhr3_, they're not really meant to be ran automagically16:45
tsdgeosbut everyone does :D16:45
dednickSaviq: :/ i have no idea how that ends as undefined... you can assign null to var surely?16:45
Saviqdednick, to var, yes - it tries to assign to UnityMenuModel16:46
Saviqdednick, so that actual error is from UnityMenuAction16:46
dednickSaviq: it's assigning a undefined to a object pointer..16:46
Saviqdednick, should be assigning null instead16:46
Saviqah it is null16:46
dednickbut it shouldnt be undefined.16:46
Saviqdednick, something then changes it to undefined16:47
Saviqdednick, but seems unrelated to the fact that I can still reproduce the issue, though16:47
dednickyeah16:47
dednickSaviq: you still have the qmenumodel update on phone right?16:47
Saviqdednick, hmm16:49
Saviqdednick, right, I might not have16:49
Saviqdednick, I probably don't - forgot about this16:49
dednick*cross fingers*16:49
Saviqdednick, yeah, that makes sense16:50
Saviqdednick, that warning, though, would be good to track down anyway16:50
Cimimzanetti, ideas?16:51
=== dandrader|afk is now known as dandrader
mzanettiCimi: still searching... In Panel/MenuContent.qml there is a fixme which might have influence on this16:52
* Cimi reads16:52
mzanettireferring to a QTBUG... Which btw waits on input from us16:52
Saviqdednick, ok, looking good, sorry for the noise16:53
dednickSaviq: yay! i'll try sort out the warning though16:54
Saviqdednick, other branch?16:54
Saviqdednick, or same?16:54
dednickSaviq: er. give me a few minutes and i might be able to put it in same.16:55
Saviqdednick, ok16:55
mhr3_tsdgeos, not much changed in 5.2 for plugindump16:58
tsdgeosok16:58
dednickSaviq: was that in tests it was complaining?17:00
Saviqdednick, no, when running17:00
mhr3_Saviq, btw re basename() re-revert, we ok to get it back? or should it wait for the ap to get committed?17:00
Saviqmhr3_, let's get a test17:00
Saviqmzanetti, we're not waiting for this, are we ↑?17:00
mhr3_i'm surprised that it still works though :P17:01
mzanettiSaviq: waiting for what?17:01
Saviqmzanetti, the baseName vs. completeBaseName thing17:02
Saviqmzanetti, in unity-scopes-shell17:02
Saviqmzanetti, was it cleanup / in-preparation for other things?17:02
dednickSaviq: no idea. it's not happening for me.17:02
dednickSaviq: i'll have to fix in another branch17:02
dednickand test on phone17:02
mzanettiSaviq: yeah... my new branch needs it and also is meant to clean stuff up.17:03
dednickSaviq: can you check if it still will connect to network if you select access point?17:03
mzanettiSaviq: but no. we can get this in now. should not wait for my current branch17:03
cwaynedavidcalle: hey, what happened to the launchpad scope?17:04
Saviqmzanetti, I know we *can* get it in17:04
Saviqmzanetti, question was whether we want to wait for ap test17:04
Saviqmzanetti, and I said yes, as there's nothing pressing us right now to get it in?17:05
mhr3_Saviq, also json-defaults ;)17:05
mzanettiok...17:05
Saviqmhr3_, yeah, tomorrow17:05
Saviqdednick, i definitely get "menuModelChanged: undefined" from onMenuModelChanged: console.log("menuModelChanged:", menuModel) in MenuItemFactory17:06
Saviqdednick, on startup17:06
Saviqdednick, and on indicator unload17:07
dednickSaviq: hm, yeah, me too17:08
davidcallecwayne, python217:08
davidcallecwayne, tied to the python2 launchpad API17:09
MacSlowok, see you folks in a few weeks!17:09
cwaynedavidcalle: ah, is launchpadlb not ported yet?17:09
Saviqdednick, anyway, doesn't affect functionality17:09
Saviqdednick, acking17:09
davidcallecwayne, I think it's blocked on some other python lib... zope maybe.17:10
cwaynedavidcalle: ah, makes sense, thanks17:10
cwayneis that a good thing to assume for most of the python ones that havent made it in yet?17:11
dednickSaviq: give me 2 secs. got it fixed17:12
Saviqdednick, oh cools17:12
davidcallecwayne, most of them, yeah, others reasons are : aggressive scraping, touchy content provider17:12
robruSaviq, ping about bug 1258655. will it be fixed today? I can land it today if it's fixed in the next 10ish hours.17:13
ubot5bug 1258655 in Mir "Abort when blanking/unblanking screen; exception thrown from mir::graphics::android::HWCCommonDevice::mode(MirPowerMode)" [Critical,In progress] https://launchpad.net/bugs/125865517:13
davidcallecwayne, and bad performance (unreliable APIs, slow, etc)17:13
Saviqrobru, kdub is working on that one17:14
Saviqrobru, he's closer to your time zone, so could very well happen17:14
dednickSaviq: done.17:14
cwaynedavidcalle: yeah, makes sense, do we have like a list or some doc on the status of individual scopes?17:14
robruSaviq, ok, thanks.17:14
Saviqrobru, bug #1253810 is almost there17:14
ubot5bug 1253810 in Unity 8 "Messages in Incoming not always display the correct date and content" [Critical,In progress] https://launchpad.net/bugs/125381017:14
robrukdub, please ping me when it's done17:14
dednickSaviq: kinda weird though...17:15
davidcallecwayne, there is a doc, a bit outdated. There is also this tool, which is not updated either, but is closer to something useful and easy to tweak. http://framli.eu/scopes_browser/index.html?scope=unity-scope-clementine17:15
dednicki'm guessing the ListView doesnt register a null model as a change when it's already a empty QVariant17:16
=== jasoncwarner__ is now known as jasoncwarner
=== pete-woods1 is now known as pete-woods
Saviqdednick, right, so you're effectively converting an invalid mainMenu.model into null17:17
=== alesage is now known as alesage|brb
=== alesage|brb is now known as alesage
Saviqdednick, it might be that ListView.model will always be undefined, even if you assign null indeed17:17
dednickSaviq: yeah, it's a bit dodge.17:17
dednickSaviq: the menuModel should probably just be q QtObject17:18
dednickSaviq: although same problem at lower level...17:18
Saviqdednick, yeah17:18
Saviqrobru, FWIW, the Mir bug is not a unity8 one, so those can be released separately17:19
davidcallecwayne, updating it right now with a script... but it's been a long time: will probably fail ;)17:19
Saviqrobru, hence marked invalid17:19
cwaynedavidcalle: :)17:19
kdubrobru, okay17:19
Saviqdednick, probably worth fixing, too:17:20
SaviqWARNING: ListItems.Standard.icon is DEPRECATED. Use iconName and iconSource instead.17:20
Saviqdednick, and well:17:21
SaviqMenuContent.qml:105:21: QML Loader: Binding loop detected for property "indexActive"17:21
Saviqdednick, indicators are pretty vocal, if you ask me!17:21
Saviq;)17:21
dednickSaviq: yeah, working on that :)17:23
Saviqok17:25
Saviqdednick, thanks for that!17:25
Saviq/afk17:25
=== alan_g is now known as alan_g|EOD
kgunnSaviq: what mir bug ?19:30
kgunn> FWIW, the Mir bug is not a unity8 one, so those can be released separately19:31
cwaynehas there been any progress on shipping scopes as click packages?19:44
mhr3cwayne, not yet20:07
mhr3cwayne, plus i'd say that primary issue is that there's no design decision on where a scope would be added20:08
cwaynemhr3: what do you mean?20:08
mhr3i mean what should happen when you install a scope20:09
mhr3where does it "go"20:09
cwaynemhr3: hm, should we get together with design? or should this wait until the api is further along anyway?20:11
mhr3we brought it up last week20:11
mhr3so it's on radar20:12
cwaynemhr3: awesome, thanks for the info :)20:15
=== _salem is now known as salem_
=== anpok_ is now known as anpok|afk
=== salem_ is now known as _salem
Saviqkgunn, the blanking one on mako22:11
Saviqkgunn, https://launchpad.net/bugs/125865522:11
ubot5Ubuntu bug 1258655 in Mir "Abort when blanking/unblanking screen; exception thrown from mir::graphics::android::HWCCommonDevice::mode(MirPowerMode)" [Critical,In progress]22:11
kgunnah22:11

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