| === kgunn is now known as Guest14808 | ||
| === _salem is now known as salem_ | ||
| === kgunn is now known as Guest35270 | ||
| === salem_ is now known as _salem | ||
| tsdgeos | Cimi: can you top approve https://code.launchpad.net/~aacid/unity8/bug1365929/+merge/234795 ? seems autopilot tests are all broken again in CI for no reason | 07:45 |
|---|---|---|
| Cimi | tsdgeos, done | 07:46 |
| tsdgeos | Cimi: you're going to the office today? | 07:48 |
| Cimi | tsdgeos, yup | 07:48 |
| tsdgeos | Cimi: can you please ask them to think on the transitions for https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1368670 ? | 07:48 |
| Cimi | tsdgeos, having my coffee now (much better at home than there :P) | 07:48 |
| ubot5 | Launchpad bug 1368670 in unity8 (Ubuntu) "[Dash] Simplify 'Manage Dash' area" [Critical,Triaged] | 07:48 |
| Cimi | tsdgeos, transitions? | 07:48 |
| Cimi | tsdgeos, for the back button? | 07:49 |
| tsdgeos | and for open | 07:49 |
| tsdgeos | Dash overview has an amazing open transition | 07:49 |
| Cimi | tsdgeos, open is like a reply command in messages | 07:49 |
| tsdgeos | where the current dash item just fades into the dash item in the list | 07:49 |
| Cimi | tsdgeos, I believe will be like in contacts | 07:49 |
| tsdgeos | thiis can't happen here | 07:49 |
| Cimi | tsdgeos, you pull the list from the bottom | 07:49 |
| tsdgeos | so i want them to tell me what needs to happen in both open and close | 07:49 |
| Cimi | tsdgeos, that bottom edge will probably go away after RTM anyway | 07:50 |
| Cimi | tsdgeos, there are other plans... | 07:50 |
| tsdgeos | good for after RTM | 07:50 |
| tsdgeos | i thought we all cared for before RTM | 07:51 |
| Cimi | tsdgeos, yes we do | 07:51 |
| tsdgeos | because if something is unusable it has to be fixed | 07:51 |
| tsdgeos | then let's make not things that are unusable | 07:51 |
| Cimi | tsdgeos, but this is not the final bottom swipe | 07:51 |
| Cimi | tsdgeos, this will eventually go as a menu button in the header | 07:51 |
| Cimi | tsdgeos, and bottom edge will probably contain feed customisation | 07:51 |
| tsdgeos | eventually is not now :) | 07:51 |
| tsdgeos | i want to know what we do now | 07:52 |
| Cimi | tsdgeos, two stages | 07:52 |
| Cimi | tsdgeos, 1st, now, we have a list of feeds in the bottom edge | 07:52 |
| Cimi | tsdgeos, 2nd, we move this list as a button on the header | 07:52 |
| Cimi | tsdgeos, and the bottom edge will become something else, related to feed customisation | 07:52 |
| Cimi | tsdgeos, for rtm we care of 1 | 07:53 |
| tsdgeos | so we need to have proper transitions for 1 | 07:53 |
| tsdgeos | don't we? | 07:53 |
| Cimi | tsdgeos, I think will be like in contacts/messaging | 07:53 |
| Cimi | tsdgeos, you pull the listview over the dash | 07:54 |
| Cimi | way simpler than now | 07:54 |
| tsdgeos | get them to confirm and write it on the bug or somewhere pelase | 07:54 |
| Cimi | tsdgeos, ok | 07:55 |
| Cimi | tsdgeos, which branches to review today? | 08:06 |
| Cimi | tsdgeos, mzanetti we have a long list of branches, we need to land land land | 08:07 |
| tsdgeos | on it i guess :d | 08:07 |
| mzanetti | Cimi: then go ahead and test silo 11 | 08:07 |
| mzanetti | and fix the issues in there :P | 08:07 |
| Cimi | afk for a bit, shower | 08:08 |
| mzanetti | :D | 08:08 |
| mzanetti | hah! I think my N4 is recovering! It flashes red again after being plugged into the wall charger for >10 hours now | 08:11 |
| tsdgeos | \o/ | 08:13 |
| tsdgeos | Cimi: for the image gallery click thing, please double check if we can kill the ¿duplicated? FIXME in qml/Dash/Previews/PreviewImageGallery.qml i tried yesterday and it seems we don't need it | 08:18 |
| mzanetti | dednick: good morning | 08:33 |
| mzanetti | dednick: testing silo11, indicators don't seem to connect | 08:33 |
| tsdgeos | marcustomlinson: https://code.launchpad.net/~marcustomlinson/unity8/handle_null_preview/+merge/234054/comments/574267 | 08:39 |
| mzanetti | dednick: ok... have to drop the indicator-polishing branch from the silo then | 08:49 |
| Cimi | tsdgeos, having look soon, going to the office in a bit | 08:50 |
| tsdgeos | ok | 08:50 |
| Cimi | packing devices... | 08:50 |
| mzanetti | Cimi: please unapprove this: I don't have permissions: https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/indicator-polishing/+merge/229083 | 08:52 |
| Cimi | mzanetti, why? | 08:53 |
| mzanetti | because it breaks the indicators | 08:54 |
| tsdgeos | dednick: are you please taking care of https://code.launchpad.net/~larsu/unity8/stop-using-statusicon/+merge/234502 ? | 08:55 |
| Cimi | mzanetti, but this morning we are supposed to merge the indicators together | 09:04 |
| dednick | mzanetti: erm. hm. that's weird | 09:11 |
| mzanetti | Cimi: ? | 09:12 |
| tsdgeos | dednick: https://code.launchpad.net/~dandrader/unity8/post-prompts-in-prompts-tweaks/+merge/233377 ? | 09:12 |
| Cimi | mzanetti, nick has a branch to go with that | 09:12 |
| Cimi | mzanetti, currently need fixing | 09:12 |
| dednick | Cimi: mzanetti is testing the silo | 09:13 |
| dednick | mzanetti: i'm actually on holiday today. i'll be out for a bit, then can try take a look | 09:13 |
| mzanetti | Cimi: ah... that might be the issue | 09:13 |
| mzanetti | Cimi: seems the other branch is not in the same silo | 09:13 |
| Cimi | mzanetti, ok | 09:13 |
| mzanetti | dednick: ^ | 09:13 |
| mzanetti | dednick: so maybe nothing you need to fix. | 09:13 |
| dednick | tsdgeos: it was approved but not top approved because it's failing CI | 09:14 |
| tsdgeos | autopilot is broken | 09:14 |
| tsdgeos | stupid autopilot and stupid CI randomly break all the time | 09:14 |
| dednick | tsdgeos: yeh. i've approved it already | 09:14 |
| dednick | top i mean | 09:14 |
| dednick | mzanetti: supposedly all the required branches are in the silo | 09:17 |
| === vrruiz_ is now known as rvr | ||
| dednick | mzanetti: can confirm it's not connecting :/ | 09:21 |
| dednick | i mean, i confirm | 09:21 |
| dednick | damnit. was working a couple of days ago | 09:22 |
| mzanetti | dednick: same here with my lifecycle fixes branch... i could swear it was working when we reviewed it | 09:23 |
| mzanetti | had to go from the silo too | 09:23 |
| dednick | mzanetti: hm. looks like my unity8 polishing branch isn't part of the silo | 09:23 |
| mzanetti | yeah... that's what cimi told me before too | 09:23 |
| dednick | file:///usr/share/unity8//Panel/Indicators/DefaultIndicatorPage.qml:177:5: Type MenuItemFactory unavailable | 09:23 |
| dednick | file:///usr/share/unity8//Panel/Indicators/MenuItemFactory.qml:454:9: Type MessageMenuItemFactory unavailable | 09:23 |
| dednick | file:///usr/share/unity8//Panel/Indicators/MessageMenuItemFactory.qml:81:13: Menus.SimpleTextMessageMenu is not a type | 09:23 |
| dednick | Shouldn't be looking for SimpleTextMessageMenu | 09:24 |
| dednick | i gota run. be back in 30 | 09:24 |
| tsdgeos | MacSlow: so no filtering at all for notification length? | 09:25 |
| tsdgeos | seems you went from 2 lines to "any lines" | 09:25 |
| tsdgeos | :D | 09:25 |
| Cimi | tsdgeos, in https://code.launchpad.net/~unity-team/unity8/croppedImageMinimumSourceSize/+merge/233319 | 09:33 |
| Cimi | when we do visible: false | 09:33 |
| Cimi | what brings it back to true? | 09:33 |
| mzanetti | tsdgeos: when you have some time, may I ask you to give this one a look? https://code.launchpad.net/~mzanetti/unity8/launcher-dont-hide-on-dash/+merge/233038 | 09:33 |
| tsdgeos | don't read my mind ;) | 09:35 |
| tsdgeos | mzanetti: the test fails | 09:35 |
| tsdgeos | may not be your fault though | 09:35 |
| tsdgeos | let's see trunk | 09:35 |
| MacSlow | tsdgeos, I don't know really know what can come through with these class-2 sms | 09:35 |
| tsdgeos | MacSlow: sure but it was 10 for snap notifications and 2 for rest, now is infinite for any? | 09:36 |
| MacSlow | tsdgeos, since nobody complained (or filed bugs) once the 10-line limit was in place, I initially thought "Just go back to this." | 09:36 |
| tsdgeos | MacSlow: honestly i don't know enought about the subject to give a better opinion, is there anyone else we should talk to? | 09:38 |
| MacSlow | tsdgeos, I was hoping to get more input from Design or the dialer-app folks. | 09:39 |
| tsdgeos | MacSlow: ping them to death :D | 09:39 |
| MacSlow | tsdgeos, I too know nothing about the sms-standard | 09:39 |
| MacSlow | tsdgeos, well there was a lengthy eMail-thread yesterday with no real output in that regard... so I assume they still try to figure it out. | 09:41 |
| tsdgeos | Cimi: in artImage? | 09:41 |
| tsdgeos | MacSlow: ok | 09:41 |
| tsdgeos | mzanetti: test also fails in master, not your dowin | 09:43 |
| tsdgeos | doing | 09:43 |
| MacSlow | tsdgeos, looks like even the folks working on the ofono-bit for sms don't know for sure the limits. So for the time being 2-line limit for non-snap-decision and unlimited for snap-decisions sounds like the best compromise. | 09:48 |
| tsdgeos | MacSlow: who creates snap-decisisons? | 09:49 |
| tsdgeos | can "random apps" do? | 09:49 |
| MacSlow | tsdgeos, no... confined/regular apps can't do that | 09:49 |
| tsdgeos | so it's mainly "us" | 09:49 |
| MacSlow | tsdgeos, correct | 09:49 |
| tsdgeos | looks the best compromise yes | 09:49 |
| tsdgeos | if you do that i'll approve | 09:49 |
| Cimi | tsdgeos, for example | 09:50 |
| tsdgeos | Cimi: noone, artImage is always non visible since it's part of a shape | 09:50 |
| Cimi | ok | 09:50 |
| Cimi | tsdgeos, cool | 09:51 |
| Cimi | tsdgeos, I don't have time to test the branch widely, did you? works well? | 09:51 |
| tsdgeos | works for me yes | 09:52 |
| tsdgeos | but honestly somebody should :D | 09:52 |
| tsdgeos | test it moar | 09:52 |
| Cimi | tsdgeos, wmall comment | 09:53 |
| Cimi | small | 09:53 |
| tsdgeos | ahhhhhhh | 09:54 |
| tsdgeos | i think i know why the autopilot tests are broken | 09:54 |
| tsdgeos | the damn notification about sd thing | 09:54 |
| tsdgeos | did that get fixed? | 09:54 |
| Cimi | tsdgeos, we want to have cards with mascot without the ubuntushape | 10:00 |
| tsdgeos | Cimi: yes | 10:06 |
| tsdgeos | Cimi: artImage is not the mascot | 10:06 |
| tsdgeos | MacSlow: does https://code.launchpad.net/~macslow/unity8/synchronous-notification/+merge/229059 have a bug or something? what do i try? | 10:37 |
| MacSlow | tsdgeos, no it's a pure feature... you can either test it with the icon-value.py Python-script from the corresponding lp:~macslow/unity-notifications/synchronous-notification branch or install lp:~macslow/indicator-sound/synchronous-notification on your device and use the volume-buttons | 10:39 |
| tsdgeos | MacSlow: i don't think we should accept new code without bugs associated to them at this point to be honest | 10:40 |
| tsdgeos | if the feature is so important that it needs to get in | 10:40 |
| tsdgeos | i want a critical bug marked as rtm | 10:40 |
| tsdgeos | what do others think? | 10:40 |
| tsdgeos | @unity: ↑ | 10:40 |
| mzanetti | tsdgeos: hah... I bet that's the easiest thing to get (the critical bug) :D | 10:41 |
| mzanetti | tsdgeos: but I agree with you | 10:41 |
| mzanetti | this has to stop | 10:41 |
| MacSlow | tsdgeos, mzanetti: don't tell me | 10:41 |
| tsdgeos | MacSlow: who "asked" for this feature? | 10:41 |
| mzanetti | but in the end its kgunn to decide | 10:41 |
| MacSlow | mzanetti, true | 10:41 |
| tsdgeos | well let's convince him | 10:42 |
| tsdgeos | per are post feature-everything | 10:42 |
| mzanetti | yeah, I told him yesterday already | 10:42 |
| tsdgeos | we want new stuff, fine, let it be in critical bugs | 10:42 |
| mzanetti | and I think he had a talk with olli about it too | 10:42 |
| MacSlow | mzanetti, implicitly I did too :) | 10:42 |
| tsdgeos | i want a paper trail when people ask why we were adding features such late | 10:42 |
| MacSlow | with kgunn not with olli | 10:42 |
| mzanetti | MacSlow: so again, who asked for this feature? and when? | 10:43 |
| MacSlow | mzanetti, design and pat | 10:43 |
| MacSlow | mzanetti, around last time in London | 10:44 |
| mzanetti | tsdgeos: uh... I guess this is required for making the SIM pin unlock behave properly | 10:44 |
| tsdgeos | mzanetti: : is it? i think it's mainly for sound notifications, no? | 10:44 |
| MacSlow | mzanetti: synchronous-notification?! no... that's for the volume-button feedback | 10:45 |
| mzanetti | well, atm we have to close the SIM pin dialog and reopen it | 10:45 |
| mzanetti | which kills the shaking animation for instance | 10:45 |
| MacSlow | mzanetti, https://www.youtube.com/watch?v=VBhxf2iQK8o | 10:45 |
| mzanetti | preview 21... duuude :D | 10:46 |
| MacSlow | mzanetti, although the visuals are of course updates now... that screencast isn't the latest | 10:46 |
| MacSlow | mzanetti, what? | 10:46 |
| MacSlow | mzanetti, 21 is the volume one... | 10:47 |
| * MacSlow looks again... | 10:47 | |
| MacSlow | maybe I miss-pasted | 10:47 |
| mzanetti | nono... video is fine I guess | 10:47 |
| MacSlow | ah ok | 10:47 |
| mzanetti | was just wondering if the notifications have so many feature that you can actually get 21 videos out of it | 10:47 |
| mzanetti | I struggled with 1 for the right edge | 10:47 |
| tsdgeos | MacSlow: so https://code.launchpad.net/~macslow/indicator-sound/synchronous-notification/+merge/229061 ? | 10:48 |
| MacSlow | mzanetti, I started counting very early... just take a look at preview 1 :) btw... preview != feature | 10:48 |
| MacSlow | tsdgeos, yeah it's basically three branches all in all | 10:49 |
| tsdgeos | MacSlow: and who is reviewing indicator-sound and the notifications one | 10:49 |
| tsdgeos | ? | 10:49 |
| MacSlow | tsdgeos, dednick would be the one for indicator-sound I guess | 10:50 |
| tsdgeos | MacSlow: then get him to do it, and address tedg's question too | 10:50 |
| === MacSlow is now known as MacSlow|lunch | ||
| tsdgeos | Cimi: answered/addressed comments in https://code.launchpad.net/~aacid/unity8/category_view_invisible_in_preview_mode/+merge/231844 | 11:17 |
| Cimi | Trevinho, c'e' un modo per disabilitare le gestures da unity? | 11:25 |
| Cimi | mouse | 11:25 |
| Cimi | mi apre la dash con due dita :'((( | 11:25 |
| Cimi | o alt tab | 11:26 |
| facundobatista | Holas | 11:26 |
| tsdgeos | MacSlow|lunch: please nominate someone to review https://code.launchpad.net/~macslow/unity8/swipe-dismiss-snap-decisions/+merge/233347 | 11:40 |
| Guest35270 | MacSlow|lunch: things are changing quickly for sure, its fine to work on the vol notif, but those guys are correct in that its not top priority | 11:55 |
| Guest35270 | https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1232633 | 11:55 |
| ubot5 | Launchpad bug 1232633 in unity8 (Ubuntu) "[system] Changing volume doesn't show feedback on the phone" [High,In progress] | 11:55 |
| Guest35270 | MacSlow|lunch: ...so for next 2-3 weeks priority should be on our "critical" + tag rtm14 | 11:56 |
| Guest35270 | it may mean, you don't get MP review love | 11:57 |
| Guest35270 | on a "high" | 11:57 |
| === MacSlow|lunch is now known as MacSlow | ||
| MacSlow | Guest35270, ok... but it's not like I only started on that just a few days ago... for notifications, due to their architecture, make me stretch between backend, frontend and apps, which makes keeping it all tight in a timely manner very hard. | 12:01 |
| MacSlow | Guest35270, incoming changes beyond certain dates don't help either | 12:02 |
| Guest35270 | MacSlow: i hear you, we're all coping... | 12:29 |
| === kgunn is now known as Guest35530 | ||
| === _salem is now known as salem_ | ||
| === facundobatista is now known as sala | ||
| === sala is now known as facundobatista | ||
| Trevinho | Cimi: no... Mi pare di no, s'era discusso riguardo aggiungere la possibilità di disabilitarlo, ma per ora è hardcoded | 12:55 |
| MacSlow | Trevinho, dude... wrong channel I bet :) | 12:57 |
| Trevinho | MacSlow: no, it's just that Cimi tempts me :D | 12:57 |
| MacSlow | Trevinho, don't give in... resist ;) | 12:58 |
| tsdgeos | mterry: what do you think of https://bugs.launchpad.net/bugs/1370473 ? | 13:01 |
| ubot5 | Launchpad bug 1370473 in unity8 (Ubuntu) "The edge demo with a pin is confusing" [Undecided,New] | 13:01 |
| tsdgeos | mterry: can you nominate someone to review https://code.launchpad.net/~mterry/unity8/tablet-security/+merge/234219 ? | 13:01 |
| mterry | tsdgeos, commented | 13:02 |
| tsdgeos | mterry: okidoki | 13:03 |
| mterry | tsdgeos, you mean can I volunteer someone to review it for me? :) Seems mean | 13:03 |
| tsdgeos | mterry: well noone has voluteered by himself in almost a week and we need someone to review :D | 13:04 |
| tsdgeos | mterry: given we don't have Michał i'd appreciate if you know who you think would have more knowledge of the code to review it | 13:04 |
| tsdgeos | mterry: also i am wondering if we need to care for this at the moment | 13:05 |
| tsdgeos | where we are time pressed by the phone | 13:05 |
| tsdgeos | if this is tablet-only | 13:05 |
| tsdgeos | mterry: maybe we can shelve it to post rtm? | 13:05 |
| mterry | tsdgeos, one of the theoretical attacks there was actually a phone that plugs into a larger screen to switch to tablet mode (not even sure if that's possible right now, but just in case) | 13:06 |
| mterry | tsdgeos, so it's not just tablet changes | 13:06 |
| tsdgeos | hmpf | 13:06 |
| tsdgeos | ok | 13:06 |
| mterry | tsdgeos, (get to locked-down emergency dialer, switch to tablet mode, dialer goes to sidestage and you are in session) | 13:07 |
| mterry | mzanetti, do you have time to review a branch? https://code.launchpad.net/~mterry/unity8/tablet-security/+merge/234219 | 13:08 |
| mzanetti | mterry: I started to look at it... but seems a big one. not sure how fast I'll get to it | 13:09 |
| mzanetti | mterry: but I'll put it on my todo | 13:09 |
| mzanetti | mterry: the other, with the pasphrase design, I belive that breaks some SIM pin lock stuff | 13:09 |
| mterry | mzanetti, guh! does it use retryText? | 13:10 |
| mzanetti | mterry: yes | 13:12 |
| mzanetti | mterry: well, not sure if it does already.. but its supposed to at least | 13:12 |
| mzanetti | mterry: not sure where Wellark is with that though | 13:12 |
| mterry | mzanetti, I see... I didn't see it being used in the code | 13:13 |
| === Malsasa_ is now known as Malsasa | ||
| === dandrader is now known as dandrader|afk | ||
| Cimi | greyback, is the word autosuggestion box | 13:42 |
| Cimi | greyback, it always counts for height | 13:42 |
| greyback | Cimi: yeah? Ok, so OSK needs fixing to correctly calculate the visible height of the OSK | 13:43 |
| greyback | nice find | 13:43 |
| Cimi | greyback, it took me 5 mins to figure out, 2 hours to install the emulator :D | 13:44 |
| Cimi | greyback, thought the emulator was slower, is quite fast! | 13:44 |
| greyback | Cimi: emulator? Don't you have phones? | 13:44 |
| Cimi | greyback, yes, but I hoped the emulator was slower to highlight the issue | 13:44 |
| greyback | Cimi: so the autosuggest box idea is a guess, based on what you are seeing? | 13:46 |
| Cimi | greyback, if I disable the autosuggest and other features (simple keyboard), the blank area follows the osk | 13:47 |
| greyback | Cimi: ok good | 13:47 |
| Cimi | greyback, if I enable the autosuggest, but in some cases is disabled (wifi password), i can see that when I close the keyboard, the blank area starts from 2-3 gu more, (where the autosuggestion box is, even if is invisible) | 13:48 |
| === dandrader|afk is now known as dandrader | ||
| Cimi | greyback, in oxide is still slow, but maybe we can do something different there | 13:48 |
| greyback | Cimi: check out lp:ubuntu-keyboard qml/Keyboard.qml and the reportKeyboardVisibleRect() function | 13:49 |
| Cimi | greyback, sure | 13:49 |
| Cimi | greyback, would it be possible to anchor the keyboard to application height? | 13:49 |
| Cimi | greyback, that way the keyboard will disappear a bit more slowly, but the issue will be completely gone | 13:50 |
| greyback | Cimi: possible yes. But resizing application windows is (1) slow and (2) not what every application wants | 13:50 |
| Cimi | greyback, oxide apparently does | 13:50 |
| Cimi | greyback, and is the worst case so far | 13:50 |
| greyback | Cimi: I can't comment on how apps do things | 13:50 |
| Cimi | greyback, oxide does, if you try with here maps, you see how the controls on screen move on top when the osk pops in | 13:52 |
| Cimi | greyback, in this case, if the application developer can decide where to anchor the osk, it might be a great workaround | 13:53 |
| greyback | Cimi: it's not possible right now | 13:53 |
| tsdgeos | mterry: i'm not sure i understand your reasoing on the LANGUAGE thing | 13:55 |
| tsdgeos | but i see what you have done works | 13:55 |
| mterry | mzanetti, so you say to be careful about the API for lockscreen, but if the notification code doesn't use the properties I change, what would be the problem? (were we waiting to hook up replyText once the backend provided it?) | 13:55 |
| tsdgeos | and have a reasoning | 13:55 |
| tsdgeos | so i think we can just approve it :D | 13:56 |
| mterry | tsdgeos, what about it doesn't make sense? | 13:56 |
| mzanetti | mterry: yes | 13:56 |
| tsdgeos | mterry: it is not that it does not make sense, it's just a long sentence! | 13:56 |
| mterry | tsdgeos, heh | 13:56 |
| tsdgeos | mterry: also i'm pretty sure those worked at some point | 13:56 |
| tsdgeos | since we used to have that problem for some other test | 13:56 |
| tsdgeos | and got fixed | 13:56 |
| mterry | tsdgeos, huh... I figured they got added for a reason, but following them through, it didn't seem to work | 13:57 |
| tsdgeos | can't really remember now what was the test that failed without them | 13:57 |
| Cimi | greyback, indeed, in Keyboard.qml you can see that wordRibbon is always taken into account regardless is visible or not | 13:57 |
| tsdgeos | let me check | 13:57 |
| === Guest35530 is now known as kgunn | ||
| greyback | Cimi: looking forward to your patch :) | 13:58 |
| Cimi | greyback, I will ping the maintainer :) | 13:58 |
| greyback | Cimi: dude come on, you're almost there, just write the fix and submit | 13:59 |
| tsdgeos | ah the time formatter test | 13:59 |
| mterry | tsdgeos, oh -- why were they working then without this change ...? | 14:01 |
| tsdgeos | mterry: because it works :D | 14:01 |
| tsdgeos | mterry: see http://paste.ubuntu.com/8365199/ | 14:01 |
| tsdgeos | ahh | 14:02 |
| tsdgeos | but it has a | 14:02 |
| tsdgeos | add_custom_target(${_test} env "LC_ALL=C" ${testCommand}) | 14:02 |
| Cimi | greyback, it can be tricky, I have to ask them | 14:03 |
| mterry | tsdgeos, yeah.. just found that myself | 14:04 |
| mterry | tsdgeos, right *after* trying to set DEFAULT_PROPERTIES... | 14:04 |
| tsdgeos | mterry: want to do something regardging that? or should i just approve? | 14:04 |
| mterry | tsdgeos, err, rather PROPERTIES ENVIRONMENT | 14:04 |
| mterry | tsdgeos, I can try to clean that up | 14:05 |
| tsdgeos | mterry: great :) | 14:06 |
| mterry | tsdgeos, what is the output of your locale? | 14:14 |
| mterry | i.e. running 'locale' | 14:14 |
| tsdgeos | mterry: http://paste.ubuntu.com/8365268/ | 14:14 |
| mterry | tsdgeos, heh, in my en_US locale, the time test didn't fail, just wanted to test that my changes work | 14:14 |
| mterry | tsdgeos, ah turns out that bit of code doesn't go through add_qml_test anyway. So I only removed the unnecessary set_test_property line but kept the manual env LC_ALL=C bit | 14:20 |
| tsdgeos | oki | 14:21 |
| mterry | tsdgeos, thanks! | 14:25 |
| Cimi | pstolowski, do you know if any scope uses the zoomable image? | 14:25 |
| pstolowski | Cimi, dunno | 14:31 |
| cwayne | Cimi: in preview? | 14:32 |
| Cimi | cwayne, yes | 14:32 |
| om26er | Hi! are these playback controls coming from Unity8 ? https://bugs.launchpad.net/ubuntu/+source/indicator-sound/+bug/1370129/+attachment/4205411/+files/sound.png | 14:33 |
| ubot5 | Launchpad bug 1370129 in indicator-sound (Ubuntu) "With music-app closed, the controls in the sound menu are dummy" [Undecided,New] | 14:33 |
| tsdgeos | josharenson: so you said you had issues with unity8 on the desktop? | 14:49 |
| josharenson | tsdgeos, yeah I can't seem to get the settings app to open | 15:00 |
| josharenson | its also taking a much longer amount of time than usual to launch the shell | 15:00 |
| tsdgeos | josharenson: in the destkop from unity8? | 15:00 |
| josharenson | yes | 15:00 |
| tsdgeos | we've never had that working afaik | 15:00 |
| josharenson | ah | 15:00 |
| * josharenson reads bug again | 15:01 | |
| tsdgeos | or have we? | 15:01 |
| tsdgeos | josharenson: wait wait, is it unity8 as the shell or unity8 run as app in another shell? | 15:01 |
| josharenson | does the store work? | 15:01 |
| josharenson | as app | 15:01 |
| josharenson | so check out https://bugs.launchpad.net/ubuntu/+source/qtmir/+bug/1354564 | 15:03 |
| ubot5 | Launchpad bug 1354564 in unity8-desktop-session (Ubuntu) "Keyboard no longer works in Unity8 desktop session" [Undecided,Confirmed] | 15:03 |
| josharenson | and I'm trying to follow steps in #13 | 15:03 |
| === kgunn is now known as Guest81961 | ||
| Cimi | cwayne, so? | 15:19 |
| === gatox is now known as gatox_lunch | ||
| === dandrader is now known as dandrader|lunch | ||
| === gatox_lunch is now known as gatox | ||
| === dandrader|lunch is now known as dandrader | ||
| alecu | Cimi: mzanetti: kgunn_: hi! in saviq's absence, who is coordinating landings in unity8? We got a fix for an annoying bug in Payments that we'd like to land soon: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1362622 | 17:57 |
| ubot5 | Launchpad bug 1362622 in unity8 (Ubuntu) "When cancelling a purchase the progress bar doesn't disappear in the preview" [Critical,In progress] | 17:57 |
| kgunn_ | alecu: so mzanetti has a silo up, do you have a reviewed/approved/tested branch ? or does it still need to go thru the review cycle ? | 18:01 |
| alecu | kgunn_: dobey made the branch, I reviewed and tested on my mako with devel-proposed. Since it's a branch for unity8 saviq usually does a review too. | 18:02 |
| mzanetti | alecu: ah ok, I'll put it on my todo | 18:03 |
| alecu | kgunn_: mzanetti: I don't mind being added as lander and doing the testing of this feature on both devel-proposed and rtm | 18:03 |
| mzanetti | alecu: ok. I'll put it into the next silo and let you know | 18:04 |
| alecu | mzanetti: great, thanks. | 18:04 |
| === dandrader is now known as dandrader|bbl | ||
| === salem_ is now known as _salem | ||
| === dandrader|bbl is now known as dandrader | ||
| === kgunn is now known as Guest93957 | ||
| josharenson | what is the correct way to install unity-notifications? | 23:56 |
| * josharenson trying make install like I should have before asking... doh | 23:57 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
