=== _salem is now known as salem_ | ||
=== salem_ is now known as _salem | ||
=== mardy_ is now known as mardy | ||
tsdgeos | Cimi: how's the review of scopes_more_things_on_memory going? | 09:13 |
---|---|---|
pstolowski | tsdgeos, hey, any luck with the ppa? | 09:30 |
tsdgeos | pstolowski: no the personal ppa's are worthless | 09:31 |
pstolowski | :( | 09:31 |
tsdgeos | need to use one that is phone enabld | 09:31 |
tsdgeos | we have one | 09:31 |
tsdgeos | but mzanetti was using it for something else | 09:31 |
tsdgeos | mzanetti: are we done with it? | 09:31 |
mzanetti | tsdgeos: sure. you approved the branches | 09:31 |
tsdgeos | mzanetti: so how do i take over? :D | 09:34 |
mzanetti | tsdgeos: I think you should have permissions | 09:34 |
=== vrruiz_ is now known as rvr | ||
mzanetti | tsdgeos: I'd suggest to just drop the existing packages | 09:35 |
mzanetti | tsdgeos: then go to your branch, create a packaging recipe and target that ppa | 09:35 |
mzanetti | that's how I do it usually... you can also manually dput if you prefer that | 09:35 |
tsdgeos | i'll dput | 09:36 |
tsdgeos | it's somethin i have some knowledge of how to try to do | 09:36 |
tsdgeos | ok packages deleted | 09:37 |
tsdgeos | now let's add stuff | 09:37 |
tsdgeos | Cimi: there's a change in card-visual-tweaks i don't understand | 09:58 |
tsdgeos | let's see if you can help me | 09:59 |
tsdgeos | Cimi: run make tryCard | 09:59 |
tsdgeos | and select Art, header, summary - overlaid | 09:59 |
tsdgeos | the "gray" area is with this patch quite bigger than without | 09:59 |
tsdgeos | and i don't see why | 09:59 |
tsdgeos | uhmmm | 10:09 |
tsdgeos | is it actually a regression of scope_more_things_memory? | 10:09 |
tsdgeos | and now i can't find a difference | 10:10 |
* tsdgeos scratches his eyes | 10:10 | |
tsdgeos | ouch that hurt | 10:10 |
tsdgeos | ok, can't see a difference anymore | 10:12 |
tsdgeos | JohnLea: ping | 10:26 |
=== karni is now known as karni-afk | ||
=== MacSlow is now known as MacSlow|lunch | ||
=== karni-afk is now known as karni | ||
facundobatista | Holas | 11:17 |
yecril71pl | Hello, what do I need to have nautilus run over ssh? | 11:41 |
=== MacSlow|lunch is now known as MacSlow | ||
=== rpadovani_ is now known as rpadovani | ||
mzanetti | seb128: hey, what's the recommended way to launch system settings from a build dir without installing? | 12:59 |
seb128 | mzanetti, hey, it's "don't" | 13:00 |
mzanetti | :( | 13:00 |
mzanetti | really? | 13:00 |
seb128 | sorry we just don't support that atm | 13:00 |
tsdgeos | pstolowski: https://launchpad.net/~unity-team/+archive/ubuntu/phone-right-edge/+packages has it | 13:00 |
seb128 | qml import paths are no fun | 13:00 |
mzanetti | seb128: hah... works quite well for me :D | 13:00 |
tsdgeos | kgunn: https://launchpad.net/~unity-team/+archive/ubuntu/phone-right-edge/+packages has the list bottom edge thing, who should be testing it? | 13:00 |
seb128 | mzanetti, you probably know qt better than us, patches are welcome ;-) | 13:00 |
mzanetti | seb128: ok | 13:01 |
seb128 | mzanetti, we didn't figure out how making it load locale plugins | 13:01 |
seb128 | mzanetti, usually we just sudo cp over the system version, which sucks but sort of work... | 13:01 |
seb128 | sudo cp; system-settings on your command line and call that back every time you do a change | 13:01 |
tsdgeos | mterry: ping | 13:03 |
mterry | tsdgeos, hello! | 13:03 |
tsdgeos | mterry: can you merge trunk over pull to referesh? | 13:03 |
mterry | tsdgeos, oh sure | 13:03 |
tsdgeos | mterry: also i was wondering about that style thing | 13:04 |
mterry | tsdgeos, no conflicts...? | 13:04 |
tsdgeos | mterry: don't know, just doesn't build as of now :D | 13:04 |
mterry | tsdgeos, oh let me test | 13:04 |
mterry | tsdgeos, the style sure, what about it? | 13:04 |
tsdgeos | mterry: so you're only removing the activityindicator? | 13:05 |
tsdgeos | or doing more things? | 13:05 |
mterry | tsdgeos, removing the activityIndicator is most of it. Also adjusting height, but I think I could have done that with default style | 13:06 |
mterry | I'm using same strings as normal style | 13:06 |
seb128 | kgunn, hey, any new about getting the unity8 rtm vcs updated? | 13:06 |
tsdgeos | mterry: because the style exposes refreshIndicator which we may just use to disable it instead of that big chunk of code? | 13:06 |
mterry | I looked into that and determined we couldn't... let me find out why | 13:07 |
mterry | Oh... I also fixed a bug in the SDK with how the text fades in/out | 13:09 |
mterry | tsdgeos, ok well besides fixing the text fading bug, the style instance is a super private variable of StyledItem (__styleInstance). And refreshIndicator is only exposed by the Ambiance theme. So hiding refreshIndicator that way would be extremely brittle | 13:12 |
tsdgeos | we do use __styleInstance in other places afaik | 13:13 |
tsdgeos | actually, no, just on a test | 13:13 |
tsdgeos | mterry: the other option is actually inheritnt from the Ambience theme | 13:14 |
tsdgeos | taht we actualyl do quite a lot | 13:14 |
tsdgeos | and then you can just set the property there to false | 13:14 |
mterry | tsdgeos, I wondered about that, but wasn't sure how. And wouldn't it require the ambience theme to be installed? | 13:15 |
tsdgeos | it would | 13:15 |
tsdgeos | but it's our default theme | 13:15 |
tsdgeos | it's a "requisite" anyway | 13:15 |
mterry | tsdgeos, not normally of u8, right? Just of Ubuntu | 13:15 |
tsdgeos | see ./qml/Dash/DashContentTabBarStyle.qml:17:import Ubuntu.Components.Themes.Ambiance 0.1 | 13:15 |
tsdgeos | or ./qml/Dash/PageHeader.qml:19:import Ubuntu.Components.Themes.Ambiance 1.1 | 13:16 |
tsdgeos | mterry: we use it, so we depend on it | 13:16 |
mterry | tsdgeos, looks like there's a bug in our packaging then, since debian/control doesn't capture that dependency :) | 13:16 |
tsdgeos | that may be | 13:16 |
mterry | tsdgeos, well anyway. That's *why* I did it like I did. If you like I can inherit from Ambiance. I'm personally not thrilled with that solution, but if we already do it... | 13:16 |
tsdgeos | mterry: i personally prefer less code than more code | 13:17 |
tsdgeos | and sure the dependency is not super awesome | 13:17 |
tsdgeos | but if it still works i'd prefer that | 13:17 |
mterry | tsdgeos, OK. I can work on that today | 13:18 |
tsdgeos | if it has other issues we can leave it | 13:18 |
mterry | tsdgeos, it will be slightly uglier because of the SDK text fading bug | 13:18 |
mterry | tsdgeos, but I can file an LP bug about that, maybe it will get fixed in time | 13:18 |
tsdgeos | mterry: ok, then leave it and add a TODO | 13:18 |
tsdgeos | we don't want ugl | 13:18 |
tsdgeos | and file the bug | 13:18 |
kgunn | seb128: yeah, just confirmed with mzanetti what we do | 13:18 |
seb128 | kgunn, thanks | 13:19 |
tsdgeos | mterry: and move it to a different file to GSV is ultra big as it is :D | 13:19 |
mterry | tsdgeos, (to be specific, the two text labels fade in/out, but the text actually changes immediately, before the fade-out. Looks really weird) | 13:19 |
tsdgeos | mterry: it = the style | 13:19 |
mzanetti | kgunn: ? | 13:19 |
tsdgeos | mterry: yeah i just saw looks very bad | 13:19 |
tsdgeos | mterry: so let's do this, keep the code with a TODO to evaluate the use of the standard style, file the bug and move the style code to a different file to clutter GSV less, works for you? | 13:20 |
mterry | tsdgeos, sure, will do | 13:20 |
tsdgeos | awesome :) | 13:20 |
kgunn | mzanetti: i'll sync up unity8/rtm from the unity8 trunk | 13:22 |
kgunn | seb needs it | 13:22 |
mzanetti | ah, right, that one | 13:22 |
kgunn | seb128: done | 13:33 |
seb128 | kgunn, mzanetti, excellent, thanks! | 13:34 |
mzanetti | seb128: btw, I've simplified this one (https://code.launchpad.net/~mzanetti/ubuntu-system-settings/update-reset-launcher/+merge/234309) as your team wished, and also am implementing a fix in the launcher to actually do the reset in real time | 13:35 |
=== karni is now known as karni-lunch | ||
seb128 | mzanetti, thanks, the CI run suggest you have a merge conflict | 13:36 |
mzanetti | can't be :D | 13:37 |
mzanetti | seb128: yeah, I had one, then I deleted everything and started from scratch because I got confused by it | 13:38 |
mzanetti | seb128: so old ci run, but code should be fine | 13:38 |
seb128 | mzanetti, ok, great, thanks | 13:38 |
seb128 | mzanetti, if you make unity8 update without restart then we can drop the "need to restart" dialog ;-) | 13:39 |
mzanetti | is there a restart dialog? didn't see that. but yeah, we can drop that | 13:39 |
mterry | tsdgeos, done | 13:45 |
tsdgeos | paulliu: ping | 13:46 |
tsdgeos | mterry: cool | 13:46 |
paulliu | tsdgeos: hi | 13:59 |
tsdgeos | paulliu: was going to say we may need to make the "mascot" of the previewheader a bit bigger | 13:59 |
tsdgeos | but let's leave it for later | 13:59 |
paulliu | tsdgeos: ok. | 14:00 |
=== _salem is now known as salem_ | ||
tsdgeos | mterry: have you tried on the phone? | 14:07 |
tsdgeos | doesn't seem to play very nice with the apps scope | 14:07 |
mterry | tsdgeos, I did. What are you seeing? | 14:07 |
tsdgeos | mterry: the bar doesn't go back to its intial height in some cases | 14:08 |
tsdgeos | say 30% | 14:08 |
tsdgeos | and then when you change to the next scope | 14:08 |
tsdgeos | the list changes vertically when there doesn't seem to be a need | 14:08 |
mterry | tsdgeos, 30% of screen? | 14:09 |
tsdgeos | not your bug most probably | 14:09 |
tsdgeos | 30% of the times | 14:09 |
tsdgeos | it's a few pixels ususally | 14:09 |
mterry | tsdgeos, it's a small gap, right? | 14:09 |
mterry | tsdgeos, yeah, that was there when I got there :) | 14:09 |
tsdgeos | noticeable enough that if you scroll horizontally it will shrink | 14:09 |
tsdgeos | mterry: what was there? | 14:09 |
mterry | tsdgeos, yes I've seen that, it was there | 14:09 |
tsdgeos | mterry: before your patch? | 14:10 |
mterry | tsdgeos, righ | 14:10 |
mterry | t | 14:10 |
mterry | let me double check on the other phone I have | 14:10 |
mterry | just to be safe, but I remember seeing that when I was first playing with pulling | 14:11 |
tsdgeos | that's weird | 14:12 |
mterry | tsdgeos, ah hold on | 14:12 |
mterry | tsdgeos, I can't reproduce on mako/utopic | 14:12 |
mterry | tsdgeos, so maybe it's either my branch or krillin/rtm, will get back to you | 14:13 |
cwayne | Cimi: hey, you seen this? https://bugs.launchpad.net/savilerow/+bug/1375822 | 14:13 |
ubot5 | Ubuntu bug 1375822 in unity8 (Ubuntu) "Settings go over header when scrolling" [Undecided,New] | 14:13 |
tsdgeos | mterry: it'd be safer if you trigger the refresh once the thing has gone back to its initial state, seems you do it a bit earlier now | 14:14 |
mterry | tsdgeos, perhaps -- I would have hoped that wouldn't screw anything up, so either delay refresh or fix that bug | 14:14 |
tsdgeos | mterry: LVWPH is not designed for lots of things happening at the same time :D it's better if you delay the thing until the body is again in place | 14:15 |
mterry | tsdgeos, OK will confirm I added the bug (seems likely at this point) and try to fix | 14:16 |
mterry | tsdgeos, good catch | 14:16 |
tsdgeos | mterry: nice ) | 14:16 |
mterry | My memories of seeing it before I mucked with the code must have been wrong. Makes me worried; I feel old and senile now | 14:17 |
mterry | tsdgeos, updated | 14:24 |
=== karni-lunch is now known as karni | ||
tsdgeos | mterry: that feels much better doesn't it ? | 14:44 |
mterry | tsdgeos, eh. It doesn't stick anymore so that's nice. But I'd still prefer instance refresh + no sticking | 14:45 |
tsdgeos | he he | 14:45 |
tsdgeos | you want it all! | 14:45 |
mterry | I'm high maintenance | 14:45 |
tsdgeos | mterry: why a rectangle? | 14:46 |
tsdgeos | can't pullToRefreshClippingRectangle be a Item ? | 14:47 |
tsdgeos | mterry: also update .pot | 14:48 |
mterry | tsdgeos, (oh we do that manually? I always think we do that automatically) | 14:48 |
tsdgeos | we don't | 14:48 |
tsdgeos | make pot_file | 14:48 |
tsdgeos | i think | 14:48 |
mterry | tsdgeos, good question re: item. I feel like I might have had a reason, but don't know what it would be, since I make the rectangle transparent | 14:48 |
tsdgeos | yeah | 14:49 |
tsdgeos | transparent rectangle | 14:49 |
tsdgeos | seems like you want an item :D | 14:49 |
tsdgeos | unless pulltothingie | 14:49 |
tsdgeos | wants a rectangle as parent | 14:49 |
tsdgeos | that'd be weird | 14:49 |
mterry | tsdgeos, let me just try replacing it... | 14:49 |
mterry | tsdgeos, no it doesn't care | 14:49 |
mterry | tsdgeos, manual pot updates inside of branches seems like a recipe for conflicts | 14:49 |
tsdgeos | mterry: correct D: | 14:50 |
mterry | I feel like there should be one per-silo update | 14:50 |
tsdgeos | yeah | 14:50 |
mterry | Yeah... pot updates inside of branches is stupid | 14:50 |
tsdgeos | now how do we do that? | 14:50 |
mterry | tsdgeos, well as part of making a silo, we could add a pot update branch... but it would have to include all the silo branches | 14:50 |
tsdgeos | yeah | 14:51 |
tsdgeos | i don't know | 14:51 |
tsdgeos | other teams do a manual push after land | 14:52 |
tsdgeos | we can do that | 14:52 |
mterry | tsdgeos, I think I was using the Rectangle during debugging, to see where it was -- the joiner point between header and content is weird | 14:52 |
* mterry is testing Item | 14:52 | |
=== gatox is now known as gatox_lunch | ||
tsdgeos | mzanetti: what do you think about a manual push after silo land to update the .pot? | 14:53 |
mzanetti | manual push to trunk? | 14:53 |
mterry | tsdgeos, branch uses Item now | 14:53 |
tsdgeos | mzanetti: yeah lots of projects do that for the .pot update, since "doesn't need review, it's just an automatical thing done manually" | 14:54 |
mzanetti | tsdgeos: I guess that might be an issue with the rtm sync though | 14:54 |
* tsdgeos shrugs | 14:55 | |
mterry | tsdgeos, I've also updated pot though I hate it | 14:55 |
tsdgeos | mterry: ok | 14:55 |
tsdgeos | let's hope we don't get much conflicts | 14:55 |
mterry | tsdgeos, it's a guaranteed conflict with any other pot update, becasue the pot header timestamp changes | 14:56 |
mterry | So only one string change branch per silo | 14:56 |
tsdgeos | mterry: vote for https://bugs.launchpad.net/cupstream2distro/+bug/1359667 | 14:56 |
ubot5 | Ubuntu bug 1359667 in Canonical Upstream To Distro [AKA CI Train] "There should be a hook mechanism available" [Undecided,Confirmed] | 14:56 |
mterry | Seems crazy | 14:56 |
tsdgeos | it is crazy | 14:58 |
mterry | mzanetti, tsdgeos, do we care about rtm sync for pot? Why does rtm need to have latest pot? | 15:00 |
tsdgeos | mterry: i don't know, kgunn, mzanetti, seb128 and more people were discussing it yesterday | 15:00 |
tsdgeos | there seems to be the fact we have a rtm branch and noone really knows why | 15:00 |
mzanetti | :D | 15:01 |
mzanetti | well... I kinda do know why | 15:01 |
mzanetti | we're just not using it, in unity8 at least | 15:01 |
tsdgeos | but someone said we were? | 15:03 |
kgunn | i hazily recall saviq wanted it so we'd run ui tests as part of landing in silos ?....but can't remember how that would be forced | 15:04 |
kgunn | seb wanted | 15:04 |
kgunn | it | 15:04 |
kgunn | updated | 15:04 |
mterry | But for all those reasons, who cares if the rtm pot file lags a release? | 15:05 |
pstolowski | tsdgeos, thanks for the ppa | 15:14 |
seb128 | mterry, because the pot before the update didn't include "Restart" and that string couldn't be translated in the rtm serie and was showing in english on e.g my french phone | 15:29 |
mterry | seb128, but that's a lag of 1 release -- which will happen anyway even if we update pot at release time in both places | 15:30 |
mterry | seb128, by the time the next rtm lands, it will get the translations from utopic series, right? | 15:30 |
* mterry might not understand how rtm translations are generated | 15:31 | |
seb128 | mterry, they are imported from the pot in the vcs rtm serie | 15:31 |
seb128 | mterry, dunno what you define 1 release | 15:31 |
seb128 | mterry, the rtm vcs didn't have a commit since septembre 9 | 15:31 |
seb128 | mterry, that was wrong/outdated in any case, no point have a rtm vcs if that doesn't contain what is in the rtm pocket | 15:32 |
seb128 | mterry, what's the issue with having the vcs updated with the actual code from the serie it should reflect? | 15:32 |
mterry | seb128, so Utopic releases U1, syncs that release to rtm as R1, then updates its pot. U2 gets released with translations from that pot, syncs as R2 which includes the pot from U1. Wouldn't rtm at that point be able to take translations that have been made between U1 and U2? | 15:33 |
seb128 | mterry, when is u8 updating its pot? | 15:34 |
mterry | seb128, oh I don't have an opinion on what should be in the rtm vcs itself -- I'm fine with it being reflected. I think unity8 is just being a bit lazy about it because we keep in sync | 15:34 |
seb128 | I don't understand why there is a vcs serie if trunk is what is used in rtm | 15:34 |
seb128 | but | 15:34 |
mterry | seb128, right now... We do it as part of the silo | 15:34 |
mterry | seb128, but I want to do it immediately after silo release | 15:35 |
seb128 | - the translations are currently defined by rtm/po/unity8.pot | 15:35 |
seb128 | - that was lacking "Restart" | 15:35 |
seb128 | -> therefor that item couldn't be translated on the rtm image | 15:35 |
seb128 | asking for the vcs to be updated was one solution | 15:35 |
seb128 | I would have been happy with another one | 15:35 |
seb128 | mterry, but yeah, translations in upstream projects suck compared to translations in ubuntu where the pot is updated during the package build | 15:36 |
mterry | hrm yeah sounds like an orthogonal but related problem from when precisely the pot is updated | 15:36 |
seb128 | we need a way to make the pot autoupdated by launchpad on commits | 15:36 |
seb128 | or daily | 15:36 |
mterry | seb128, tsdgeos pointed me at bug 1359667 | 15:37 |
ubot5 | bug 1359667 in Canonical Upstream To Distro [AKA CI Train] "There should be a hook mechanism available" [Undecided,Confirmed] https://launchpad.net/bugs/1359667 | 15:37 |
seb128 | we keep having outdated pot | 15:37 |
=== gatox_lunch is now known as gatox | ||
dandrader | dednick, could you chime in here? https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1373966/comments/4 | 16:17 |
ubot5 | Ubuntu bug 1373966 in unity8 (Ubuntu) "Trusted prompt sessions get stuck on screen blank" [Critical,Triaged] | 16:17 |
dednick | dandrader: yup. just did | 16:35 |
dednick | dandrader: dont know if what i said will help though | 16:35 |
dandrader | dednick, what's a "prompt provider" and a "trust helper"? | 16:38 |
dednick | dandrader: heh. | 16:38 |
dednick | dandrader: trust helper is what starts and manages a trust session. | 16:39 |
dednick | prompt provider is the child ui. | 16:39 |
dednick | dandrader: trust helper does not have any stake in unity8 | 16:39 |
dednick | dandrader: the application (web browser, etc) asks a trust helper service to start a trust session with a prompt provider (login ui) | 16:41 |
dednick | so ends our complicated nomenclature judo | 16:42 |
dandrader | dednick, I have a faint recollection that you sent me a URL with a diagram of this whole thing.... | 16:44 |
dandrader | some time ago | 16:44 |
dandrader | or was it me just asking that same question months ago | 16:45 |
dednick | dandrader: https://docs.google.com/a/canonical.com/drawings/d/1Sx3Q6IugYY0DjrQfN3em6PBhyeNotXtYAa4UioVdT8c/edit | 16:45 |
dandrader | dednick, thanks! | 16:46 |
dednick | dandrader: or this is probably more accurate now: https://docs.google.com/a/canonical.com/drawings/d/1IFSCWt_lOe5VPDf7PdplVwyHwimChtJKp1iW5UdlkHY/edit | 16:47 |
dandrader | ah, ok | 16:48 |
=== alan_g is now known as alan_g|EOD | ||
=== dandrader is now known as dandrader|afk | ||
=== dandrader|afk is now known as dandrader | ||
dandrader | racarr, ping | 19:19 |
dandrader | racarr, commented on some of your pending MPs | 19:25 |
=== dandrader is now known as dandrader|bbl | ||
ted | mterry, Can we turn your branch into an MR so I can make a silo to put things together? | 20:12 |
ted | mterry, It can be WIP. | 20:12 |
ted | mterry, It seems my phone doesn't have enough space to get all the deps to build U8. It's grown. | 20:12 |
mterry | ted, OK. I'm in the middle of some sort of awful internet problems. I'll try to push and propose. I'm also debugging new changes to the branch, to get the content itself to adjust | 20:18 |
ted | mterry, K, I just want to get all the indicators and U8 together so we can test them. | 20:19 |
ted | mterry, Also, do you know if there's a system settings branch to add the MessagesWelcomeScreen value setting? | 20:19 |
mterry | ted, no I don't know | 20:21 |
ted | Wow, system settings has a bunch of proposed branches. | 20:27 |
mterry | ted, guh. I need to go to a coffee shop. My home internet is so awful I can't even push a branch | 20:28 |
mterry | ted, in meantime, feel free to take that junk branch, put it in a team branch in unity8 space and make a fake WIP proposal | 20:28 |
mterry | else I can hopefully do it when I'm back up | 20:28 |
ted | K | 20:29 |
ted | mterry, I can do that, but another idea is to SSH somewhere with screen and do it. | 20:29 |
racarr | dandrader|bbl: Thanks :) will iterate on them | 20:31 |
ted | mterry, Hmm, there's a "showAllUI" do you know what that is? | 20:32 |
mterry | ted, yeah but these 5 bytes per second I'm getting, I need to move elsewhere | 20:33 |
mterry | ted, OK back from Starbucks | 20:44 |
ted | mterry, I didn't get a chance to push yet. Do you want to? | 20:46 |
mterry | ted, sure | 20:46 |
racarr | dandrader|bbl: Should be all updated :) | 20:48 |
racarr | thanks | 20:48 |
mterry | ted, https://code.launchpad.net/~mterry/unity8/greeter-profiles-test/+merge/236613 | 20:50 |
=== salem_ is now known as _salem | ||
=== dandrader_ is now known as dandrader | ||
dandrader | racarr, you forgot this one https://code.launchpad.net/~mir-team/platform-api/support-state-changing/+merge/235545/comments/579554 | 23:13 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!