| === TheMuso` is now known as TheMuso | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| === duflu_ is now known as duflu | ||
| didrocks | tsdgeos: hey, did you see the email on the unity8 maguro tests failing (they are multiple of them starting on image 181) | 09:14 |
|---|---|---|
| didrocks | that can be due to latest Mir | 09:15 |
| tsdgeos | didrocks: i did see it, and in my evilness i decided to unsee it :D | 09:15 |
| tsdgeos | didrocks: can't we just deprecate maguro already and be happy? | 09:16 |
| mhr3 | tsdgeos, it seems we're not displaying the activity indicator when waiting for a video thumbnail | 09:17 |
| mhr3 | expected? | 09:18 |
| mhr3 | Cimi, ^? | 09:18 |
| mhr3 | that's for video widget | 09:18 |
| Cimi | mhr3, I missed that | 09:18 |
| mhr3 | Cimi, isn't it part of lazy image itself? | 09:19 |
| mhr3 | Cimi, if not, pls add | 09:19 |
| tsdgeos | yeah that's what i though | 09:19 |
| tsdgeos | by using a LazyImage | 09:19 |
| Cimi | dunno then | 09:19 |
| tsdgeos | that we use | 09:19 |
| tsdgeos | we should get it | 09:19 |
| tsdgeos | Cimi: please investigate why we don't :D | 09:19 |
| Cimi | mhr3, test the conditions pls | 09:19 |
| tsdgeos | maybe | 09:19 |
| tsdgeos | visible: state === "ready" | 09:19 |
| tsdgeos | is breaking it | 09:19 |
| tsdgeos | probably we need to move that to the image and mousearea, but not to the lazy image | 09:20 |
| tsdgeos | so we get the spinning wheel | 09:20 |
| didrocks | asac: happy if we discare the unity8 failures on maguro and promote 181 if dogfooding is good? | 09:21 |
| didrocks | asac: seems people want to not take it into account even if I multiple times said it would be even worse with the emulator, so better to check for issues | 09:21 |
| Cimi | tsdgeos, https://code.launchpad.net/~cimi/unity8/units8.previews_RatingStars/+merge/206239 | 09:22 |
| tsdgeos | didrocks: asac: about maguro, yes, there is/was at least a test that i could reproduce all the time back on monday/tuesday and i know it could be worse on the emulator *but* what i was seeing may be caused by a lot of causes, one of them Qt having a bug on slower stuff (we know qt5.0 scene graph loop is bad) so before investing more time in getting it fixed i want to have Qt 5.2 first, and then if it still happens we can devote time to properly | 09:23 |
| tsdgeos | investigate it. My problem is doing it *now*, not investigating it | 09:23 |
| tsdgeos | that was a big long ^_^ | 09:23 |
| tsdgeos | didrocks: asac: did i made sense in that sentence? | 09:23 |
| didrocks | let me reread… it's… long :p | 09:23 |
| tsdgeos | Cimi: saw it, will do in a bit | 09:23 |
| didrocks | tsdgeos: if you just take a look at the output and can sense "this is a test only issue, no behavioral broken itself", I'm happy | 09:24 |
| tsdgeos | didrocks: i can't say that | 09:26 |
| didrocks | sad we even don't have results on 182 to know if some are reliably failing | 09:27 |
| tsdgeos | as said, the test failure on maguro (at least the one i debugged a bit at beginning of the week) is stuff that should not be happening | 09:27 |
| tsdgeos | but as trivial as "something that should be index 1 of a list is index 2" | 09:27 |
| tsdgeos | so i don't want to debug that in an ancient Qt | 09:27 |
| didrocks | tsdgeos: maguro isn't officially deprecated though, but beh… | 09:28 |
| tsdgeos | didrocks: i know, i'm not saying that it should not be investigated, i just don't want to investigate it now | 09:29 |
| tsdgeos | after we update to qt 5.2 | 09:29 |
| tsdgeos | i'm happy to do | 09:29 |
| didrocks | tsdgeos: that would mean we block promotion until we get 5.2? | 09:29 |
| tsdgeos | didrocks: that's for you guys to decide :D | 09:30 |
| tsdgeos | didrocks: i do not think that this bugs are new | 09:31 |
| tsdgeos | s/bugs/test failures | 09:31 |
| tsdgeos | but the data says otherwise | 09:31 |
| didrocks | they didn't show as failures | 09:31 |
| didrocks | but ok, let's see | 09:31 |
| tsdgeos | so i can be just speaking crap | 09:31 |
| tsdgeos | mhr3: this https://code.launchpad.net/~aacid/unity8/misc_previewlist_fixes/+merge/206365 should fix the previewlist opening/closing problems you were having yesterday, can you check | 09:32 |
| asac | so the problem is that you guys decay our stuff | 09:32 |
| asac | shoudl we try removing MIR again? | 09:32 |
| asac | etc.? | 09:32 |
| asac | should we ignore? | 09:32 |
| asac | tsdgeos: you already said the same last week | 09:33 |
| asac | e.g. its not new | 09:33 |
| asac | but there is no indication of that being right | 09:33 |
| tsdgeos | asac: i know i'm repeating myself | 09:33 |
| asac | we made an exception last week to give you more time to investigate | 09:33 |
| asac | now you say you cant make it now | 09:34 |
| asac | there needs to be a STOP at some point | 09:34 |
| asac | or REVERT | 09:34 |
| tsdgeos | asac: sure | 09:34 |
| asac | tsdgeos: please check these failures out... if you can proof them to be SoC/maguro specific | 09:34 |
| asac | then we can ignore | 09:34 |
| tsdgeos | asac: i can't do that and MWC at the same time | 09:34 |
| asac | but htey are most likely genuine | 09:34 |
| asac | races etc. | 09:34 |
| asac | tsdgeos: i am not sure what you do for MWC | 09:35 |
| tsdgeos | asac: let me repeat again "qt 5.0 qml is racy as hell and i don't want to debug it because it has been fixed already" | 09:35 |
| asac | tsdgeos: we mght not get 5.2 | 09:35 |
| asac | you dont know that yet | 09:35 |
| tsdgeos | so i am not going to spend my time on that unless my manager tells me | 09:35 |
| asac | (most likely we will get it) | 09:35 |
| tsdgeos | speak to my manager | 09:36 |
| asac | kgunn: ^ | 09:36 |
| asac | :) | 09:36 |
| tsdgeos | thanks | 09:36 |
| seb128 | asac, not getting 5.2 seems like that would be a mistake | 09:36 |
| asac | seb128: sure it would be | 09:36 |
| asac | just sayin gthat whatever isnt reality might never happen (even if unlikely and brutal) | 09:36 |
| tsdgeos | asac: or speak to saviq when he's back on monday, i'm just a regular guy that shouldn't and don't want to take this decisions | 09:36 |
| tsdgeos | greyback: morning, seen the mail by kevin about the sdk failure on demo-stuff ppa? | 09:37 |
| asac | tsdgeos: my understanding was that you are saviq's placeholder | 09:37 |
| asac | while he is gone | 09:37 |
| tsdgeos | asac: i am saviq's placeholder | 09:37 |
| greyback | tsdgeos: yeah just now | 09:37 |
| asac | tsdgeos: so you can make the decisions as he could | 09:37 |
| tsdgeos | asac: that's a bit of a overstatement :D I don't have all the knowledge about all the other components he has, i've just been raised temporarily because we needed someone | 09:38 |
| asac | tsdgeos: so what do you propose? as a leader you must have a clear suggestion | 09:38 |
| tsdgeos | asac: i do have a clear suggestion, ignore test failures that happen in maguro but not on mako until 5.2 lands | 09:39 |
| seb128 | asac, imho it doesn't make sense spend time to workaround toolkit issues in our code, especially if getting a workaround is not trivial, when we know those issues are bugs fixed in the new version | 09:39 |
| tsdgeos | asac: i think i've been pretty clear in that | 09:39 |
| asac | noone has proven that | 09:39 |
| asac | its qt | 09:39 |
| asac | its a guess | 09:39 |
| asac | and hope | 09:39 |
| tsdgeos | asac: noone has proven that it is not | 09:39 |
| asac | right,k hence investigation is needed | 09:39 |
| asac | we had many many flaky things fixed | 09:40 |
| seb128 | or moving on with qt 5.2 is needed | 09:40 |
| asac | on maguro | 09:40 |
| tsdgeos | asac: and we have proven that there are billions of problems in qt | 09:40 |
| asac | never was the toolkit cited as the reason | 09:40 |
| seb128 | it's creating an increasing number of resources wastes | 09:40 |
| asac | try how the APs succeed with qt 5.2 then | 09:40 |
| asac | do they? | 09:40 |
| asac | that could easily make us rest | 09:40 |
| tsdgeos | asac: i don't know, do they? | 09:40 |
| tsdgeos | Cimi: you wokring on the video stuff? | 09:59 |
| tsdgeos | Cimi: the RatingStars.qml we weren't using it yet? | 10:03 |
| Cimi | tsdgeos, we were using it | 10:03 |
| Cimi | tsdgeos, thus I added the compatibility layer | 10:03 |
| tsdgeos | ah righ | 10:03 |
| Cimi | tsdgeos, mainly old previews | 10:03 |
| tsdgeos | just saw it now | 10:04 |
| tsdgeos | Cimi: should then tst_RatingStars.qml still be there and not rename/Rework it to proof the old way still works? | 10:05 |
| tsdgeos | and add a tst_Rating.qml with the new code | 10:05 |
| Cimi | tsdgeos, the tst_Rating is tested | 10:06 |
| tsdgeos | Cimi: yes | 10:06 |
| tsdgeos | Cimi: but you deleted the tst_RatingStars.qml | 10:06 |
| tsdgeos | can we leave it there while we don't delete the compatibility stuff with the old scopes? | 10:07 |
| Cimi | tsdgeos, how do I revert the previous file and add the new one? | 10:08 |
| tsdgeos | good question :D | 10:09 |
| Cimi | tsdgeos, I'll simply add the file | 10:09 |
| tsdgeos | ok | 10:09 |
| Cimi | and who cares | 10:09 |
| Cimi | tsdgeos, at this point I can remove the compatibility layer and have double widget | 10:10 |
| tsdgeos | ? | 10:11 |
| Cimi | tsdgeos, copy back RatingStars | 10:11 |
| tsdgeos | ah | 10:11 |
| tsdgeos | sure | 10:11 |
| tsdgeos | that is even better :D | 10:11 |
| tsdgeos | be sure not to break old-scopes | 10:11 |
| Cimi | tsdgeos, actually | 10:11 |
| Cimi | tsdgeos, my new rating stars provides new assets | 10:11 |
| Cimi | boring | 10:11 |
| Cimi | :-\ | 10:11 |
| Cimi | tsdgeos, readied old stuff | 10:15 |
| Cimi | *readded | 10:15 |
| tsdgeos | Cimi: the test too? | 10:17 |
| Cimi | nope | 10:17 |
| Cimi | tsdgeos, forgot to run bzr add | 10:18 |
| Cimi | tsdgeos, done | 10:18 |
| greyback | anyone familar with LP builds? Getting weirdest error with: https://launchpad.net/~unity-team/+archive/demo-stuff/+recipebuild/652566 | 10:26 |
| tsdgeos | nope :/ | 10:27 |
| greyback | ah, seems to be a more general fail. #ubuntu-devel discussing it now | 10:27 |
| greyback | @unity: don't update your machine right now, there seems to be a broken libgcc1 package in the repo atm | 10:29 |
| tsdgeos | \o/ | 10:29 |
| tsdgeos | not | 10:29 |
| tsdgeos | let's revert everything! | 10:29 |
| === _salem is now known as salem_ | ||
| tsdgeos | Cimi: the ratings widget has the same problem vidoe had with the height | 11:09 |
| Cimi | tsdgeos, ah sorry dude | 11:10 |
| Cimi | tsdgeos, will fix | 11:10 |
| tsdgeos | Cimi: no worries, that's why we have people reviewing :) | 11:11 |
| Cimi | tsdgeos, pushed | 11:11 |
| tsdgeos | Cimi: it ain't working | 11:12 |
| tsdgeos | Cimi: apply the patch i said and see the difference between clip enabled and not enabled | 11:13 |
| tsdgeos | unless it's on purpose that the line edit and button are not part of the height | 11:13 |
| tsdgeos | which i think is a bit weird :D | 11:13 |
| Cimi | what? | 11:14 |
| Cimi | why is not working with implicitHeight? | 11:14 |
| Cimi | tsdgeos, | 11:20 |
| tsdgeos | Cimi: it's not that implicitHeight is not working, i'd say that your childrenRect is unhappy | 11:20 |
| tsdgeos | let me have a look | 11:21 |
| Cimi | tsdgeos, how do I test it? | 11:21 |
| tsdgeos | Cimi: see the comment on the review | 11:21 |
| tsdgeos | Cimi: i'd say your reviewContainer is missing a implicitheight too | 11:21 |
| Cimi | I'm looking | 11:22 |
| Cimi | tsdgeos, works for you? | 11:22 |
| tsdgeos | nope :D | 11:22 |
| tsdgeos | Cimi: http://paste.ubuntu.com/6930678/ ? | 11:29 |
| Cimi | tsdgeos, I ant it to stretch | 11:30 |
| Cimi | *want | 11:30 |
| Cimi | mmm | 11:30 |
| tsdgeos | Cimi: but it is not stretching :D | 11:31 |
| tsdgeos | Cimi: see your own tryPreviewRating without any change | 11:31 |
| tsdgeos | you have it set to anchors.fill: parent | 11:31 |
| Cimi | tsdgeos, I understand why its not stretching | 11:31 |
| tsdgeos | and it's just not height enough | 11:32 |
| Cimi | but I'd like it to stretch | 11:32 |
| tsdgeos | besides this is the implicitheight | 11:32 |
| Cimi | tsdgeos, in case you want two line of comment | 11:32 |
| tsdgeos | doesn't mean it can't adapt to more height | 11:32 |
| tsdgeos | it just gives a default one | 11:32 |
| tsdgeos | so if it should not prevent you from stretching | 11:32 |
| mzanetti | do we need some unreleased uitk version for the new-scopes stuff? | 11:34 |
| mzanetti | getting this: nativeOrientationAngle: 0 // stop contents being re-rotated when shell in landscape | 11:34 |
| mzanetti | but qtdeclarative5-ubuntu-ui-toolkit-plugin seems to be up to date | 11:35 |
| mzanetti | err. this is the error: Cannot assign to non-existent property "nativeOrientationAngle" | 11:35 |
| mzanetti | greyback: you might know this? ^ | 11:36 |
| greyback | mzanetti: yeah, I'm trying to get it built into the PPA | 11:36 |
| mzanetti | greyback: and it seems to be rotated on the desktop now | 11:38 |
| greyback | mzanetti: oh, that's not supposed to happen | 11:38 |
| tsdgeos | mzanetti: greyback: maybe because of the missing uitk stuff? | 11:39 |
| Cimi | tsdgeos, but if I disable the anchors, yes | 11:40 |
| tsdgeos | Cimi: diff? | 11:40 |
| Cimi | tsdgeos, your | 11:40 |
| greyback | tsdgeos: mzanetti: possibly, checking now | 11:41 |
| Cimi | tsdgeos, even mine doesn't stretch though | 11:43 |
| tsdgeos | Cimi: that's what i was saying :D | 11:43 |
| Cimi | hah | 11:43 |
| mzanetti | ok... going for lunch then and will try again. greyback, please let me know when the uitk package is ready | 11:45 |
| greyback | mzanetti: will do. LP giving me grief | 11:45 |
| mzanetti | *try again later | 11:45 |
| tsdgeos | Cimi: http://paste.ubuntu.com/6930736/ | 11:46 |
| tsdgeos | Cimi: this works here with both | 11:46 |
| tsdgeos | anchors.fill: parent and no width defined | 11:46 |
| tsdgeos | and with width: parent.width no anchors defined and no height defined | 11:46 |
| Cimi | tsdgeos, nope | 11:50 |
| Cimi | tsdgeos, maybe sdk bug | 11:50 |
| tsdgeos | nope what? | 11:50 |
| Cimi | tsdgeos, try typing in the entry when it's filled | 11:51 |
| Cimi | stretched | 11:51 |
| tsdgeos | lol | 11:51 |
| * greyback bbiab | 11:52 | |
| tsdgeos | Cimi: so maybe just have my first patch and consider stretching for later? | 11:52 |
| Cimi | tsdgeos, I want to fix the stretching if possible | 11:52 |
| Cimi | tsdgeos, let me see | 11:52 |
| tsdgeos | ok, work on that | 11:52 |
| Cimi | tsdgeos, review the tests | 11:53 |
| Cimi | tsdgeos, not sure they are done in the correct way | 11:53 |
| Cimi | tsdgeos, like, logically correct to put logic inside tests | 11:53 |
| tsdgeos | ok | 11:53 |
| Cimi | tsdgeos, for me is fine, but I'd like an opinion | 11:53 |
| tsdgeos | will do | 11:53 |
| Cimi | mhr3, btw, I received nexus 4 yesterday | 11:54 |
| Cimi | mhr3, guess which was the first thing I did? | 11:54 |
| mhr3 | Cimi, i don't want to know! :P poor n4, it's naked now! | 11:55 |
| Cimi | ehehe | 11:55 |
| Cimi | mhr3, thinking of getting a prepaid | 11:56 |
| Cimi | mhr3, and try ubuntu on it outside dev | 11:56 |
| Cimi | mhr3, is there a way to disable 3d data? | 11:56 |
| Cimi | *3g | 11:56 |
| mhr3 | Cimi, you think it would work out-of-the-box? :) | 11:56 |
| tsdgeos | mhr3: when you have some time i'd appreciate a quick test to see if https://code.launchpad.net/~aacid/unity8/misc_previewlist_fixes/+merge/206365 fixes your preview opening/closing problems | 11:56 |
| mhr3 | Cimi, i mean it might... but well :) | 11:57 |
| Cimi | mhr3, still buggered? | 11:57 |
| mhr3 | tsdgeos, am just about to try it | 11:57 |
| mhr3 | tsdgeos, i need https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-replacing-preview/+merge/206354 :) | 11:57 |
| tsdgeos | mhr3: you want me to review it or you want me to make unity8 work with it? | 11:59 |
| mhr3 | tsdgeos, review | 11:59 |
| mhr3 | tsdgeos, it doesn't change much of the shell-facing api | 11:59 |
| tsdgeos | mhr3: but does it work? :D | 12:00 |
| mhr3 | tsdgeos, ie nothing breaks | 12:00 |
| mhr3 | yes :) | 12:00 |
| tsdgeos | amazing | 12:00 |
| mhr3 | tsdgeos, your branch does breaks the header though | 12:00 |
| mhr3 | after closing preview it's not there | 12:00 |
| tsdgeos | sure, it's because of the scrolling of the view | 12:00 |
| tsdgeos | you can scroll it back | 12:01 |
| tsdgeos | but you're right it doesn't make sense | 12:01 |
| tsdgeos | since there's enough space | 12:01 |
| tsdgeos | it should be visible | 12:01 |
| mhr3 | yep | 12:02 |
| mhr3 | tsdgeos, is there a way to make the animation super-slow? | 12:03 |
| tsdgeos | mhr3: the opening one? | 12:03 |
| mhr3 | yea | 12:03 |
| mhr3 | when opening preview for a scope it's super weird | 12:03 |
| mhr3 | i want to see what's going on | 12:04 |
| tsdgeos | i'd say openeffect | 12:04 |
| tsdgeos | but let me check | 12:04 |
| tsdgeos | i don't see anything weird though with this branch :S | 12:04 |
| mhr3 | tsdgeos, i meant specifically scopes scope is odd (probably because of no-art) | 12:04 |
| tsdgeos | sure i've been using scopes scope | 12:04 |
| tsdgeos | mhr3: yes qml/Dash/DashContentOpenEffect.qml | 12:09 |
| tsdgeos | - duration: 200 | 12:09 |
| tsdgeos | + duration: 2000 | 12:09 |
| mhr3 | yea, the first frame make everything jump | 12:10 |
| mhr3 | makes | 12:10 |
| tsdgeos | yep | 12:10 |
| tsdgeos | going to see why that happens | 12:10 |
| tsdgeos | that'll fix the closing too most probably | 12:10 |
| mhr3 | it works fine for the first row of results | 12:10 |
| mhr3 | interesting | 12:10 |
| tsdgeos | yeah because there's some calculation in there going wrong | 12:11 |
| tsdgeos | positionListView in PreviewListView.qml is a bit of too much calculation :D | 12:11 |
| === MacSlow is now known as MacSlow|lunch | ||
| tsdgeos | mhr3: pushed | 12:20 |
| mhr3 | same branch? | 12:20 |
| gatox | mhr3, hi, do you know if it is possible to return a preview with the new scopes api when an action is triggered in a preview and the perform_action method is called?? (for example, in the click scope move from the Install preview to Progress preview) | 12:22 |
| mhr3 | gatox, it is, but scope-tool doesn't support it yet | 12:23 |
| gatox | mhr3, we have been trying to figure it out how to do that yesterday, but no luck | 12:23 |
| gatox | mhr3, ahhhhh ok | 12:23 |
| mhr3 | pstolowski, can you explain pls ^ trying to prepare all the mps now | 12:23 |
| pstolowski | gatox, you need to respond with ActivationResponse with status=ShowPreview | 12:24 |
| pstolowski | gatox, and then you get a call to preview() | 12:24 |
| gatox | pstolowski, but that is not working yet, right? | 12:24 |
| pstolowski | gatox, so it's an extra round-trip (unlike the old API, where you could pass serialized preview right away) | 12:25 |
| pstolowski | gatox, not supported by scope-tool, right | 12:25 |
| gatox | pstolowski, ack, thanks | 12:25 |
| tsdgeos | mhr3: yes | 12:25 |
| Cimi | tsdgeos, maybe because textfield is supposed to be a single line | 12:27 |
| tsdgeos | Cimi: maybe, there's a text area? | 12:28 |
| Cimi | tsdgeos, it's another option | 12:28 |
| Cimi | tsdgeos, not sure how it is themed | 12:28 |
| tsdgeos | mhr3: i don't think that i can honestly review https://code.launchpad.net/~mhr3/unity-scopes-shell/preview-replacing-preview/+merge/206354 | 12:30 |
| tsdgeos | mhr3: i can say you're missing a const :D | 12:30 |
| tsdgeos | in some functions | 12:30 |
| tsdgeos | but i have almost no knowledge of how that thing works | 12:31 |
| tsdgeos | i don't see any obvious coding mistake | 12:31 |
| mhr3 | tsdgeos, just run it :) | 12:31 |
| tsdgeos | but given that tests pass, that was to be expected | 12:31 |
| tsdgeos | mhr3: and do anything special? i don't know how to get an install/progress widget, none of the scopes i have seem to do it | 12:32 |
| tsdgeos | mhr3: or just do a test run of stuff? | 12:32 |
| mhr3 | tsdgeos, yea, replacing previews working is guaranteed by the test | 12:33 |
| mhr3 | famous last words :) | 12:33 |
| mhr3 | tsdgeos, or do what saviq does, get coverage report and see if the change lines are covered :) | 12:34 |
| kgunn | tsdgeos: mhr3 mzanetti ...i am so sorry to leave broken ppa's for your morning, did we recover ? | 12:35 |
| mhr3 | kgunn, no, lp is borked :/ | 12:36 |
| tsdgeos | kgunn: greyback was working on it | 12:36 |
| tsdgeos | but seems we have a broken gcc or soemthing that makes it hard to recover | 12:36 |
| greyback | kgunn: LP having problems, and a bad libgcc1 broke everything for a while, but things seem to be moving slowly again | 12:36 |
| kgunn | thanks...felt bad going to bed leaving it that way | 12:36 |
| greyback | kgunn: sleep more important | 12:37 |
| mhr3 | gatox, we have a branch with which scope-tool can do it, but lp is borked, can't rebuild the demo ppa | 12:40 |
| gatox | mhr3, ack..... any estimate for that?? | 12:41 |
| mhr3 | gatox, scratch that, lp works, it's building | 12:41 |
| gatox | mhr3, great | 12:42 |
| mhr3 | soon in demo ppa near you | 12:42 |
| tsdgeos | mhr3: approved, want me to top approve too? | 12:43 |
| mhr3 | tsdgeos, if it works for you, sure | 12:43 |
| tsdgeos | mhr3: did you have time to look at the second fix for the previewlist preview open/close thing? | 12:43 |
| mhr3 | not yet | 12:43 |
| mhr3 | tsdgeos, yea, now it seems to actually work :) | 12:55 |
| kgunn | mhr3 demo-stuff works ? | 12:55 |
| mhr3 | kgunn, no, tsdgeos's branch | 12:56 |
| mhr3 | kgunn, what's broken there anyway? | 12:56 |
| kgunn | mhr3: :) thot i was crazy... | 12:56 |
| kgunn | ubuntu-ui-toolkit | 12:56 |
| kgunn | i just cleaned and rekicked...let's see | 12:56 |
| mhr3 | oh :/ | 12:56 |
| === alan_g is now known as alan_g|lunch | ||
| mhr3 | gatox, built for desktop > https://code.launchpad.net/~unity-team/+archive/demo-stuff/+sourcepub/3913988/+listing-archive-extra | 13:05 |
| gatox | mhr3, thanks | 13:09 |
| kgunn | alrigth ubutnu-ui-toolkit seems to have built!...soon in the demo-stuff | 13:21 |
| kgunn | greyback_: ^ | 13:21 |
| greyback_ | \o/ | 13:21 |
| mzanetti | kgunn: nope, not recovered yet. the merge conflicts between new scopes and right edge are quite bad... still trying to fix them | 13:22 |
| kgunn | mzanetti: thanks...(and actually ui-toolkit is only part of the demo-stuff...for n7 rotation) | 13:47 |
| kgunn | starting to need a crib sheet :) | 13:47 |
| mzanetti | kgunn: well, the new scopes stuff depends on it | 13:47 |
| mzanetti | kgunn: which means I need to get it merged with my branches too | 13:47 |
| kgunn | mzanetti: oh crap...so we have to pull it in | 13:48 |
| kgunn | it==ui-toolkit | 13:48 |
| mzanetti | kgunn: right now I tied to run the new scopes stuff on my desktop and it doesn't even start without the unreleased uitk stuff | 13:48 |
| kgunn | really!?! | 13:48 |
| mzanetti | kgunn: commenting away the bits that make it fail, makes the shell start again, but rotated on the desktop | 13:48 |
| mzanetti | which in turn tries to load the unfinished sidestage stuff from my branch | 13:49 |
| kgunn | mzanetti: well...true we didn't test the effect | 13:49 |
| kgunn | i blame greyback_ | 13:49 |
| mzanetti | :D | 13:49 |
| mzanetti | I do so too :P | 13:49 |
| kgunn | he's an easy target | 13:49 |
| mzanetti | nah... we'll get it fixed | 13:49 |
| mzanetti | but yeah, need to the get the uitk stuff into the ppa I believe | 13:50 |
| mzanetti | greyback_: right? ^ | 13:50 |
| greyback_ | what did I do now? | 13:50 |
| mzanetti | greyback_: just kidding. you didn't do anyhting | 13:51 |
| greyback_ | mzanetti: http://pastebin.ubuntu.com/6931260/ will at least un-rotate on desktop | 13:51 |
| === alan_g|lunch is now known as alan_g | ||
| mzanetti | kgunn: what's the plan actually? to have the new scopes + right edge on all phones? | 13:56 |
| mzanetti | kgunn: or will there be different phone images? | 13:56 |
| kgunn | mzanetti: yes...that's the plan | 13:56 |
| === MacSlow|lunch is now known as MacSlow | ||
| kgunn | mzanetti: check my thinking but demo-stuff will be for n10/n7 | 13:56 |
| kgunn | and phone-right-edge will be for n4 | 13:56 |
| mzanetti | ok... so the right edge branch needs still some fixing to work with the new scopes... | 13:57 |
| * mzanetti is on it | 13:57 | |
| kgunn | mzanetti: thanks sir | 13:57 |
| kgunn | no pressure but mark will be reviewing early next week | 13:57 |
| mzanetti | :D | 13:58 |
| mzanetti | kgunn: what's the ETA on new scopes landing into trunk? | 13:58 |
| tsdgeos | pete-woods: ping | 14:00 |
| pete-woods | tsdgeos: hi | 14:00 |
| tsdgeos | pete-woods: hud-service is at 100% | 14:00 |
| tsdgeos | pete-woods: what do i do? | 14:00 |
| mzanetti | kgunn: anyways, this branch rebases right-edge-2 on new-scopes: lp:~mzanetti/unity8/right-edge-based-on-new-scopes | 14:00 |
| mzanetti | kgunn: should make it start up again with that, still some broken animations tho | 14:01 |
| pete-woods | tsdgeos: can you attach gdb to it? | 14:01 |
| tsdgeos | i guess | 14:01 |
| tsdgeos | let me | 14:01 |
| pete-woods | is it gdb -p PID ? | 14:02 |
| kgunn | mzanetti: ta... | 14:02 |
| gatox | mhr3, do i need any particular unity branch to use previes activation to show a new preview? or upgrading the system is enough (on desktop) | 14:02 |
| kgunn | i was gonna test dandrader's copy/paste from the landing silo....then i'll turn my eyes to that | 14:02 |
| tsdgeos | pete-woods: http://paste.ubuntu.com/6931318/ http://paste.ubuntu.com/6931320/ | 14:03 |
| tsdgeos | pete-woods: let me see if i can get debug symbols for libdbusmenu-qt5.so.2 and for hud-service | 14:03 |
| mhr3 | gatox, new-scopes is enough | 14:03 |
| pete-woods | tsdgeos: thanks | 14:03 |
| tsdgeos | pete-woods: http://paste.ubuntu.com/6931329/ http://paste.ubuntu.com/6931331/ http://paste.ubuntu.com/6931338/ | 14:06 |
| tsdgeos | pete-woods: the DBusMenuImporter::slotMenuAboutToShow and waitForWatcher seem to be the offenders | 14:06 |
| tsdgeos | sil2100: ↑↑↑ is that you? | 14:06 |
| === dandrader is now known as dandrader|afk | ||
| tsdgeos | oh | 14:08 |
| tsdgeos | now that i see | 14:08 |
| tsdgeos | i also have a few | 14:08 |
| tsdgeos | QVariantMap DBusMenuExporterDBus::getProperties(int, const QStringList&) const: Condition failed: action | 14:09 |
| tsdgeos | bool DBusMenuExporterDBus::AboutToShow(int): Condition failed: menu | 14:09 |
| tsdgeos | in my console | 14:09 |
| tsdgeos | no clue where they come from | 14:09 |
| tsdgeos | but seem related | 14:09 |
| pete-woods | tsdgeos: thanks for this debugging info - very helpful! | 14:09 |
| tsdgeos | pete-woods: can i kill it now? | 14:10 |
| pete-woods | sure | 14:10 |
| pete-woods | tsdgeos: if they are in your console, it makes me think it's an app that you started from there that is the culprit | 14:10 |
| pete-woods | obviously HUD should be resilient to it | 14:10 |
| pete-woods | but it would still be very handy if we could find the exact app | 14:11 |
| tsdgeos | fg says it may be kate | 14:11 |
| tsdgeos | which is the only app i ran from there and is still running now | 14:12 |
| Debolaz | I suppose you get this question a lot, but when is Unity 8 planned to be ready for desktop use? | 14:12 |
| tsdgeos | Cimi: you there? | 14:13 |
| tsdgeos | mhr3: ping | 14:15 |
| === dandrader|afk is now known as dandrader | ||
| kgunn | Debolaz: it'll be some time before unity8 is full fledged on desktop..however there are efforts under way to create a desktop preview this cycle...so look for the 14.04 rel | 14:23 |
| sil2100 | tsdgeos: wazzup? | 14:26 |
| tsdgeos | sil2100: hud-service spinin 100% cpu on libdbusmenu-qt5, read a bit up | 14:27 |
| tsdgeos | sil2100: afair where you involved in that? or not anymore? | 14:27 |
| sil2100 | tsdgeos: yes, but hm, this code didn't change since like a year | 14:28 |
| tsdgeos | ok | 14:28 |
| Cimi | tsdgeos, sorry, irc clinet crashed and didn't realise | 14:35 |
| mzanetti | tsdgeos: hey, I think there are 2 regressions in the new-scopes stuff in regard to the shell background. | 14:42 |
| mzanetti | tsdgeos: 1st, there isn't a background any more at all which makes running apps shine through the dash | 14:42 |
| tsdgeos | mzanetti: what do you mean there isn't a background? | 14:43 |
| tsdgeos | mzanetti: there's the white thing | 14:43 |
| Cimi | tsdgeos, also | 14:43 |
| mzanetti | tsdgeos: but its not opaque | 14:43 |
| Cimi | tsdgeos, with textarea | 14:43 |
| Cimi | tsdgeos, shall it make 3 lines tall? | 14:43 |
| Cimi | I make it | 14:43 |
| mzanetti | tsdgeos: 2nd, the dummy image for checking the background settings is gone which will break the greeter wallpaper in certain circumstances | 14:44 |
| tsdgeos | Cimi: works for me | 14:44 |
| tsdgeos | mzanetti: ok, i'll have a look unless you beat me to it :D | 14:44 |
| mzanetti | tsdgeos: this is what I mean: http://i.imgur.com/00KTKsq.png with 2st | 14:45 |
| mzanetti | 1st... | 14:45 |
| Cimi | tsdgeos, lineHeight * 3 is bad? | 14:45 |
| Cimi | or paintedHeight | 14:45 |
| Cimi | whatever | 14:45 |
| Cimi | not sure what works | 14:45 |
| tsdgeos | Cimi: try if it works first and then we can discuss how bad it looks :D | 14:45 |
| Cimi | tsdgeos, nevermind | 14:49 |
| Cimi | tsdgeos, textArea doesn't have lineHeight | 14:49 |
| Cimi | tsdgeos, so it's not possible | 14:49 |
| Cimi | tsdgeos, tried contentHeight / lineCount but it can't be done or it loops | 14:49 |
| kgunn | dandrader: ok, i got your qml file...but how do i run on the phone? (e.g. particularly directory? command line incantation?) | 14:50 |
| Cimi | tsdgeos, ok, pushed | 14:52 |
| Cimi | @unity: for the nexus 4 wifi, it doesn't work yet if you flash ubuntu from android 4.4. All you have to do it is either flash ubuntu when you're on android 4.3, or flash the radio from android 4.3 https://www.copy.com/s/JniBu/4.3%20files%20%28mako%29/radio-mako-m9615a-cefwmazm-2.0.1700.84.zip | 14:53 |
| tsdgeos | Cimi: cool, so what i said (or was trying to say at least :D) | 14:57 |
| dandrader | kgunn, qmlscene [SOME_QML_FILE].qml --desktop_file_hint=[SOME_EXISTING_DESKTOP_FILE_TO_FOOL_UNITY8] | 15:00 |
| tsdgeos | greyback_: how's that fixed uitk coming? | 15:00 |
| greyback_ | tsdgeos: should be in | 15:01 |
| tsdgeos | greyback_: where? | 15:01 |
| tsdgeos | in demo-stuff? | 15:01 |
| dandrader | kgunn, thus: qmlscene copy-paste.qml --desktop_file_hint=/usr/share/click/preinstalled/com.ubuntu.calendar/current/calendar-app.desktop | 15:01 |
| greyback_ | tsdgeos: yeah | 15:01 |
| tsdgeos | not getting it | 15:01 |
| tsdgeos | greyback_: kgunn: you need to rebuild, trusty has a newer version, so it wins | 15:01 |
| greyback_ | tsdgeos: seems archive has newer version entirely, do apt-cache showpkg to see what version and maybe switch | 15:02 |
| tsdgeos | greyback_: that may help me, but not anyone else installing the ppa, we need to get that working | 15:02 |
| greyback_ | tsdgeos: yep | 15:02 |
| kgunn | dandrader: uh...doesn't seem to work for me | 15:03 |
| kgunn | at least...doesn't do what i expected | 15:03 |
| greyback_ | kgunn: what you think? Can we replace the date string in the package name with "demo-stuff" to avoid this? | 15:04 |
| kgunn | greyback_: yes...its only for demo | 15:04 |
| kgunn | should be fine | 15:04 |
| dandrader | kgunn, what do you get? | 15:04 |
| dandrader | kgunn, it's a pretty ugly test app, I warn you :) | 15:05 |
| kgunn | dandrader: so i hit "copy all" words highlight...but then no matter what "paste" doesn't result in anything | 15:05 |
| dandrader | kgunn, hmmm, then it's not working. are you sure you have both MPs there? | 15:05 |
| dandrader | kgunn, the one for unity-mir and the other for platform-api? | 15:06 |
| kgunn | in fact dandrader if i hit "paste" while the text is highlighted...it just erases all highlighted text (like a delete) | 15:06 |
| kgunn | yep | 15:06 |
| dandrader | kgunn, a better way is to copy. close the app. run it again and then paste | 15:06 |
| Cimi | tsdgeos, replied | 15:06 |
| kgunn | dandrader: well....let me do a full dist upgrade just in case | 15:06 |
| kgunn | dandrader: i'll try your trick first | 15:06 |
| dandrader | kgunn, but from what you told me, I can already tell you the clipboard is not working | 15:07 |
| kgunn | ...mmm, your trick didn't work | 15:07 |
| kgunn | let me dist-upgrade | 15:07 |
| dandrader | kgunn, so your image must be missing one of my patches... | 15:08 |
| kgunn | dandrader: i installed one by one...so just in case | 15:08 |
| kgunn | btw, i'm using the ppa from here | 15:08 |
| tsdgeos | Cimi: tx | 15:08 |
| kgunn | https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFlCc1VzeVZzWmdBZS11WERjdVc3dmc&usp=sharing#gid=24 | 15:08 |
| kgunn | in the landing-006 sheet | 15:09 |
| tsdgeos | Cimi: did you fix the height thing too? | 15:11 |
| Cimi | yep | 15:11 |
| * tsdgeos reviews again | 15:13 | |
| === dandrader is now known as dandrader|afk | ||
| tsdgeos | Cimi: still not working | 15:16 |
| Cimi | ouch | 15:16 |
| Cimi | tsdgeos, I used your patch | 15:16 |
| tsdgeos | Cimi: set clip to true and see how the text field is cut instead of shrinked | 15:16 |
| Cimi | ok | 15:17 |
| tsdgeos | Cimi: my patch worked when it was not a textarea i guess :D | 15:17 |
| Cimi | tsdgeos, I'll quickly go out for errads | 15:17 |
| tsdgeos | or maybe didn't work and i didn't realize | 15:17 |
| Cimi | tsdgeos, there is a break with the rain | 15:17 |
| tsdgeos | run! | 15:17 |
| Cimi | I'll do this soom | 15:17 |
| Cimi | ahah | 15:17 |
| Cimi | yeah | 15:17 |
| Cimi | pretty shitty today | 15:17 |
| kgunn | Cimi: i'm gonna break your heart after weeks of 0 C, we've got sunshine & 23 C | 15:23 |
| === alan_g is now known as alan_g|tea | ||
| mzanetti | kgunn: Cimi: complaining about weather? this is happening in the area I come from: http://www.youtube.com/watch?v=f5waSw2mMfY | 15:24 |
| mzanetti | kgunn: right-edge + new-scopes working on my phone now | 15:25 |
| mzanetti | kgunn: should I upload a new package to the ppa or will that conflict with what you're doing? | 15:25 |
| kgunn | mzanetti: @weather...deadly but beautiful | 15:25 |
| tsdgeos | mzanetti: i don't know how to solve that background problem without having the image two times :/ | 15:26 |
| kgunn | mzanetti: if you want to push to | 15:26 |
| tsdgeos | mzanetti: or maybe i can add the image also to the scope tool | 15:26 |
| tsdgeos | that'd work too | 15:26 |
| mzanetti | tsdgeos: which image do you mean? the background (white) one? | 15:26 |
| tsdgeos | mzanetti: yes | 15:26 |
| kgunn | mzanetti: if you could merge to lp:~unity-team/unity8/merge-new-scopes-right-edge-2 | 15:27 |
| tsdgeos | mzanetti: the problem is not that the white thing is transparent, the problem is that we make the dash transparent while "bringing it up", which in turns makes the image transparent because it is inside the dash | 15:27 |
| kgunn | then we can trigger another build | 15:27 |
| mzanetti | kgunn: ack | 15:27 |
| tsdgeos | mzanetti: so i'd need to move it out of the dashh, but then that'd make the scope-tool not have a background, then i guess i can add the backgrond to the scope tool too | 15:27 |
| kgunn | its not interfering with me...i'm on copy/paste landing effort | 15:27 |
| kgunn | mzanetti: just wanted to note...i created a seperate branch to push to (for merging right-edge-2 & new-scopes) | 15:28 |
| kgunn | i should've told you in the mail...but i was getting goofy last night and forgot | 15:28 |
| mzanetti | kgunn: ah ok. no prob. will push to there | 15:29 |
| kgunn | mzanetti: did that in order to keep automerging/building with new-scopes updates | 15:29 |
| kgunn | or somethign like that | 15:29 |
| kgunn | ...i can't rememebr...but i had a good reason last night :D | 15:29 |
| mzanetti | sure, makes sense... I just didn't know and pushed to yet another branch. but no probl, really | 15:29 |
| kgunn | i think it was conflicts maybe... | 15:30 |
| mzanetti | tsdgeos: also, this one breaks the greeter if the settings have a bad image path; http://paste.ubuntu.com/6931738 | 15:30 |
| tsdgeos | mzanetti: propose a merge :) | 15:31 |
| mzanetti | tsdgeos: so the background image in the shell is gone for good, right? | 15:32 |
| tsdgeos | mzanetti: the background image for the shell is the white paper | 15:32 |
| === dandrader|afk is now known as dandrader | ||
| kgunn | dandrader|afk: bad news my friend...double checked, using ppa's from silo build i referenced earlier, it still doesn't paste | 15:32 |
| kgunn | dandrader: can you try using ppa:ci-train-ppa-service/landing-006 on top of latest image | 15:33 |
| kgunn | (or image greater than 181) | 15:33 |
| kgunn | 181 has mir0.1.5 | 15:33 |
| dandrader | kgunn, hmmm, ok, will try that out | 15:34 |
| kgunn | dandrader: oh shit... | 15:34 |
| kgunn | nvmd | 15:34 |
| kgunn | i just checked...the ppa is empty! | 15:34 |
| dandrader | kgunn, what? | 15:34 |
| kgunn | wtf | 15:34 |
| dandrader | hahahaha | 15:34 |
| dandrader | what a relief :) | 15:35 |
| kgunn | hmmmm.... | 15:35 |
| tsdgeos | mzanetti: https://code.launchpad.net/~aacid/unity8/new-scopes-dash-background-opacity/+merge/206516 | 15:44 |
| tsdgeos | kgunn: greyback_: dandrader: i forced the "good" uitk and now i can't unlock the greeter anymore | 15:46 |
| tsdgeos | kgunn: greyback_: dandrader: do you mind if we revert the change the new-scopes branch until it works? | 15:46 |
| dandrader | tsdgeos, don't know what you're talking about | 15:48 |
| greyback_ | tsdgeos: what device? | 15:48 |
| tsdgeos | greyback_: my computer | 15:48 |
| greyback_ | tsdgeos: and you've the demo-stuff ppa installed on it? | 15:48 |
| tsdgeos | dandrader: i i move the greeter left and it keeps rebounding | 15:48 |
| tsdgeos | greyback_: yes | 15:48 |
| dandrader | tsdgeos, and that's with what unity8? | 15:49 |
| greyback_ | tsdgeos: ok, I just want to reproduce before reverting | 15:49 |
| tsdgeos | dandrader: the one that is in demo-stuff | 15:49 |
| tsdgeos | new-scopes branch | 15:49 |
| tsdgeos | ah wait | 15:50 |
| tsdgeos | maybe i forgot to recompiel? | 15:50 |
| tsdgeos | ah yes | 15:50 |
| dandrader | tsdgeos, well, uitk is unrelated to borked DragHandle/DDA (greeter unlock) | 15:50 |
| tsdgeos | if i recompile i get it in vertical :D | 15:50 |
| === alan_g|tea is now known as alan_g | ||
| tsdgeos | so yes, can we please get new-scopes to the working place it was before? | 15:51 |
| greyback_ | tsdgeos: yep, my bad, need to do a new build | 15:51 |
| tsdgeos | people is going to give me shit because of it | 15:51 |
| tsdgeos | and honestly i've had enough :D | 15:51 |
| greyback_ | tsdgeos: what branch of unity8? | 15:52 |
| tsdgeos | greyback_: new-scopes | 15:52 |
| dandrader | tsdgeos, I'm innocent! never touched that new-scopes branch | 15:52 |
| greyback_ | tsdgeos: checking... | 15:52 |
| tsdgeos | dandrader: well, but kgunn merged your stuff there | 15:52 |
| greyback_ | tsdgeos: would you please merge latest lp:~unity-team/unity8/rotate-n7 just to check? | 15:53 |
| tsdgeos | greyback_: that brings some sanity in :) | 15:54 |
| mzanetti | tsdgeos: ScopeTool.qml is not used in unity8, right? only if you run the standalone scopetool | 15:54 |
| tsdgeos | mzanetti: right | 15:54 |
| tsdgeos | greyback_: want me to push that? | 15:54 |
| greyback_ | tsdgeos: yeah please | 15:54 |
| mzanetti | tsdgeos: approved | 15:55 |
| tsdgeos | tx | 15:57 |
| kgunn | greyback_: do we need to change the unity8 recipe to merge lp:~unity-team/unity8/rotate-n7 | 15:57 |
| kgunn | into the unity8 package builds | 15:57 |
| kgunn | thot i did that.... | 15:58 |
| greyback_ | kgunn: IMO let's use all new-scopes from now on, else we'll end up with more conflicts/confusion | 15:58 |
| kgunn | greyback_: sure.... | 15:58 |
| kgunn | and good...i didn't do it | 15:59 |
| === gatox is now known as gatox_lunch | ||
| kgunn | dandrader: it workie! | 16:23 |
| kgunn | going to find a browser person to bother | 16:23 |
| dandrader | \o/ | 16:23 |
| kgunn | hey hey... dandrader it even pastes into notepad | 16:24 |
| tsdgeos | greyback_: how do we get the demo-stuff uitk version number to be bigger than the released one? | 16:28 |
| greyback_ | tsdgeos: already done, have kicked off a build | 16:28 |
| tsdgeos | cool! | 16:28 |
| tsdgeos | mzanetti: are you proposing a merge with that "invalid background thing"? | 16:29 |
| tsdgeos | karni: there? | 16:29 |
| dandrader | kgunn, nice. I recall i had problems trying to make that "paste" pop-up to show itself | 16:29 |
| kgunn | hey dandrader so i was able to copy text from cnet into the messaging app | 16:29 |
| kgunn | so... voodoo fix ? | 16:29 |
| tsdgeos | pete-woods: actually, now that i think, it can't be kate since that's not qt5 based, so it has to be something else | 16:30 |
| dandrader | kgunn, who knows, maybe I had a borked ubuntu-ui-toolkit installation or something.... | 16:30 |
| kgunn | dandrader: great work | 16:30 |
| tsdgeos | pete-woods: so either unity8 or assistant, those are basically the only qt5 apps i run afaik | 16:31 |
| kgunn | dandrader: i did find one thing you can't do...paste into the google search box | 16:31 |
| pete-woods | tsdgeos: it's got to be some qt application (4 or 5) or firefox, they are the only things still using dbusmenu | 16:31 |
| tsdgeos | pete-woods: but the backtrace said libdbusmenu-qt5, no? | 16:31 |
| pete-woods | it's probably not firefox, as I'd have noticed that by now | 16:31 |
| pete-woods | tsdgeos: that's just on the HUD end, not the client | 16:31 |
| tsdgeos | ahhhhh | 16:32 |
| tsdgeos | trueth | 16:32 |
| pete-woods | tsdgeos: I've found libdbusmenu-qt to be "somewhat twitchy" so far, not really looking forward to trying to figure this out | 16:34 |
| pete-woods | I have the feeling that the library has only really been extensively tested for exporting menus, not importing | 16:34 |
| mzanetti | tsdgeos: I was looking into it and it seems we've a copy of that error recovery inside the Greeter now and load the background image for the greeter from AccountsService | 16:37 |
| tsdgeos | mzanetti: may be, Saviq did some stuff in there | 16:38 |
| mzanetti | tsdgeos: so the question is actually why we still load the background from gsettings and not just set the jpg files to greeter.defaultBackground | 16:38 |
| tsdgeos | mzanetti: no idea tbh, let's just leave it for monday when Saviq is back :D | 16:38 |
| tsdgeos | Cimi: https://code.launchpad.net/~aacid/unity8/show_video_playback_screenshot_loading_indicator/+merge/206525 | 16:40 |
| Cimi | tsdgeos, k | 16:41 |
| Cimi | tsdgeos, I fixed the bug | 16:41 |
| Cimi | tsdgeos, not sure about the mouse are enabled | 16:41 |
| tsdgeos | Cimi: well, we don't show the play button, it'd be weird to actually play | 16:41 |
| Cimi | tsdgeos, we maybe still want to eat events? | 16:42 |
| Cimi | tsdgeos, ah right | 16:42 |
| Cimi | tsdgeos, maybe we want to change onclicked? | 16:42 |
| tsdgeos | Cimi: i can do that if you prefer | 16:42 |
| Cimi | tsdgeos, what do you think? | 16:42 |
| tsdgeos | Cimi: i am not sure, i think that ideally what i have should work, but not sure tbh | 16:43 |
| Cimi | tsdgeos, it works, but events are going through or not? | 16:43 |
| tsdgeos | yours is probably a bit safer even it is "less declarative" | 16:43 |
| greyback_ | damn, now why didn't you upload.... https://code.launchpad.net/~unity-team/+recipe/ubuntu-ui-toolkit-daily | 16:44 |
| tsdgeos | Cimi: they are | 16:44 |
| tsdgeos | Cimi: but they are also going thorugh in the header or text for example | 16:44 |
| tsdgeos | Cimi: so it should not be a problem | 16:45 |
| tsdgeos | no? | 16:45 |
| Cimi | dunno :D | 16:46 |
| mzanetti | tsdgeos: ack | 16:48 |
| tsdgeos | Cimi: well you didn't add a big mousearea catching clicks and doing nothing to PreviewTextSummary, no? | 16:48 |
| tsdgeos | so they go thoguth | 16:49 |
| tsdgeos | so i don't se why we should make them not go in this case | 16:49 |
| Cimi | k | 16:49 |
| mzanetti | kgunn: Just flashed a fresh image, added the ppa, upgraded and rebooted. Flicking through an app spread now! | 16:57 |
| kgunn | mzanetti: \o/ | 16:58 |
| kgunn | can't thank you guys enough | 16:58 |
| cwayne | mzanetti, with new-scopes too? | 16:58 |
| mzanetti | cwayne: yip yip | 16:58 |
| cwayne | :D | 16:59 |
| cwayne | mzanetti, what's the ppa? | 16:59 |
| mzanetti | its secret :P | 16:59 |
| mzanetti | cwayne: here you go: https://launchpad.net/~unity-team/+archive/phone-right-edge/+packages | 16:59 |
| cwayne | mzanetti, :P thanks | 17:00 |
| cwayne | mzanetti, would i need this and the demo-stuff ppa? | 17:05 |
| mzanetti | cwayne: no, only this | 17:06 |
| cwayne | mzanetti, tried, got this: E: Unable to locate package unity-api | 17:07 |
| cwayne | E: Unable to locate package unity-plugin-scope | 17:07 |
| mzanetti | cwayne: hmm... not sure where that comes from. I tested on a fresh image and it worked fine | 17:08 |
| cwayne | mzanetti, did you just do a dist-upgrade? i took the package list from the ppa description | 17:08 |
| Cimi | tsdgeos, was thinking | 17:09 |
| Cimi | tsdgeos, we can setup a wiki page | 17:09 |
| Cimi | tsdgeos, like https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 | 17:09 |
| Cimi | tsdgeos, already with the answers | 17:09 |
| mzanetti | cwayne: I just did apt-get upgrade | 17:09 |
| Cimi | tsdgeos, so we can easily copy paste | 17:09 |
| Cimi | :P | 17:09 |
| cwayne | mzanetti, ack, ill try that thanks | 17:09 |
| === gatox_lunch is now known as gatox | ||
| mzanetti | cwayne: and made sure I have this version of unity8: http://paste.ubuntu.com/6932240 | 17:10 |
| kgunn | cwayne: phone-right-edge is only for phone/n4....and demo-stuff will be for tablet/n10,n7 | 17:10 |
| cwayne | right | 17:11 |
| * tsdgeos EODs | 17:12 | |
| tsdgeos | tty on monday! | 17:12 |
| cwayne | mzanetti, hm, seems i still have the old unity-scope-click | 17:14 |
| cwayne | looks like it may need to be copied over from demo-stuff, not seeing it in phone-right-edge | 17:15 |
| mzanetti | kgunn: hmm.. looks like cwayne is right... the new scopes stuff in unity8 is loaded, but no backends seem to be loaded. | 17:20 |
| mzanetti | do we need some more packages in the ppa? | 17:21 |
| === dandrader is now known as dandrader|lunch | ||
| pete-woods | tsdgeos: I've got an experimental libdbusmenu-qt branch you could try here: https://code.launchpad.net/~pete-woods/libdbusmenu-qt/remove-busy-watcher/+merge/206538 | 17:45 |
| pete-woods | ah, EOD | 17:46 |
| karni | tsdgeos: I guews I'm late to reply to your ping | 17:50 |
| cwayne | mzanetti, so far it works with just adding the unity-scope-click from the demo-stuff ppa | 17:51 |
| cwayne | looks nice too :) good job | 17:51 |
| karni | hrm... I did change my nick to karni-holiday, but only on canonical irc | 17:51 |
| === alan_g is now known as alan_g|EOW | ||
| tsdgeos | karni: so i did suggest a different way to fix the resolution thing, did you see it? | 18:18 |
| * tsdgeos has to go | 18:21 | |
| * karni looks | 18:33 | |
| === karni is now known as karni-holiday | ||
| kgunn | mhr3: you around ? | 18:37 |
| karni-holiday | Cimi: Do you plan to look for better fix than this? https://code.launchpad.net/~unity-team/unity8/new-scopes-carousel-bounds-patch/+merge/206307 | 18:43 |
| karni-holiday | Cimi: If not, blocking on an awkward carousel behavior fix on tablet for MWC will not help :( | 18:44 |
| karni-holiday | Cimi: I'm leaving this with you guys. cwayne, please note that either my fix should go in to avoid that weird behavior on tablet for MWC demoing or Cimi is the only person that can help (he marked my MP as "Needs fixing", and I don't have a better fix). | 18:46 |
| === dandrader|lunch is now known as dandrader | ||
| ssweeny | mhr3, ping, do you know if it's possible to disable previews entirely for a new-style scope? i tried just returning null but that opens a blank preview | 19:04 |
| ssweeny | *returning null from the preview() function that is | 19:05 |
| karni-holiday | ssweeny: searched through a number of source files, can't find the binding glue, sorry | 19:05 |
| cwayne | ssweeny, worst case scenario couldn't we just have some text in the previews? | 19:07 |
| ssweeny | cwayne, probably, but for some of them it makes no sense to have anything | 19:08 |
| Cimi | karni-holiday, we have to increase the number of items | 19:28 |
| Cimi | karni-holiday, this is a regression | 19:28 |
| Cimi | we have 0 regressions policy | 19:28 |
| Cimi | we need to switch back to grid for less than 7 items | 19:29 |
| Cimi | and provide a fix for 7-10 | 19:29 |
| Cimi | kgunn, who is management here? https://code.launchpad.net/~unity-team/unity8/new-scopes-carousel-bounds-patch/+merge/206307 | 19:30 |
| Cimi | solution is | 19:31 |
| Cimi | first we revert the patch that reverts to grid when there are 7 or less items | 19:32 |
| Cimi | second we work on improving drag over bounds | 19:32 |
| karni-holiday | Cimi: please follow up with cwayne. | 19:34 |
| karni-holiday | you dont revert the patch to item count. you'll break more than the carousel. | 19:34 |
| karni-holiday | anyway, not here. sorry. | 19:35 |
| Cimi | karni-holiday, we have to! | 19:38 |
| Cimi | we don't add regressions | 19:38 |
| Cimi | the merge should have been blocked at review | 19:39 |
| karni-holiday | you want to revert to <=6 limit in new-scopes? | 19:39 |
| Cimi | I want to have grid till 6 items | 19:40 |
| karni-holiday | ok, plewse raise this with cwayne and achiang asap | 19:40 |
| Cimi | as it was before | 19:40 |
| Cimi | if we manage to fix the carousel with 5 items, we can reapply the item count | 19:40 |
| karni-holiday | That is not possie without consulting with ^ | 19:40 |
| Cimi | it should | 19:40 |
| Cimi | we can just change code in unity | 19:40 |
| Cimi | isn't it? | 19:40 |
| karni-holiday | Im on my pho e, please talk to cwayne. | 19:41 |
| karni-holiday | *phone | 19:41 |
| Cimi | ok | 19:41 |
| Cimi | cwayne, ping me when you're back | 19:41 |
| karni-holiday | thank you, Cimi | 19:41 |
| Cimi | karni-holiday, I can work on this next week, but I doubt I can make the carousel work with 1-5 elements | 19:41 |
| Cimi | karni-holiday, that's why we have to have grid for few elements... | 19:42 |
| Cimi | and UX-wise, is also way better | 19:42 |
| Cimi | a carousel with 1-5 elements does not make sense | 19:42 |
| Cimi | makes the interface slower to operate | 19:42 |
| Cimi | what I can possibly fix is the drag over bounds | 19:43 |
| Cimi | but I have to dig it properly on monday | 19:43 |
| ssweeny | Cimi, are we talking about <7 elements on tablet profile or in all cases falling back to grid? | 20:40 |
| Cimi | ssweeny, on tablet should be 7 or more | 20:41 |
| Cimi | ssweeny, phone maybe 5 | 20:41 |
| kgunn | Cimi: you need anything from me? | 20:41 |
| ssweeny | Cimi, in that case i agree. 5 looks pretty reasonable to me on a phone but on a 10-inch tablet in landscape it would be weird | 20:42 |
| Cimi | kgunn, yep | 20:43 |
| Cimi | kgunn, I'm trying to understand what's going on, what happened | 20:43 |
| kgunn | heh | 20:43 |
| Cimi | kgunn, with the carousel minimum shown items | 20:43 |
| Cimi | kgunn, with oren we agreed we wanted filter grid for 6 or less | 20:44 |
| kgunn | Cimi: so big picture...everything is ok atm, building, running from the demo-stuff & phone-right-edge ppa | 20:44 |
| kgunn | we need to focus on the tweaks/bug fixes now | 20:44 |
| kgunn | landing is somewhat secondary....primary is getting fixes into branches feeding those ppas | 20:45 |
| kgunn | ...so | 20:45 |
| kgunn | for this carousel issue...we should just use our brains and pick the right limit | 20:45 |
| kgunn | Cimi: can you determine screen size from w/in carousel ? | 20:46 |
| kgunn | then set your limit based on that ? | 20:46 |
| kgunn | so you could keep your limit 6 on the phone...8 on the tablet ? | 20:47 |
| * kgunn wonders if i'm even tracking with the issue properly | 20:47 | |
| cwayne | Cimi, i think we'd be okay with 5 on the phone, 7 on the tablet | 20:49 |
| cwayne | if that works for you :) | 20:49 |
| ssweeny | kgunn, we've been working with 5-item carousels on the phone and that's been going well for us | 20:49 |
| ssweeny | yeah what cwayne said :) | 20:49 |
| kgunn | makes sense to me | 20:50 |
| kgunn | Cimi: possible ? | 20:50 |
| kgunn | realizing its past your beer time Cimi | 20:50 |
| Cimi | kgunn, nope | 20:50 |
| Cimi | kgunn, I was on the loo :) | 20:50 |
| Cimi | hah | 20:50 |
| Cimi | anyway | 20:51 |
| Cimi | I'll have a look next week | 20:51 |
| kgunn | Cimi: ok, please do...surely from the sdk somewhere you can get dpi and pixel width of the parent... | 20:51 |
| Cimi | on monday morning I'll try to see if there's a clever way to detect | 20:51 |
| cwayne | don't we just check if units.gu > 80 | 20:51 |
| Cimi | i depends on rotation... I'll ask next week | 20:52 |
| Cimi | best to know with the rest of the team which is the proper way of doing it | 20:52 |
| cwayne | oh, i didn't realize we were gonna have shell rotation | 20:52 |
| cwayne | cool! | 20:52 |
| Cimi | cwayne, not sure if it works yet :) | 20:52 |
| Cimi | but at least I now have the steps | 20:53 |
| Cimi | glad we agree on having the grid with less items | 20:53 |
| Cimi | I have to go, girlfriend is upset | 20:54 |
| Cimi | enjoy weekend guys! | 20:54 |
| cwayne | you too Cimi! | 20:54 |
| === dandrader_ is now known as dandrader | ||
| === salem_ is now known as _salem | ||
| === jhodapp is now known as jhodapp|afk | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
