| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| === shiznix_ is now known as shiznix | ||
| darklight_ | Any chance of seeing this fixed https://bugs.launchpad.net/ubuntu/+source/compiz/+bug/1305438 ? which is also tied to https://answers.launchpad.net/ubuntu/+question/243724 | 07:09 |
|---|---|---|
| ubot5 | Ubuntu bug 1305438 in compiz (Ubuntu) "hot corners stop working time to time" [Undecided,Confirmed] | 07:09 |
| Saviq | darklight_, it's probably best if you post it on the ML | 07:13 |
| darklight_ | Saviq, this one unity-design@lists.launchpad.net ? | 07:18 |
| Saviq | darklight_, no, ubuntu-devel would be a better choice, I also added a "unity" task to the bug, as this seems to be caused by the unity plugin, not by compiz itself | 07:18 |
| Saviq | according to the comments | 07:18 |
| Saviq | MacSlow, you still had old tags in your branch, please make sure to strip all your local checkouts and any outstanding ones you have in LP | 07:21 |
| Saviq | paulliu, same for you ↑ | 07:21 |
| Saviq | http://people.canonical.com/~msawicz/unity8/strip-u8-tags.sh for the rescue | 07:21 |
| MacSlow | Saviq, *sigh* | 07:21 |
| Saviq | MacSlow, remember you have to strip them on both sides - remote and local separately, as tag deletion doesn't carry over "bzr pull/push", as opposed to any new ones, which will happily get pushed all over the place... | 07:22 |
| MacSlow | Saviq, I know... I did all local and remote branches... will receck again | 07:23 |
| Saviq | MacSlow, I'm scanning the remote ones now | 07:23 |
| MacSlow | Saviq, ok | 07:23 |
| Saviq | MacSlow, but you don't have any | 07:24 |
| didrocks | Saviq: oh, you've done a script :p | 07:24 |
| Saviq | didrocks, yeah, we're *almost* there | 07:24 |
| MacSlow | Saviq, I'm rechecking local ones | 07:24 |
| didrocks | Saviq: well done man! that wasn't easy :) | 07:24 |
| Saviq | didrocks, yeah, on that note | 07:24 |
| Saviq | didrocks, any idea about "7.85+14.10.20140428.2-0ubuntu1 ?" | 07:24 |
| Saviq | where did that tag come from? | 07:24 |
| didrocks | you mean, it was never released? | 07:25 |
| Saviq | didrocks, it doesn't point to any revision | 07:25 |
| Saviq | the ? | 07:25 |
| Saviq | it looks like it got tagged by the train but somehow didn't actually get released | 07:26 |
| didrocks | Saviq: I wonder if we didn't get a rerelease after pushing to -proposed | 07:26 |
| didrocks | weird, doesn't seem to be the case though… | 07:27 |
| Saviq | didrocks, branches are scrapped on each Build, though aren't they? | 07:27 |
| didrocks | Saviq: yeah, they are | 07:27 |
| Saviq | didrocks, yeah, so it's something weird | 07:27 |
| Saviq | didrocks, anyway, looks like a one-off error, if I notice it again will let you know | 07:27 |
| didrocks | Saviq: please do, I'm quite puzzled | 07:28 |
| paulliu | Saviq: ok | 07:29 |
| Saviq | paulliu, you definitely have them in lp:~paulliu/unity8/logout | 07:29 |
| paulliu | Saviq: yes. | 07:29 |
| tsdgeos | waht, compiz is at 250% of cpu :D | 07:33 |
| tsdgeos | i shall reboot | 07:33 |
| Saviq | mzanetti, lp:~unity-team/unity8/fix-lvwph-xvfb is obsolete isn't it? | 07:38 |
| Saviq | dednick, lp:~nick-dedekind/unity8/indicators.hint-interval: 366 tags | 07:39 |
| Saviq | dednick, please strip | 07:39 |
| Saviq | Wellark, please use http://people.canonical.com/~msawicz/unity8/strip-u8-tags.sh on lp:~kaijanmaki/unity8/indicator-root-state-icons-fix and on any local checkout | 07:39 |
| MacSlow | Saviq, on lp:~macslow/unity8/sound-hint-support I've had these tags http://pastebin.ubuntu.com/7403087 still, which probably came from trunk (branched off yesterday) | 07:44 |
| Saviq | MacSlow, those are correct | 07:44 |
| MacSlow | ah ok | 07:44 |
| Saviq | MacSlow, we only don't like the 300 tags that came from lp:unity via our old lp:unity/8.0 | 07:45 |
| Saviq | MacSlow, all of them will have ? in `bzr tags` | 07:45 |
| Saviq | there's actually one like that in trunk, but that one's current, and we don't care enough | 07:45 |
| tsdgeos | wtf bitcoin spam in queries :S | 07:49 |
| tsdgeos | karni: that bug you oepened, can you give the templates, components, cardData that is used for ot? | 07:50 |
| tsdgeos | it? | 07:50 |
| mzanetti | Saviq: right, yes, it is. | 07:54 |
| Saviq | MacSlow, bug #1315419 is not ours, it's the service that decides to stop waiting for the password, isn't it? | 08:09 |
| ubot5 | bug 1315419 in unity8 (Ubuntu) "Timeout when trying to enter a long passphrase" [Undecided,Confirmed] https://launchpad.net/bugs/1315419 | 08:09 |
| MacSlow | Saviq, no there's a timeout on every notification, which bites there... so it's currently ours | 08:09 |
| Saviq | MacSlow, ok | 08:09 |
| Saviq | MacSlow, please comment on the bug, I added a unity-notifications task | 08:10 |
| Saviq | biab | 08:20 |
| mhr3 | dpm, ping? could we have the scopes docs updated? | 08:43 |
| mhr3 | dpm, just grab the latest snapshot from utopic | 08:43 |
| dednick | Saviq: done | 08:49 |
| karni | tsdgeos: you mean the "image should match width" in preview bug? | 08:57 |
| tsdgeos | karni: yep | 08:58 |
| tsdgeos | maybe you did | 08:58 |
| tsdgeos | for some reason i seem to get only the first email of those bugs | 08:58 |
| tsdgeos | and not the followups | 08:58 |
| karni | tsdgeos: I can provide some data. And I believe template has nothing to do with the preview. I'll update the bug. | 08:59 |
| tsdgeos | ah it's preview? | 08:59 |
| * tsdgeos can't read | 08:59 | |
| karni | "scope result preview", yes | 08:59 |
| karni | its not a card | 08:59 |
| tsdgeos | sorry :/ | 09:00 |
| karni | tsdgeos: no need :) | 09:00 |
| dpm | mhr3, I'll get in touch with mhall119 to update the API site, he's taking care of that now | 09:05 |
| mhr3 | dpm, k, thx | 09:05 |
| dpm | mhr3, np, sent you an e-mail to you both | 09:07 |
| tsdgeos | hmmm | 09:12 |
| tsdgeos | the .sh suffixes for build and run are unfortunate | 09:12 |
| tsdgeos | i need to type more now! | 09:12 |
| mhr3 | tsdgeos, isn't it still ./r<Tab> ? :) | 09:17 |
| tsdgeos | mhr3: nope | 09:17 |
| tsdgeos | because there's builddir and run_on_device | 09:17 |
| mhr3 | ah.. sucks indeed | 09:18 |
| mhr3 | two more keypresses | 09:18 |
| tsdgeos | my productivity suddenly plumetted :D | 09:20 |
| tsdgeos | thanks we have ↑ so i can just still use that | 09:22 |
| Makay | hi | 09:29 |
| Makay | I wrote a short article about my "Unity redesign" idea. This is a small but very important correction in the current layout. My article is hungarian but i hope you will understand the images. http://androbit.net/articles/428/new_ubuntu_desktop_layout.html | 09:35 |
| seb128 | Makay, hey, you should open specific bugs in launchpad | 09:37 |
| seb128 | but things like the controls being in the corner is done for a reason | 09:37 |
| seb128 | it's easier to access a corner than to access 15 pixels right of a corner | 09:38 |
| seb128 | in one case you can throw the mouse in the direction, in the other one you need to aim | 09:38 |
| mzanetti | Makay: also, if your last image would contain the same amount of indicators than the first, I'm not sure if it would still look that good | 09:40 |
| === vrruiz_ is now known as rvr | ||
| Makay | mzanetti: yes, this is a problem but the current system tray (with panel) is ugly too | 09:48 |
| Cimi | tsdgeos, Saviq I just branched unity, I am running builddir/tools/unity-scope-tool, after having started scope-ui-starting | 09:57 |
| Cimi | I only have "my apps", and rest seems unresponsive | 09:57 |
| Cimi | no scopes | 09:57 |
| Cimi | file:///home/cimi/Development/unity8/carousel-shadow/qml/ScopeTool.qml:149:23: Unable to assign null to QString | 09:58 |
| Cimi | file:///home/cimi/Development/unity8/carousel-shadow/qml/Dash/DashContent.qml:101: TypeError: Cannot read property 'loaded' of null | 09:58 |
| Saviq | Cimi, "having started scope-ui-starting"? | 10:01 |
| Cimi | sure | 10:01 |
| Saviq | Cimi, you mean initctl emit? | 10:01 |
| Cimi | yes | 10:01 |
| mhr3 | Cimi, unresponsive how? | 10:05 |
| Cimi | mhr3, that error | 10:06 |
| mhr3 | Cimi, you see "Music" but nothing there? | 10:06 |
| Cimi | so no scopes | 10:06 |
| Cimi | yes | 10:06 |
| Cimi | looks like a service is not working | 10:06 |
| mhr3 | Cimi, do `restart scope-registry` | 10:06 |
| mhr3 | and run the tool then | 10:06 |
| Cimi | nope | 10:07 |
| Cimi | I have file:///home/cimi/Development/unity8/carousel-shadow/qml/Dash/DashContent.qml:101: TypeError: Cannot read property 'loaded' of null | 10:07 |
| Cimi | every time I switch | 10:07 |
| mhr3 | Cimi, hm, pastebin ~/.cache/upstart/scope-registry.log | 10:09 |
| Cimi | http://paste.ubuntu.com/7403713/ | 10:09 |
| Cimi | the death is when you asked me to restart | 10:10 |
| mhr3 | yea, looks fine | 10:10 |
| mhr3 | Cimi, try this `stop scope-registry && stop smart-scopes-proxy && start smart-scopes-proxy` | 10:11 |
| Cimi | now? | 10:12 |
| Cimi | start all of them? | 10:12 |
| mhr3 | Cimi, just run the tool | 10:13 |
| mhr3 | the last start should start everything up | 10:14 |
| Cimi | no | 10:14 |
| mhr3 | Cimi, so what do you see in the tool now? | 10:14 |
| Cimi | same error | 10:15 |
| Cimi | same stuff | 10:15 |
| mhr3 | Cimi, are you on t or u? | 10:15 |
| Cimi | u | 10:15 |
| mhr3 | Cimi, latest everything? | 10:15 |
| Cimi | sure | 10:16 |
| mhr3 | damn | 10:16 |
| Cimi | correct | 10:16 |
| Cimi | :) | 10:16 |
| mhr3 | i'm running out of ideas | 10:16 |
| mhr3 | Cimi, are you sure your changes didn't break something? ;) | 10:18 |
| Cimi | mhr3, if I run unity8 works | 10:20 |
| Cimi | mhr3, is the scope tool that doesn't | 10:20 |
| Cimi | I'll run whole unity... but neh | 10:20 |
| mhr3 | i'm having issues with the header, but i guess that's cause i'm on T | 10:22 |
| mhr3 | hmm, and ./run actually doesn't work | 10:22 |
| mhr3 | yey me | 10:22 |
| Cimi | mhr3, oh sorry | 10:25 |
| Cimi | I am on t not u | 10:25 |
| Cimi | those stupid letters :) | 10:26 |
| Cimi | 14.04 | 10:26 |
| tsdgeos | mhr3: there's no run anymore :D | 10:26 |
| tsdgeos | it's called ./run.sh | 10:26 |
| mhr3 | tsdgeos, yea, that | 10:26 |
| tsdgeos | mhr3: define "doesn't work" | 10:26 |
| mhr3 | tsdgeos, hangs | 10:27 |
| mhr3 | i see it uses upstart now | 10:27 |
| tsdgeos | it does for a while yes | 10:27 |
| mhr3 | i suppose it doesn't like something in there | 10:27 |
| tsdgeos | mhr3: make sure that if you have a "local" unity8.conf you update it with the contents of the file in data/ | 10:28 |
| === dandrader is now known as dandrader|afk | ||
| mhr3 | tsdgeos, where do i need to install a local one? | 10:28 |
| tsdgeos | mhr3: you don't need to install it, i'm saying if you have a local one | 10:28 |
| mhr3 | i dont | 10:29 |
| tsdgeos | are you sure? | 10:29 |
| mhr3 | but i think it might work if i did :) | 10:29 |
| tsdgeos | it seems to me you're using an old one and not the new one that doesn't wait on indicators starting | 10:29 |
| tsdgeos | but may be something else | 10:29 |
| tsdgeos | anyway, you should be on U | 10:29 |
| mhr3 | tsdgeos, wasn't the --no-wait thing an sru? | 10:30 |
| tsdgeos | mhr3: not sure, maybe :D | 10:30 |
| mhr3 | but yea, it's not there | 10:30 |
| mhr3 | i still find it weird that the scopes-ui emit works without no-wait and the indicator one doesn't | 10:31 |
| mhr3 | what's the deal there? | 10:31 |
| tsdgeos | there's some indicator that hangs | 10:36 |
| tsdgeos | that's my understanding | 10:37 |
| Saviq | mhr3, yeah, it's something that reacts to indicator-services-start that doesn't return | 10:50 |
| Saviq | mhr3, also, we probably want to wait for the registry at least to start before starting the UI, something we don't care for indicators | 10:50 |
| Saviq | otherwise we might miss the 300ms mark | 10:50 |
| mhr3 | Saviq, there's another timeout in the plugin | 10:51 |
| mhr3 | could just make it smarter | 10:51 |
| Saviq | mhr3, sure, if you make it smarter to retry, we'll be able to --no wait indeed ;) | 10:52 |
| === dandrader|afk is now known as dandrader | ||
| Saviq | or maybe even react to scope-registry started | 10:53 |
| mhr3 | Saviq, are we delaying the startup now or something? | 10:53 |
| Saviq | mhr3, we're only starting unity8 after emit scope-ui-starting returns | 10:53 |
| Saviq | mhr3, which means that everything listening to that event completes startup | 10:54 |
| Saviq | mhr3, that'd be only smart-scope-proxy atm AFAICT | 10:54 |
| mhr3 | Saviq, right, but is there actual data? that we're making the startup slower by 500ms or something? | 10:55 |
| Saviq | mhr3, you'd have to ask ogra | 10:55 |
| mhr3 | otherwise, wishlist :) | 10:55 |
| mhr3 | there were some talks to merge the two registries into one | 10:56 |
| mhr3 | would make things better | 10:56 |
| mhr3 | marcustomlinson, ^ right? ;) | 10:56 |
| marcustomlinson | mhr3: right | 10:56 |
| Saviq | mhr3, I never even requested it, I just explained why we do what we do, and you said that you could make it smarter ;) | 11:05 |
| tsdgeos | dednick: ping | 11:05 |
| dednick | tsdgeos: plop | 11:05 |
| tsdgeos | dednick: having a look at https://code.launchpad.net/~nick-dedekind/unity8/MenuContentActivator.bindingLoop/+merge/217899 | 11:05 |
| tsdgeos | dednick: can't find anywhere the contentChanged signal is used :-S | 11:06 |
| tsdgeos | ah it's probably part of an assignment | 11:06 |
| dednick | menuActivator.content[index].active | 11:07 |
| mhr3 | Saviq, right :) i could make the timing out smarter, but there are more correct fixes... that would take longer though | 11:07 |
| dednick | tsdgeos: yeah. it's one of those QMLListProperty thingos | 11:07 |
| tsdgeos | dednick: can i copy your comment into the commit message? | 11:10 |
| tsdgeos | makes things much easier if someone reads the commit log in the future | 11:11 |
| dednick | tsdgeos: sure | 11:11 |
| dednick | tsdgeos: done | 11:12 |
| tsdgeos | ah no | 11:13 |
| tsdgeos | i didn't mean that one | 11:13 |
| tsdgeos | i meant the other one | 11:13 |
| tsdgeos | :D | 11:13 |
| tsdgeos | "If we're already asking for the index, we know it exists already. No need to send a changeNotification on an implied creation." | 11:13 |
| tsdgeos | i'll add both | 11:13 |
| tsdgeos | :D | 11:13 |
| dednick | tsdgeos: :) | 11:13 |
| === MacSlow is now known as MacSlow|lunch | ||
| MacSlow|lunch | Saviq, I can't mark LP #1315419 as a duplicate of LP #1295762 because of timeout-error on LP | 11:32 |
| ubot5 | Launchpad bug 1315419 in unity8 (Ubuntu) "Timeout when trying to enter a long passphrase" [Undecided,Confirmed] https://launchpad.net/bugs/1315419 | 11:32 |
| ubot5 | Launchpad bug 1295762 in unity-notifications "snap decision timeout needs to be determined by the requesting app" [Undecided,Triaged] https://launchpad.net/bugs/1295762 | 11:32 |
| MacSlow|lunch | Saviq, could you try? | 11:33 |
| Saviq | heh | 11:33 |
| MacSlow|lunch | no pun intended :) | 11:33 |
| Saviq | MacSlow|lunch, done, took three tries for me, too | 11:33 |
| MacSlow|lunch | gave up after five | 11:34 |
| karni | Hey guys, I'm looking at d.u.com cplusplus unity-scopes documentation - where can I find available PreviewWidget types? | 11:39 |
| karni | http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/unity-scopes/classunity_1_1scopes_1_1_preview_widget.html | 11:39 |
| Saviq | MacSlow|lunch, huh, apparently you managed to dupe it in the end, just got an email ;) | 11:40 |
| Saviq | karni, http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/unity-scopes/previewwidgets.html | 11:40 |
| Saviq | karni, top level "Preview widget types" ;) | 11:40 |
| karni | Saviq: lovely! | 11:40 |
| karni | Saviq: yeah.. *me hides* | 11:41 |
| karni | Saviq: What's the difference between pushing widgets one by one VS adding them to PreviewWidgetList and pushing that? | 11:41 |
| greyback | Saviq: can I ask for a volunteer to review: https://code.launchpad.net/~gerboland/unity-mir/shutdown-crash-fix3/+merge/218394 | 11:41 |
| Saviq | karni, mhr3 can confirm, but they're sent to the UI as soon as you send them, so if you push them one by one they should get to the UI slightly faster | 11:42 |
| karni | Saviq: that'd make sense. wondering if there's any more magic. thanks! | 11:43 |
| Saviq | karni, you can also make a conscious decision, knowing that you can push a base preview straight away, and then request additional data from any service you need to ask and supplement the preview | 11:43 |
| mhr3 | karni, sorry, irc issues, what's the problem? | 11:43 |
| Saviq | mhr3, no problem | 11:43 |
| karni | mhr3: no problem | 11:43 |
| Saviq | <karni> What's the difference between pushing widgets one by one VS adding them to PreviewWidgetList and pushing that? | 11:43 |
| mhr3 | it's faster :) | 11:43 |
| mhr3 | flicker-free | 11:44 |
| mhr3 | or well.. before Saviq kicks me... it *can* be flicker-free | 11:44 |
| karni | :) | 11:46 |
| davmor2 | mhr3: Flicker-free is like matrix spoons and internet cake it doesn't exist ;) | 11:47 |
| Saviq | tsdgeos, tests/plugins/Dash/cardcreatortest.cpp: bad whitespace in line 47 | 11:47 |
| tsdgeos | meh | 11:47 |
| Saviq | tsdgeos, how are you getting that with QtC? | 11:47 |
| tsdgeos | i'm not using QtC | 11:47 |
| Saviq | I can't get my QtC to mess up whitespace ;D | 11:47 |
| Saviq | tsdgeos, that explains it :D | 11:48 |
| mhr3 | karni, for clarity, adding all the widgets to a list and pushing that is flicker-free option | 11:48 |
| karni | davmor2: haha | 11:48 |
| karni | mhr3: ack | 11:48 |
| Saviq | mhr3, but slower, as it's sent in one go, right? so if you're requesting data from a remote API, it makes sense to send them one by one after having sent a base one for display? | 11:49 |
| karni | Saviq: yes | 11:49 |
| mhr3 | Saviq, sure, if it'd take you too long to populate them all, it's better to chunk it, but if you don't have such a problem, single push is better | 11:50 |
| Saviq | mhr3, k | 11:50 |
| mhr3 | Saviq, now, i need renderer for annotations | 11:51 |
| Saviq | mhr3, since when do we have annotations designed? :P | 11:52 |
| mhr3 | Saviq, since forever :P | 11:52 |
| Saviq | mhr3, except they've been rejected by a certain gentleman | 11:52 |
| mhr3 | alright, i'll ask mike about it | 11:53 |
| Saviq | mhr3, in CPT Mark said that he wanted them to be rich and context-dependent | 11:54 |
| dednick | tsdgeos: :) | 11:54 |
| Saviq | mhr3, not just plain links | 11:54 |
| dednick | tsdgeos: bleh. sorry | 11:54 |
| tsdgeos | dednick: don't be sorry for being happy :D | 11:54 |
| mhr3 | Saviq, i thought he meant that both should be possible | 11:54 |
| === _salem is now known as salem_ | ||
| karni | mhr3: I think Unity scopes api doc is missing instruction of how to add a preview action with *custom* uri | 12:01 |
| karni | as long as result uri is same as the action uri, it's easy, because you don't need to provide it | 12:01 |
| karni | optional {"uri", Variant("uri")} doesn't resolve to mapped uri value, and {"uri", "uri"} doesn't compile. I suppose VariantBuilder is not the way to go if we want custom action uri? | 12:03 |
| === alan_g is now known as alan_g|lunch | ||
| mhr3 | karni, what do you mean, "doesn't resolve to mapped uri"? | 12:04 |
| Cimi | Saviq, tsdgeos how do I export GRID_UNIT_PX for the new run script? | 12:04 |
| Saviq | Cimi, initctl set-env --global GRID_UNIT_PX=blah | 12:05 |
| karni | mhr3: if there's "uri" field in the result, that Variant("uri") action button resolves to <scope path>/uri instead of _real uri from the result_ | 12:05 |
| Cimi | Saviq, works thx | 12:05 |
| karni | mhr3: it does not properly map, preview just takes hardcoded "uri", not *the* uri from the result | 12:05 |
| karni | mhr3: suppose, in the result I have "more_from_category_uri" and I want an action button that launches that uri | 12:06 |
| karni | mhr3: {"uri", Variant("more_from_category_uri")} will result in this action: <scope path>/more_from_category_uri instead, say, http://foo.shop.com/category/2 | 12:07 |
| mhr3 | karni, so just do {"uri", Variant("http://foo...")} ? | 12:07 |
| karni | mhr3: I thought these things should resolve. that's why we do header.add_attribute_mapping("text", "title"); and not ..._mapping("text", "THE title"); | 12:08 |
| mhr3 | but you can only map whole fields, not subfields | 12:09 |
| mhr3 | you'd need to create an actions-like field in the result | 12:09 |
| mhr3 | if you wanted to map that | 12:09 |
| * karni stares at what mhr3 is writing ;) | 12:10 | |
| dednick | tsdgeos: bleh. sorry | 12:10 |
| karni | mhr3: you want something like {"uri", Variant(result["custom_uri"])} ? | 12:10 |
| dednick | baaaa! | 12:10 |
| mhr3 | karni, no, when creating the result, you'd have to do: | 12:10 |
| karni | mhr3: can I even access the result? | 12:10 |
| mhr3 | karni, VariantBuilder vb; vb.add_tuple({"uri", "http://..."}); result['actions'] = vb.end(); | 12:11 |
| mhr3 | karni, and then in preview w.add_attribute_mapping("actions", "actions"); | 12:11 |
| mhr3 | should have named it foo in result, then _mapping("actions", "foo") | 12:11 |
| karni | mhr3: I see. That sort of imposes Preview format on the result set. | 12:12 |
| mhr3 | karni, right | 12:12 |
| mhr3 | karni, that's the price to pay if you want to map it | 12:12 |
| Saviq | greyback, sorry, I looked at what you wrote and thought "sure", broken xchat-gnome didn't send my thoughts to you ;) | 12:12 |
| greyback | Saviq: np | 12:13 |
| karni | mhr3: and if I don't want to map it, is there another way? you said {"uri", Variant("http://foo...")} - how do I achieve that? can I access the result from within the preview? | 12:13 |
| mhr3 | karni, you need to do it manually, but yea | 12:13 |
| karni | mhr3: I'm sorry I'm asking so many questions, but we're paving the dev story | 12:13 |
| mhr3 | karni, ie, copy the result, in preview() | 12:13 |
| karni | mhr3: so 'result' is available in the scope of Preview::run? | 12:14 |
| karni | in the Preview, in general | 12:14 |
| mhr3 | karni, not by default, you need to do it manually ^ | 12:14 |
| Saviq | greyback, while working on this, did you notice unity8 taking a long time (like ½ a minute) to shut down after "initctl stop unity8"? | 12:15 |
| Saviq | I would expect the same on Ctrl+C actually | 12:15 |
| mhr3 | karni, but yea, would be nice to support it out-of-box | 12:15 |
| mhr3 | pstolowski, could we ^? | 12:15 |
| karni | mhr3: ok. that's way more dirty than enforcing preview format on the result. I'll go that route. | 12:15 |
| karni | mhr3: thank you | 12:16 |
| mhr3 | karni, well, one has to go over the wire, one is just a mem copy... so... | 12:16 |
| greyback | Saviq: I notice that from time to time yes, but I didn't think it was unity8 that was blocking | 12:16 |
| Saviq | greyback, it is, it's just waiting for *something* on shutdown (probably the join()) | 12:17 |
| greyback | Saviq: with that patch, worst case shutdown time is around 5 seconds (hit Ctrl+C just when Qt is starting up) | 12:17 |
| Saviq | greyback, ok sounds good | 12:17 |
| karni | mhr3: by dirty I mean the need to change scope-preview header and source file in a way that should not be needed. the less it is required to successfully write the scope, the better. so, I'll just reformat the result and map it as "actions", "actions" with add_attribute_mapping | 12:17 |
| Saviq | greyback, no checklist in the MP? | 12:18 |
| greyback | Saviq: not merging to trunk, but devel branch | 12:18 |
| Saviq | greyback, ah | 12:18 |
| Saviq | forgot about that | 12:18 |
| greyback | Saviq: I plan to fix bug 1305128 and then do a release | 12:19 |
| ubot5 | bug 1305128 in Unity 8 "Apps don't always get focused when started with upstart-app-launch" [Critical,In progress] https://launchpad.net/bugs/1305128 | 12:19 |
| Saviq | greyback, glad, will help | 12:19 |
| Saviq | (I will) | 12:19 |
| greyback | ta | 12:19 |
| Saviq | greyback, doesn't build | 12:28 |
| Saviq | greyback, SIGINT, SIGTERM not declared in this scope, missing signal.h? | 12:28 |
| greyback | Saviq: probably. /me more curious why it builds for me | 12:29 |
| pstolowski | mhr3, karni we could provide result in preview's query run(), but the only backwards-compatible way of doing this that I can think of is to make it an attribute of PreviewQueryBase, and set it before calling run() (so it would only be available while inside run(), and i'm not sure I like it) | 12:29 |
| Saviq | greyback, FWIW same error in CI | 12:29 |
| mhr3 | pstolowski, overloaded constructor/ | 12:30 |
| mhr3 | ? | 12:30 |
| pstolowski | mhr3, well, yes, but then you have a class that can be partially initialized if you use the default ctor | 12:30 |
| mhr3 | pstolowski, mark deprecated :) | 12:31 |
| karni | mhr3: does this make sense to you? action works (launches browser), label does not show up: http://paste.ubuntu.com/7404288/ | 12:32 |
| karni | mhr3: I used a vector, because I want more general code that shows you can add more than one button | 12:32 |
| karni | plus, mapping would expect an array there so.. VariantArray is actually std::vector<Variant> .. | 12:32 |
| mhr3 | karni, that's what the add_tuple does | 12:32 |
| mhr3 | karni, ie the variant builder is already creating an array | 12:33 |
| karni | mhr3: actions is an array, that coutains tuples that map to buttons | 12:33 |
| karni | [ { button attributes }, { button attributes }, ... ] | 12:33 |
| karni | mhr3: so, you have an array of variant builders, no? | 12:34 |
| mhr3 | karni, no, variantbuidler is building an array already | 12:34 |
| karni | mhr3: I guess the preview actiono works only because it actually matches the result uri | 12:34 |
| mhr3 | karni, if you didn't need an array you'd just use a variantmap | 12:34 |
| karni | okay, let me try it then :) | 12:35 |
| mhr3 | karni, yea :) | 12:35 |
| greyback | Saviq: fix pushed | 12:35 |
| Saviq | greyback, k | 12:35 |
| karni | mhr3: good call, man. thanks | 12:35 |
| pstolowski | mhr3, I think I've reached the limit of 'mark as deprecated' for today ;) | 12:35 |
| karni | mhr3: I'm happy to see std::vector<Variant> go away. I love STL, but this didn't look right. | 12:36 |
| mhr3 | karni, but if you changed the VariantBuilder to VariantMap and kept the vector, it would actually work | 12:37 |
| karni | mhr3: gotcha :)! | 12:37 |
| pstolowski | yay for loosely typed apis ;) | 12:38 |
| mhr3 | anyway, /me lunch | 12:39 |
| Saviq | greyback, so any idea how you built it? dirty build? | 12:43 |
| greyback | Saviq: must've been | 12:43 |
| Saviq | greyback, kk | 12:44 |
| Saviq | greyback, btw, you said you were discussing with mirfolk that this whole quit architecture could use some love, that patch is then only valid until that happens, right? | 12:46 |
| greyback | Saviq: this was their suggestion on how it should be done | 12:47 |
| Saviq | greyback, ok | 12:47 |
| dednick | baaaa! | 12:48 |
| dednick | 3 times today. not going well | 12:48 |
| Saviq | dednick, wassup there? :D | 12:48 |
| Saviq | dednick, you didn't ping tsdgeos this time ;) | 12:49 |
| dednick | Saviq: yeah. not concentrating! | 12:49 |
| Saviq | greyback, awesome, this seems to fix the shutdown time indeed, will speed up autopilot tests considerably | 12:56 |
| === alan_g|lunch is now known as alan_g | ||
| greyback | Saviq: good to hear. Did you check the startup-wizard shuts down cleanly too? | 12:58 |
| Saviq | greyback, not yet, running the u8 suite now | 12:59 |
| greyback | ok | 12:59 |
| Saviq | Cimi, pointer on how to test welcome wizard? | 12:59 |
| karni | Is the organic grid only good for scope results with different text length, or can one achieve the "image organic layout" like in the gallery app? | 13:11 |
| Saviq | karni, organic grid is _only_ good for images | 13:12 |
| Saviq | karni, different text length is vertical journal | 13:12 |
| === MacSlow|lunch is now known as MacSlow | ||
| Saviq | MacSlow, hey, because of lp:~unity-team/unity-system-compositor u-s-c gains a dependency on libandroid-properties-dev, could that be avoided? | 13:13 |
| Saviq | MacSlow, otherwise we lose arm64 as an arch on which u-s-c built | 13:13 |
| MacSlow | Saviq, not if we want to keep having artwork-assets correctly scaled based on the provided per-device-grid-unit | 13:14 |
| MacSlow | Saviq, that dependency is due to having to read the grid-unit from android-properties | 13:15 |
| MacSlow | Saviq, it's already nasty having to do this... dropping the dependency on libandroid-properties-dev would make it worse... despite I don't know of the top off my head how to solve getting the grid-unit-value in such a case | 13:16 |
| Saviq | Mirv, ↑ | 13:17 |
| Saviq | wonder if we can have optional deps... | 13:17 |
| Saviq | or well, maybe libandroid-properties could be built for arm64.. | 13:21 |
| MacSlow | Saviq, why is it not available by default? | 13:24 |
| Saviq | MacSlow, well, it only makes sense for android... which is only available for x86 + armhf | 13:24 |
| Saviq | MacSlow, feels like we need to abstract it somehow | 13:25 |
| Saviq | MacSlow, I'll ask around | 13:26 |
| MacSlow | Saviq, ok | 13:26 |
| Cimi | Saviq, you add the upstart file | 13:45 |
| Cimi | Saviq, and reboot phone | 13:45 |
| Saviq | Cimi, "the upstart file" found where? | 13:45 |
| Cimi | Saviq, you might also want to remove the file from .config/ubuntu-system-settings/wizard-has-run after than | 13:46 |
| Cimi | Saviq, inside wizard folder of a ubuntu system settings branch | 13:46 |
| Saviq | Cimi, the wizard is in the image already? | 13:46 |
| Saviq | just without the upstart job? | 13:47 |
| Cimi | Saviq, yeah I think so | 13:47 |
| Cimi | is part of system settings | 13:47 |
| Saviq | k let's see | 13:47 |
| Cimi | I don't know which version is here | 13:47 |
| Saviq | Cimi, "creating surface at (0, 58) with size (768, 1222) with title 'Window 1'" doesn' | 13:48 |
| Saviq | t sound right | 13:48 |
| Cimi | I recommend https://code.launchpad.net/~unity-team/ubuntu-system-settings/wizard.wifi/+merge/212675 or https://code.launchpad.net/~cimi/ubuntu-system-settings/wizard.privacy | 13:48 |
| Cimi | Saviq, wizard.wifi has the updated mir support | 13:49 |
| Saviq | Cimi, k | 13:49 |
| Cimi | Saviq, pending review since weeks.. | 13:49 |
| Saviq | Cimi, sounds like you should bug people to review :P | 13:49 |
| Cimi | I did | 13:49 |
| Saviq | Cimi, bug them more | 13:50 |
| Saviq | Cimi, what're you up to today btw? | 13:50 |
| Cimi | seb128, my french friend | 13:50 |
| Saviq | Cimi, any word on new infographics? | 13:50 |
| * seb128 looks at Saviq | 13:50 | |
| * Saviq is invisible to seb128 | 13:50 | |
| seb128 | Saviq, I don't need more being bugged thanks :p | 13:50 |
| seb128 | just more hours in a day | 13:51 |
| Saviq | seb128, sure you do! ;) | 13:51 |
| seb128 | lol | 13:51 |
| seb128 | ok ok, I can have a look to some u-s-s pending changes now | 13:51 |
| Saviq | see! :D | 13:51 |
| seb128 | ;-) | 13:51 |
| Cimi | Saviq, I did the shadow of the carousel but looks bad | 13:51 |
| Cimi | Saviq, I think I'll wait for the new ubuntushape | 13:52 |
| Cimi | Saviq, can we bug zoltan more? | 13:52 |
| Cimi | Saviq, doing infographics this afternoon | 13:52 |
| Saviq | Cimi, you can bug whomever you want ;) | 13:52 |
| Cimi | Saviq, but like together | 13:52 |
| Saviq | Cimi, cool thanks | 13:52 |
| Cimi | Saviq, we can't wait a year for that | 13:52 |
| Saviq | Cimi, ok, I'll do that later today/early tomorrow, depending on when he's around | 13:53 |
| Cimi | Saviq, problem with the shadow and the ubuntushape is that if I add a png with the current ubuntushape | 13:54 |
| Cimi | Saviq, the bright/white highlight line for the inset effect at the bottom of the ubuntushape clashes with the shadow put below | 13:54 |
| Cimi | so looks prob 10% better and 5% worse | 13:55 |
| Saviq | mhm | 13:55 |
| Saviq | Cimi, is ofono-qt a new dependency you added in one of your branches? it's missing from debian/control apparently | 14:00 |
| Cimi | Saviq, mmm | 14:00 |
| Cimi | Saviq, let me check | 14:00 |
| Cimi | Saviq, we need ofono for simcard | 14:00 |
| Cimi | Saviq, but I believe that was already in system settings | 14:01 |
| Cimi | Saviq, https://code.launchpad.net/~cimi/ubuntu-system-settings/wizard.sim-detection/+merge/211976 | 14:01 |
| Cimi | Saviq, should be already there | 14:02 |
| Saviq | Cimi, hmm indeed it's there, why does cmake complain then :/ | 14:03 |
| === dandrader is now known as dandrader|afk | ||
| Cimi | Saviq, maybe was simply missing from phonemanager | 14:07 |
| Saviq | Cimi, looks like it's a cross-building issue | 14:08 |
| === dandrader|afk is now known as dandrader | ||
| mhall119 | mhr3: ping | 14:27 |
| mhr3 | mhall119, pong | 14:28 |
| mhall119 | mhr3: hey, can we setup the scopes api to auto-publish new docs? | 14:29 |
| mhr3 | mhall119, what's necessary for that? | 14:31 |
| mhall119 | mhr3: I have a script you can run against the doxygen output, and an auth token to let you push it to the server | 14:32 |
| greyback | dandrader: hey, <mlankhorst> was asking about bug 1307701 on #ubuntu-desktop, perhaps you can help him? | 14:32 |
| ubot5 | bug 1307701 in unity (Ubuntu) "xserver mouse pointer emulation from touch breaks with QML app." [High,In progress] https://launchpad.net/bugs/1307701 | 14:32 |
| mhall119 | we just need to integrate that into your release process so that it runs whenever you update the packages containing them | 14:32 |
| dandrader | greyback, we've been talking. that's the bug I've been working on for quite a few days now | 14:33 |
| mhr3 | mhall119, hm, need to think where that could be done, will get back to you once meeting ends :) | 14:33 |
| greyback | dandrader: yes that I knew. Ok, I wasn't aware you two were already talking | 14:33 |
| mhall119 | mhr3: ok, let me know if you want to do a hangout to discuss it | 14:33 |
| karni | mhr3: Do you know if emblem image would work when overlay is enabled? Our team discussed that, it is a valid use case to require an overlay on the card image, to surface provider of given result. Be it music, clothes, perfumes or music equipment stores, suppose we use large cards with overlay enabled - we'd still want to show the shop/distributor logo in there. | 14:45 |
| karni | mhr3: If you're not the proper person to ask, I'll ping someone else :) | 14:45 |
| kgunn | where be mterry ? | 14:46 |
| Saviq | kgunn, didn't see him today yet | 14:46 |
| davmor2 | kgunn: he's heard you were after him so wisely hid? ;) | 14:47 |
| kgunn | probably :) | 14:47 |
| Saviq | kgunn, actually IRC says he was here, last exited ~1.5h ago | 14:48 |
| kgunn | Saviq: he had a lightdm config change up for landing (to move the dir location)...did that get landed ? ...i didn't get silo until late friday..but i don't see it now on the ci-train backend | 14:48 |
| kgunn | spreadsheet still looks wonky | 14:49 |
| Saviq | kgunn, wrong spreadsheet | 14:49 |
| Saviq | kgunn, there's a new one https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFVHQ3FuMDJGLUZCamJfSjYzbWh3Wnc#gid=0 | 14:49 |
| * kgunn still behind on mails | 14:49 | |
| Saviq | kgunn, but yeah, I landed that change | 14:50 |
| kgunn | thanks sir! | 14:50 |
| Saviq | Cimi, can you please look into bug #1316629 with karni | 14:59 |
| ubot5 | bug 1316629 in Unity 8 "Title of text preview widget does not show" [Undecided,New] https://launchpad.net/bugs/1316629 | 14:59 |
| Cimi | Saviq, noticed the issue, yes I can | 15:00 |
| dednick | greyback: hey. how does unity8 know what y position to bound apps to (ie displacement of the panel) | 15:02 |
| greyback | dednick: unity-mir does that, see ApplicationManager::placeSession | 15:03 |
| dednick | greyback: ah. manually calced panelHeight... | 15:03 |
| greyback | dednick: yep :( | 15:03 |
| Saviq | seb128, was system settings X-buildable at any point? | 15:04 |
| dednick | greyback: umm. possible to change to a property set by unity8? | 15:04 |
| greyback | dednick: sure. What's the problem you're trying to solve? | 15:04 |
| dednick | greyback: we have a new "active call hint" which basically displaces the panel by another 3gu (only when we have a call active) | 15:05 |
| Saviq | seb128, the aftermath of trying to install the deps for it looks crazy http://paste.ubuntu.com/7404996/ :D | 15:05 |
| seb128 | Saviq, yes, for most of the trusty cycle (and in trusty release afaikà | 15:05 |
| greyback | dednick: oh man, that's not going to be easy - the qtcompositor work would help you a lot with that | 15:05 |
| seb128 | Saviq, is that trusty or utopic? | 15:06 |
| Saviq | seb128, utopic | 15:06 |
| dednick | greyback: yeah. active apps are going to be a pain... not really wure what the ui story is though. If we're supposed to displace apps or what... | 15:06 |
| dednick | guess i should check with design | 15:06 |
| greyback | dednick: how urgent is this? | 15:06 |
| seb128 | Saviq, I guess we had regressions there then :/ maybe xnox can help you | 15:07 |
| Saviq | seb128, indeed... | 15:07 |
| dednick | greyback: um. dunno. i don't thin it's on a "urgent list" | 15:07 |
| Saviq | xnox, another X-build log... this one's heavy http://paste.ubuntu.com/7404996/ | 15:07 |
| dednick | greyback: i'll just put it through and log a bug for now :) | 15:08 |
| dednick | "undocumented feature" | 15:08 |
| greyback | dednick: I just don't want you working hacking on a tricky solution with the existing unity8/mir design, that the QtCompositor work will make much much easier | 15:08 |
| dednick | greyback: no matter. deisgn will probably change their mind and decide they just want to colourize the panel anyway.... | 15:09 |
| greyback | dednick: :) | 15:09 |
| mhr3 | karni, i think we wanted to add a special image_attribution attribute | 15:10 |
| mhr3 | karni, and that would work no matter what overlay is set to | 15:11 |
| karni | mhr3: sounds great to me. please treat that as clear guidance from the phone delivery team this is a feature we would really want to have. | 15:11 |
| mhr3 | Saviq, is that right^? | 15:11 |
| karni | mhr3: where would the overlay be located? upper right? (bottom is taken by the overlay) | 15:12 |
| mhr3 | don't think there are designs for that | 15:12 |
| Saviq | mhr3, I don't think we agreed ultimately, but yeah something like this would be needed (/!\ in the spec /!\) | 15:12 |
| karni | mhr3: okay, I'm happy to pull some strings | 15:12 |
| mhr3 | mikenagle_, ^^ | 15:13 |
| Saviq | seb128, xnox, false alarm, it looks like trunk builds fine, but the wizard branches don't | 15:15 |
| seb128 | Saviq, ok, "good" ;-) | 15:15 |
| Saviq | seb128, ok no, I take that back, it's not working even in trunk, but I don't understand what's happening :| | 15:26 |
| * Saviq tries with a fresh chroot | 15:26 | |
| Saviq | alecu, it'd have to be Thu or earlier than that, I can't go past 4pm UTC either tomorrow | 15:28 |
| === gatox is now known as gatox_lunch | ||
| karni | I have a UI question/feedback regarding Previews: I have a preview, with "More..." (collapsed 2 buttons) and "Buy" at the bottom. When I press "More...", I imagine the screen would scroll, so that I can see those additional buttons. Instead, they overflow at the bottom of the screen, and I have to manually scroll down to see them. Should I file a bug? | 15:33 |
| karni | Actually, I'll file it, and you guys decide what you want to do with it :) | 15:34 |
| Saviq | karni, yeah, the combo button is a hack, waiting for a proper SDK implementation | 15:34 |
| Saviq | karni, but file a bug nevertheless, please write "should use expandable components from the SDK" | 15:35 |
| karni | ack. then certainly will log a bug | 15:35 |
| karni | ok! | 15:35 |
| karni | https://bugs.launchpad.net/unity8/+bug/1316660 | 15:37 |
| ubot5 | Ubuntu bug 1316660 in Unity 8 "Scope preview allows buttons overflow bottom edge" [Undecided,New] | 15:37 |
| tsdgeos | Saviq: fginther: any idea of what's up with https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1780/console ? | 15:48 |
| Saviq | tsdgeos, trusty | 15:48 |
| tsdgeos | ah | 15:48 |
| tsdgeos | obvious :D | 15:48 |
| tsdgeos | can we get it to utopic? | 15:48 |
| Cimi | karni, I think unity8 code is fine, you sure is the property properly sent? https://bugs.launchpad.net/unity8/+bug/1316629 | 15:49 |
| ubot5 | Ubuntu bug 1316629 in Unity 8 "Title of text preview widget does not show" [Undecided,New] | 15:49 |
| fginther | tsdgeos, work to move it is in progress, it's a little more work as it requires some VM setup. I'm hoping to have it working tomorrow | 15:49 |
| mhr3 | fginther, hey, any chance to migrate ci T->U for unity-scope-mediascanner? | 15:50 |
| karni | Cimi: Isn't the code that I pasted correct? If it is, and I can see the "text" field (the "price" key mapps to correct value) - I should be able to see the hardcoded "title" field (with value "Price") | 15:50 |
| tsdgeos | fginther: cool :) | 15:50 |
| Cimi | karni, I think it is, but unity might not receive this title thing | 15:51 |
| karni | Cimi: what do you mean by unity might not receive this title thing? | 15:51 |
| karni | Cimi: wouldn't that be the bug I reported? | 15:51 |
| Saviq | mhr3, we need to exted unity-scope-tool to understand previews | 15:51 |
| Saviq | ↑↑ | 15:51 |
| Cimi | karni, http://bazaar.launchpad.net/~unity-team/unity8/trunk/view/head:/qml/Dash/Previews/PreviewTextSummary.qml#L45 | 15:51 |
| karni | Cimi: I can see "3.14 EUR" as "text", but can't see "title" as "Price. so, unity receives the data, but doesn't show the title | 15:52 |
| * karni looks | 15:52 | |
| fginther | mhr3, that was migrated yesterday. All projects should be building on utopic now | 15:52 |
| Cimi | karni, I have this widgetData["title"], maybe this property is not defined | 15:52 |
| Saviq | karni, sure, it might be that it's not unity8 at fault but unity-scopes-shell | 15:52 |
| mhr3 | fginther, hmm - https://code.launchpad.net/~jamesh/unity-scope-mediascanner/scope-tests/+merge/217731 | 15:52 |
| karni | Saviq: It might be. That's why I came here to ask for your teams opinion :) | 15:52 |
| Saviq | karni, can you add some debugs in there to make sure widgetData["title"] is populated? | 15:52 |
| mhr3 | fginther, didn't cut it? | 15:52 |
| fginther | mhr3, i'll rerun that one now | 15:53 |
| mhr3 | Saviq, what do you mean? | 15:53 |
| Cimi | Saviq, karni that is my first guess, preview is tested to dinamically add/remove title and works | 15:53 |
| karni | hrm | 15:53 |
| mhr3 | Saviq, udnerstand how? | 15:53 |
| Saviq | mhr3, I mean that we need to display the preview data in the tool | 15:53 |
| Saviq | mhr3, and allow modifying it | 15:53 |
| Saviq | seb128, bug #1316667 | 15:53 |
| ubot5 | bug 1316667 in ubuntu-system-settings (Ubuntu) "0.1+14.10.20140428-0ubuntu1 does not cross-build on utopic" [Undecided,New] https://launchpad.net/bugs/1316667 | 15:53 |
| mhr3 | Saviq, hmmm | 15:53 |
| karni | Cimi: yeah, that code looks fine | 15:53 |
| mhr3 | Saviq, would be nice | 15:53 |
| Saviq | mhr3, would be indeed! ;) | 15:53 |
| seb128 | Saviq, thanks | 15:53 |
| mhr3 | Saviq, should do the same for results though | 15:54 |
| Saviq | mhr3, sure | 15:54 |
| seb128 | Saviq, u-s-s didn't change much, not sure it's ours | 15:54 |
| mhr3 | Saviq, and i don't think sdk has some kind of dataview component | 15:54 |
| Saviq | seb128, I'm sure it's not | 15:54 |
| seb128 | Saviq, said differently, it might stalled reported here | 15:54 |
| karni | Saviq: if you mean to add debug logs to unity8, I'm afraid I don't have time for that ATM to build it all. But if there's noone else to try it, I guess I have no choice. | 15:55 |
| Saviq | seb128, sure, but I'm actually unable to understand what did change, I can install all of those deps manually just fine (not that it builds, but that's another topic - it fails to locate some packages via pkg-config) | 15:55 |
| Saviq | karni, no need to build | 15:55 |
| Saviq | karni, just change in the qml | 15:55 |
| karni | Saviq: oh, that's what you mean | 15:55 |
| karni | sure | 15:56 |
| Saviq | karni, http://paste.ubuntu.com/7405273/ | 15:56 |
| seb128 | Saviq, we need xnox to help | 15:56 |
| seb128 | or maybe Laney (but he's off today) | 15:56 |
| Saviq | seb128, indeed | 15:56 |
| Saviq | karni, there's a BUT | 15:57 |
| Saviq | karni, it works in the scopes scope | 15:57 |
| karni | ORLY | 15:57 |
| Saviq | karni, if you long-press Amazon, it displays Description | 15:57 |
| karni | Saviq: You're talking about the widget of type "header"? | 15:58 |
| karni | Saviq: or about what is under the "Search" button | 15:58 |
| Saviq | karni, what are you talking about :D | 15:58 |
| Saviq | karni, I thought you talked about widget type "text" | 15:58 |
| karni | Saviq: widget of type "text" that has "text" field required, and optional "title" field | 15:58 |
| karni | correct | 15:58 |
| Saviq | karni, yeah, under the Search button | 15:58 |
| karni | I'm in the preview of Amazon scope | 15:58 |
| Saviq | karni, "Description" is the title | 15:58 |
| Saviq | karni, of that widget | 15:58 |
| karni | Saviq: "This is an Ubuntu search...." that is the title of "text": "Online Shopping...", yes? | 15:59 |
| Saviq | karni, no | 15:59 |
| Saviq | karni, "Description", literal | 15:59 |
| mhr3 | karni, http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/unity-scopes/previewwidgets.html#text | 15:59 |
| karni | I see no "Description" on that page | 16:00 |
| * Saviq screenshots | 16:00 | |
| karni | I flashed trusty proposed today | 16:00 |
| karni | mhr3: You're showing me the documentation I pointed you at :D? | 16:00 |
| karni | title Optional string | 16:00 |
| karni | text A string containing the text | 16:00 |
| karni | that is exactly what I'm talking about - optional "title" does not work for me, mhr3 | 16:00 |
| Saviq | no I'm not :| | 16:00 |
| Saviq | karni, TRUSTY? :P | 16:00 |
| Saviq | karni, you're so old school | 16:00 |
| karni | Saviq: that was to mhr3 | 16:00 |
| karni | Saviq: lol | 16:00 |
| mhr3 | karni, oh did you... sorry just read the bottom :) | 16:00 |
| mhr3 | karni, anyway, that works and is used :) | 16:01 |
| Saviq | screenshot no worky :| | 16:01 |
| karni | ok guys, let me see the amazon code, and I'll take it from there | 16:01 |
| karni | if I have a "text" widget with title and text, only text field shows up | 16:01 |
| Cimi | karni, if you not set text, what happens? | 16:02 |
| karni | if you want, mark the bug as incomplete, I'll make sure it's complete | 16:02 |
| karni | Cimi: well, I haven't tried not setting text on a text widget, but I can try | 16:02 |
| Saviq | karni, ah you're doing mapping | 16:02 |
| karni | I think nothing will show up, let me try | 16:02 |
| Saviq | karni, you don't want that | 16:02 |
| karni | Saviq: title is hardcoded, text is mapped | 16:02 |
| mhr3 | karni, wait... it actually doesn't work | 16:02 |
| karni | I'm lost you guys :D | 16:02 |
| Cimi | karni, one mistake I can think of the backend is replacing the title settings with only the text settings | 16:02 |
| Cimi | karni, or rather try adding title after text | 16:03 |
| Saviq | karni, you're in mhr3's hands | 16:03 |
| karni | Cimi: I can try that | 16:03 |
| karni | Saviq: thanks pal | 16:03 |
| karni | :) | 16:03 |
| Saviq | karni, but, IIUC | 16:03 |
| Saviq | karni, you should add_attribute_value | 16:03 |
| Saviq | karni, not add_attribute_mapping | 16:03 |
| karni | :O | 16:03 |
| Saviq | karni, mapping maps from the result to preview | 16:03 |
| Saviq | karni, value sets the value | 16:04 |
| karni | that could have been it, and would explain why nothing showed up. f**k | 16:04 |
| mhr3 | Saviq, do you actually see "Description" in scopes scope? | 16:04 |
| mhr3 | cause i don't | 16:04 |
| Saviq | mhr3, I do | 16:04 |
| Saviq | mhr3, on desktop, too | 16:04 |
| Saviq | on phone, too | 16:04 |
| mhr3 | maybe my version is just old | 16:04 |
| Saviq | I mean in tool as well | 16:04 |
| karni | Cimi: the code I pasted here is then incorrect https://bugs.launchpad.net/unity8/+bug/1316629 | 16:04 |
| mhr3 | and i forgot my phone at home | 16:04 |
| ubot5 | Ubuntu bug 1316629 in Unity 8 "Title of text preview widget does not show" [Undecided,New] | 16:04 |
| * karni tries _value() | 16:05 | |
| Saviq | karni, IIUC price.add_attribute_mapping("title", "Price"); tries to map a "Price" attribute of the _result_ to the preview, but the result has no such attribute | 16:05 |
| karni | Saviq: that would be correct, Sir | 16:05 |
| karni | Cimi: Marked bug as Invalid | 16:07 |
| karni | Consider kicking me from this channel. Man I'm sneaky it took three people to notice my bug in 3 lines of code. I'm sorry! | 16:07 |
| Cimi | Saviq, https://code.launchpad.net/~cimi/unity8/preview-text-summary-bottom-padding/+merge/217903 | 16:18 |
| Saviq | Cimi, good bugging! will get to it tomorrow | 16:19 |
| karni | Saviq: I justed asked Cimi about that very bug. I suspect there's no chance this will get into image tomorrow? | 16:19 |
| karni | Saviq: asking cose of demo Thu morning. If not, I can avoid using that for now. | 16:20 |
| Saviq | karni, yeah, no chance | 16:20 |
| karni | Saviq: ack, np | 16:20 |
| Saviq | karni, it's qml only, you could just patch it in on device ;) | 16:20 |
| karni | Saviq: heheheh, true | 16:20 |
| karni | I'm not the one doing the demo, so I'll refrain from dirty patches :) but thanks! | 16:21 |
| Cimi | Saviq, naybe we cab have it in a silo>? | 16:21 |
| karni | as far as I'm concerned, don't worry, really. | 16:22 |
| karni | Cimi: allow me to file a bug and link to that branch, so I can point to it from code. don't want to be pasting MP links in there :) | 16:25 |
| Cimi | ok | 16:25 |
| karni | Cimi: in lp, can you please link bug https://bugs.launchpad.net/unity8/+bug/1316683 to branch https://code.launchpad.net/~cimi/unity8/preview-text-summary-bottom-padding | 16:27 |
| ubot5 | Ubuntu bug 1316683 in Unity 8 "Text preview widget has too much padding" [Medium,Triaged] | 16:27 |
| Cimi | done | 16:28 |
| karni | thank you | 16:28 |
| === gatox_lunch is now known as gatox | ||
| === redj_ is now known as redj | ||
| mhr3 | mhall119, ping? | 17:56 |
| mhall119 | mhr3: pong | 18:05 |
| mterry | Saviq, another change in silo 002 please: remove the lightdm branch and rekick USC and unity8 | 19:08 |
| * mterry goes and tests split autopilot tests now that QA landed the change | 19:09 | |
| mhr3 | alecu, could you check out click from 005? | 19:10 |
| alecu | mhr3: sure thing | 19:10 |
| mhr3 | alecu, although... i can do it, going to check the rest anyway | 19:10 |
| Saviq | mterry, doing, will add the egl prerequisite explicitly, too | 19:16 |
| mterry | Saviq, OK | 19:16 |
| Saviq | mterry, seen my comment on it? | 19:16 |
| mterry | Saviq, no? Let me look | 19:17 |
| mterry | Saviq, noted. We can respin the silo for that and whatever autopilot fixes u8 requires later | 19:18 |
| Saviq | mterry, yup | 19:18 |
| Saviq | mterry, build kicked, will not notify you about it since we'll be missing the arm64 build of u-s-c | 19:21 |
| Saviq | (bot will not) | 19:21 |
| Saviq | josharenson, hey, bugs should only get marked "fix committed" when things are in trunk (or well, maybe devel that we don't use in unity8) | 19:58 |
| Saviq | josharenson, so "in progress" is usually the last status before landing a change | 19:58 |
| josharenson | saviq, alright. Were there any bzr hooks (like putting 'fixes lp:12345' in commit) or procedural issues that I missed? | 19:59 |
| Saviq | josharenson, --fixes lp:12345 when committing, yeah, it would have linked automagically | 20:00 |
| greyback | Saviq: "UK was off, so Gerry was off, too." <- tut tut tut | 20:00 |
| Saviq | greyback, ;) | 20:00 |
| Saviq | josharenson, but even if you missed that, there's a "Link a related branch" link on the right here: https://bugs.launchpad.net/unity8/+bug/1269282 | 20:00 |
| ubot5 | Ubuntu bug 1269282 in Unity 8 "Missing documentation / -help from unity8 binary" [High,In progress] | 20:00 |
| Saviq | josharenson, and a corresponding "Link a bug report" halfway down https://code.launchpad.net/~josharenson/unity8/doc_args | 20:01 |
| Saviq | josharenson, has the same effect - the bug will be marked released for the Ubuntu task | 20:02 |
| Saviq | when the package gets into distro | 20:02 |
| josharenson | saviq, so link the branch after its released? | 20:03 |
| Saviq | josharenson, no, link it as soon as it's up | 20:03 |
| Saviq | josharenson, if you commit with --fixes lp:12345 | 20:03 |
| Saviq | josharenson, LP will link it for you | 20:03 |
| josharenson | ack | 20:03 |
| Saviq | josharenson, then, when it's released, LP will mark the bug Fix Released | 20:03 |
| josharenson | saviq, oh thats convenient and fancy | 20:03 |
| Saviq | josharenson, yeah, it will also put the changelog and info where the package was fixed in a comment | 20:04 |
| Saviq | josharenson, one unfortunate distinction is that LP is made for upstream projects as well as distros | 20:04 |
| Saviq | josharenson, which means a bug can affect the upstream project, a distro package of that project, both, or even more (like target a particular distro release) | 20:05 |
| Saviq | josharenson, distro generally means Ubuntu for us | 20:05 |
| josharenson | ok | 20:06 |
| Saviq | josharenson, so e.g. https://bugs.launchpad.net/unity8/+bug/1281728 | 20:06 |
| ubot5 | Ubuntu bug 1281728 in unity8 (Ubuntu) "Unity8 random freeze on demo image" [Critical,Incomplete] | 20:06 |
| Saviq | josharenson, actually has 4 "tasks" - upstream Mir, upstream Unity8 and Ubuntu packages for both | 20:07 |
| josharenson | saviq, i see | 20:07 |
| Saviq | josharenson, which could in theory be disjoint (they are for Mir, for example), but for unity8 there could be only one | 20:07 |
| Saviq | josharenson, the adverse effect is that LP only closes the package bugs, not the upstream ones | 20:07 |
| Saviq | josharenson, and well, we have to disjoing bug lists... | 20:08 |
| Saviq | josharenson, ideally we would only have Ubuntu bugs, but that'd mean some of us would need to get on https://wiki.ubuntu.com/UbuntuBugControl | 20:08 |
| Saviq | to get the rights to triage Ubuntu bugs | 20:08 |
| Saviq | but well, maybe that's a route forward, I need to think | 20:08 |
| Saviq | end-of-lecture :) | 20:09 |
| josharenson | haha thanks | 20:09 |
| Saviq | mterry, u-s-c and u8 built in silo 002 | 20:11 |
| Saviq | I'm cancelling the build job | 20:11 |
| mhr3 | alecu, i don't see any .ini translations for click, is that expected? | 20:19 |
| mhr3 | alecu, maybe ping dobey about that :) | 20:21 |
| mhr3 | mhall119, and ping once again? | 20:37 |
| === jhodapp is now known as jhodapp|afk | ||
| mhall119 | mhr3: pong once again | 21:14 |
| mhr3 | mhall119, lucky, was about to close this window for today :) | 21:15 |
| mhr3 | mhall119, so the doc | 21:15 |
| mhr3 | s | 21:15 |
| mhr3 | mhall119, i was wondering where could we hook the upload, but i don't really see any place without pushing the key to bzr or something like that | 21:16 |
| mhall119 | mhr3: we don't want the key somewhere public | 21:16 |
| mhr3 | figured | 21:17 |
| mhr3 | what would you suggest? | 21:17 |
| mhall119 | mhr3: I was thinking it would go into the jenkins configs or something internal | 21:17 |
| mhr3 | mhall119, maybe the actual publisher that pushes stuff to distro? | 21:18 |
| mhr3 | mhall119, then projects could just be marked as "exports_docs" and when pushing such a project, it would push the docs as well | 21:18 |
| mhall119 | mhr3: that would work too | 21:19 |
| mhr3 | but didier is the one who'd be able to make that happen | 21:19 |
| === salem_ is now known as _salem | ||
| mhr3 | mhall119, let's schedule a meeting with him tomorrow? | 21:19 |
| mhall119 | as long as the doxygen output is available and it can run python with access to the internet (or at least a site on prodstack) | 21:19 |
| mhall119 | mhr3: tomorrow is good, I have an hour block at 11am US/Easter where I'm unavailable but otherwise I'm free | 21:20 |
| mhr3 | mhall119, my afternoon is clear too | 21:21 |
| mhr3 | mhall119, sent invite, time ok? or too early for you? | 21:24 |
| mhall119 | mhr3: that works for me | 21:25 |
| mhr3 | mhall119, eh, sorry, was looking at tuesday instead of wed | 21:27 |
| mhr3 | you got overlap there | 21:27 |
| mhr3 | mhall119, ok to move it 30mins later? | 21:27 |
| mhall119 | mhr3: no need, that's a core apps IRC meeting that popey runs, I'm not needed there | 21:29 |
| mhr3 | mhall119, ok | 21:30 |
| mhr3 | so talk to you tomorrow | 21:30 |
| Saviq | josharenson, did you see my comment https://code.launchpad.net/~josharenson/unity8/doc_args/+merge/217840/comments/520182 ? | 22:00 |
| josharenson | saviq, got it | 22:01 |
| Saviq | josharenson, can't merge your branch until that's done, tags in bzr are rather viral... | 22:01 |
| josharenson | saviq there is no change after running that script? | 22:05 |
| josharenson | saviq, like I removed the tags, but what is there to push now? | 22:14 |
| Saviq | josharenson, no pushing | 22:14 |
| Saviq | josharenson, you need to push it on every copy of the branch separately | 22:15 |
| Saviq | josharenson, tags in bzr are unfortunately orthogonal to commits | 22:15 |
| Saviq | josharenson, which means whenever you push / pull, you populate all the ones that didn't exist in the branch you push/pull to | 22:15 |
| Saviq | or maybe should I say parallel, actually | 22:16 |
| josharenson | saviq, sure, but how do I make lp aware of this? | 22:16 |
| Saviq | josharenson, you run the script on lp:~josharenson... | 22:16 |
| josharenson | oh duh | 22:16 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!