[07:25] tsdgeos, hey, one thing I noticed on overview → go to Ubuntu Store, bottom swipe (so temp scope to overview) [07:25] tsdgeos, the overview seems to cross-fade with... the dash? [07:25] tsdgeos, feels like it should just be there at full-opacity already [07:26] let me see [07:28] tsdgeos, minor nitpick, Icon { name: "ubuntu-store-symbolic", color: "white" } would give you the app store icon (for the "Store" button in the bottom bar) colourized [07:32] ok [07:36] meh just when I thought the issue with lost input is gone, it happens again [07:38] Saviq: i don't get any scope :S [07:38] mzanetti, yeah, it's not :| [07:38] tsdgeos, ?? [07:38] tsdgeos, silo 11? [07:38] no [07:38] distro + dash_overview [07:39] uh oh [07:40] http://paste.ubuntu.com/7986661/ [07:41] good day for mhr3 not being with us anymore [07:43] ok, i manually started smart-scopes-proxy and seems better now [07:44] Saviq: does "App Store" button work for you in overview? [07:44] does nothing here [07:44] :S [07:44] tsdgeos, yes, works fine [07:44] oh man [07:45] let's get silo 11 in case it has something for that [07:45] which i don't think [07:46] tsdgeos, yeah shouldn't matter [07:46] * Saviq tries on desktop with no silo 11 [07:46] maybe my scopes are just borked [07:46] and i need a restart or something to work [07:47] i'm just doing performQuery [07:47] and expecting the scopes scope to come back to me with a "oh sure let's show this scope" [07:47] tsdgeos, did you restart scope-registry as well? [07:47] sure [07:49] Saviq, http://paste.ubuntu.com/7986711/ [07:49] card_emblems :( [07:49] Cimi, they will all be the same conflict most probably [07:50] nothing [07:50] let me try weaving [07:50] Cimi, but not sure what you want me to do, that's life [07:50] let me try to restart :S [07:50] Saviq, kill me would be nic [07:50] nice [07:52] Saviq, I work on branches for month, other branches get reviewed but not mines [07:52] so I spend ages fixing conflicts and fixing bugs that those branches introduce [07:52] Cimi, dude, I did review that before, you were away [07:52] Cimi, stop whining, all of us have to deal with conflicts [07:53] Cimi, and sorry if your branch has lower priority than other, large merges, but that's just how it is [07:53] Cimi, there's 10 of us working on the same project, there's no way to not have conflicts [07:56] Cimi, please fix those, I'll review your branch today, was planning to anyway [07:56] niet [07:57] no workie for the click in scope in overview [07:57] Cimi, and sorry for being blunt, but it's really not nice to hear you complain about conflicts for 2 weeks every day [07:57] Cimi, even if it's not true, that's how it feels :/ [07:57] Saviq, because my branch sits in the dust and I have to do every day extra work [07:57] I'd rather work on the features instead fixing conflicts [07:57] Cimi, care to see others' branches how long they've been in the queue? [07:58] Cimi, do you think someone's targeting your branches in particular? [07:58] hah no :) [07:59] tsdgeos, works here [07:59] yeah [07:59] i'll debug it [07:59] see what's happening [07:59] Cimi, imagine tsdgeos when he had to rebase dash overview on top of dash as app, think that was an easy task? did he mention it once? [08:00] tsdgeos, actually the "temp to overview" dash leaking through doesn't happen on desktop ?¿ [08:00] Saviq, I'd complain if it wasn't probably the 5th time I have to rebase emblems... [08:00] hmm or on mako [08:00] I wouldn't [08:01] Cimi, it's bad timing, is all, you came back during the sprint, when all of those things were already happening [08:01] Saviq: ?¿ [08:01] tsdgeos, ooor on my other device, any more... [08:02] :D [08:02] and back [08:02] whoa that's confusing [08:03] apps → store → overview → store... hmm, why can't I bottom-swipe any more? ah, because that is opened on top of the overview... [08:03] the navigation dots are quite ugly, were they design approved? [08:03] Cimi, the bar is a wrong colour [08:03] ok [08:03] Cimi, and they will have to change between light and dark based on the bar colour [08:04] Cimi, so yeah, not compleet [08:04] Saviq: aha, i know why it doesn't work for me [08:04] because i have the store as a "favorite" [08:05] tsdgeos, [08:05] so fixing that [08:07] tsdgeos, ok yeah, dash leaking through is happening until something fixes it... until something breaks it again [08:07] tsdgeos, like start → store → overview already shows it [08:09] booo [08:09] ok [08:09] let me see [08:09] that worked fine at some point [08:09] tsdgeos, I'm not even sure we need the opacity change there at all... [08:09] that probably broke when i did the shader thing [08:10] well, that probably is the shader thing basically [08:10] that shouldn't be showing at all :D [08:10] yeah, thought so [08:10] yeap [08:11] yeah refactoring [08:11] hehe [08:12] ok pushed [08:12] should be better now === vrruiz_ is now known as rvr [08:40] +1 [08:47] Saviq: handle is in [08:47] tsdgeos, awesomes, checking [08:47] looks a bit out of place tbh [08:48] tell me what you think and how you'd improve it [08:48] tsdgeos, well, the asset looks wrong for one [08:49] tsdgeos, is visible in preview where you can't swipe [08:49] tsdgeos, possible to fade out on bottom swipe instead of invisible? [08:50] Saviq: looks ultra bad [08:50] Saviq: comment out the enabled: overviewController.progress == 0 [08:50] in DAsh.qml [08:50] Saviq: well it's the asset i got :D [08:52] tsdgeos, sure ;) [08:52] Saviq: preview enabledness removed [08:54] ktx [08:54] Saviq, conflicts resolved [08:55] Cimi, k, will review asap [08:55] thanks :) [08:58] tsdgeos, in temp scopes it doesn't go away when you scroll up [08:58] no? [08:58] that's weird [08:58] or down [08:58] whichever way that is [08:59] yep [08:59] checking [08:59] tsdgeos, maybe it should slide down out of view instead of opacity? [09:00] Saviq: i think opacity works good enough for the vertical movement [09:00] Saviq: but maybe down works for when actually opening the overview? [09:00] so you have two different behaviour [09:00] becuase it's using it vs "hiding" it? [09:00] tsdgeos, yeah that might be [09:01] let me see why temp scopes doesn't work [09:01] kk [09:01] and i'll impement both [09:01] and then we can play with it [09:04] Saviq: temp scope fixed [09:07] kk [09:09] mzanetti, when you want to talk... [09:10] Cimi: need to try some more things... [09:11] ok [09:11] dandrader: hey, is your branch already in a state where I could use it as a prereq? [09:11] Saviq, https://code.launchpad.net/~fboucault/ubuntu-ui-toolkit/flickable_right_speed/+merge/229343 [09:11] mzanetti, yes! [09:11] cool [09:11] Cimi, yeah, I know, what about it? [09:11] tsdgeos, ideas? https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1348557 [09:11] Ubuntu bug 1348557 in Ubuntu UI Toolkit "Make scrolling speed resolution independent" [High,Confirmed] [09:12] Saviq, cannot be implemented that way [09:12] mzanetti, just finishing updating the vkb mock and will propose it [09:12] Saviq, florian ran out of ideas [09:12] ah, cool. I'll do my daily review then in the meantime [09:12] Cimi, ugh, attached properties don't go through? [09:12] right, damn [09:12] although weird [09:13] apparently only patching qt [09:15] Cimi, TBH I think that's really what we should do, actually fix the original bug upstream, not work around it [09:23] Saviq, should be a RTM requirementr [09:25] Saviq: shouldn't this happen automagically? https://code.launchpad.net/~pitti/unity8/update-pot/+merge/230072 [09:26] mzanetti, if we do this on every push, you'll get the comment change all the time [09:26] mzanetti, with line numbers [09:26] mzanetti, I think ideally I'd like that to happen per release [09:26] Saviq: no... I meant some launchpad bot doing this [09:26] mzanetti, no, it only does translations [09:27] ah [09:27] mzanetti, it can't do .pot file since it can't know how to do it [09:27] mzanetti, but yeah, a pre-build hook in train would probably be best for this [09:27] * Saviq files a bug [09:27] ah... right... that's why KDE has the convention of a "messages.sh" script in each repo, so the bot knows what to do [09:28] Saviq: anyways, I'll approve that branch then... looks ok [09:28] yup [09:34] tsdgeos, shall I rebuild the silo or are you doing the tab show/hide? [09:34] Saviq: ah, i already did that [09:34] tsdgeos, ok, pulling ;) [09:35] tsdgeos, sounds like you're off the hook for overview now, then [09:35] Saviq: until you find more stuff to fix ;) [09:35] tsdgeos, yeah, but I didn't ;) [09:36] ok, let's see if i can get a new image on $thephone and SIM unlocking works [09:36] tsdgeos, FWIW I wouldn't have made the tab "follow" the gesture, rather a binary shown/hidden, but that's minor [09:37] it was easir this way :D [09:37] tsdgeos, how difficult do you think it would be to hook up the activity indicator to the overview? [09:37] tsdgeos, ok, fine [09:37] Saviq: in which regard, for search? [09:37] tsdgeos, yes [09:38] hopefully not much [09:38] let me see [09:39] tsdgeos, it should be as easy as || overview.processing [09:39] yeah [09:41] yeah done [09:42] awesomes [09:45] Saviq: hey, about this: https://code.launchpad.net/~macslow/unity8/fix-1305885/+merge/227726 [09:45] Saviq: Mirco says we can't have qmltests for this kind of stuff [09:46] which seems odd... [09:46] Saviq: should we approve it nevertheless and add a TODO item for after RTM to properly mock out the stuff? [09:47] mzanetti, yeah [09:47] ack [09:47] tsdgeos, looks great, nothing else I can see broken [09:47] lol... our alarm ringtone really sounds like a lullaby [09:48] mzanetti, Saviq: also added a corresponding comment [09:49] Saviq: great! === shiznix_ is now known as shiznix [09:50] mzanetti, how do you delete an alarm? [09:50] Cimi: --wipe :D [09:50] Cimi: dunno tbh :) [09:50] ahah [09:50] indeed [09:50] what a bad UX [09:51] Cimi: swipe it to the right [09:51] undiscoverable [09:52] and many times you flick down instead right [09:52] it should be in a repeater not listview [09:53] Cimi: yeah, right [09:53] Cimi: however, the clock app is being "rebooted" atm [09:53] as in rewritten from scratch [09:56] Cimi, hmm, I removed emblem in tryCard from the data part (not template), emblem still there? [09:57] Saviq, emblems everywhere! [09:57] Saviq, I'll have a look... [09:57] Cimi, also, if emblem is defined, you always leave the space for it (wrap title where emblem would be) [09:57] Cimi, we can have a category with optional emblems, and we should only wrap title around the emblem where it's actually there, should be 0-wide otherwise [09:58] Cimi, also tryDash broken? [09:59] god [09:59] /food [09:59] all broken now [10:01] lol [10:01] i rebooted the phone [10:01] opened the greeter "too soon" [10:02] and got the "Unity Dash" booting dots animating [10:03] Saviq, testDash breaks because of the change to attributes we did [10:04] var hasAttributes = hasTitle && components["attributes"] || false; -> var hasAttributes = hasTitle && components["attributes"]["field"] || false; [10:06] Cimi: are you basing your stuff on top of dash_overview? or trunk? [10:08] tsdgeos, trunkl [10:09] MacSlow: hmm... I just realize you're reaching out of context again... [10:09] MacSlow: there is no "notification" in NotificationMenuItemFactory.qml [10:09] Cimi: you may try dash_overview, it has some substantial changes too and is next-thing-to-land afaik [10:09] mzanetti, ? [10:09] Cimi: but confirm with Saviq first [10:10] mzanetti, being on another branch atm [10:10] tsdgeos, we need to move the processing indicator in Components [10:10] have you done that already? [10:10] MacSlow: well, open NotificationMenuItemFactory.qml and search for "notification" [10:10] there isn't such a thing [10:10] yet you're accessing it [10:11] which is bad practice and makes the code a mess in the long run [10:11] why not just adding the accepted() signal to the NotificationMenuItemFactory and connecting to that? [10:11] mzanetti, well the whole UnityMenuModel is something that's not meant to be kept around in notifications [10:14] make testDash fails on trunk for me [10:14] let me try to dist-upgrade first.. [10:26] yeah testDash is broken in trunk... [10:26] qmltestrunner::Dash::test_processing_indicator() and qmltestrunner::Dash::test_setCurrentScope() [10:27] Cimi: fix it :P [10:29] waiting saviq for a question on attributes [10:31] tsdgeos, are you exporting the processing indicator to Components? [10:32] or somewhere under qml/Dash/ ? [10:32] Cimi: which processing indicator? [10:32] the orange line? [10:32] tsdgeos, yup [10:32] there is only one [10:32] doesn't need to be exported [10:32] tsdgeos, we need it in the overview [10:32] it is in the overview [10:32] cool [10:32] there is only one [10:32] anyway [10:33] tsdgeos, thought it was outside of it [10:33] fine then [10:33] it is outside of it [10:33] there's only one [10:33] but it tells it to show [10:36] Cimi: not sure i'm explaining myself tbh [10:39] tsdgeos, when is your branch going to be merged? (really)? [10:39] tsdgeos, wondering if scope settings will be reviewed/merged before yours or not [10:40] and which needs to rebase... [10:40] Cimi: before dash_overview? [10:40] dash_overview should go in today afaik [10:40] ok [10:40] there is lot of things connected [10:40] I'll rebase on yours [10:41] Cimi, not sure what guidance do you need from me? [10:41] Saviq, testDash is broken in trunk for me [10:41] can you test? [10:41] could be due to scope v4 or dash indicator [10:41] then I have another quesitojn [10:42] Cimi, indeed two failures [10:42] Cimi, I think those get fixed with dash overview though [10:42] ehm... folks... -> http://pastebin.ubuntu.com/7987781 [10:42] we said components["attributes"] were always defined, and the way to distinguish was with components["attributes"]["field"], but in testDash this fails, where can we see how the mock processes the data? [10:42] MacSlow, yeah, merge trunk [10:43] trying to compile unity8 and really just branched [10:43] Saviq, I'm pretty sure I did just that... [10:43] mzanetti, dist-upgrade [10:43] MacSlow, trunk already asks for v4 of the scopes api, so you're not up to date [10:43] MacSlow, you may try removing builddir/CMakeCache.txt in case cmake got confused [10:43] Cimi, he is upgraded, but cmake's looking for v3 instead of v4 [10:44] Saviq, on components["attributes"]["field"] ? [10:45] Cimi, fake_categories.cpp needs to always return ["attributes"] [10:45] ok [10:45] I'll add that to card emblems [10:47] Saviq, which value for that? [10:47] Cimi, just an empty map will suffice [10:47] Cimi, see what art is doing [10:48] gatox, hey, you pung yesterday, you got along with the indicator in the end? [10:48] Cimi, yeah, testDash is fixed in dash overview [10:50] Saviq, yes, thx, i was able to find it in unity [10:52] Saviq, I've been able to compile unity8 now... but I can't start it any more... -> http://pastebin.ubuntu.com/7987829 (also deleting that file LightDM) doesn't help. What changed in the way unity8 is meant to be started? [10:52] gatox, truth be told I didn't understand your branch from a quick look, was it that the Payments button does give up a "processing" boolean that we should show the activity indicator on? [10:53] MacSlow, you don't have unity8 installed [10:53] MacSlow, you need it installed or copy the data/unity8{,-dash}.conf to ~/.config/upstart/ [10:53] MacSlow, but ./run.sh isn't working too great yet with the dash as separate process, I've a branch that needs some love [10:54] Saviq, hm... didn't touch my ~/.config/upstart in weeks... wonder how the unity8.conf was wiped [10:55] MacSlow, you don't need it there if unity8 is installed, you must've removed it somehow [10:55] Saviq, i propose the branch to have jenkins building it for me, because i was having issues building it on the phone yesterday... i wasn't able to test it yet, but i think it works... the idea is to have a signal to show the overlay indicator as when the preview is loading, when the preview is launching the pay-service or anything in the future, because right now opening applications take a while (and pay service needs to open pay-ui and [10:55] sometimes online accounts), and for some moments you have a preview where it seems nothing is going on, and it's confusing for the user [10:56] gatox, yeah, thing is that should already be happening, at least for actions that involve the scope (like pressing install) [10:56] gatox, I'll have a closer look, thanks [10:58] Saviq, ah ok, so this branch shouldn't be necessary?? let me know if we need to do something or wait for that to happen in unity, thx [11:01] gatox, this might be a special case actually (pay being a special button, opening urls taking a moment) [11:01] gatox, so it might still make sense indeed [11:03] Saviq, fixed categories mock [11:03] in emblems [11:09] Saviq, will we merge overview today? [11:09] Saviq, wondering where to rebase settings === facundobatista_ is now known as facundobatista [11:10] Cimi, rebase on overview, yeah, even if not today, that's the next thing that's going in [11:10] Saviq, yes but if albert is holiday next week [11:10] Saviq, we might have settings in before overview [11:10] Cimi, no, because we won't wait for Albert to fix any issues in there, we need this in asap [11:10] Holas [11:10] ok [11:10] \o [11:11] dednick, on that note, dash overview could use a re-review, would you be able to do it today? [11:12] Saviq: can do [11:14] gtg guys, mobo replacement [11:14] biab === MacSlow is now known as MacSlow|lunch [11:17] pete-woods: ping [11:20] greyback, Saviq, any success on the date time issue? [11:21] olli: I've tracked down root cause to be a workaround I added to qtubuntu for bug 1346633 [11:21] bug 1346633 in QtMir "[enhancement] Autopilot testing needs to know screen coordinates of widgets on screen" [Undecided,New] https://launchpad.net/bugs/1346633 [11:21] olli: so I'm working on a different workaround [11:22] heh [11:22] greyback, thx! [11:22] how bad is it... hours, days...? [11:22] olli: I'll keep you posted. In the mean time, SK guys has a workaround ready https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/pickerpanel-workaround/+merge/230065 [11:22] olli: hours hopefuly [11:23] greyback, OK [11:25] Saviq, just to be sure.... should i keep working on my branch, testing it, etc? or should wait until you tell me that is being covered or not by unity already? [11:26] tsdgeos: hi [11:27] pete-woods: your'e the usermetrics guy, right? [11:27] tsdgeos: yep [11:27] pete-woods: i see in the metrics manager you accept a domain [11:27] is it mandatory that messages go untranslated and you translate them [11:27] or can messages come in translated too? [11:27] or is domain not the translation domain? [11:27] :D [11:27] tsdgeos: nope, feel free to do whatever really [11:28] you're correct, it is the translation domain [11:28] ok, so "" and a translated mesage is also ok [11:28] tsdgeos: it's preferable to do untranslated [11:28] as if you change language then things will be translated [11:28] but it's not the end of the world if it comes in translated [11:29] pete-woods: will a reboot fix it? [11:29] tsdgeos: to fix it would require whoever is writing the data to re-run [11:29] ah [11:29] ok, so better untranslated + domain [11:29] tx === dandrader is now known as dandrader|afk [11:44] can I create a binding from the loaded component of a loader to its loader? [11:47] Saviq: could I steal 10 mins of your time? [11:47] I'd need some second opinions [11:48] Cimi: sure === lool- is now known as lool [11:49] can I do sth like readonly property bool processing: item.hasOwnProperty("processing") ? item.processing : false ? [11:49] Loader { readonly property bool processing: item.hasOwnProperty("processing") ? item.processing : false } [11:50] creating binding inside onLoaded breaks on unloaded state [11:51] Cimi: yeah, should work I guess. You might want to add "item && item.hasOwnProp..." [11:52] oh yeah correct [11:52] or maybe status === Loader.Ready instead item? [11:52] whatever you prefer [11:52] should end up being the same [11:53] Saviq, ping? [11:58] on one of my Unity 8 desktops all I get is a box that says "Hello" and has a text input ... anyone know what that's all about? === alan_g is now known as alan_g|lunch [12:10] dandrader|afk: when you get back, I need your input on https://code.launchpad.net/~gerboland/qtubuntu/fix_1351024/+merge/230094 ASAP [12:20] bregma: I think that's a password prompt [12:20] I get that when I run unity8 directly atm [12:21] not sure exactly why, though [12:21] well, it lack a certain amount of usefulness at the moment [12:22] fortunately my other test machine always brings up the online accounts app, which barfs all over and dies immediately, bypassing the greeter/lockscreen and ringing up the dash [12:22] sometimes three wrongs do make a right === MacSlow|lunch is now known as MacSlow [12:29] Saviq, pete-woods some settings seems to be undefined by default value... what shall we do in the UI? true or false? [12:30] unless model.value is expected to be undefined and we want .defaultValue or so === greyback is now known as greyback|lunch [12:33] back [12:34] mzanetti, gatox, back [12:36] Saviq, hi... i just wanted to know if i should keep working on that branch or expect unity to solve the button issue for payment too [12:36] gatox, if in current trunk it doesn't, there's nothing in queue to fix that [12:37] Saviq, ack, i'll have this working then [12:38] Cimi: you shouldn't have to handle it in the UI [12:38] Cimi: which settings, specifically are like that? [12:40] pete-woods: is there a plan to update the youtube scope in the store to the new api? [12:40] or are you waiting for a promoted image [12:40] cwayne: it's already been updated [12:41] it has been in the store a while now [12:41] huh, i didnt see an update, weird [12:41] oh well, thanks :D [12:41] np :) [12:41] 1.0.7 is what you're looking for [12:45] excellent, thanks again [12:45] Cimi: do you expect emblems to land with dash-overview? [12:45] cwayne, not up to me [12:45] cwayne, under reviews.. [12:45]