Saviq | o/ | 07:04 |
---|---|---|
tsdgeos | Saviq: let's land stuff :D | 07:27 |
Saviq | tsdgeos, dash-as-app is next in store, and there's still two issues to solve | 07:27 |
tsdgeos | ok :/ | 07:28 |
tsdgeos | Saviq: who's working on them? | 07:29 |
Saviq | tsdgeos, Daniel and myself | 07:29 |
Saviq | tsdgeos, or you could take over the second issue (meaning can't launch apps from dash from silo 7) | 07:29 |
Saviq | tsdgeos, we suspect click scope not sending proper appid:///, application:/// urls | 07:30 |
tsdgeos | Saviq: well i'm doing altnav, but if this is more important i can have a look if you want | 07:30 |
Saviq | tsdgeos, nah, do your stuff | 07:30 |
tsdgeos | oki | 07:30 |
Saviq | tsdgeos, we need to wait for Daniel anyway | 07:30 |
tsdgeos | oki | 07:31 |
tsdgeos | Saviq: this is old, no ? https://code.launchpad.net/~unity-team/unity8/grid-see-more/+merge/226297 disapprove? | 07:35 |
Saviq | tsdgeos, done | 07:36 |
Saviq | mhr3, hey, the trace here bug #1351801 looks rather similar to bug #1348702, don't you think? | 07:36 |
ubot5 | bug 1351801 in lrt-crashes "lrt test found crash file _usr_bin_unity8.32011.crash" [Critical,New] https://launchpad.net/bugs/1351801 | 07:36 |
ubot5 | bug 1348702 in unity-scopes-api "unity8 crashed with SIGSEGV" [High,Fix committed] https://launchpad.net/bugs/1348702 | 07:36 |
mhr3 | hmm, doesn't look completely the same | 07:37 |
Saviq | mhr3, symbols are not complete, though | 07:57 |
mhr3 | Saviq, yea, just had a look at the raw .crash file, looks very similar there, but the retrace is weird | 07:57 |
Saviq | mhr3, agreed | 07:59 |
tsdgeos | ah! | 08:25 |
tsdgeos | i know what's making the test_departments unstable | 08:25 |
tsdgeos | now onto a fix | 08:26 |
mhr3 | tsdgeos, hey, so question - could we land the nav + altnav in its current form and then look into what would be needed to properly support navigating from the middle of the tree? | 08:35 |
tsdgeos | mhr3: well, there's no altnav :D | 08:35 |
tsdgeos | so i guess no | 08:35 |
mhr3 | tsdgeos, after that then | 08:35 |
tsdgeos | sure, when there's altnav, hopefully at the end of today | 08:36 |
tsdgeos | we can land it | 08:36 |
tsdgeos | and then explore the other stuff | 08:36 |
mhr3 | k | 08:36 |
jgdx | h | 08:36 |
Cimi | Saviq, https://code.launchpad.net/~fboucault/ubuntu-ui-toolkit/flickable_right_speed | 08:40 |
mhr3 | Cimi, is there a branch with the scope settings? | 08:43 |
Cimi | mhr3, I am starting now | 08:43 |
Cimi | mhr3, will finish this week | 08:43 |
mhr3 | Cimi, let me know once there's something testable | 08:44 |
Cimi | mhr3, tomorrow you coming here? | 08:45 |
tsdgeos | Saviq: what's the thing that regenerates tests/mocks/Unity/Unity.qmltypes ? | 08:45 |
tsdgeos | qmlplugindump? | 08:46 |
mhr3 | Cimi, yep, and you'll get told off if it aint ready then! :P | 08:46 |
tsdgeos | do we have a way to update it? or just call it manually? | 08:46 |
Cimi | mhr3, you are gonna leave, why you care that much? :) | 08:46 |
Cimi | mhr3, you wanna stay? :) | 08:46 |
mhr3 | Cimi, who said i will be doing the telling off? :) | 08:48 |
tsdgeos | mhr3: i think i'm going to split it in two branches | 09:01 |
tsdgeos | one in lp:~unity-team/unity8/scopes-v4 that is mostly getting it to compile with scopes-v4 + renaming | 09:01 |
tsdgeos | and another one dependent on that that actually implements alt nav | 09:02 |
mhr3 | tsdgeos, sounds good to me, at least we can land the v4 stuff | 09:02 |
tsdgeos | easier to review i'd say | 09:02 |
tsdgeos | also if we want to land v4 we can do now yes | 09:02 |
tsdgeos | Saviq: do we use CachingNetworkManagerFactory at all? | 09:34 |
Saviq | tsdgeos, yes | 09:34 |
tsdgeos | we do | 09:34 |
tsdgeos | damn files that have classe isnide with different names :D | 09:35 |
Saviq | ;) | 09:35 |
tsdgeos | i guess we don't care it leaks | 09:35 |
tsdgeos | since it's app-long lived | 09:35 |
Saviq | tsdgeos, well, it's shell-lived, if that's what you mean | 09:36 |
tsdgeos | yeah | 09:36 |
tsdgeos | shell is an app ;) | 09:36 |
Saviq | not sure we shouldn't care about leaks in the shell ;) | 09:37 |
tsdgeos | Saviq: it's only a leak when you shut down | 09:37 |
Saviq | ah ok | 09:37 |
tsdgeos | which who cares :D | 09:37 |
tsdgeos | i mean it's technically one | 09:37 |
tsdgeos | but bleh | 09:37 |
paulliu | cwayne1: ping | 09:42 |
tsdgeos | mhr3: did we decide that there would only be altnavigation if there was navigation? | 09:49 |
mhr3 | tsdgeos, no | 09:49 |
mhr3 | tsdgeos, right now you can have altnav and no nav | 09:50 |
tsdgeos | ok, not a big issue either | 09:50 |
tsdgeos | just to know what's possible :D | 09:50 |
tsdgeos | anyone thinks how i could do this if at all? http://paste.ubuntu.com/7950889/ | 10:52 |
mhr3 | tsdgeos, connect to both :) | 10:53 |
tsdgeos | sure | 10:53 |
mhr3 | and ignore one :) | 10:53 |
tsdgeos | that i have | 10:53 |
tsdgeos | but is lame | 10:53 |
tsdgeos | in the "oh look how cool is this" world | 10:53 |
mhr3 | but works :) | 10:53 |
tsdgeos | sure | 10:53 |
Saviq | tsdgeos, other than dynamically creating that object, don't think so | 10:54 |
mhr3 | tsdgeos, you could always generate some qml ;P | 10:54 |
tsdgeos | hmmm | 10:55 |
tsdgeos | good idea | 10:55 |
tsdgeos | not | 10:55 |
=== MacSlow is now known as MacSlow|lunch | ||
facundobatista | Holas | 11:06 |
mhr3 | Saviq, seen http://paste.ubuntu.com/7950960/ ? | 11:07 |
mhr3 | Saviq, media-hub / dbus-cpp crash in u8 | 11:07 |
tsdgeos | Saviq: so we have any visuals on how the double navigation has to look like? | 11:11 |
tsdgeos | i understand it's half-half for buttons | 11:11 |
tsdgeos | the list is still full width | 11:11 |
tsdgeos | but i'm interested in how the buttons look when the list is open | 11:11 |
tsdgeos | to show the list belongs to the left or the right | 11:11 |
tsdgeos | is it only the ^ merker or there's something else? | 11:12 |
cwayne1 | paulliu: (late) pong | 11:14 |
paulliu | cwayne1: What data are you actually passed to attributes? | 11:16 |
cwayne1 | paulliu: 2014/08/04 07:16:52 [{4.00mi } {(4) http://s3-media2.fl.yelpcdn.com/assets/2/www/img/ccf2b76faa2c/ico/stars/v1/stars_large_4.png}] | 11:17 |
paulliu | cwayne1: is that a rating? | 11:17 |
cwayne1 | paulliu: its suppose to be a number of ratings, and then the image I get from the API | 11:17 |
mhr3 | tsdgeos, page 5 https://docs.google.com/a/canonical.com/document/d/1rtLO2iSm0rfTtoI7iKtGt2pVlf_65FOFZWG8rpFN2U4/edit | 11:23 |
tsdgeos | hmmmm | 11:24 |
tsdgeos | is it whiter because it's amazon? | 11:26 |
mhr3 | tsdgeos, don't think so | 11:28 |
mhr3 | tsdgeos, maybe it's whiter cause there's two | 11:29 |
tsdgeos | sure | 11:29 |
tsdgeos | but the selected one | 11:29 |
mhr3 | tsdgeos, or maybe it's supposed to be whiter even now when you activate it | 11:29 |
tsdgeos | gets white | 11:29 |
tsdgeos | but in a blue scope | 11:29 |
tsdgeos | that'd be weird? | 11:29 |
mhr3 | tsdgeos, similarly weird than having white departments in a blue scope? | 11:32 |
tsdgeos | correct :D | 11:33 |
tsdgeos | i always expected people would complain about that | 11:33 |
paulliu | cwayne1: very weird. Actually I cannot reproduce your bug. | 11:33 |
cwayne1 | paulliu: even with the click I sent? | 11:33 |
paulliu | cwayne1: wait. Let me try your click | 11:34 |
mhr3 | cwayne1, are there any scopes that override the background color? | 11:34 |
cwayne1 | mhr3: i know of one, the rest only override the pageheader background | 11:34 |
mhr3 | cwayne1, and it overrides it to black i guess? | 11:35 |
cwayne1 | uh, i think light blue | 11:35 |
mhr3 | cwayne1, oh.. we're wondering what do departments look like in that situation, cause they'll be white, right? | 11:36 |
cwayne1 | hah this one doesnt have depts | 11:36 |
cwayne1 | but no, they inherit the color | 11:36 |
paulliu | cwayne1: btw, how to let yelp scope to get some data? I saw empty. | 11:36 |
mhr3 | cwayne1, oh? | 11:36 |
mhr3 | cwayne1, in that case it's solved i guess :) | 11:37 |
cwayne1 | paulliu: should just work.. i wonder if it doesn't have data for wherever you are? | 11:37 |
mhr3 | tsdgeos, so yes, make it blue then | 11:37 |
cwayne1 | mhr3: id check with Saviq, but i recall it inheriting the color | 11:37 |
paulliu | cwayne1: Do I need to login to somewhere? | 11:37 |
cwayne1 | paulliu: nope | 11:38 |
paulliu | cwayne1: I'm in blue-fin office right now. Should be good. | 11:38 |
tsdgeos | mhr3: cwayne1: oh yes, they alrady inherit the colors | 11:38 |
tsdgeos | mhr3: so i guess i'll just put some opacity layer on the non open one and bedone | 11:38 |
tsdgeos | until someone complains :D | 11:38 |
cwayne1 | paulliu: agh crap. bluefin's squid proxy breaks it | 11:39 |
cwayne1 | paulliu: can you run it on mobile data instead? | 11:39 |
paulliu | cwayne1: I can't. I don't have a SIM card here. | 11:40 |
cwayne1 | damnit | 11:40 |
paulliu | cwayne1: I'll be back to Taiwan tomorrow. Let's see if it works then. | 11:41 |
cwayne1 | paulliu: cool, thanks. it just seems weird to me that it works pefectly in scope-tool, but doesnt on device | 11:43 |
paulliu | cwayne1: yeah. I'll check it on devices. | 11:44 |
dandrader | damn, unity 7 on my desktop is freezing right after entering the password | 11:45 |
dandrader | bregma_, any tips? ^ | 11:48 |
paulliu | dandrader: upgrade in console? | 11:48 |
paulliu | dandrader: maybe it is not freezing. Just very slow. | 11:48 |
dandrader | upgraded & rebooted already | 11:49 |
Saviq | tsdgeos, mhr3, yeah design for that is somewhat inconsistent | 11:59 |
Saviq | the amazon document isn't really visual design | 11:59 |
Saviq | tsdgeos, we need to introduce a background color behind the nav buttons in any case | 11:59 |
mhr3 | tsdgeos, opacity layer? eek? the color is supposed to be customizable | 12:00 |
mhr3 | s/eek\?/eek!/ | 12:00 |
dandrader | paulliu, that solved it... http://www.webupd8.org/2012/10/how-to-reset-compiz-and-unity-in-ubuntu.html although now I can only make it work by running "unity-reset" :/ | 12:02 |
=== MacSlow|lunch is now known as MacSlow | ||
Saviq | dandrader, hey, any progress on shell locking up with dash as app? | 12:27 |
kgunn | was just wondering too ^ | 12:27 |
Saviq | oh there's two elopios | 12:28 |
Saviq | elopio, I just got silo 7 installed and I can run apps from dash just fine, isn't that what you reported not working for you? | 12:28 |
dandrader | Saviq, kgunn, not today as so far I've been just struggling with getting unity7 on my desktop to work again and copying the work form laptop to desktop | 12:30 |
dandrader | from | 12:30 |
dandrader | Saviq, kgunn apart from that I'm not stuck. but updating tests, mocks has taken quite a bit of my time last Friday and it's still not done | 12:33 |
mhr3 | jhodapp, looks familiar? http://paste.ubuntu.com/7950960/ | 12:41 |
kgunn | dandrader: ack | 12:42 |
jhodapp | mhr3, not really...when are you getting that? | 12:43 |
Saviq | mhr3, anywhere we should look at for paulliu_ to be able to implement interactive: false? | 12:44 |
Saviq | mhr3, or will it just be if (template["interactive"]) { }? | 12:46 |
=== _salem is now known as salem_ | ||
tsdgeos | mhr3: i found a nicer way to make it nice | 12:57 |
tsdgeos | wait for lats bits | 12:57 |
mhr3 | jhodapp, u8 crash, without retrace it was pointing into libmedia-hub-client | 12:58 |
mhr3 | Saviq, let's make it "non-interactive", so default == false | 12:59 |
Saviq | paulliu_, ok so ↑, please in GenericScopeView check whether template["non-interactive"] is true, and ignore taps / long presses | 13:00 |
Saviq | paulliu_, base off of Cimi's touchdown branch so that you can disable the touchdown effect, too | 13:00 |
mhr3 | pstolowski, fyi ^ | 13:00 |
=== pete-woods is now known as pete-woods|lunch | ||
paulliu_ | Saviq: ok. got it. | 13:01 |
pstolowski | mhr3, Saviq fine by me, thanks! | 13:02 |
jhodapp | mhr3, is that when doing a song preview? | 13:03 |
mhr3 | jhodapp, it actually happened when i was closing everything | 13:03 |
jhodapp | mhr3, what do you mean? closing what exactly? | 13:03 |
mhr3 | jhodapp, preview with the audio components was long dead at that point | 13:03 |
tsdgeos | mhr3: have a look at https://code.launchpad.net/~aacid/unity8/alt_nav_support/+merge/229448 | 13:04 |
mhr3 | jhodapp, well, long in computer terms, a second... | 13:04 |
mhr3 | ish | 13:05 |
jhodapp | mhr3, right, but what were you closing? | 13:05 |
mhr3 | jhodapp, the entire scope | 13:05 |
Saviq | mhr3, about https://code.launchpad.net/~unity-team/unity-scopes-shell/drop-appid-workaround/+merge/228427 | 13:06 |
Saviq | mhr3, should we even have the if(application) still? | 13:06 |
jhodapp | mhr3, interesting...seems like things might be getting destroyed in the wrong order then...it was trying to get the playback status but perhaps the media-hub instance was already destroyed | 13:07 |
mhr3 | Saviq, i won't cry if we drop it :) | 13:07 |
mhr3 | jhodapp, sounds about right | 13:07 |
mhr3 | jhodapp, or maybe it just got the reply after it was destroyed | 13:08 |
jhodapp | mhr3, right exactly | 13:08 |
Saviq | mhr3, how about activateApplication then? | 13:08 |
jhodapp | mhr3, I know that ricmm and mandel had a lot of problems with gcc4.9 with crashers and such | 13:08 |
mhr3 | Saviq, i never needed it, was for you guys | 13:09 |
Saviq | mhr3, yeah, drop in v4? | 13:09 |
mhr3 | Saviq, fine with me | 13:09 |
tsdgeos | mhr3: https://code.launchpad.net/~aacid/unity8/alt_nav_support/+merge/229451 with the proper prerequisite | 13:10 |
Saviq | mhr3, will activateUri ever get a scope:// uri still? | 13:11 |
* Saviq will drop activateUri() altogether if follows that path | 13:11 | |
mhr3 | tsdgeos, file:///home/miso/projects/unity8/qml/ScopeTool.qml:75:9: Cannot assign to non-existent property "model" | 13:11 |
tsdgeos | ah right | 13:12 |
tsdgeos | need to update scopetool | 13:12 |
tsdgeos | mhr3: just remove that line i think | 13:12 |
tsdgeos | or give me a sec to fix it | 13:12 |
mhr3 | Saviq, yes | 13:13 |
mhr3 | Saviq, but activateUri isn't Q_INVOKABLE, is it? | 13:13 |
Saviq | mhr3, yeah, is not | 13:13 |
mhr3 | Saviq, so leave my methods alone :) | 13:14 |
tsdgeos | mhr3: fixed | 13:15 |
mhr3 | ty | 13:15 |
tsdgeos | wait n | 13:15 |
=== tedg` is now known as tedg | ||
tsdgeos | Could not acquire lock | 13:15 |
mhr3 | nothing to pull | 13:15 |
tsdgeos | who has the lock on my branch? | 13:15 |
tsdgeos | anyhow pushed now | 13:15 |
mhr3 | only you can :) | 13:15 |
tsdgeos | mhr3: ↑ | 13:15 |
tsdgeos | i think wrong timed ctrl+c did it | 13:16 |
mhr3 | tsdgeos, hm, i get no departments in the tool now | 13:17 |
mhr3 | tsdgeos, http://paste.ubuntu.com/7951864/ | 13:18 |
mhr3 | oh wait... maybe i don't have correct shell plugin installed :) | 13:18 |
tsdgeos | looks like it | 13:19 |
tsdgeos | mhr3: also for the "hidden" trick to work we may need the shell plugin to work like the ui code "needs" | 13:19 |
tsdgeos | or maybe not | 13:20 |
mhr3 | what does that mean? | 13:20 |
tsdgeos | and it's just my plugin code being crap :D | 13:20 |
tsdgeos | probably not | 13:20 |
tsdgeos | or maybe yes | 13:20 |
tsdgeos | don't know :D | 13:20 |
Saviq | mhr3, re-review https://code.launchpad.net/~unity-team/unity-scopes-shell/drop-appid-workaround/+merge/228427 please? | 13:21 |
tsdgeos | mhr3: basically you need to return the hidden "node" as currentId and once i query for it change the currentid to the selected one | 13:21 |
tsdgeos | mhr3: something like this http://paste.ubuntu.com/7951882/ | 13:21 |
Saviq | mhr3, I'll leave the handler be until we land v4 then | 13:23 |
mhr3 | Saviq, k | 13:24 |
mhr3 | tsdgeos, that is weird, can't we do something less weird? | 13:26 |
tsdgeos | mhr3: thing is the code was never designed to do jumps | 13:27 |
tsdgeos | so it doesn't know how it can be currently in a child without even being in the aprt | 13:27 |
tsdgeos | parent | 13:27 |
tsdgeos | so i need to get the parent first | 13:27 |
mhr3 | tsdgeos, btw i'm loosing the state when playing with both navs, not sure if it's the ui or the plugin... or the scope | 13:27 |
tsdgeos | mhr3: it's separate items in the ui | 13:28 |
tsdgeos | should not break there (i think) | 13:28 |
tsdgeos | mhr3: but if you think it may be because of the ui give me a way to reproduce the problem | 13:30 |
cwayne1 | what is alt-nav-support? | 13:36 |
Saviq | cwayne1, aka sort order | 13:36 |
cwayne1 | i.e. sort by price or rating? | 13:36 |
Saviq | cwayne1, i.e. whatever the scope sends | 13:38 |
Saviq | cwayne1, but yes, it's meant to be used for secondary navigation (next to departments) | 13:39 |
mhr3 | tsdgeos, looks like plugin issue | 13:39 |
cwayne1 | Saviq: and that's separate from filters right | 13:39 |
Saviq | cwayne1, yes | 13:40 |
tsdgeos | mhr3: oki | 13:40 |
=== Estilanda_ is now known as Estilanda | ||
mhr3 | tsdgeos, shoot... i f-ed up | 13:43 |
tsdgeos | mhr3: can be fixed? | 13:43 |
mhr3 | tsdgeos, my brilliant idea of dropping loadDepartment() and just using using performQuery, means that i'd need to mangle the queries in one nav everytime you switch state in the other | 13:44 |
tsdgeos | :D | 13:45 |
Saviq | mhr3, you mean in the cache you keep? eek | 13:45 |
mhr3 | fsck | 13:46 |
mhr3 | tsdgeos, will you kill me if i ask to revert that? | 13:49 |
tsdgeos | mhr3: so what do you want to do? | 13:50 |
mhr3 | tsdgeos, ie drop the *query props and role from the nav interface, and keep using loadDepartment() instead of performQuery() | 13:50 |
mhr3 | tsdgeos, or like setNavState(bool alt, string state_id); | 13:51 |
tsdgeos | mhr3: one loadDepartment or two loadDepartment? | 13:51 |
mhr3 | i'd still prefer a single method, but need to distinguish for which one is it, so has to have extra param at least | 13:52 |
tsdgeos | mhr3: if it has to be done, it has to be done | 13:53 |
tsdgeos | please ping me when you've updated the api so i can follow up | 13:53 |
mhr3 | tsdgeos, ok... sorry about that :/ | 13:53 |
tsdgeos | no worries, what has to be done has to be done | 13:58 |
elopio | Saviq: that was when I compiled the branch myself. I couldn't reproduce it from the silo. | 13:59 |
elopio | On the silo, my problem was showing the dash when an app was loading. Can you give that a try? | 13:59 |
Saviq | elopio, yeah, that we're onto | 13:59 |
Saviq | elopio, great, means that's our last issue, dandrader is on it | 13:59 |
elopio | Saviq: ok, thanks. | 13:59 |
Saviq | elopioI | 13:59 |
Saviq | I'm just reviewing your branch, few small comments coming up | 14:00 |
elopio | Saviq: great. After my meetings I'll take care of them. | 14:04 |
mhr3 | tsdgeos, pushed query drop | 14:04 |
mhr3 | Saviq, also activateApplication drop | 14:04 |
Saviq | mhr3, ktx | 14:05 |
tsdgeos | Saviq: mhr3: and what do we do with activateApplication dependant code in the shell? | 14:05 |
mhr3 | Saviq, oh yea, maybe we should have left it for later? | 14:05 |
Saviq | tsdgeos, we remove it in v4 | 14:06 |
Saviq | tsdgeos, there's only 4 lines | 14:06 |
tsdgeos | Saviq: no replacement? | 14:06 |
Saviq | tsdgeos, dash-as-app replaces it | 14:06 |
tsdgeos | ok | 14:06 |
Saviq | tsdgeos, by launching directly with Qt.openUrlExternally | 14:06 |
mhr3 | tsdgeos, but without dash-as-app it's no good | 14:06 |
Saviq | mhr3, dash-as-app lands before anything else in unity8 | 14:06 |
mhr3 | ok | 14:06 |
mhr3 | Saviq, and if it needs to be reverted? | 14:07 |
Saviq | mhr3, it doesn't :P | 14:07 |
* mhr3 remembers greeter :P | 14:07 | |
Saviq | mterry, mhr3 is here for you ↑ | 14:07 |
mterry | :) | 14:08 |
pstolowski | tsdgeos, hey, is there a silo with https://code.launchpad.net/~aacid/unity8/scopeActivatePreview/+merge/228624 by any chance? | 14:09 |
tsdgeos | Saviq: ↑ ¿ | 14:09 |
Saviq | pstolowski, not reviewed yet, no | 14:09 |
pstolowski | ack, thanks | 14:10 |
mzanetti | https://code.launchpad.net/~unity-team/unity-api/scopes-v4/+merge/228346/comments/556463 | 14:18 |
mzanetti | hehe.. when I proposed this the answer was: If changing we should add activate* for the others too :P | 14:18 |
=== pete-woods|lunch is now known as pete-woods | ||
Saviq | mzanetti, go away | 14:21 |
mzanetti | Saviq: in an hour | 14:22 |
Saviq | mzanetti, didn't get your sentence anyway :P | 14:22 |
mzanetti | Saviq: well. I proposed exactly that when I started with dash-as-app | 14:22 |
Saviq | mzanetti, should've made it happen then ;) | 14:23 |
mzanetti | and mhr3's opinion was that the others should have activate*() signals too instead of removing the one for apps | 14:23 |
Saviq | mzanetti, apparently he doesn't care any more for some reason ;) | 14:24 |
mzanetti | yeah... I guess so :( | 14:24 |
mhr3 | mzanetti, i mostly wanted single place that deals with it | 14:28 |
mhr3 | if that's url-dispatcher... so be it | 14:29 |
mzanetti | mhr3: yeah, same here :) | 14:29 |
mzanetti | mhr3: no worries... just had to grin a little when the mail notification showed up on my screen before | 14:29 |
mhr3 | but imo it would be cleaner if there was just a heyHandleThis(url) signal | 14:29 |
mzanetti | I'm not really here today anyways | 14:29 |
tsdgeos | mhr3: pushed the api changes | 14:34 |
MacSlow | mzanetti, aren't you on vacation? :) | 14:40 |
tedg | Saviq, Is there a reason the packages in the dash-as-app silo aren't built? Can we click the build button? | 14:40 |
mzanetti | MacSlow: yup | 14:41 |
Saviq | tedg, they are | 14:41 |
Saviq | tedg, it was a reconf just to drop qtdeclarative from it | 14:41 |
Saviq | tedg, I can kick a WATCH_ONLY if you want ;) | 14:41 |
tedg | Heh, okay. | 14:42 |
Saviq | done | 14:42 |
Saviq | since you're the 2nd person asking me that today ;) | 14:42 |
=== pete-woods is now known as pete-woods|at-do | ||
Saviq | elopio, yikes there's three of you nwo | 14:44 |
Saviq | now | 14:44 |
Saviq | elopio, reviewed dash as app ap | 14:44 |
* Saviq restarts xchat, got confused with elopio | 14:44 | |
elopio | I could use two more of myself :) | 14:44 |
Wellark | dednick: can we go forward with this one? AFAIK it will conflict with your indicator-polish branch as it ccontains the same fix separately: https://code.launchpad.net/~unity-api-team/unity8/default-indicator-page-loader-visible-fix/+merge/228943 | 14:47 |
Wellark | how close are you to land your branch? | 14:47 |
Saviq | Wellark, check the status on that MP again ;) | 14:49 |
elopio | Saviq: I'll make the corrections, thanks for the detailed review. Any suggestions about the is_job_running function? | 14:49 |
elopio | I could just try to stop the dash without checking if it was running. | 14:50 |
Wellark | Saviq: :D | 14:51 |
Wellark | right | 14:51 |
Wellark | Saviq, dednick: thanks! | 14:51 |
Wellark | Estilanda: could you approve the design changes in here: https://code.launchpad.net/~unity-api-team/unity8/modeminfo/+merge/225159 | 14:53 |
Saviq | elopio, I think you need to check for just start/ instead | 14:53 |
Saviq | elopio, if you just stop, and it's not started, you'll get a bad exit code | 14:53 |
=== dandrader_ is now known as dandrader|lunch | ||
tsdgeos | mterry: ping | 14:57 |
mterry | tsdgeos, hello! | 14:58 |
tsdgeos | mterry: i can swipe the greeter to the left in the first screen | 14:58 |
tsdgeos | known bug? | 14:58 |
tsdgeos | or shall i file one? | 14:58 |
mterry | tsdgeos, uh, known behavior. I haven't asked design whether it's a bug or not | 14:58 |
mterry | tsdgeos, the swipe left behavior was added after the edge demo. I'm not sure it's awful to allow it? | 14:59 |
mterry | But design might think differently,sure. Worth filing a bug against ubuntu-ux, assigning to esti maybe, and confirming with them | 14:59 |
tsdgeos | it feels quite confusing | 15:00 |
tsdgeos | that you're telling me how to swipe from right | 15:01 |
tsdgeos | and i can also swipe from left | 15:01 |
tsdgeos | none of the other screens let me do taht | 15:01 |
mhr3 | tsdgeos, pushed the plugin, things seems to work fine now | 15:12 |
tsdgeos | goodie | 15:13 |
tsdgeos | Saviq: https://code.launchpad.net/~cimi/unity8/card_emblems_single_container/+merge/223899 approved or rejected? | 15:14 |
tsdgeos | ah rejected | 15:14 |
* tsdgeos can't read | 15:14 | |
tsdgeos | mhr3: did you try the hidden stuff? | 15:16 |
mhr3 | tsdgeos, kindof... i didn't notice it not working :) | 15:16 |
tsdgeos | ok :D | 15:16 |
Saviq | @unity when reviewing, please make sure to click "claim review" at the top so that you don't clash | 15:17 |
* tsdgeos does that when really reviewing | 15:17 | |
Saviq | yup, good | 15:17 |
tsdgeos | the rest i have comments on are either "needs fixing" because of merges or "abstain" because of fixed meregs | 15:17 |
=== gatox is now known as gatox_lunch | ||
=== pete-woods|at-do is now known as pete-woods | ||
Cimi | Saviq, best way to put settings is like PreviewListView on right of the scope? | 15:43 |
Cimi | or we have other ideas? | 15:43 |
Saviq | Cimi, probably, yeah, just a page on top of the scope | 15:44 |
Cimi | Saviq, fading in or sliding like previews? | 15:44 |
Saviq | Cimi, sliding, same | 15:44 |
Cimi | ok | 15:44 |
Saviq | Cimi, anyway, who am I to say ;) | 15:44 |
Saviq | Cimi, you've got designers closer than I do | 15:44 |
Cimi | Saviq, yeah, just wondering if you had chat on it | 15:45 |
Saviq | Cimi, truth is we really need to make all these transitions consistent, using a PageStack or so, they're all over the place now | 15:45 |
Saviq | but well, SDK doesn't support animated pagestacks :| | 15:45 |
Saviq | yet | 15:45 |
mhr3 | Saviq, think we'll need a Q_INVOKABLE void forceSearch() on the Scope object | 15:51 |
Saviq | mhr3, you mean a refresh()? | 15:51 |
mhr3 | Saviq, yea | 15:51 |
Saviq | mhr3, yeah, thought so | 15:51 |
mhr3 | Saviq, for those refresh button | 15:51 |
mhr3 | s | 15:51 |
Saviq | +2 | 15:51 |
Saviq | (because we have two of those) :P | 15:51 |
mhr3 | Saviq, adding to v4 :) | 15:51 |
Saviq | yup | 15:52 |
mhr3 | Saviq, slot or invokable? | 15:52 |
Saviq | mhr3, same thing, really | 15:52 |
mhr3 | can slot be virtual actually? | 15:52 |
mhr3 | never tried that | 15:52 |
Saviq | mhr3, yeah it can | 15:52 |
=== gatox_lunch is now known as gatox | ||
Cimi | tsdgeos, is this edge demo or welcome wizard? https://bugs.launchpad.net/unity8/+bug/1352408 | 16:09 |
ubot5 | Launchpad bug 1352408 in Unity 8 "Welcome wizard lets you swipe from the left in the "Swipe from the right" screen" [Undecided,New] | 16:09 |
Saviq | Cimi, edge demo indeed | 16:10 |
Cimi | ok | 16:10 |
tsdgeos | Cimi: right sorry | 16:11 |
dednick | Saviq: how do i get ubuntu-app-launch to recognise an arbitrary process as an application? | 16:14 |
Saviq | dednick, you don't | 16:14 |
dednick | Saviq: say that i want to. | 16:15 |
dednick | :) | 16:15 |
Saviq | dednick, if you want for qtmir to recognize it as an app, pass --desktop_file_hint=/path/to/desktop/file | 16:15 |
Saviq | but that's something that happens in qtmir, not in ual | 16:15 |
Saviq | dednick, there's no way to inject processes into ual | 16:15 |
dednick | Saviq: what i mean is, how does ual know about apps? | 16:15 |
dednick | app-ids | 16:15 |
Saviq | dednick, it starts them | 16:15 |
dednick | Saviq: right. so an upstart config? | 16:16 |
Saviq | dednick, there's a job or two, application-click, application-legacy | 16:16 |
Saviq | dednick, that UAL passes some args to | 16:16 |
Saviq | dednick, dpkg -L ubuntu-app-launch | 16:16 |
dednick | Saviq: thanks | 16:17 |
Saviq | o/ | 16:19 |
=== boiko_ is now known as boiko | ||
Cimi | dednick, mhr3 tomorrow lunch? | 16:31 |
dednick | honest! | 16:31 |
mhr3 | burgers! | 16:31 |
dednick | Cimi: i think that's a yes | 16:31 |
Cimi | hah | 16:32 |
=== dandrader|lunch is now known as dandrader | ||
elopio | Saviq: I pushed changes for all your comments. | 16:55 |
Saviq | elopio, thanks | 17:04 |
=== alan_g is now known as alan_g|EOD | ||
kgunn | elopio: what's the easiest means to repro the freeze bug on dash-as-app silo ? | 20:13 |
kgunn | i'm assuming ap test ? | 20:13 |
kgunn | i can't get it to happen manually | 20:14 |
elopio | kgunn: I could reproduce it manually a lot easier than with autopilot. | 20:24 |
elopio | but with autopilot, run the test: | 20:24 |
elopio | unity8.application_lifecycle.tests.test_url_dispatcher.URLDispatcherTestCase.test_swipe_out_application_started_by_url_dispatcher | 20:24 |
kgunn | elopio: its basically right edge app switcher right ? | 20:26 |
elopio | kgunn: no, left edge to show the dash. | 20:52 |
kgunn | elopio: hehe, that might explain it :)....lemme play some more, pretty sure i've been doing that as well | 20:53 |
elopio | kgunn: Saviq said dandrader was already fixing it. | 20:55 |
kgunn | elopio: yeah, but hasn't....i'm testing lastest proposed with silo7 | 20:58 |
kgunn | i'll run the ap test in a moment...but i can't get this to do it manually | 20:58 |
kgunn | elopio: just to make sure that's on n4 right ? | 20:59 |
kgunn | that you're seeing this issue ? | 20:59 |
elopio | kgunn: yes, nexus 4. | 21:01 |
elopio | I haven't tried since the silo was ready, but I guess not much has changed. | 21:01 |
elopio | launch an app, and while it's on the black screen loading, swipe from the left to show the dash. | 21:01 |
Saviq | yeah I've been able to reproduce reliably, and that is what the one failing test triggers as well | 21:03 |
mhall119 | Saviq: mhr3: does searching in a scope limit the results based on a selected category? or is it up to the scope author to do that? | 21:03 |
Saviq | mhall119, scope author | 21:03 |
mhall119 | ok | 21:03 |
mhr3 | mhall119, the client can impose limits | 21:03 |
mhr3 | but i don't think it does atm | 21:04 |
mhr3 | Saviq, oh, did you try to call me czech number last week? | 21:04 |
mhr3 | s/me/my/\ | 21:04 |
Saviq | mhr3, yeah I did ;) | 21:04 |
mhr3 | i guess i noticed a bit too late | 21:04 |
mhr3 | ie 15 minutes ago :P | 21:05 |
Saviq | mhr3, nice latency ; | 21:05 |
Saviq | ;) | 21:05 |
mhr3 | Saviq, that's the better latency, before this i checked it about 3months ago :P | 21:05 |
Saviq | mhr3, ROFL just saw your directory photo, you have a Polish beer^Wyellowish fluid's brand umbrella behind you ;D | 21:06 |
mhr3 | surprise, surprise, it was taken in poland :) | 21:07 |
Saviq | yeah, the "stacyjka" name behind you kind of gave it away ;) | 21:07 |
mhr3 | Lublin ;) | 21:07 |
kgunn | elopio: sorry, interenet probs...do you have the bug handy for dash as app silo ? | 21:38 |
elopio | kgunn: I haven't reported a bug for it, just mentioned it on the mail thread. Should I file one for you? | 21:39 |
kgunn | elopio: it might be helpful to track things...b/c i can not for the life of me repro...i need to double check all the packages etc | 21:40 |
elopio | kgunn: https://bugs.launchpad.net/unity8/+bug/1352571 | 21:59 |
ubot5 | Launchpad bug 1352571 in Unity 8 "With dash-as-app, showing the dash while an app is loading freezes" [Undecided,Confirmed] | 21:59 |
elopio | kgunn: please assign it to Daniel. I can't do that. | 21:59 |
kgunn | yep thanks elopio | 22:01 |
=== salem_ is now known as _salem |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!