Saviq | tsdgeos, hey, one thing I noticed on overview → go to Ubuntu Store, bottom swipe (so temp scope to overview) | 07:25 |
---|---|---|
Saviq | tsdgeos, the overview seems to cross-fade with... the dash? | 07:25 |
Saviq | tsdgeos, feels like it should just be there at full-opacity already | 07:25 |
tsdgeos | let me see | 07:26 |
Saviq | 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:28 |
tsdgeos | ok | 07:32 |
mzanetti | meh just when I thought the issue with lost input is gone, it happens again | 07:36 |
tsdgeos | Saviq: i don't get any scope :S | 07:38 |
Saviq | mzanetti, yeah, it's not :| | 07:38 |
Saviq | tsdgeos, ?? | 07:38 |
Saviq | tsdgeos, silo 11? | 07:38 |
tsdgeos | no | 07:38 |
tsdgeos | distro + dash_overview | 07:38 |
tsdgeos | uh oh | 07:39 |
tsdgeos | http://paste.ubuntu.com/7986661/ | 07:40 |
tsdgeos | good day for mhr3 not being with us anymore | 07:41 |
tsdgeos | ok, i manually started smart-scopes-proxy and seems better now | 07:43 |
tsdgeos | Saviq: does "App Store" button work for you in overview? | 07:44 |
tsdgeos | does nothing here | 07:44 |
tsdgeos | :S | 07:44 |
Saviq | tsdgeos, yes, works fine | 07:44 |
tsdgeos | oh man | 07:44 |
tsdgeos | let's get silo 11 in case it has something for that | 07:45 |
tsdgeos | which i don't think | 07:45 |
Saviq | tsdgeos, yeah shouldn't matter | 07:46 |
* Saviq tries on desktop with no silo 11 | 07:46 | |
tsdgeos | maybe my scopes are just borked | 07:46 |
tsdgeos | and i need a restart or something to work | 07:46 |
tsdgeos | i'm just doing performQuery | 07:47 |
tsdgeos | and expecting the scopes scope to come back to me with a "oh sure let's show this scope" | 07:47 |
Saviq | tsdgeos, did you restart scope-registry as well? | 07:47 |
tsdgeos | sure | 07:47 |
Cimi | Saviq, http://paste.ubuntu.com/7986711/ | 07:49 |
Cimi | card_emblems :( | 07:49 |
Saviq | Cimi, they will all be the same conflict most probably | 07:49 |
tsdgeos | nothing | 07:50 |
Cimi | let me try weaving | 07:50 |
Saviq | Cimi, but not sure what you want me to do, that's life | 07:50 |
tsdgeos | let me try to restart :S | 07:50 |
Cimi | Saviq, kill me would be nic | 07:50 |
Cimi | nice | 07:50 |
Cimi | Saviq, I work on branches for month, other branches get reviewed but not mines | 07:52 |
Cimi | so I spend ages fixing conflicts and fixing bugs that those branches introduce | 07:52 |
Saviq | Cimi, dude, I did review that before, you were away | 07:52 |
Saviq | Cimi, stop whining, all of us have to deal with conflicts | 07:52 |
Saviq | Cimi, and sorry if your branch has lower priority than other, large merges, but that's just how it is | 07:53 |
Saviq | Cimi, there's 10 of us working on the same project, there's no way to not have conflicts | 07:53 |
Saviq | Cimi, please fix those, I'll review your branch today, was planning to anyway | 07:56 |
tsdgeos | niet | 07:56 |
tsdgeos | no workie for the click in scope in overview | 07:57 |
Saviq | 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 |
Saviq | Cimi, even if it's not true, that's how it feels :/ | 07:57 |
Cimi | Saviq, because my branch sits in the dust and I have to do every day extra work | 07:57 |
Cimi | I'd rather work on the features instead fixing conflicts | 07:57 |
Saviq | Cimi, care to see others' branches how long they've been in the queue? | 07:57 |
Saviq | Cimi, do you think someone's targeting your branches in particular? | 07:58 |
Cimi | hah no :) | 07:58 |
Saviq | tsdgeos, works here | 07:59 |
tsdgeos | yeah | 07:59 |
tsdgeos | i'll debug it | 07:59 |
tsdgeos | see what's happening | 07:59 |
Saviq | 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? | 07:59 |
Saviq | tsdgeos, actually the "temp to overview" dash leaking through doesn't happen on desktop ?¿ | 08:00 |
Cimi | Saviq, I'd complain if it wasn't probably the 5th time I have to rebase emblems... | 08:00 |
Saviq | hmm or on mako | 08:00 |
Cimi | I wouldn't | 08:00 |
Saviq | Cimi, it's bad timing, is all, you came back during the sprint, when all of those things were already happening | 08:01 |
tsdgeos | Saviq: ?¿ | 08:01 |
Saviq | tsdgeos, ooor on my other device, any more... | 08:01 |
tsdgeos | :D | 08:02 |
Saviq | and back | 08:02 |
Saviq | whoa that's confusing | 08:02 |
Saviq | 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 |
Cimi | the navigation dots are quite ugly, were they design approved? | 08:03 |
Saviq | Cimi, the bar is a wrong colour | 08:03 |
Cimi | ok | 08:03 |
Saviq | Cimi, and they will have to change between light and dark based on the bar colour | 08:03 |
Saviq | Cimi, so yeah, not compleet | 08:04 |
tsdgeos | Saviq: aha, i know why it doesn't work for me | 08:04 |
tsdgeos | because i have the store as a "favorite" | 08:04 |
Saviq | tsdgeos, <facepalm> | 08:05 |
tsdgeos | so fixing that | 08:05 |
Saviq | tsdgeos, ok yeah, dash leaking through is happening until something fixes it... until something breaks it again | 08:07 |
Saviq | tsdgeos, like start → store → overview already shows it | 08:07 |
tsdgeos | booo | 08:09 |
tsdgeos | ok | 08:09 |
tsdgeos | let me see | 08:09 |
tsdgeos | that worked fine at some point | 08:09 |
Saviq | tsdgeos, I'm not even sure we need the opacity change there at all... | 08:09 |
tsdgeos | that probably broke when i did the shader thing | 08:09 |
tsdgeos | well, that probably is the shader thing basically | 08:10 |
tsdgeos | that shouldn't be showing at all :D | 08:10 |
Saviq | yeah, thought so | 08:10 |
tsdgeos | yeap | 08:10 |
tsdgeos | yeah refactoring | 08:11 |
tsdgeos | hehe | 08:11 |
tsdgeos | ok pushed | 08:12 |
tsdgeos | should be better now | 08:12 |
=== vrruiz_ is now known as rvr | ||
Saviq | +1 | 08:40 |
tsdgeos | Saviq: handle is in | 08:47 |
Saviq | tsdgeos, awesomes, checking | 08:47 |
tsdgeos | looks a bit out of place tbh | 08:47 |
tsdgeos | tell me what you think and how you'd improve it | 08:48 |
Saviq | tsdgeos, well, the asset looks wrong for one | 08:48 |
Saviq | tsdgeos, is visible in preview where you can't swipe | 08:49 |
Saviq | tsdgeos, possible to fade out on bottom swipe instead of invisible? | 08:49 |
tsdgeos | Saviq: looks ultra bad | 08:50 |
tsdgeos | Saviq: comment out the enabled: overviewController.progress == 0 | 08:50 |
tsdgeos | in DAsh.qml | 08:50 |
tsdgeos | Saviq: well it's the asset i got :D | 08:50 |
Saviq | tsdgeos, sure ;) | 08:52 |
tsdgeos | Saviq: preview enabledness removed | 08:52 |
Saviq | ktx | 08:54 |
Cimi | Saviq, conflicts resolved | 08:54 |
Saviq | Cimi, k, will review asap | 08:55 |
Cimi | thanks :) | 08:55 |
Saviq | tsdgeos, in temp scopes it doesn't go away when you scroll up | 08:58 |
tsdgeos | no? | 08:58 |
tsdgeos | that's weird | 08:58 |
Saviq | or down | 08:58 |
Saviq | whichever way that is | 08:58 |
tsdgeos | yep | 08:59 |
tsdgeos | checking | 08:59 |
Saviq | tsdgeos, maybe it should slide down out of view instead of opacity? | 08:59 |
tsdgeos | Saviq: i think opacity works good enough for the vertical movement | 09:00 |
tsdgeos | Saviq: but maybe down works for when actually opening the overview? | 09:00 |
tsdgeos | so you have two different behaviour | 09:00 |
tsdgeos | becuase it's using it vs "hiding" it? | 09:00 |
Saviq | tsdgeos, yeah that might be | 09:00 |
tsdgeos | let me see why temp scopes doesn't work | 09:01 |
Saviq | kk | 09:01 |
tsdgeos | and i'll impement both | 09:01 |
tsdgeos | and then we can play with it | 09:01 |
tsdgeos | Saviq: temp scope fixed | 09:04 |
Saviq | kk | 09:07 |
Cimi | mzanetti, when you want to talk... | 09:09 |
mzanetti | Cimi: need to try some more things... | 09:10 |
Cimi | ok | 09:11 |
mzanetti | dandrader: hey, is your branch already in a state where I could use it as a prereq? | 09:11 |
Cimi | Saviq, https://code.launchpad.net/~fboucault/ubuntu-ui-toolkit/flickable_right_speed/+merge/229343 | 09:11 |
dandrader | mzanetti, yes! | 09:11 |
mzanetti | cool | 09:11 |
Saviq | Cimi, yeah, I know, what about it? | 09:11 |
Cimi | tsdgeos, ideas? https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1348557 | 09:11 |
ubot5 | Ubuntu bug 1348557 in Ubuntu UI Toolkit "Make scrolling speed resolution independent" [High,Confirmed] | 09:11 |
Cimi | Saviq, cannot be implemented that way | 09:12 |
dandrader | mzanetti, just finishing updating the vkb mock and will propose it | 09:12 |
Cimi | Saviq, florian ran out of ideas | 09:12 |
mzanetti | ah, cool. I'll do my daily review then in the meantime | 09:12 |
Saviq | Cimi, ugh, attached properties don't go through? | 09:12 |
Saviq | right, damn | 09:12 |
Saviq | although weird | 09:12 |
Cimi | apparently only patching qt | 09:13 |
Saviq | Cimi, TBH I think that's really what we should do, actually fix the original bug upstream, not work around it | 09:15 |
Cimi | Saviq, should be a RTM requirementr | 09:23 |
mzanetti | Saviq: shouldn't this happen automagically? https://code.launchpad.net/~pitti/unity8/update-pot/+merge/230072 | 09:25 |
Saviq | mzanetti, if we do this on every push, you'll get the comment change all the time | 09:26 |
Saviq | mzanetti, with line numbers | 09:26 |
Saviq | mzanetti, I think ideally I'd like that to happen per release | 09:26 |
mzanetti | Saviq: no... I meant some launchpad bot doing this | 09:26 |
Saviq | mzanetti, no, it only does translations | 09:26 |
mzanetti | ah | 09:27 |
Saviq | mzanetti, it can't do .pot file since it can't know how to do it | 09:27 |
Saviq | mzanetti, but yeah, a pre-build hook in train would probably be best for this | 09:27 |
* Saviq files a bug | 09:27 | |
mzanetti | 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:27 |
mzanetti | Saviq: anyways, I'll approve that branch then... looks ok | 09:28 |
Saviq | yup | 09:28 |
Saviq | tsdgeos, shall I rebuild the silo or are you doing the tab show/hide? | 09:34 |
tsdgeos | Saviq: ah, i already did that | 09:34 |
Saviq | tsdgeos, ok, pulling ;) | 09:34 |
Saviq | tsdgeos, sounds like you're off the hook for overview now, then | 09:35 |
tsdgeos | Saviq: until you find more stuff to fix ;) | 09:35 |
Saviq | tsdgeos, yeah, but I didn't ;) | 09:35 |
tsdgeos | ok, let's see if i can get a new image on $thephone and SIM unlocking works | 09:36 |
Saviq | tsdgeos, FWIW I wouldn't have made the tab "follow" the gesture, rather a binary shown/hidden, but that's minor | 09:36 |
tsdgeos | it was easir this way :D | 09:37 |
Saviq | tsdgeos, how difficult do you think it would be to hook up the activity indicator to the overview? | 09:37 |
Saviq | tsdgeos, ok, fine | 09:37 |
tsdgeos | Saviq: in which regard, for search? | 09:37 |
Saviq | tsdgeos, yes | 09:37 |
tsdgeos | hopefully not much | 09:38 |
tsdgeos | let me see | 09:38 |
Saviq | tsdgeos, it should be as easy as || overview.processing | 09:39 |
tsdgeos | yeah | 09:39 |
tsdgeos | yeah done | 09:41 |
Saviq | awesomes | 09:42 |
mzanetti | Saviq: hey, about this: https://code.launchpad.net/~macslow/unity8/fix-1305885/+merge/227726 | 09:45 |
mzanetti | Saviq: Mirco says we can't have qmltests for this kind of stuff | 09:45 |
mzanetti | which seems odd... | 09:46 |
mzanetti | Saviq: should we approve it nevertheless and add a TODO item for after RTM to properly mock out the stuff? | 09:46 |
Saviq | mzanetti, yeah | 09:47 |
mzanetti | ack | 09:47 |
Saviq | tsdgeos, looks great, nothing else I can see broken | 09:47 |
mzanetti | lol... our alarm ringtone really sounds like a lullaby | 09:47 |
MacSlow | mzanetti, Saviq: also added a corresponding comment | 09:48 |
tsdgeos | Saviq: great! | 09:49 |
=== shiznix_ is now known as shiznix | ||
Cimi | mzanetti, how do you delete an alarm? | 09:50 |
mzanetti | Cimi: --wipe :D | 09:50 |
mzanetti | Cimi: dunno tbh :) | 09:50 |
Cimi | ahah | 09:50 |
Cimi | indeed | 09:50 |
Cimi | what a bad UX | 09:50 |
mzanetti | Cimi: swipe it to the right | 09:51 |
Cimi | undiscoverable | 09:51 |
Cimi | and many times you flick down instead right | 09:52 |
Cimi | it should be in a repeater not listview | 09:52 |
mzanetti | Cimi: yeah, right | 09:53 |
mzanetti | Cimi: however, the clock app is being "rebooted" atm | 09:53 |
mzanetti | as in rewritten from scratch | 09:53 |
Saviq | Cimi, hmm, I removed emblem in tryCard from the data part (not template), emblem still there? | 09:56 |
Cimi | Saviq, emblems everywhere! | 09:57 |
Cimi | Saviq, I'll have a look... | 09:57 |
Saviq | Cimi, also, if emblem is defined, you always leave the space for it (wrap title where emblem would be) | 09:57 |
Saviq | 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:57 |
Saviq | Cimi, also tryDash broken? | 09:58 |
Cimi | god | 09:59 |
Saviq | /food | 09:59 |
Cimi | all broken now | 09:59 |
tsdgeos | lol | 10:01 |
tsdgeos | i rebooted the phone | 10:01 |
tsdgeos | opened the greeter "too soon" | 10:01 |
tsdgeos | and got the "Unity Dash" booting dots animating | 10:02 |
Cimi | Saviq, testDash breaks because of the change to attributes we did | 10:03 |
Cimi | var hasAttributes = hasTitle && components["attributes"] || false; -> var hasAttributes = hasTitle && components["attributes"]["field"] || false; | 10:04 |
tsdgeos | Cimi: are you basing your stuff on top of dash_overview? or trunk? | 10:06 |
Cimi | tsdgeos, trunkl | 10:08 |
mzanetti | MacSlow: hmm... I just realize you're reaching out of context again... | 10:09 |
mzanetti | MacSlow: there is no "notification" in NotificationMenuItemFactory.qml | 10:09 |
tsdgeos | Cimi: you may try dash_overview, it has some substantial changes too and is next-thing-to-land afaik | 10:09 |
MacSlow | mzanetti, ? | 10:09 |
tsdgeos | Cimi: but confirm with Saviq first | 10:09 |
MacSlow | mzanetti, being on another branch atm | 10:10 |
Cimi | tsdgeos, we need to move the processing indicator in Components | 10:10 |
Cimi | have you done that already? | 10:10 |
mzanetti | MacSlow: well, open NotificationMenuItemFactory.qml and search for "notification" | 10:10 |
mzanetti | there isn't such a thing | 10:10 |
mzanetti | yet you're accessing it | 10:10 |
mzanetti | which is bad practice and makes the code a mess in the long run | 10:11 |
mzanetti | why not just adding the accepted() signal to the NotificationMenuItemFactory and connecting to that? | 10:11 |
MacSlow | mzanetti, well the whole UnityMenuModel is something that's not meant to be kept around in notifications | 10:11 |
Cimi | make testDash fails on trunk for me | 10:14 |
Cimi | let me try to dist-upgrade first.. | 10:14 |
Cimi | yeah testDash is broken in trunk... | 10:26 |
Cimi | qmltestrunner::Dash::test_processing_indicator() and qmltestrunner::Dash::test_setCurrentScope() | 10:26 |
mzanetti | Cimi: fix it :P | 10:27 |
Cimi | waiting saviq for a question on attributes | 10:29 |
Cimi | tsdgeos, are you exporting the processing indicator to Components? | 10:31 |
Cimi | or somewhere under qml/Dash/ ? | 10:32 |
tsdgeos | Cimi: which processing indicator? | 10:32 |
tsdgeos | the orange line? | 10:32 |
Cimi | tsdgeos, yup | 10:32 |
tsdgeos | there is only one | 10:32 |
tsdgeos | doesn't need to be exported | 10:32 |
Cimi | tsdgeos, we need it in the overview | 10:32 |
tsdgeos | it is in the overview | 10:32 |
Cimi | cool | 10:32 |
tsdgeos | there is only one | 10:32 |
tsdgeos | anyway | 10:32 |
Cimi | tsdgeos, thought it was outside of it | 10:33 |
Cimi | fine then | 10:33 |
tsdgeos | it is outside of it | 10:33 |
tsdgeos | there's only one | 10:33 |
tsdgeos | but it tells it to show | 10:33 |
tsdgeos | Cimi: not sure i'm explaining myself tbh | 10:36 |
Cimi | tsdgeos, when is your branch going to be merged? (really)? | 10:39 |
Cimi | tsdgeos, wondering if scope settings will be reviewed/merged before yours or not | 10:39 |
Cimi | and which needs to rebase... | 10:40 |
tsdgeos | Cimi: before dash_overview? | 10:40 |
tsdgeos | dash_overview should go in today afaik | 10:40 |
Cimi | ok | 10:40 |
Cimi | there is lot of things connected | 10:40 |
Cimi | I'll rebase on yours | 10:40 |
Saviq | Cimi, not sure what guidance do you need from me? | 10:41 |
Cimi | Saviq, testDash is broken in trunk for me | 10:41 |
Cimi | can you test? | 10:41 |
Cimi | could be due to scope v4 or dash indicator | 10:41 |
Cimi | then I have another quesitojn | 10:41 |
Saviq | Cimi, indeed two failures | 10:42 |
Saviq | Cimi, I think those get fixed with dash overview though | 10:42 |
MacSlow | ehm... folks... -> http://pastebin.ubuntu.com/7987781 | 10:42 |
Cimi | 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 |
Saviq | MacSlow, yeah, merge trunk | 10:42 |
MacSlow | trying to compile unity8 and really just branched | 10:43 |
MacSlow | Saviq, I'm pretty sure I did just that... | 10:43 |
Cimi | mzanetti, dist-upgrade | 10:43 |
Saviq | MacSlow, trunk already asks for v4 of the scopes api, so you're not up to date | 10:43 |
Saviq | MacSlow, you may try removing builddir/CMakeCache.txt in case cmake got confused | 10:43 |
Saviq | Cimi, he is upgraded, but cmake's looking for v3 instead of v4 | 10:43 |
Cimi | Saviq, on components["attributes"]["field"] ? | 10:44 |
Saviq | Cimi, fake_categories.cpp needs to always return ["attributes"] | 10:45 |
Cimi | ok | 10:45 |
Cimi | I'll add that to card emblems | 10:45 |
Cimi | Saviq, which value for that? | 10:47 |
Saviq | Cimi, just an empty map will suffice | 10:47 |
Saviq | Cimi, see what art is doing | 10:47 |
Saviq | gatox, hey, you pung yesterday, you got along with the indicator in the end? | 10:48 |
Saviq | Cimi, yeah, testDash is fixed in dash overview | 10:48 |
gatox | Saviq, yes, thx, i was able to find it in unity | 10:50 |
MacSlow | 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 |
Saviq | 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:52 |
Saviq | MacSlow, you don't have unity8 installed | 10:53 |
Saviq | MacSlow, you need it installed or copy the data/unity8{,-dash}.conf to ~/.config/upstart/ | 10:53 |
Saviq | 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:53 |
MacSlow | Saviq, hm... didn't touch my ~/.config/upstart in weeks... wonder how the unity8.conf was wiped | 10:54 |
Saviq | MacSlow, you don't need it there if unity8 is installed, you must've removed it somehow | 10:55 |
gatox | 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 |
gatox | 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:55 |
Saviq | gatox, yeah, thing is that should already be happening, at least for actions that involve the scope (like pressing install) | 10:56 |
Saviq | gatox, I'll have a closer look, thanks | 10:56 |
gatox | 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 | 10:58 |
Saviq | gatox, this might be a special case actually (pay being a special button, opening urls taking a moment) | 11:01 |
Saviq | gatox, so it might still make sense indeed | 11:01 |
Cimi | Saviq, fixed categories mock | 11:03 |
Cimi | in emblems | 11:03 |
Cimi | Saviq, will we merge overview today? | 11:09 |
Cimi | Saviq, wondering where to rebase settings | 11:09 |
=== facundobatista_ is now known as facundobatista | ||
Saviq | Cimi, rebase on overview, yeah, even if not today, that's the next thing that's going in | 11:10 |
Cimi | Saviq, yes but if albert is holiday next week | 11:10 |
Cimi | Saviq, we might have settings in before overview | 11:10 |
Saviq | Cimi, no, because we won't wait for Albert to fix any issues in there, we need this in asap | 11:10 |
facundobatista | Holas | 11:10 |
Cimi | ok | 11:10 |
Saviq | \o | 11:10 |
Saviq | dednick, on that note, dash overview could use a re-review, would you be able to do it today? | 11:11 |
dednick | Saviq: can do | 11:12 |
Saviq | gtg guys, mobo replacement | 11:14 |
Saviq | biab | 11:14 |
=== MacSlow is now known as MacSlow|lunch | ||
tsdgeos | pete-woods: ping | 11:17 |
olli | greyback, Saviq, any success on the date time issue? | 11:20 |
greyback | olli: I've tracked down root cause to be a workaround I added to qtubuntu for bug 1346633 | 11:21 |
ubot5 | 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 |
greyback | olli: so I'm working on a different workaround | 11:21 |
olli | heh | 11:22 |
olli | greyback, thx! | 11:22 |
olli | how bad is it... hours, days...? | 11:22 |
greyback | 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 |
greyback | olli: hours hopefuly | 11:22 |
olli | greyback, OK | 11:23 |
gatox | 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:25 |
pete-woods | tsdgeos: hi | 11:26 |
tsdgeos | pete-woods: your'e the usermetrics guy, right? | 11:27 |
pete-woods | tsdgeos: yep | 11:27 |
tsdgeos | pete-woods: i see in the metrics manager you accept a domain | 11:27 |
tsdgeos | is it mandatory that messages go untranslated and you translate them | 11:27 |
tsdgeos | or can messages come in translated too? | 11:27 |
tsdgeos | or is domain not the translation domain? | 11:27 |
tsdgeos | :D | 11:27 |
pete-woods | tsdgeos: nope, feel free to do whatever really | 11:27 |
pete-woods | you're correct, it is the translation domain | 11:28 |
tsdgeos | ok, so "" and a translated mesage is also ok | 11:28 |
pete-woods | tsdgeos: it's preferable to do untranslated | 11:28 |
pete-woods | as if you change language then things will be translated | 11:28 |
pete-woods | but it's not the end of the world if it comes in translated | 11:28 |
tsdgeos | pete-woods: will a reboot fix it? | 11:29 |
pete-woods | tsdgeos: to fix it would require whoever is writing the data to re-run | 11:29 |
tsdgeos | ah | 11:29 |
tsdgeos | ok, so better untranslated + domain | 11:29 |
tsdgeos | tx | 11:29 |
=== dandrader is now known as dandrader|afk | ||
Cimi | can I create a binding from the loaded component of a loader to its loader? | 11:44 |
mzanetti | Saviq: could I steal 10 mins of your time? | 11:47 |
mzanetti | I'd need some second opinions | 11:47 |
mzanetti | Cimi: sure | 11:48 |
=== lool- is now known as lool | ||
Cimi | can I do sth like readonly property bool processing: item.hasOwnProperty("processing") ? item.processing : false ? | 11:49 |
Cimi | Loader { readonly property bool processing: item.hasOwnProperty("processing") ? item.processing : false } | 11:49 |
Cimi | creating binding inside onLoaded breaks on unloaded state | 11:50 |
mzanetti | Cimi: yeah, should work I guess. You might want to add "item && item.hasOwnProp..." | 11:51 |
Cimi | oh yeah correct | 11:52 |
Cimi | or maybe status === Loader.Ready instead item? | 11:52 |
mzanetti | whatever you prefer | 11:52 |
mzanetti | should end up being the same | 11:52 |
gatox | Saviq, ping? | 11:53 |
bregma | 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? | 11:58 |
=== alan_g is now known as alan_g|lunch | ||
greyback | dandrader|afk: when you get back, I need your input on https://code.launchpad.net/~gerboland/qtubuntu/fix_1351024/+merge/230094 ASAP | 12:10 |
pete-woods | bregma: I think that's a password prompt | 12:20 |
pete-woods | I get that when I run unity8 directly atm | 12:20 |
pete-woods | not sure exactly why, though | 12:21 |
bregma | well, it lack a certain amount of usefulness at the moment | 12:21 |
bregma | 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 |
bregma | sometimes three wrongs do make a right | 12:22 |
=== MacSlow|lunch is now known as MacSlow | ||
Cimi | Saviq, pete-woods some settings seems to be undefined by default value... what shall we do in the UI? true or false? | 12:29 |
Cimi | unless model.value is expected to be undefined and we want .defaultValue or so | 12:30 |
=== greyback is now known as greyback|lunch | ||
Saviq | back | 12:33 |
Saviq | mzanetti, gatox, back | 12:34 |
gatox | 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 |
Saviq | gatox, if in current trunk it doesn't, there's nothing in queue to fix that | 12:36 |
gatox | Saviq, ack, i'll have this working then | 12:37 |
pete-woods | Cimi: you shouldn't have to handle it in the UI | 12:38 |
pete-woods | Cimi: which settings, specifically are like that? | 12:38 |
cwayne | pete-woods: is there a plan to update the youtube scope in the store to the new api? | 12:40 |
cwayne | or are you waiting for a promoted image | 12:40 |
pete-woods | cwayne: it's already been updated | 12:40 |
pete-woods | it has been in the store a while now | 12:41 |
cwayne | huh, i didnt see an update, weird | 12:41 |
cwayne | oh well, thanks :D | 12:41 |
pete-woods | np :) | 12:41 |
pete-woods | 1.0.7 is what you're looking for | 12:41 |
cwayne | excellent, thanks again | 12:45 |
cwayne | Cimi: do you expect emblems to land with dash-overview? | 12:45 |
Cimi | cwayne, not up to me | 12:45 |
Cimi | cwayne, under reviews.. | 12:45 |
Cimi | pete-woods, youtube model.value | 12:45 |
Saviq | Cimi, did you fix the issues I mentioned? | 12:45 |
Saviq | in emblems? | 12:45 |
pete-woods | Cimi: okay, this is in mediascanner I take it? | 12:46 |
Cimi | Saviq, which one? | 12:46 |
Cimi | pete-woods, no idea | 12:46 |
Cimi | I have this show youtube in video scope true | false | 12:47 |
Cimi | model.value is undefined for it | 12:47 |
Cimi | unless i set it | 12:47 |
Cimi | Saviq, ah the 0 padding? | 12:48 |
pete-woods | Cimi: okay, reproduced it, seems to be a bug | 12:48 |
Cimi | Saviq, I try it in half hour | 12:48 |
Cimi | taking lunch break | 12:48 |
=== dandrader|afk is now known as dandrader | ||
Saviq | Cimi, and the fact that it didn't "go away" when I edited the data field in tryCard | 12:50 |
pete-woods | Cimi: okay, it's because there was a bug in the previous version of the settings | 12:50 |
pete-woods | Cimi: leading to corrupt settings db -> rm -rf ~/.local/share/videoaggregator/ | 12:50 |
dandrader | greyback|lunch, still need my input there or does Timo's review do it/ | 12:51 |
dandrader | ? | 12:51 |
Saviq | MacSlow, like the branch much more without the regex ;) | 12:54 |
facundobatista | does anybody know if there is a way to indicate that a widget should NOT have a preview? that is, ignore when the user does a long tap on it; thanks! | 12:54 |
MacSlow | Saviq, I've updated the relevant branches for the text-filter... it needs another look now (https://code.launchpad.net/~macslow/unity-notifications/fix-1335787/+merge/227334, https://code.launchpad.net/~macslow/unity8/fix-1335787/+merge/230087) | 12:54 |
MacSlow | Saviq, I understand... still I had to skip 7 tests which can't be covered (converted) via QTextDocument | 12:54 |
Saviq | MacSlow, on the test, please use the _data approach as we do in QML | 12:55 |
Saviq | MacSlow, an old 4.8 tutorial http://qt-project.org/doc/qt-4.8/qtestlib-tutorial2.html | 12:56 |
tsdgeos | facundobatista: ping | 12:57 |
facundobatista | tsdgeos, pong | 12:57 |
tsdgeos | facundobatista: how do i translate the 7digital scope | 12:57 |
tsdgeos | ? | 12:57 |
facundobatista | tsdgeos, lp:ubuntu-rest-scopes, see "locale" dir | 12:57 |
facundobatista | tsdgeos, it uses gettext | 12:58 |
dandrader | greyback|lunch, will review it as well anyway | 12:58 |
greyback|lunch | dandrader: yes please review anyway, since oyu know the code | 13:00 |
tsdgeos | facundobatista: oki | 13:02 |
facundobatista | tsdgeos, any doubt there, just let me know | 13:03 |
Saviq | MacSlow, FWIW, this kind of "I may be plaintext, but I also can be html" is just broken by design | 13:06 |
MacSlow | Saviq, yeah | 13:06 |
tsdgeos | facundobatista: https://code.launchpad.net/~aacid/ubuntu-rest-scopes/update_pot/+merge/230105 | 13:06 |
Saviq | MacSlow, either you accept HTML and all <> need to be escaped, or not, and they will be broken | 13:06 |
Saviq | MacSlow, so I'm actually happy with the results you got with QTextDocument | 13:06 |
MacSlow | Saviq, yeah | 13:07 |
facundobatista | tsdgeos, what is that? | 13:07 |
MacSlow | Saviq, just finished adapting the test to the _data() scheme | 13:07 |
tsdgeos | facundobatista: updates the pot file | 13:07 |
tsdgeos | so we can actually translate stuff | 13:07 |
facundobatista | mmm | 13:07 |
tsdgeos | facundobatista: basically i ran ./build/get_translatable_texts.sh | 13:08 |
MacSlow | Saviq, pushed | 13:08 |
Saviq | MacSlow, kk | 13:08 |
=== alan_g|lunch is now known as alan_g | ||
Saviq | MacSlow, I think you could actually use the newRow("") to store the actual string | 13:09 |
Saviq | MacSlow, this way you'll get better feedback on test output on which case failed | 13:10 |
tsdgeos | Saviq: how's silo 11 going? | 13:10 |
tsdgeos | plan to land today? | 13:11 |
Saviq | tsdgeos, hoping to | 13:11 |
MacSlow | Saviq, yeah... but blows up the result-output a bit... even if all tests (of text-filter) pass | 13:11 |
facundobatista | tsdgeos, thanks | 13:11 |
MacSlow | Saviq, but then... hardly an issue really :) | 13:11 |
Saviq | tsdgeos, they were expecting scope favouriting to work, too, had to crush their dreams on that (shoulda kinda have mentioned it) | 13:11 |
Saviq | MacSlow, blows how? by printing the strings? | 13:12 |
tsdgeos | Saviq: oh | 13:12 |
MacSlow | Saviq, well it spits out 26 more lines now... initially it was only one | 13:12 |
Saviq | MacSlow, well, that's the whole idea ;) | 13:13 |
Saviq | MacSlow, every case there is now treated as a separate test | 13:13 |
MacSlow | Saviq, certainly looks more impressive now with 36 passed test all together :) | 13:14 |
Saviq | exactly! | 13:14 |
tsdgeos | facundobatista: buaaaarns, some bot unaaccepted my update | 13:17 |
tsdgeos | Saviq: on $thephone on first boot seems the shell is displaced like 1 pixel right | 13:18 |
tsdgeos | bringing back the greeter usually fixes it | 13:18 |
tsdgeos | wrong, blanking the screen fixes it | 13:19 |
facundobatista | tsdgeos, let's check now | 13:19 |
Saviq | tsdgeos, yeah,known bug | 13:23 |
Saviq | unknown cause yet | 13:23 |
tsdgeos | Saviq: ok, also 5 min ago, restart was not restarting the phone but just the shell | 13:23 |
tsdgeos | is it known or shall i file a bug? | 13:23 |
Saviq | didn't hear that no | 13:23 |
Cimi | Saviq, I don't know why if I unsed the data you still see the emblem... | 13:23 |
Cimi | *unset | 13:24 |
tsdgeos | problem is that stupid me i shut down | 13:24 |
Cimi | is weird | 13:24 |
tsdgeos | so i can't repro anymore | 13:24 |
Cimi | if you set to an invalid data it hides, valid one it shows.,.. | 13:24 |
=== greyback|lunch is now known as greyback | ||
Cimi | Saviq, might be something not updated: if you set the data to an invalid png then you remove the data, the emblem is gone | 13:25 |
Cimi | Saviq, if you don't set to an invalid image first and remove the data, it uses the previous emblem | 13:25 |
Saviq | Cimi, rather something looks at the url and only changes if it evals to true | 13:26 |
Saviq | Cimi, and "" does not eval to true | 13:26 |
Cimi | Saviq, same issue with mascot | 13:26 |
Saviq | Cimi, ok, let's leave that, unimportant as will never happen IRL | 13:26 |
Cimi | Saviq, however, changing in CardCreator.js fixes that | 13:27 |
Cimi | source: cardData["emblem"]; -> source: cardData["emblem"] || ""; | 13:28 |
Cimi | we can do the same for mascot | 13:28 |
Cimi | source: cardData["mascot"]; -> source: cardData["mascot"] || ""; | 13:28 |
bregma | Saviq, could I get you or one of your evil minions to review https://code.launchpad.net/~bregma/unity8/merge-desktop-upstart-job/+merge/230016 so we can get the Unity 8 desktop back up and on to its next shock of the unknown ASAP? | 13:28 |
bregma | and, possibly include the MP in the next Unity 8 ci-train landing? | 13:29 |
Saviq | Cimi, sounds good, yeah, maybe the json parser gives up null or undefined and source isn't reset with that | 13:29 |
Saviq | Cimi, so good fix | 13:29 |
Saviq | bregma, will do | 13:29 |
Saviq | bregma, ah on that note, can you drop the surfaceflinger part?\ | 13:29 |
Cimi | Saviq, in case of null image, shall we reserve space or not? | 13:29 |
Saviq | Cimi, no | 13:30 |
bregma | I could, but I have no way to test that | 13:30 |
Saviq | bregma, there isn't a session like that any more :) | 13:30 |
Cimi | Saviq, mascot reserves space though | 13:30 |
Saviq | bregma, and I will test it | 13:30 |
Cimi | Saviq, try messing up with the url; | 13:30 |
Saviq | Cimi, I don't think it does in fact (it should not) | 13:30 |
Saviq | Cimi, if it does - bug | 13:30 |
Cimi | Saviq, width and height are static for mascot | 13:31 |
Cimi | width: units.gu(6); \n height: units.gu(5.625); \n\ | 13:31 |
Saviq | Cimi, feels like a bug | 13:32 |
Cimi | I can do sth like source status === Image.Ready ? .. : 0 | 13:32 |
Cimi | without source | 13:32 |
Cimi | I can fix that in emblems if you like | 13:32 |
Cimi | (fixed) | 13:33 |
Saviq | Cimi, yeah, invalid image should just not exist in there, if we managed to get it done with GridLayout or something, that would've happened automagically | 13:49 |
Cimi | Saviq, not fixed for mascot | 13:49 |
Cimi | thoght was easy... the calculations with sourceSize causes binding loops | 13:49 |
Cimi | cause | 13:49 |
cwayne | are there any scopes that currently utilize the alt nav support? | 13:50 |
Saviq | cwayne, it's not landed yet in unity8 at least ;) | 13:50 |
cwayne | Saviq: well we can still see the code and see if it makes sense to start integrating them into our scopes :) | 13:51 |
=== _salem is now known as salem_ | ||
Saviq | cwayne, sure, dunno though | 13:51 |
Saviq | tsdgeos, did you see any real scope using dual nav? | 13:51 |
tsdgeos | Saviq: nope :/ | 13:52 |
pete-woods | cwayne: I've seen the amazon scope doing it | 13:52 |
Cimi | Saviq, gonna commit this (and fixes tests that will break) http://paste.ubuntu.com/7989005/ | 13:52 |
tsdgeos | all testing is based solely on my fake plugin impl | 13:52 |
pete-woods | cwayne: although obviously that support depends on stuff that hasn't landed yet | 13:52 |
cwayne | right | 13:52 |
Saviq | Cimi, from a quick skim looks fine, although are we sure cardData will always be there? | 13:53 |
Saviq | Cimi, on creation I mean | 13:53 |
Cimi | Saviq, I thought it was | 13:54 |
Cimi | Saviq, you told me we always create it or not? | 13:54 |
Cimi | might have misunderstood | 13:54 |
Saviq | Cimi, yeah, we do, but it might not be bound in time | 13:54 |
Saviq | Cimi, like if that binding evaluates before cardData is bound from the result | 13:54 |
Cimi | Saviq, because I removed it from attributes | 13:54 |
Cimi | I can add them everywhere | 13:55 |
Saviq | Cimi, yeah, now I think about it again, we need at least a check for cardData, because that might be null on creation | 13:55 |
Cimi | ok | 13:55 |
Cimi | adding all of them | 13:55 |
Saviq | Cimi, if it's defined then ["attributes"] will be there too | 13:55 |
Saviq | I mean always | 13:55 |
Saviq | so we don't need to check for that, but the cardData object itself might be null on creation | 13:55 |
Cimi | Saviq, all pushed - apart the fix for not allocating 0 size in case of broken mascot image | 13:59 |
Saviq | Cimi, yeah, ok, unrelated bug | 14:00 |
Cimi | (which requires lil more thought) | 14:00 |
Saviq | jeez that grew | 14:00 |
Saviq | Cimi, whitespaces again :| | 14:00 |
Cimi | oh fuck | 14:01 |
Saviq | Cimi, can you add a prerequisite that removes all the spaces and ; from cardCreator | 14:01 |
Cimi | damn qtcreATOR | 14:01 |
Cimi | ok | 14:01 |
Saviq | (; where it doesn't make sense, so outside of JS) | 14:01 |
Cimi | nope | 14:02 |
Cimi | let me revert that | 14:02 |
Cimi | Saviq, but why we added whitespace at end of line in first place? | 14:03 |
Saviq | Cimi, readability in cardCreator I expect | 14:03 |
Saviq | Cimi, so that you don't get lines that all end with \n\ | 14:03 |
Cimi | ok | 14:03 |
Saviq | or actually end with\n\ | 14:03 |
Saviq | Cimi, the card creator tests leave the result as a file on failure, shouldn't you just copy those onto the expected ones | 14:04 |
Saviq | Cimi, and check the diff that it makes sense | 14:04 |
Cimi | Saviq, spacing issues fixed I think | 14:09 |
Cimi | diff seems sane now on LP | 14:09 |
Saviq | Cimi, still 1200 lines for a small emblem ;P | 14:10 |
Cimi | pffff | 14:10 |
Cimi | :D | 14:10 |
Saviq | Cimi, '' vs. "" changes?? | 14:11 |
Saviq | 200- var mascotShapeCode = ""; | 14:11 |
Saviq | 201- var mascotCode = ""; | 14:11 |
Saviq | 202+ var mascotShapeCode = ''; | 14:11 |
Saviq | 203+ var mascotCode = ''; | 14:11 |
Cimi | Saviq, code consistency | 14:11 |
Cimi | Saviq, we always used '' apart here | 14:11 |
Saviq | Cimi, you really don't want this to be reviewed do you | 14:11 |
Cimi | ? | 14:11 |
Cimi | what's the problem? | 14:12 |
Saviq | Cimi, having an MP with a bunch of unrelated changes doesn't help review speed | 14:12 |
Cimi | bunch = 2 lines? | 14:12 |
Saviq | Cimi, there's a whole slew of renames too | 14:12 |
Saviq | and there's more "" to '' elsewhere | 14:12 |
Cimi | Saviq, you're right about the '' | 14:13 |
Cimi | Saviq, but it's a easy review | 14:13 |
Saviq | Cimi, well, it's not easy any more :P | 14:13 |
Saviq | tsdgeos, you stuck in SIM land or can help with hopefully a small thing? | 14:17 |
tsdgeos | Saviq: i am trying to rebuild gallery not to crash | 14:17 |
tsdgeos | but if it's quick, sure | 14:17 |
Saviq | tsdgeos, we need to forward processing from GSVs open on top of overview | 14:18 |
Saviq | tsdgeos, if you open a temp scope in overview, no processing indicator | 14:18 |
Saviq | tsdgeos, should be another || gsv.processing | 14:18 |
Saviq | or so | 14:18 |
tsdgeos | correct | 14:18 |
tsdgeos | let me see | 14:18 |
tsdgeos | Saviq: pushed | 14:21 |
Saviq | tsdgeos, awesomes, thanks | 14:21 |
Saviq | mzanetti, could I volunteer you for https://code.launchpad.net/~mterry/unity8/wrong-password-handling/+merge/229184 please? | 14:22 |
Saviq | another one for evaluation: keyboard persists after you open a scope from search | 14:26 |
Saviq | tsdgeos, ↑ | 14:26 |
Saviq | tsdgeos, if you think easy, please fix, otherwise we'll do it | 14:26 |
tsdgeos | i fixed that | 14:26 |
tsdgeos | someone broke it :/ | 14:27 |
Saviq | bastards | 14:27 |
tsdgeos | was probably me :D | 14:27 |
tsdgeos | with other page header changes | 14:28 |
tsdgeos | let me see if it's easy to fix | 14:28 |
Saviq | Cimi, nice timing ;) | 14:36 |
Cimi | Saviq, hah | 14:36 |
Cimi | Saviq, bloody xchat without highlights for words with @ | 14:37 |
Cimi | missed the ping | 14:37 |
dednick | Saviq: which silo is dash overview? | 14:38 |
dednick | or tsdgeos: ^ | 14:38 |
tsdgeos | dednick: 11 | 14:39 |
dednick | tsdgeos: ta | 14:39 |
Saviq | dednick, there's a few fixes on top of the silo now | 14:40 |
Saviq | smaller ones | 14:40 |
pete-woods | Saviq: if you want any extra help testing that silo, just give me a shout | 14:42 |
Saviq | pete-woods, we've plenty of hands, it's going into user testing on Monday ;) | 14:43 |
Saviq | pete-woods, but thanks, you can test it out anyway | 14:43 |
Saviq | Cimi, were you not rebasing emblems on dash_overview? | 14:45 |
Cimi | Saviq, no | 14:45 |
pete-woods | okay, just wanted to help (this silo is particularly exciting) | 14:45 |
Cimi | Saviq, scope settings | 14:45 |
Saviq | Cimi, ah | 14:45 |
Saviq | Cimi, looks like you want to rebase emblems, too, 3 conflicts | 14:46 |
Cimi | Saviq, or dash overview... | 14:46 |
Cimi | :D | 14:46 |
Saviq | Cimi, or... not | 14:46 |
Cimi | Saviq, which one gets first? | 14:46 |
Saviq | Cimi, overview | 14:46 |
Cimi | ok | 14:46 |
Saviq | Cimi, small, easy conflicts though | 14:47 |
Cimi | Saviq, dash overview has whitespace changes too | 14:48 |
Saviq | Cimi, are we really playing the blame game now? | 14:49 |
=== gatox is now known as gatox_lunch | ||
Cimi | Saviq, yeah! | 14:49 |
Cimi | Saviq, anyway pushing now | 14:49 |
Saviq | Cimi, well, I'm not | 14:49 |
Saviq | Cimi, did you have to rename anchors to artAnchors etc? were they clashing? | 14:50 |
Cimi | Saviq, some of them were | 14:51 |
Cimi | Saviq, so I changed name for readability | 14:51 |
Saviq | Cimi, and btw , ' is only used when there's quotes inside the string already, to avoid escaping it | 14:51 |
dandrader | dednick, any qmltest for the prompt session stuff? | 14:51 |
Saviq | Cimi, everywhere else we actually use " | 14:51 |
Saviq | Cimi, so TBH I'd rather ' be an exception than a rule in that particular file | 14:52 |
Cimi | Saviq, I asked that question at the sprint | 14:52 |
Cimi | difference / what we prefer | 14:52 |
Saviq | Cimi, it doesn't make a difference, so *changing* it doesn't make sense to me | 14:52 |
dednick | dandrader: not at the moment. | 14:52 |
Saviq | just for the sake of changing it | 14:52 |
dednick | dandrader: on my todo list | 14:53 |
Saviq | especially in a branch that doesn't deal with that wholesome, but actually implements things | 14:53 |
Cimi | Saviq, https://code.launchpad.net/~cimi/unity8/card_emblems_2/+merge/230126 | 14:58 |
Cimi | rebased | 14:58 |
Saviq | Cimi, ok, I'm reviewing the original one first anyway | 14:59 |
Cimi | Saviq, i can change '' nack to "" | 14:59 |
Cimi | back | 14:59 |
Saviq | Cimi, don't, just don't do such changes unnecessarily, they fit in a separate MP that only deals with such changes | 14:59 |
Cimi | ok | 14:59 |
Saviq | Cimi, maybe remove the superseding MP for now, I'm halfway through the review in the non-superseded one | 15:00 |
Saviq | Cimi, this way comments should go over | 15:00 |
Saviq | Cimi, or wait, maybe they'll go over anyway, let's see | 15:00 |
greyback | opinion wanted: do we consider the orange line under the indicators bar to be part of the panel? Or is it separate? | 15:05 |
Cimi | greyback, I'd say panel | 15:06 |
greyback | Cimi: me too | 15:07 |
Saviq | Cimi, reviewied | 15:10 |
Saviq | -i | 15:10 |
Saviq | greyback, panel for sure | 15:11 |
greyback | Saviq: ok | 15:11 |
Cimi | Saviq, When emblem is declared, but invalid, there's no right text margin. ? | 15:11 |
Saviq | Cimi, text reaches right edge | 15:11 |
tsdgeos | Saviq: commited the unfocus | 15:11 |
Cimi | has 1 gu here | 15:11 |
Saviq | Cimi, should have 1gu margin on the right | 15:11 |
Cimi | maybe in particular cases | 15:11 |
Saviq | Cimi, when emblem invalid or not there? | 15:11 |
greyback | panelHeight is not including the orange line, and so we're actually drawing over the top 2DP of app surfaces | 15:12 |
Saviq | Cimi, it's fine when it's not declared | 15:12 |
Saviq | greyback, ouch | 15:12 |
Saviq | tsdgeos, thanks | 15:12 |
Cimi | Saviq, you right | 15:12 |
Saviq | I right! | 15:12 |
Cimi | Saviq, font hinting got me | 15:12 |
Cimi | thought was padding | 15:13 |
tsdgeos | alecu: ping | 15:13 |
Saviq | Cimi, we have a bug there btw, there should be no margins if there's no background and touchdown only covers art | 15:13 |
alecu | tsdgeos: hi | 15:13 |
tsdgeos | alecu: i'm pretty sure that this is wrong | 15:13 |
tsdgeos | alecu: http://paste.ubuntu.com/7989543/ | 15:14 |
tsdgeos | you can't have "static" + translations | 15:14 |
tsdgeos | since the translation will get run on "start" time | 15:14 |
tsdgeos | before anything is set | 15:14 |
tsdgeos | and you end up with no translation | 15:14 |
tsdgeos | which is what i'm seeing | 15:14 |
tsdgeos | am i making sense? | 15:14 |
alecu | tsdgeos: yes, I think I understand | 15:15 |
tsdgeos | want a patch? | 15:15 |
alecu | tsdgeos: sure! or a bug should be enough | 15:15 |
tsdgeos | i have 45 min without much to do | 15:15 |
tsdgeos | so i'll give you a patch :D | 15:15 |
Cimi | Saviq, no bg? | 15:16 |
tedg | greyback, So I'm looking at app startup, and I have a case where it's taking Unity 640ms to execute the focus observers. Do you know why that could be? | 15:16 |
Saviq | Cimi, say art + title | 15:16 |
alecu | tsdgeos: thanks! | 15:16 |
Saviq | Cimi, title is unnecessarily squeezed 2gus from the sides | 15:17 |
Saviq | Cimi, when there's no background and touchdown will only affect art | 15:17 |
Cimi | Saviq, paste me components? | 15:19 |
greyback | tedg: focus observers? Can you elaborate please. Some data would be nice | 15:19 |
tsdgeos | alecu: hmm, i'll open a bug :D | 15:19 |
tedg | greyback, The observer for the focus and starting events with UAL. | 15:19 |
greyback | tedg: I always like questions along the lines "your code is slow, do you know why?" :D | 15:19 |
tsdgeos | alecu: not as easy as i thought | 15:19 |
tedg | greyback, Let me put the data somewhere | 15:19 |
Cimi | ok got it | 15:19 |
Saviq | Cimi, but that bug's in trunk anyway | 15:20 |
Cimi | how should is bt? | 15:20 |
greyback | tedg: thanks. Did you acquire it with that method your blogged about? | 15:20 |
greyback | http://gould.cx/ted/blog/Measuring_Upstart_App_Launch_using_LTTng | 15:20 |
tedg | greyback, Yes, I've got a tar of it. | 15:20 |
Cimi | Saviq, let me see if I can quickly fix it | 15:20 |
tsdgeos | alecu: https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+bug/1354501 | 15:21 |
ubot5 | Ubuntu bug 1354501 in unity-scope-click (Ubuntu) "Do not use class static translatable messages" [Undecided,New] | 15:22 |
tedg | greyback, http://chinstrap.canonical.com/~ted/dialer-start-20140808-095853.tar | 15:22 |
alecu | tsdgeos: I'm just testing it on my device in spanish, and you are right, it looks very weird. | 15:22 |
tedg | greyback, You can just run the untar'd dir: babeltrace dialer-start-20140808-095853 | 15:22 |
alecu | tsdgeos: thanks for the bug | 15:22 |
tsdgeos | alecu: no prob | 15:22 |
tedg | greyback, Looking at the "observer_start" and "observer_finish" events | 15:23 |
greyback | screw you SSO | 15:23 |
=== dandrader is now known as dandrader|lunch | ||
tedg | Heh, yes. | 15:24 |
tedg | greyback, So this is just clicking on the dialer in the dash | 15:25 |
=== om26er is now known as om26er|dinner | ||
dednick | um. woops. just managed to delete my entire working folder. goodbye code. | 15:28 |
greyback | ouch | 15:28 |
greyback | tedg: ok interesting. I'll have to dig to see what's so slow there | 15:28 |
dednick | good news is that i've got lots of space now. | 15:28 |
tedg | greyback, Cool, thanks! | 15:29 |
Cimi | Saviq, fixed but doesn't look that nice | 15:32 |
Saviq | Cimi, that is the design everywhere, though, that they line up on the sides | 15:32 |
Cimi | ok | 15:33 |
Cimi | I will push | 15:33 |
tedg | greyback, So I did a run of the clock app and it only took 65ms. | 15:38 |
tedg | greyback, Not sure what the difference is. | 15:38 |
tedg | greyback, The only thing I can think of is that clock is a confined/click where dialer is unconfined/legacy. | 15:38 |
greyback | tedg: that's an astute observation, can you see if other unconfined apps have the same slowdown? | 15:40 |
tedg | greyback, Just a sec, grabbing a couple more dialer runs to see if it was just a fluke. | 15:40 |
tedg | greyback, Did two dialer runs, one after another. The first took 600ms, the second less than 60ms. Not sure it's a confinement issue. | 15:44 |
=== gatox_lunch is now known as gatox | ||
tedg | greyback, Hmm, browser took ~150ms. Feel like I'm not getting consistent results, any idea of what I should stop/start/do per run? | 15:47 |
greyback | tedg: tbh what you're doing is the most realistic, so we need to figure out why the lack of consistency | 15:49 |
greyback | tedg: I think it's probably a good idea to add lttng tracepoints to qtmir | 15:49 |
tedg | greyback, Ah, that's a good idea | 15:51 |
Cimi | Saviq, pushed... required more time to deal with emblems, subtitles ect | 15:57 |
Cimi | Saviq, Indent please. ? | 16:01 |
Cimi | your diff comment | 16:01 |
Saviq | Cimi, there's no indentation in { } | 16:03 |
Cimi | Saviq, gotcha - fixed | 16:03 |
Cimi | Saviq, are we sure of emblem requirement? | 16:04 |
Saviq | Cimi, not sure what you mean | 16:05 |
=== dandrader|lunch is now known as dandrader | ||
Cimi | Saviq, using statusicon | 16:05 |
Saviq | and apparently don't care... stupid ^W | 16:05 |
Saviq | Cimi, yes, it needs to be colourized | 16:05 |
Saviq | Cimi, well, you can actually use Icon | 16:05 |
Saviq | Cimi, passing source: to it | 16:05 |
Saviq | Cimi, since it's meant to be square | 16:05 |
Cimi | Saviq, trying | 16:06 |
Cimi | Saviq, icon does not have source | 16:09 |
Cimi | Saviq, only set of named images from the iconset | 16:10 |
Cimi | Saviq, I can try with qt graphical effects | 16:12 |
Cimi | but we will have to convert the palette colors into HSL then getting just H | 16:16 |
Saviq | mzanetti a bit of a last minute thing... dash-as-app, dash is foreground, long-left edge swipe, can we ~easily make launcher stay on screen? | 16:21 |
mzanetti | Saviq: hmm... depends a bit on the ~easily | 16:21 |
Saviq | mzanetti, as in, Monday morning for user testing ;) | 16:22 |
Cimi | yes please | 16:22 |
Saviq | mzanetti, they start 9am in London | 16:22 |
Cimi | I hate it disappears | 16:22 |
Cimi | line 246 Launcher.qml | 16:23 |
Cimi | if we have a way to check from launcher if we are in the dash... doable easily | 16:24 |
Cimi | Saviq, do we? | 16:24 |
mzanetti | Cimi: ApplicationManager.get(0).appId == "unity8-dash" | 16:24 |
Saviq | Cimi, sure we do, launcher is shell | 16:24 |
mzanetti | Cimi: however, you need to update it manually as "0" doesn't change | 16:25 |
=== om26er|dinner is now known as om26er | ||
Saviq | mzanetti, don't we have like focusedApp though? | 16:25 |
Saviq | mzanetti, being a shortcut to 0? | 16:25 |
mzanetti | err. right | 16:25 |
mzanetti | ApplicationManager.focusedApplicationId | 16:25 |
mzanetti | Saviq: trying to get away from that though | 16:26 |
mzanetti | but I guess ok for the hotfix | 16:26 |
Saviq | mzanetti, yeah right, we have two focused apps at any given time.. | 16:26 |
Saviq | (on tablet) | 16:26 |
mzanetti | Saviq: yes, for once. | 16:26 |
mzanetti | Saviq: also, ApplicationManager shouldn't care in the long run | 16:26 |
Saviq | trueth | 16:27 |
mzanetti | Saviq: we just tell it which apps to suspend | 16:27 |
mzanetti | Cimi: are you on it or should I fix it? | 16:28 |
Cimi | mzanetti, I am not on it, but that's the line to edit... | 16:28 |
mzanetti | ack | 16:28 |
Cimi | that you knoew | 16:28 |
Cimi | knew | 16:28 |
mzanetti | Cimi: wrong line :P | 16:29 |
mzanetti | even wrong file :D | 16:29 |
Saviq | lol | 16:29 |
mzanetti | ah no... I guess we could just add it here | 16:29 |
Cimi | :P | 16:30 |
mzanetti | nah... Shell.qml is better | 16:30 |
Saviq | mzanetti, think so, shouldn't the onDraggingChanged have a different behavior if dash is foreground? | 16:31 |
Saviq | mzanetti, sure, Launcher shouldn't know directly, but Shell can tell it | 16:31 |
mzanetti | Saviq: which onDraggingChanged | 16:31 |
mzanetti | ? | 16:31 |
mzanetti | Saviq: found it... | 16:32 |
Saviq | mzanetti, Launcher:244 | 16:32 |
mzanetti | Saviq: why should that have a different behavior? | 16:32 |
Saviq | mzanetti, maybe I'm just reading it wrong ;) | 16:32 |
mzanetti | Saviq: actually I'm not entirely happy with that code any more... | 16:32 |
Saviq | mzanetti, anyway, you know your way around it | 16:32 |
mzanetti | Saviq: its a bit of a mix between the launcher knowing about stuff and the shell controlling the launcher | 16:32 |
Saviq | mzanetti, yup indeed | 16:32 |
Saviq | mzanetti, we need a hotfix, not a proper solution, for now ;) | 16:33 |
mzanetti | that's the thing... one writes it to not know about anything and the next one hotfixes something into it... | 16:33 |
Saviq | mzanetti, that won't even land in trunk | 16:33 |
Saviq | mzanetti, we just need it in the silo | 16:33 |
mzanetti | http://paste.ubuntu.com/7990053 | 16:33 |
Cimi | Saviq, btw seen I fixed the rest of the things? | 16:34 |
Cimi | apart the Icon which doesn't support source | 16:34 |
mzanetti | still need to compile and test... | 16:34 |
Saviq | Cimi, yes it does (ugh, maybe not in trunk yet?) | 16:34 |
Cimi | Saviq, not on my pc | 16:35 |
Cimi | it fails | 16:35 |
Cimi | mzanetti, I still think we want to change the status to visible | 16:35 |
Saviq | Cimi, https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1284233 | 16:35 |
Cimi | mzanetti, thus I said line 246 | 16:35 |
ubot5 | Ubuntu bug 1284233 in Ubuntu UI Toolkit "Can't use uris for Icon source" [Undecided,Fix released] | 16:35 |
mzanetti | Cimi: ? | 16:35 |
Saviq | Cimi, Icon has a source property since Monday | 16:35 |
Cimi | Saviq, in staging maybe | 16:36 |
mzanetti | :D | 16:36 |
Saviq | Cimi, RELEASED | 16:36 |
Saviq | Cimi, 1.1.1179+14.10.20140804-0ubuntu1 | 16:36 |
Cimi | OKM | 16:36 |
Cimi | OK | 16:36 |
Cimi | OKOK | 16:36 |
Cimi | OK | 16:36 |
Cimi | :D | 16:36 |
Saviq | Cimi, https://launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/1.1.1179+14.10.20140804-0ubuntu1 | 16:37 |
Cimi | it's friday late afternoon, I started at 8:30am and I am well cooked... | 16:37 |
Cimi | mzanetti, <Saviq> mzanetti a bit of a last minute thing... dash-as-app, dash is foreground, long-left edge swipe, can we ~easily make launcher stay on screen? | 16:37 |
Cimi | mzanetti, does your code keep the launcher on screen? | 16:38 |
mzanetti | Cimi: yes | 16:38 |
mzanetti | but just figured it keeps it on the screen a bit too often | 16:38 |
Cimi | cool then | 16:38 |
Saviq | mzanetti, yeah, it needs to switch to visible state indeed | 16:40 |
Saviq | mzanetti, it has to be the same exactly like a short swipe | 16:40 |
mzanetti | yep... but there's still an issue | 16:40 |
Saviq | mzanetti, aanyway | 16:41 |
Saviq | mzanetti, we have an hour on Monday morning | 16:41 |
Saviq | before UK wakes up | 16:41 |
mzanetti | not so sure any more if I can do this in an hour :D | 16:41 |
Saviq | :| | 16:41 |
=== dpm is now known as dpm-afk | ||
Saviq | mzanetti, that's fine, if we can't, we can't, they had could've let us know earlier | 16:42 |
mzanetti | well, I'll try | 16:42 |
Cimi | Saviq, mistery solved - CardCreator.js was importing 0.1 not 1.1 | 16:42 |
mzanetti | muahahaha | 16:43 |
mzanetti | scnr | 16:43 |
Saviq | ok, I gotta unglue... | 16:46 |
mzanetti | Saviq: Cimi: hotfix... not to be landed for real: http://paste.ubuntu.com/7990208 | 16:51 |
Cimi | Saviq, Using icon now, added colorisation and fixed png asset for test | 16:52 |
Cimi | (pushed) | 16:52 |
Cimi | mzanetti, hah, line 246 :P /troll | 16:53 |
mzanetti | Cimi: yeah... but we really don't want to land this as is | 16:53 |
Cimi | of course | 16:54 |
Cimi | mzanetti, also in spread? | 16:54 |
Cimi | but enjoy weekend guys | 16:54 |
Saviq | o/ | 16:55 |
* greyback EOW | 17:11 | |
=== alan_g is now known as alan_g|EOW | ||
mzanetti | dednick: hey, still around? | 17:24 |
=== boiko_ is now known as boiko | ||
Saviq | mzanetti, aaaghhg /build/buildd/unity8-8.00+14.10.20140808.1/qml/Launcher/Launcher.qml: bad whitespace in line 249 ;P | 17:41 |
mzanetti | we all love that test :) | 17:41 |
Saviq | mzanetti, I'm fixing | 17:42 |
mzanetti | I'm on it Saviq | 17:42 |
Saviq | too late | 17:42 |
mzanetti | meh | 17:42 |
Saviq | mzanetti, aren't you using QtCreator? it strips any trailing whitespace here | 17:43 |
mzanetti | Saviq: I am for most stuff. But for such hotfixes its too much effort to configure qtcreator | 17:43 |
mzanetti | still not using colocated branches | 17:43 |
mzanetti | Saviq: I need to change the builddir, add -j9 to makeopts and if I want run to work add -mousetouch and whatnot | 17:44 |
mzanetti | hence I often just ./build.sh, nano and run.sh | 17:44 |
Saviq | oh jeez | 17:46 |
Saviq | mzanetti, well, yeah, I'm not using QtC to build/run anyway, but yeah colo branches FTW | 17:46 |
Saviq | but nano? ;) | 17:47 |
mzanetti | Saviq: yeah, why not? | 17:47 |
mzanetti | gets the job done | 17:47 |
mzanetti | and I mostly just have to press one key to do something instead of 3 per command :P | 17:48 |
mzanetti | the only nasty thing is on *wrt, where it mostly autocompletes to nandwrite 'cause nano isn't installed :D | 17:49 |
=== om26er_ is now known as om26er | ||
Cimi | mzanetti, sudo nano ftw! | 19:44 |
mzanetti | Cimi: ? | 20:06 |
Cimi | mzanetti, I ran it many times :) | 20:49 |
=== salem_ is now known as _salem |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!