/srv/irclogs.ubuntu.com/2013/12/10/#ubuntu-unity.txt

=== thomi_ is now known as thomi
SaviqMirv, hey, do you think there will be a rc2 of Qt 5.2?08:12
SaviqMirv, also, do you have branches for all the packages in qt5-beta2 somewhere?08:13
=== iahmad is now known as iahmad|afk
tsdgeosSaviq:08:22
tsdgeosHi all,08:22
tsdgeosWe have new packages available for your testing & verification in http://download.qt-project.org/snapshots/qt/5.2/5.2.0/2013-12-09_204/08:22
tsdgeossubject: Qt 5.2 final candidate packages available08:23
Saviqtsdgeos, right ;)08:23
tsdgeosdoesn't look like there'll be an RC208:23
Saviqtsdgeos, "final candidate" sounds very much like rc2 ;D08:23
tsdgeosmaybe08:24
tsdgeosi just think it's packages we'll flip the switch tomorrow if noone complains about08:24
tsdgeosand not a more formal RC208:24
tsdgeosbut i'm still not at 100% :D08:24
Saviqtsdgeos, ;)08:24
tsdgeosso may not be reading it correctly08:24
Saviqtsdgeos, good news, at least, is that both crashes we've encountered (unity-api and my test with JSON parsing)08:25
Saviqtsdgeos, are gone with release08:25
MirvSaviq: no formal rc release I think (http://download.qt-project.org/development_releases/qt/5.2/). all packaging is under https://code.launchpad.net/~kubuntu-packagers/kubuntu-packaging08:25
tsdgeosSaviq: awesome08:25
MirvSaviq: that's great news.08:26
SaviqMirv, seen there's a "final candidate" tarballs around?08:26
MirvSaviq: tsdgeos: I have set around 50+ recipe builds to qt5-beta2, a lot of build failures in some packages, at least something has changed regarding running tests (opengl tests are tried to be run in gsettings-qt on builders)08:26
MirvSaviq: sure, there are tarballs, but no rc release tarballs08:27
SaviqMirv, right, only snapshots08:27
MirvSaviq: yeah, date coded snapshots which all are called "final candidate builds"08:28
MirvSaviq: it seems the patched qtdeclarative is now there, so I'm clicking rebuild for unity-api08:28
SaviqMirv, next time we need to make it so we have a recipe just following the release branch ;)08:28
SaviqMirv, cool08:28
MirvSaviq: the problem with automated Qt packaging is that during development the files tend to change all the time in at least qtbase/qtdeclarative, so *.install files are outdated, and same goes for symbols files (which I've now mostly temporarily removed simply until final release)08:29
SaviqMirv, yeah, I know, just a dream there :)08:30
MirvSaviq: I'll probably cc: you later today with my tinkerings around the opengl es on x86..08:30
SaviqMirv, great, thanks08:30
Saviqtsdgeos, there's like 6 newer builds in here http://download.qt-project.org/snapshots/qt/5.2/5.2.0/ - does it go regardless of the lack of changes?08:31
Saviqtsdgeos, i.e. is 204 that you linked different from 209 do you think?08:31
tsdgeoshmm08:32
Saviqno commits in qtdeclarative at least since 5.1208:33
tsdgeosi guess they might change08:33
tsdgeosno idea how to find out which one though08:34
tsdgeosother than git logging08:35
tsdgeosCimi: did i convince you in https://code.launchpad.net/~aacid/unity8/search_history_pointer_correct_pointer_position ?08:39
tsdgeosCimi: also do you have time for https://code.launchpad.net/~aacid/unity8/dash_show_home_no_index/+merge/197714 ?08:40
SaviqMirv, segfault :/08:40
SaviqMirv, built fine here yesterday with that patch...08:40
Mirvso it seems :(08:43
=== iahmad|afk is now known as iahmad
SaviqMirv, hum08:46
SaviqPatch correct_geometry_for_2x_pixmaps.patch does not apply (enforce with -f)08:46
SaviqMirv, that's on freshly branched lp:~kubuntu-packagers/kubuntu-packaging/qtdeclarative-opensource-src08:46
SaviqMirv, how could it build, then...08:46
SaviqMirv, <facepalm>08:47
SaviqMirv, it's only the packaging branch :)08:47
MirvSaviq: yes, that's still how all Qt packaging is handled also in Debian :) wget .orig.tar.xz from qt5-beta2 PPA09:05
SaviqMirv, yeah, never did much with merge-style packaging09:05
SaviqMirv, but managed my way around it09:05
MirvSaviq: btw regarding qtwebkit 5.2, add_experimentalDevicePixelRatio.patch is again disabled since it didn't apply.. I guess rsalveti or someone should look at it09:06
MirvSaviq: interesting. there was an autobuild of webbrowser-app in qt5-beta2 with the latest 5.2 fix, and it seems it also segfaults on i386 and armhf, but not on amd64 when running tests09:07
SaviqMirv, right, sounds like unity-api then09:07
SaviqMirv, aah, so it never segfaulted on amd64 did it?09:15
SaviqMirv, I'm on amd64 <facepalm/> :/09:15
SaviqMirv, anyway - you might find http://bazaar.launchpad.net/~unity-team/kubuntu-packaging/qtdeclarative-opensource-src-5.2-rc2/revision/95 useful - the patch conflicted with the 209 snapshot, so once there's a proper release, that patch should apply fine09:17
MirvSaviq: :) yep, never segfaulted on amd64.09:23
Mirvthanks for that, I'll fetch that already so that I don't forget about it, because maybe I won't build another qtdeclarative until the final is out09:23
SaviqMirv, cheers09:24
Mirvmerged and pushed09:24
Saviqmzanetti, pushed a fix to new-scopes-integrate-card09:30
Saviqmzanetti, apparently 38/2 != 18.5, but 19!09:31
mzanetti:D09:31
mzanettiSaviq: cheers. will give it another run09:31
mzanettiah... wait09:31
mzanettidid you see the other one?09:31
Saviqmzanetti, looking09:32
Saviqmzanetti, that you mean https://code.launchpad.net/~saviq/unity8/new-scopes-integrate-card/+merge/197930/comments/459628 ?09:32
Saviqmzanetti, that's what I just fixed09:32
Saviqmzanetti, doesn't segfault when passing...09:32
mzanettiyip. all passing here09:32
mzanettiSaviq: ok. good to go I'd say09:32
Saviqmzanetti, hit it!09:32
mzanettiSaviq: happroved09:33
Saviqmzanetti, thanks /me merges09:33
nic-doffaySaviq, got any tasks floating around?09:34
Saviqnic-doffay, yeah, review https://code.launchpad.net/~bfiller/unity8/fix-password-autocaps/+merge/198319 please?09:35
nic-doffaySaviq, cool09:35
Saviqnic-doffay, test on the phone please09:35
nic-doffaySaviq, yep09:35
mhr3there's no default implementation of qhash for strings?09:36
mhr3as in std::string09:36
Saviqmhr3, I'd imagine they want QString...09:37
Saviqtsdgeos, feelin' better btw?09:38
tsdgeosyeah09:38
tsdgeosnot 100%09:38
tsdgeosbut rest really helped09:38
mhr3Saviq, they do, but i dont09:38
tsdgeosmhr3: then define your qHash operator for std::string09:38
Saviq09:38
Saviqthat just wraps it into QString ;)09:38
tsdgeoslol09:38
tsdgeosslow++09:38
mhr3think i'll just go for std::map09:39
Saviqor that09:39
mhr3oh wait, there's unordered_map now09:39
mhr3that should have the algorithmic complexity as qhash, no?09:40
mhr3the same*09:40
Saviqmhr3, card integration landed09:40
tsdgeosmhr3: yeah09:40
Saviqmhr3, yeah sounds sane09:41
mhr3Saviq, coolio, lp:unity8? :)09:41
Saviqmhr3, weell now let's not get ahead of ourselves ;)09:41
mhr3Saviq, would be nice to get it in there ;)09:41
Saviqmhr3, there's too many FIXMEs in there still09:41
Saviqmhr3, that need to go before getting into trunk09:41
tsdgeosSaviq: want me to add the code for the case in which you change the width of a column in a vJournal with items on it and so i need to force resize on them?09:41
tsdgeosor we're not planning to do that and i can leave a TODO?09:42
mhr3Saviq, so chop, chop, fix them ;P09:42
Saviqtsdgeos, I think it'd be good, as we might scale the window without refreshing the model09:42
tsdgeosok09:42
tsdgeosdoing09:43
Saviqmhr3, s/icon/art/ anyone?09:43
mhr3Saviq, got it in a branch, but adding real tests in the same branch... so it's... "fun"09:43
Saviqmhr3, OMG REAL TESTS?09:43
mhr3RIGHT?!09:44
Saviqmhr3, btw, so if you export the FORCE_NEW_SCOPES=1, Unity 0.1 becomes the same as Unity 0.2?10:00
mhr3Saviq, pretty much, yea10:01
Saviqmhr3, so if we wanted to merge new-scopes into unity8, how could we differentiate between the two when selecting a renderererererer?10:01
Saviqmhr3, we could just go for .hasOwnProperty("template") or so10:02
mhr3Saviq, well the new scopes give you the an object for category.renderer, old ones just a string :)10:02
=== rachelliu_ is now known as rachelliu
Saviqmhr3, yeah, which should not be "renderer" btw, but "template" :P10:03
mhr3Saviq, i thought it wouldn't like undefined10:03
Saviqmhr3, it can deal fine with undefined10:03
mhr3but sure, if a bit of code is added to handle it10:03
Saviqmhr3, or we can make it deal with it10:03
Saviqmhr3, yeah, that10:03
mhr3Saviq, but yea, the ultimate idea is to have all the code sitting in trunk but disabled by default10:04
mhr3makes dev on it easy10:05
Saviqmhr3, I'm just worried that will hold us back too much10:05
Saviqmhr3, like the above ↑ you'd name it template, but didn't want to break old stuff, so went for renderer10:06
mhr3right10:06
mhr3there are pros and cons10:06
Saviqmhr3, and then we'll build all kind of bolierplate code to make it work both ways10:06
mhr3as always :)10:06
Saviqand then after we flip the switch we'll have to rip that all out again10:06
mhr3Saviq, ultimately in unity8 pretty much all the changes will be limited to the new card renderer10:07
Saviqmhr3, that is true, but that's the thing - we could rip out half of the code there already10:07
mhr3this renderer <> template snafu doesn't sound like a big deal10:08
Saviqmhr3, sure, that's just one example10:08
greybackMirv: hey, are the QtQuick Controls packaged in the Qt5.2 PPA?10:08
mhr3Saviq, imo the cost of keeping compability with both isn't that high10:09
mhr3at least not right now10:09
mhr3things will get more fun once we do previews and stuff10:09
mhr3for me it's the classical let's not diverge from trunk unless we have to10:10
Saviqmhr3, yeah of course - but that's what I mean is holding us back, for which I'm not sure we have time - or want it to10:11
* tsdgeos finds a bug in the verticalJournal implementation10:12
tsdgeosdamn unittests!10:12
mhr3Saviq, i can work eitherway really, it's a have-a-look-dear-manager feature :P10:13
Mirvgreyback: no, sorry. raising on my todo list.10:14
mhr3Saviq, so if you want to do gutting of code from the branch, feel free to10:14
mhr3Saviq, i don't think that's worth it, will just complicate merging10:15
greybackMirv: it's not critical for me, so no need to re-prioritize just for me10:15
mhr3i mean, it should be right before merging to trunk10:15
Mirvok, only slightly notching it up then ;)10:18
dednickmhr3: ping10:23
mhr3dednick, pong10:23
Saviqmhr3, yeah10:25
dednickmhr3: howdy. just looking back into the dee thing i left off. I agree with not linking having a new base class.10:26
dednickmhr3: although i may have to change the DeeModelface if we want to do it properly10:26
dednickmhr3: so that we can get the multiplexer from the filter model backend.10:27
mhr3dednick, what do you have in mind? cause yea, i'm not sure what would be the nice way to do it10:27
mhr3dednick, also, how was capetown?10:28
dednickmhr3: nice and warm. although i was only there for 2 days. Hiked table mountain.10:28
mhr3dednick, oh? just 2 days? did the city frighten you or something? :)10:30
dednickmhr3: hehe. no, just had other places to go :) Want to spend more time there next time around though. Really good climbing on table mountain.10:31
mhr3did you do actual climbing there as well?10:32
dednickmhr3: no, just some hiking.10:32
SaviqMirv, .install updates for qtdeclarative: http://bazaar.launchpad.net/~unity-team/kubuntu-packaging/qtdeclarative-opensource-src-5.2-rc2/revision/9710:32
mhr3dednick, will want some more tips, heading there as well in a month ;)10:34
tsdgeosSaviq: https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/1371/?#showFailuresLink still happening?10:34
Saviqtsdgeos, that's new actually10:34
tsdgeosis it?10:34
Saviqtsdgeos, if you look at the videos10:34
Saviqtsdgeos, unity8 never shows up10:34
tsdgeosok10:34
Saviqtsdgeos, https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/1371/artifact/results/autopilot/artifacts/10:34
tsdgeosmay be that i broke something in the merge10:34
Saviqtsdgeos, I don't think so, seen that for other branches yesterday10:35
dednickmhr3: awesome. you going for sprint?10:35
mhr3dednick, yep10:35
tsdgeosSaviq: ah10:35
Saviqtsdgeos, "init: unity8 main process (11206) killed by TERM signal"10:35
Saviqtsdgeos, is what ends up in .xsession-errors10:35
Saviqtsdgeos, no idea where that comes from :/10:35
tsdgeosSaviq: so we're crashing again?10:36
Saviqtsdgeos, TERM, not crashing10:36
Saviqtsdgeos, and there's no .crash file around10:36
tsdgeosright10:36
Saviqtsdgeos, it's like it's killed before it really starts10:36
tsdgeosCimi: so what comment would you add?10:36
tsdgeosCimi: are you going to ask for a comment in all the lines that do something right?10:36
dednickmhr3: lucky! You must hike up a route called India Venster. book through http://www.hiketablemountain.co.za/ . They are really nice guides.10:36
Saviqgreyback, ↑10:37
tsdgeosbecause i am a bit against literate programming tbh10:37
MirvSaviq: thanks10:37
Saviqmhr3, you staying past the sprint?10:37
mhr3Saviq, nope, but will be there the whole weekend before it10:38
Saviqmhr3, cool, we're staying a few days past with greyback10:38
mhr3Saviq, and arriving on sunday?10:39
Saviqmhr3, yes10:39
greybackmhr3: yep10:39
dednickcan I come?10:39
dednick:)10:39
dednickits cold here10:39
mhr3i'm flying there on friday, so should have most of saturday and whole sunday free10:39
mhr3a few more designers are doing that as well10:40
RAOFYou guys have *another* sprint?10:41
SaviqRAOF, design one10:41
SaviqRAOF, we're just there to say NO10:41
Saviqfrom time to time10:42
RAOF:)10:42
Saviqand get fired10:42
RAOFAll of the things we saw last week looked perfectly fine from Mir's point of view (ie: they didn't involve us doing anything different ☺).10:42
RAOFAs the people implementing them you may have other ideas :P10:42
SaviqRAOF, yes, there's kgunn that will be saying NO for you guys ;)10:43
mhr3nice summary :)10:46
mhr3dednick, the link looks good, will check it out :)10:48
mhr3dednick, anyway back to dee, i was thinking about something like - http://paste.ubuntu.com/6550461/10:48
mhr3dednick, the nice thing about it, is that the usage of a FilterModel is just an impl detail10:49
mhr3and there could even be a "demultiplex-changeset" property10:50
mhr3default TRUE10:50
mhr3the not-so-nice thing is that doing things a bit more lazily would need more thought10:53
mhr3so for example the impl wouldn't keep a filtermodel unless you called _get_bin for it yet10:54
mhr3s/yet//10:54
Cimitsdgeos, link to bugreport10:55
Cimitsdgeos, on that line10:55
tsdgeosCimi: why do you want a link to a bugreport that has nothing to do with the code?10:55
tsdgeosthe code is not a workaround10:55
tsdgeosthe code is correct code that works by itself10:55
tsdgeossure it happens that the old code should have worked and does not10:56
tsdgeosand that is a bug10:56
Cimitsdgeos, ah10:56
tsdgeosbut the new new code is correct10:56
tsdgeosas i already mentioned in the comment i made10:56
Cimiok so don't add comment10:56
CimiI thought was a workaround10:56
seb128Cimi, hey, could you review/ack the change on https://code.launchpad.net/~larsu/overlay-scrollbar/hide-scrollbars-before-unmapping/+merge/197415 ?10:58
Cimiseb128, I wasn't a fan of that but I'll have a look10:59
tsdgeosCimi: https://code.launchpad.net/~aacid/unity8/search_history_pointer_correct_pointer_position/+merge/197423/comments/45754610:59
seb128Cimi, if you are not a fan please find another way to fix bug #1086494, those warnings are annoying10:59
seb128Cimi, thanks10:59
ubot5bug 1086494 in overlay-scrollbar "GtkScrolledWindow is mapped but visible child GtkScrollbar is not mapped" [Undecided,In progress] https://launchpad.net/bugs/108649410:59
Cimiseb128, yeah will think about it10:59
seb128Cimi, thanks10:59
Cimiseb128, it's weird11:00
Cimiseb128, dunno why mapped children should be mapped if parent is mapped :-\11:00
Saviqmzanetti, what we talked about yesterday https://qt.gitorious.org/qt/qtdeclarative/commit/75a6f86f685f1a5ce6cb91212641fe446a37be2e11:01
seb128larsu, ^11:01
Cimiseb128, if that's the warning, you should unman not hide11:01
Cimi*unmap11:01
mzanettiSaviq: nice one :)11:01
Saviqmzanetti, we need a qmltryrunner that does that11:01
seb128Cimi, well, I'm running larsu's fix for a week and it made those warning go away without side effect11:02
seb128but maybe11:02
seb128I'm not picky on the solution11:02
Cimiseb128, hide unmaps and change visible flag as well11:02
nic-doffay_Saviq, is this still the same write issue as before? (Getting this on run_on_device -s) W: Not using locking for read only lock file /var/lib/dpkg/lock11:02
seb128I just want gedit to stop spamming stdout11:02
Cimiseb128, unmaps just unmaps but does not change visible gtk flag11:02
Saviqnic-doffay_, very possible, did you make your device writable?11:02
seb128ok11:02
Cimiseb128, better to use unmap if possible11:02
seb128let's way for larsu to comment11:02
seb128he understand those stuff better than me11:03
larsuCimi: unmap is not enough, you'd also need to unrealize11:03
nic-doffay_Saviq, no I assumed that step wouldn't be required...11:03
mzanettiSaviq: you mean exporting the QTestRootObject?11:03
nic-doffay_What's the command?11:03
larsuCimi: that's what the warning is about11:03
Saviqnic-doffay_, it alwaysis11:03
Saviqnic-doffay_, building unity8 on the device requires installing packages on the phone - so you need writable11:03
Saviqmzanetti, yeah11:03
Saviqmzanetti, although... that might mean the tests will actually run under qmlscene, too ;)11:04
mzanettiSaviq: hmm... but shouldn't happen in the QtTest import?11:04
Saviqmzanetti, oh well, right11:05
Saviqmzanetti, it's interesting why they'd do it in the runner and not the plugin indeed11:05
Cimilarsu, it's about map, not realize11:05
mzanettiSaviq: not sure what I'm missing... but I think this commit does it in QtTest11:06
mzanettiso I'm not sure why exactly it fails11:06
mzanettiunless we have a version where this commit is just not in yet11:06
Saviqmzanetti, we do11:06
Saviqmzanetti, right, so... more investigation to be had11:07
* Saviq finds it confusing that libqt5declarative5 comes from qtquick1....11:09
larsuCimi: I think it complains about realize when only unmapping and remapping (I vaguely remember trying that first)11:09
larsuCimi: feel free to fix it differently, but I don't want to spend more time on it, as the solution I proposed seems to work fine11:09
seb128larsu, Cimi: bug #1086494 (the title has it)11:09
ubot5bug 1086494 in overlay-scrollbar "GtkScrolledWindow is mapped but visible child GtkScrollbar is not mapped" [Undecided,In progress] https://launchpad.net/bugs/108649411:09
larsuand makes seb128 happy :)11:09
dednickmhr3: why are there void func pointers on the end of interfaces in dee?11:10
larsuseb128: ya, I think I tried unmapping-only first and then it complains about realize11:10
dednickmhr3: is it byte alignment?11:10
Saviq/food11:10
mhr3dednick, it's for extending the interface without having to break the abi11:10
seb128larsu, +1 on not spending more time on it, that solution works, if Cimi has a better one I guess you are fine commenting on it though?11:11
dednickmhr3: ah, so add a function and remove a void?11:11
mhr3dednick, yep11:11
dednickmhr3: smartypants11:11
Cimiseb128, larsu I just don't see the point why we should change visible gtk flag for that11:11
mhr3dednick, seen the pastebin?11:11
CimiI guess we can play with realise and map as well11:11
mhr3dednick, that wouldn't need changes to DeeModel itself11:12
Cimiif we call unrealise, we call unmap11:12
seb128Cimi, can you propose a patch with what you describe?11:12
larsuCimi: okay, please do that, then11:12
dednickmhr3: ah, no i hadnt seen it.11:13
Cimiseb128, can do later11:13
seb128thanks11:13
mhr3dednick, so stop ignoring me! :P11:13
nic-doffay_Saviq, yeah it's working now, that's weird though because usually I haven't had any write issues after a flash.11:14
dednickmhr3: so basically just keep a static map of backend->Demultiplexer?11:15
mhr3dednick, not exactly, this allows you to even have multiple demultiplexers on top of the same model11:16
mhr3dednick, and you'd get the individual FilterModels by calling the _get_bin() method11:17
mhr3dednick, eh yea, i'm missing self ptr in the get_bin()11:17
=== rachelliu_ is now known as rachelliu
dednickmhr3: um, so how do we use the same demultiplexer across multiple filter models if it's not using the same one for the given backend?11:21
dednicksince we're not exposing DeeDemultiplexer to unity anymore that is.11:23
mhr3dednick, since the usage of FilterModel is just an impl detail, it's up to the Demultiplexer itself to ensure that the models(bins) have proper data11:23
mhr3dednick, and eh, the idea is that the Demultiplexer itself would be a public class in the lib11:24
mhr3but it's a generalized data demultiplexer, not just a changeset-* signals demultiplexer11:25
dednickmhr3: so we're still creating the demultiplexer in unity?11:25
mhr3yes11:25
mhr3and using it to get the per-category models using its get_bin()11:26
dednickmhr3: then what is the point of changing it? who cares if it's generalised?11:26
mhr3it's just an idea11:27
dednickmhr3: "on the other it's very specific functionality which is now exposed as a base dee class" :)11:27
dednickmhr3: i thought the idea was not to expose.11:28
mhr3splitting data from one model into multiple is a valid use case11:28
mhr3and the demuliplexer seems like a correct way to do it11:28
mhr3what i didn't like about the previous approach is that it was too loose - just demultiplexing the signals, and having to manually "tag" each model that should use it11:29
dednickmhr3: i c11:30
mhr3dednick, that all being said, maybe you have a better idea with the change to DeeModel? i'm not sure what exactly you want to do there11:31
dednickmhr3: i'm still unclear as to how the FilterModel knows it has a demultiplexer if it isn't told.11:31
mhr3dednick, fwiw the usage of filtermodel is optional, it's up to the demultiplexer to split the data, it could just as well create new DeeSequenceModels and maintain the rows itself (using the bin_func)11:33
mhr3of course that would be silly and expensive :)11:33
mhr3but the point remains, the split is up to it, not necessarily the submodel11:34
mhr3and since this will be inside dee, you could just add internal methods to filtermodel11:35
=== _salem is now known as salem_
dednickmhr3: i'm still confused... How would we create the demultiplexer in unity (using dee_multiplexer_new) and have the BinFunc in dee if it's internal to FilterModel.11:54
dednickmhr3: you are very confusing...11:54
mhr3dednick, would be easy with a whiteboard, how come you're not at the office?11:55
mhr3and how come i'm not at the office? :P11:55
mhr3dednick, the bin func is something the user of the demult has to provide11:56
mhr3otherwise it'd be pretty useless11:56
dednickmhr3: right. but then what are the internal methods in FilterModel?11:56
dednickmhr3: because i'm lazy11:56
dednickmaybe i'll just leave this and come in tomorrow11:57
mhr3dednick, dunno really what the internal method would be, whatever is needed to make it work :)11:57
mhr3afaict it might not even be necessary to add an internal method11:58
dednickmhr3: lol. ok11:58
mhr3dednick, but agreed, let's postpone till tomorrow11:59
mhr3i'll have a deeper look at it meanwhile :)11:59
nic-doffay_Saviq, want to top approve this? The fix is working for me.12:07
tsdgeoserrrrr12:07
Saviqnic-doffay_, sure, can you add a "needs-ap-test" to the bug pleaes12:07
Saviqplease, even12:07
tsdgeosun errrr12:07
Saviqtsdgeos, ?12:07
Saviqtsdgeos, -12:07
Saviqdednick, added more data to https://bugreports.qt-project.org/browse/QTBUG-3435112:13
Saviqdednick, btw, there's "provide missing info" at the top, that changes the status back from NEEDINFO to REPORTED12:13
Saviqdednick, it might slip through the cracks otherwise12:13
dednickSaviq: ah. thanks12:14
Saviqdednick, btw, https://code.launchpad.net/~nick-dedekind/unity8/StrFTimeFormatter/+merge/192343 is correct still, is it? meaning we haven't decided on anything that would make this not needed?12:15
tsdgeosCimi: mzanetti: we broke the delegateRange stuff when merging the dashrenderer stuff12:17
Saviqdednick, only question there - will this work fine with encoding? as in does g_date_time_format cough up UTF8 - and if so, shouldn't we go for QString::fromUtf8?12:17
Cimidelegaterange?12:17
mzanettimeh12:17
tsdgeosdelegateCreationBegin12:18
mzanettireally?12:18
tsdgeosyep12:18
tsdgeosit's assigned in the wrong direction12:18
tsdgeosnever gets to the filter grid12:18
Saviq:/12:18
tsdgeosi guess i'll go over all the changes from that merge12:19
tsdgeosbecause it's broken lots of things12:19
tsdgeosso a post review won't hurt12:19
Saviqtsdgeos, list down what test could be added please12:19
dednickSaviq: my opinion is that it's the correct way to do it. But yes, should be using fromUtf812:19
tsdgeosSaviq: it's hard to test what test could be added12:20
tsdgeosbecause the test would be "make sure the item of the list gets that assigned"12:20
tsdgeosand that is happening12:20
Saviqtsdgeos, I understand12:20
tsdgeosis just that the item is not correctly passing that to its children12:20
tsdgeoswhich is an implementation detail12:20
Saviqtsdgeos, just when going over it again - try to see if there's anything else worth testing12:20
tsdgeosi.e. we could not have had that test before the change because the change is that introduced it12:20
Saviqtsdgeos, not necessarily that particular thing12:20
tsdgeossure12:21
Saviqlarsu, dednick, where does the time format come from, btw? locale? are the choices hardcoded in the indicator?12:21
Saviqlarsu, I mean for the datetime indicator, sorry for the lack of context12:22
larsuSaviq: don't know, let me check12:23
Saviqnic-doffay_, you can add tags to bugs, just below the description12:23
Saviqnic-doffay_, this way you can then find all of them with that tag by: https://bugs.launchpad.net/bugs/+bugs?field.tag=needs-ap-test12:24
larsuSaviq: ugh, right now datetime sends a string :(12:24
larsucontaining the time itself12:24
Saviqlarsu, do you know about the messaging indicator then?12:25
Saviqlarsu, I'm looking at https://code.launchpad.net/~nick-dedekind/unity8/StrFTimeFormatter/+merge/19234312:25
larsuSaviq: messaging indicator sends unix timestamps12:26
Saviqlarsu, and the format?12:26
larsuSaviq: I don't think they send it12:26
Saviqhrmpf /me is lost12:27
larsuwhy?12:27
Saviqwhy would we need the TimeFormatter then12:27
Saviqinstead of using Qt's internal formatting based on locale12:27
Saviqas well, I assume LC_TIME is what it's for12:27
larsuya, it changes the time when the timezone changes12:28
larsualso (in the future (maybe)) when coming back from suspend12:28
Saviqlarsu, sure, but that has nothing to do with the formatting12:28
Saviqlarsu, as long as we take the timestamp12:28
Saviqand format it according to LC_TIME12:28
Saviqthat should do12:28
Saviqdednick, thoughts ↑?12:29
Saviqif nothing gives us an explicit time format12:29
dednicklarsu: datetime sends x-canonical-time & x-canonical-time-format12:29
nic-doffay_Saviq, cool12:29
larsuSaviq: you want the time string to change when the timezone changes12:29
larsuthis is all that the Timeformatter thing does12:29
dednickwhich is unit time12:29
dednick*unix12:30
larsudednick: ah, datetime doesn't send strings anymore? It does on my desktop...12:30
Saviqdednick, right, so I'm trying to find out where does the format string come from :)12:30
dednicklarsu: i mean for appointments12:30
Saviqdednick, if it comes from locale12:30
Saviqand well - if it doesn't... shouldn't it?12:31
dednickSaviq: it's hardcoded12:31
dednickSaviq: in appointments12:32
dednickSaviq: well, appointments have this funky format where it can be daily, or single, and changes based on proximity to event.12:33
Saviqdednick, right, I'm just wondering if it's the right thing to do12:33
Saviqdednick, the proximity thing should be system-wide12:34
Saviqdednick, so SDK-ish12:34
dednickSaviq: i'm sure design would tell us if it wasnt. :)12:34
dednickit's same i desktop12:34
Saviqdednick, wherever you wanted "human readable time", you'd just use a time formatter from the SDK12:34
Saviqdednick, and tell it to do smart things12:34
Saviqdednick, doing it in the indicator seems weird12:35
larsuSaviq: ya, I could totally see this being in the sdk12:35
Saviqdednick, we should just get a timestamp from them12:35
dednickSaviq: right. so basically what this thing is, but without the format string12:35
Saviqdednick, well, more, if we said it's meant to be smart about formatting12:35
Saviqdednick, and do "yesterday" / "tomorrow" and such12:35
dednickSaviq: sure.12:37
Saviqdednick, but it really feels like the indicators should just cough up timestamps12:37
Saviqdednick, and let the UI decide what's best for each use case12:37
Saviqdednick, btw, we had something doing the smart thing for the people scope12:38
Saviqdednick, pure JS12:38
dednickSaviq: yeah, i think i did mention that before. But it's being used in desktop as well, so we cant really get right of it easily.12:38
dednicks/right/rid12:38
Saviqdednick, well, we can just ignore it :)12:38
larsudednick: what do you mean? We should always port towards using timestamps only12:42
larsuand I think datetime is the only one left12:42
dednickSaviq: what would we do about the format gsettings? they're @ com.canonical.indicator.datetime . It applies to both time label & calendar entries.12:43
Saviqdednick, it feels weird to have it only for the indicator12:44
Saviqdednick, it should be a system-wide setting12:44
larsuSaviq: afaik, this is not possible, as different places in the ui want to refer to times differently12:45
Saviqlarsu, yeah, that, why?12:45
Saviqlarsu, I think other than it being "long", "short", "human-readable" etc.12:45
Saviqlarsu, which can be derived from locale, AFAICT12:46
larsuSaviq: don't remember. I think it was mpt who told me at some point12:46
larsuor another designer...12:46
Saviqlarsu, but nothing should explicitly chose a time format...12:46
Saviqmpt, hey, we're pondering on why would you want different time formatting in different places - other then referring to those formats by descriptive names like "long", "short", "human-readable" etc.12:47
dednickSaviq: because we dont want the time to be "Now"12:48
dednick:)12:48
dednickhey Saviq, what is the time? reply: "now"12:48
Saviqdednick, that's why I'm saying - a few different formats, but not arbitrary ones12:49
Saviqdednick, but derived from the locale12:49
Saviqmpt, is it really necessary for things to come up with arbitrary time formats? doesn't that result in inconsistency in the UI? and make it confusing when you need/want to change this - changing the time global time formatting would not affect, say, the indicator12:49
Saviqdednick, I mean TimeFormatter { timestamp: 123456; format: "short" }12:50
Saviqdednick, instead of  TimeFormatter { timestamp: 123456; format: "hh:mm" }12:50
Saviqdednick, as hh:mm is not locale-independent12:50
dednickSaviq: right, well i didnt mean abritrary. but there was no central place to get this global thing from. I agree.12:52
dednickBut for the time indicator, it's using a label for the widget, which i guess is going to have to change to a timestamp.12:52
Saviqdednick, oh yes12:53
Saviqdednick, fortunately it can, just by adding a new prop - while keeping the other for desktop12:53
Saviqdednick, actually BUT12:53
Saviqdednick, the label should not be coming from the indicator at all, should it...12:53
Saviqdednick, we just need to know it's the current time12:54
Saviqdednick, so that indicator does not need to send up new stuff at all12:54
dednickSaviq: well we need it for desktop unless we port that as well.12:54
Saviqdednick, yeah, what I'm saying - we can just leave the indicator be12:54
Saviqdednick, but ignore what it's coming up with12:54
Saviqdednick, and just format the time ourselves12:54
dednickSaviq: you mean to use QTime::getCurrentTime or whatever?12:55
Saviqdednick, and not even get the current time from the indicator - as why would we12:55
Saviqdednick, yeah12:55
dednickSaviq: we use it to sync time on greeter as well.12:55
dednickbut i guess that can just be QTime as well.12:56
Saviqdednick, only thing we might need from the indicator is the timezone12:56
Saviqdednick, and well, we need https://blueprints.launchpad.net/ubuntu-ui-toolkit/+spec/time-component, too12:56
dednickSaviq: we sort that out in the formatter12:56
dednickSaviq: the tz i mean12:57
Saviqdednick, the only thing the indicator gives us right now, is timely updates without the above component12:57
Saviqdednick, yeah ok12:57
Saviqaaanyway12:57
Saviqthat's a bigger task12:57
dednickSaviq: should we put the formatter in toolkit then?12:59
Saviqdednick, ultimately, yes - not in the state it's in atm probably12:59
Saviqdednick, it needs to be much more complete12:59
dednickSaviq: yay. wont take months to get through12:59
Saviqdednick, so *while* we care about time formats from the indicators, we'll merge your things, but I'll be filing bugs against projects13:00
Saviqthis may even be a blueprint, FWIW13:00
dednickok13:00
Saviqdednick, so, UTF8?13:01
dednickSaviq: yeah, changing now13:01
mzanettikgunn: Why "Opinion"? https://bugs.launchpad.net/bugs/124093913:01
ubot5Ubuntu bug 1240939 in ofono (Ubuntu) "[ofono] PinRequired doesn't change to "puk" any more on 3 invalid attempts" [Undecided,Fix released]13:01
mptSaviq, I don’t understand the premise of the question. What things are coming up with arbitrary time formats?13:05
Saviqmpt, the datetime indicator has a gsetting, for one13:05
Saviqmpt, the events coming from the messaging indicator send them up, too, depending on the time proximity of the event13:06
=== greyback is now known as greyback|lunch
Saviqdednick, larsu, do you know they have those formats hardcoded for events, or are they constructed based on the locale ↑↑?13:07
mptSaviq, is there a way to implement the settings shown in <https://wiki.ubuntu.com/TimeAndDate?action=AttachFile&do=view&target=settings-clock.png> that is better than using a gsetting?13:08
dednickSaviq: it's definately generated manually.13:09
Saviqmpt, well, datetime indicator is slightly trickier, as it's more about user preference rather than locale13:10
Saviqmpt, and yeah we can have those settings without actually trying to come up with a datetime string13:10
larsuSaviq: the messaging indicator uses relative times. The strings for time periods are translated normally13:10
larsuthere's nothing in the locale that can help us there13:11
larsudatetime is a different matter13:11
larsucharles might know why we're not using locale there13:11
Saviqlarsu, well, for human-readable you still want the same resolution13:11
Saviqlarsu, so saying, "tomorrow" is not enough13:11
larsumaybe because we want to be able to change it dynamically?13:11
Saviqlarsu, but for "tomorrow, 12:00am", you need the locale13:12
larsuSaviq: the same resolution as what?13:12
Saviqlarsu, the human-readable thing usually needs to carry as much information as "a format"13:12
Saviqif you have "2013.12.11 12:00am", you can't just convert that into "tomorrow"13:12
Saviqyou need "tomorrow, 12:00am"13:12
Saviqso that's where locale comes into play as well13:12
larsuthe messaging menu uses relative times13:13
Saviqlarsu, isn't "tomorrow" relative?13:13
Saviqwhat would it do with the 12:00am?13:13
larsuI mean relative as in "24 hours ago"13:13
Saviqsure, and past that, it's going to be yesterday13:13
larsuno13:13
Saviq36 hours ago?13:14
larsuit does "days ago!13:14
larsu"days ago"13:14
Saviqwell, it might be fine for messages13:14
Saviqit's not for events13:14
larsuright, and that's what my original point was: every ui component needs it a bit differently13:14
larsuso having a few gsettings might not be enough13:15
Saviqlarsu, doesn't mean it needs to deal with all of it internally13:15
larsubut then, ianad13:15
Saviqlarsu, we just need a flexible enough time formatter13:15
larsuwhich is what the MR is about, no?13:15
Saviqnot if it deals with arbitrary time formats13:15
Saviqwhich it does13:15
Saviqcoming with the events13:16
dandraderdoes the battery indicator icon turn to "charging" (that lightning bolt) when you plug your Nexus 10 to your pc's usb port?13:16
dandraderwith me it happens only when I plug to the actual charger13:16
mptSaviq, as I recall, ted used a datetime string so that geeks could, if they wanted to, use a datetime format that wasn’t represented by the checkboxes. I don’t really mind one way or the other, as long as the checkboxes work.13:16
Saviqdandrader, no, might be it's not getting enough umph13:17
Saviqmpt, right, sure, we can leave the gsetting option there, but I'm after a generic solution here13:17
dednickSaviq: pushed fromUtf8 change13:18
mptSaviq, what do you mean by a generic solution?13:18
dandraderSaviq, but I suppose it still charges, albeit slowly, right?13:19
Saviqdandrader, possible both ways, really13:19
Saviqmpt, that app authors can just use a simple-as-possible API for displaying time in their apps - one that's consistent with what the user chose as their locale13:19
Saviqmpt, or their preferred time format13:20
Saviqmpt, instead of everyone and their mother inventing a time format just-right for their usage13:20
Saviqmpt, that completely disregards any locale/user-preference there might be13:20
seb128Saviq, you want to standardize stuff like "display the current month as part of the time"?13:21
seb128that feels weird13:21
Saviqseb128, no13:21
seb128gettext/the locale should already give you a 24/12 format adapted to your locale13:22
* Saviq needs to write this down13:23
Saviqseb128, I'm after: user selects a locale or their preferred time formatting in the settings app13:23
seb128Saviq, yeah, you need some concrete examples I think13:23
seb128Saviq, like "display the month with the time" (what the indicator currently have)13:23
Saviqseb128, everywhere where there's time displayed, it will adhere to the user-selected preference above13:23
Saviqseb128, datetime indicator is very specialized13:24
seb128or if not what preferences do you see?13:24
mptSaviq, as I understand it, problems with the GNU locale system include (1) not being overridable in a way that works across apps, (2) not updating instantly in every app, and (3) not standardizing either relative dates (e.g. “Yesterday”) or relative times (e.g. “2 minutes ago”). Maybe Qt/QML fixes some of those? A new API might fix the others.13:24
seb128we don't have any "time format preference" atm13:24
Saviqseb128, yeah, we just use locale - but we might let people select a different format than the locale comes up with, right?13:24
mptSaviq, oh, and (4) requiring everyone to use the Gregorian calendar.13:24
Saviqmpt, (1) not sure why? you can override LC_TIME and assuming everything uses locale (as opposed to arbitrary format strings), all should work13:25
seb128Saviq, how different? I never felt the need to change my time format, just wondering how common/useful that is13:25
Saviqmpt, (2) yeah, that's a problem13:25
Saviqmpt, (3) that's exactly something I'd like to solve on the SDK level13:26
Saviqmpt, (4) yeah... can't help you there...13:26
Saviqseb128, people that want their phone in US English, but can't read 01/01/2013, for example13:26
seb128Saviq, looking to my android, it doesn't seem they have a time format preference13:26
seb128that seems a geeky usage (and it's date, not time)13:27
Saviqseb128, doesn't matter, really - if we just say your selected locale is your time format, that's fine, too13:27
Saviqseb128, everything I was talking about applies13:27
Saviqseb128, in the N9 I have separate "Language" and "Regional format" selection13:28
Saviqseb128, and can change 12h vs. 24h separately13:29
mptSaviq, (1) for example if I’m CTO on a military base and I want 24-hour time everywhere, there’s no radio button I can toggle that will set it in LibreOffice and Thunderbird and XChat. If that’s *only* because there isn’t a GUI for it, rather than because the underlying setting doesn’t exist or because some of those apps would ignore it, then wonderful. :)13:30
seb128Saviq, right, so basically you want to create a "get_time()" that calls e.g strftime with the right parameter, depending of a gsettings config (e.g "show 24 hours" would use %R)13:31
Saviqseb128, yeah, something like that13:32
Saviqseb128, the app developer should never know13:32
Saviqseb128, they should just say "I want the long date-time format", "I want the short time format", "I want the human-readable date-time format", "I want the human-readable time format"13:32
seb128seems like a JDI thing then? e.g we need a bug with a list of options to support and somebody to write the code13:33
seb128or was there something to discuss?13:33
Saviqseb128, JDI?13:33
Saviqah Just Do It13:33
SaviqYOLO13:33
seb128just do it13:33
seb128;-)13:33
seb128well, you just listed 4 cases13:34
Saviqseb128, yeah, this grew from "indicators should not give up time formats" to a more generic "GNU locale is bad" conversation ;)13:34
Saviqseb128, sure, there's more13:34
Saviqseb128, much more13:34
seb128soon you are back to having strftime and % options :p13:34
mptSaviq, (3) I would be keen to review or contribute to that API design. It’s bothered me for about seven years.13:34
Saviqmpt, will13:34
mptI could help collect use cases, for example.13:34
Saviqmpt, that would be fantastic13:34
Saviqmpt, let me file a blueprint13:35
mptok13:35
Saviqmpt, https://blueprints.launchpad.net/ubuntu-ui-toolkit/+spec/time-formatter13:44
nic-doffay_dandrader, any idea when this is ready  to land? https://code.launchpad.net/~dandrader/unity8/runningAppsEndClose/+merge/19625713:47
nic-doffay_dandrader, nm13:47
nic-doffay_merged already.13:47
dandradernic-doffay_, not only merged, but also released as well :)13:48
nic-doffay_dandrader, haha yeah I shot off those questions while reading the diff :P13:49
nic-doffay_dandrader, can I use it to fix this: https://bugs.launchpad.net/unity8/+bug/121303413:50
ubot5Ubuntu bug 1213034 in Unity 8 "Can't dismiss keyboard by tapping outside of search entry" [High,Opinion]13:50
dandradernic-doffay_, to remove focus from the text input field?13:50
mzanettiSaviq: what do I need to do in order to make an application show up? Context: I dropped Stage.qml and trying to reimplement it13:50
Saviqmzanetti, "show up"? you need to focus it13:51
mzanettiSaviq: so it is started, I get ApplicationManager.focusedApplicationIdChanged(). but I don't see it on the screen13:51
Saviqmzanetti, you also need to hide the dash I think13:51
mzanettimhm... /me tries13:51
Saviqmzanetti, IIRC the shell is always on top of apps (only way we can get the launcher on top of apps)13:52
dandradernic-doffay_, using the PressedOutsideNotifier seems a bit of an overkill for that. I think what we need is to make the dash background or something take/accept input focus when touched13:52
nic-doffay_Saviq, ^13:52
mzanettiSaviq: right... I thought something like this. just didn't find the place in the old Stage where that happens. will search more13:52
Saviqdandrader, well, it's not just about the dash13:52
Saviqdandrader, but say you pull the launcher in, pull the panel in13:53
Saviqdandrader, and that's exactly what we need - see, here's an area where I'm at (and my popover)13:53
Saviqdandrader, if you touch anything else, I want to unfocus13:53
dandraderSaviq, hmm, right. so you want to lose focus regardless, even if noone else is interested in it13:54
Saviqdandrader, yes13:54
dandraderSaviq, nic-doffay_  therefore having it working differently than a traditional text entry field when using a mouse13:55
dandraderwhere is loses focus only once another focusable widget is clicked13:55
Saviqdandrader, yeah, because we want to hide the OSK13:55
Saviqdandrader, so it's more about that than the mouse13:55
Saviqdandrader, as soon as you've stopped typing - you don't want the OSK13:56
Saviqdandrader, and to bring it back - you want an action from the user on the field they want to type in13:56
dandraderSaviq, is this behavior specific for this search text entry or is it for any text entry in general?13:57
Saviqdandrader, probably not specific for us, no13:57
Saviqdandrader, not even for the "we have a drop-down" case, since that component should be available in the SDK, too13:57
dandraderSaviq, I'm asking because there's (or might be) this case where you want to vertically scroll your ui without unfocusing the text entry.13:58
Saviqdandrader, not sure - could be a prop on the text entry, FWIW13:59
Saviqdandrader, the OSK is something you usually want to get rid of13:59
Saviqdandrader, when looking at other things13:59
Saviqdandrader, just 'cause it takes a lot of screen real estate14:00
Saviqdandrader, what usecase do you have in mind?14:00
tsdgeosCimi: i remerged https://code.launchpad.net/~aacid/unity8/dash_show_home_no_index/+merge/19771414:01
dandraderSaviq, just recalling how it worked witht the N9 when on a web page, for instance. you're typing on a text entry but could peek up or down the page for some information relevant to what you're typing. then the next char you type make the page scroll back to have the focused text entry on screen again14:02
Cimiok14:02
Saviqdandrader, right, but OTOH it hides when typing in the address bar - as soon as you touch the history14:03
dandraderSaviq, so, in that case, hiding the vkb (i.e. unfocusing the text entry) is not right or wrong, I think. But it's a design decision that should be consistent throughout the system14:04
dandrader"that case" meaning my example use case14:04
Saviqdandrader, right, care to file a bug against ubuntu-ui-toolkit and ubuntu-ux?14:05
dandraderSaviq, to ask about the "when to unfocus" behavior?14:06
Saviqdandrader, it probably is basically about "what area do I consider being «in focus» for my text field"14:06
Saviqdandrader, yeah, for the dash it's definitely search entry + search history14:06
Saviqdandrader, but for a website it might be the whole website14:07
Saviqdandrader, it might be indeed a case by case thing14:07
dandraderSaviq, will file the bug14:09
Saviqdandrader, thanks, please provide a few use cases as you can think of, that justify one or the other behavior14:10
Saviqdednick, you need to merge trunk into the formatter14:10
=== rachelliu_ is now known as rachelliu
SaviqCimi, when you see a failure like https://code.launchpad.net/~aacid/unity8/dash_show_home_no_index/+merge/197714/comments/45983114:11
SaviqCimi, don't auto-re-approve - it's usually an issue - either a conflict or whitespace14:11
CimiSaviq, I approved again because albert merged trunk14:12
SaviqCimi, oh14:12
SaviqCimi, it wasn't there in the list of comments14:12
SaviqCimi, interesting14:12
SaviqCimi, sorry, reapproved14:13
CimiSaviq, your automatic launchpad bot is failing :P14:13
SaviqCimi, yeah, that's 'cause lp is failing - there should be a comment listing the new revision :P14:13
=== dandrader is now known as dandrader|afk
dednickSaviq: thanks. will check now14:19
Saviqmzanetti, you said something about haptic feedback / vibration? did you read about it coming, somewhere?14:20
mzanettiSaviq: my galaxy nexus vibrates on every touch since the last flash14:21
mzanettitrusty-proposed that is14:21
Saviqhm interesting14:21
Saviqmzanetti, wonder if it's something low-level that got enabled14:22
nic-doffay_Saviq, so I'm assuming I should hold off on that then?14:23
mzanettiSaviq: I thought so... weird thing is, it doesn't do any more. but no clue what I did14:23
nic-doffay_re the keyboard dismissal...14:23
Saviqnic-doffay_, no, please implement locally14:23
Saviqnic-doffay_, if/when we get it from the SDK, we'll just use it14:23
Saviqnic-doffay_, but let's have it in the mean time14:23
Saviqmzanetti, lol14:23
nic-doffay_Saviq, cool14:24
mzanettiSaviq: well, I'm sure it did on friday when I last flashed14:24
* mzanetti flashes again14:24
=== dandrader|afk is now known as dandrader
Saviqmzanetti, tsdgeos, /me and greyback|lunch have a meeting over standup14:31
mzanettigreyback|lunch: o/14:31
Saviqmzanetti, tsdgeos, and nic-doffay_ can't speak, so he put his stuff in the doc14:31
=== greyback|lunch is now known as greyback
Saviqtvoss, mtg?14:32
mzanettigreyback: what's the difference between ApplicationManager's focusedApplicationIdChanged() and focusRequested() signals?14:32
tsdgeosSaviq: standup?14:32
tvossSaviq, https://plus.google.com/hangouts/_/canonical.com/application14:32
greybackmzanetti: focusRequested is when for some reason Mir changes the focused application in a way that shell wasn't responsible - e.g. an application started not by shell14:32
greybackmzanetti: that signal is listened to by shell so that shell can decide if that new focus decision is correct or might need over-riding14:33
mzanettigreyback: it's missing in unity-api14:33
greybackmzanetti: it doesn't belong there really. It's a hack. It should go away once Mir looses the concept of focus14:34
mzanettigreyback: huh? you mean the shell is responsable for it14:34
Saviqgreyback, https://plus.google.com/hangouts/_/canonical.com/application14:34
mzanettianyways. ok. it means I have to keep it for now I guess... thanks14:35
dednickCimi: you in office tomorrow?14:40
Cimiafternoon14:41
tsdgeossorry internet broke14:42
tsdgeosstandup finished i guess?14:42
mzanettitsdgeos: no prob14:42
* mzanetti goes to pick up his car from the service... bbiab14:43
tsdgeoscollapsing still broken ^_^15:06
* tsdgeos hides15:06
kgunnmterry: yo15:12
mterrykgunn, hello!15:12
kgunnmterry: got time in about 15 min to chat with me and greyback ?15:12
mterryk15:12
kgunngreyback: ?15:12
kgunndednick_: meant to say ....welcome back15:14
dednick_kgunn: thanks!15:14
dandraderSaviq, nic-doffay_ https://bugs.launchpad.net/ubuntu-ux/+bug/125959615:16
ubot5Ubuntu bug 1259596 in Ubuntu UX "When should a text entry lose focus?" [Undecided,New]15:16
Saviqdandrader, cheers15:17
dandraderSaviq, nic-doffay_ regardless of the decision, I'm thinking that it would be best if the unfocus action is taken not by the text entry itself but by the entity that has given focus to it in the first place (the input handling logic?). Otherwise we might get some race conditions: E.g. user taps outside focused entry onto yet another focusable item.15:19
dandradernot sure if this centralized approach is actually doable, though15:21
dandraderas we might end up having to change code in QQuickWindow ....15:22
Saviqdandrader, I don't even think if it's that complicated15:26
Saviqdandrader, all the use cases I can come up with, it's actually the text area itself that knows whether it makes sense for it to be focused when you tap outside of it (or outside of an area)15:27
greybackkgunn: am there15:34
=== alan_g is now known as alan_g|tea
dandraderSaviq, right, because the item itself sets its focus property to true (in hope of getting activeFocus) when pressed. Thus it would just set it back to false upon "pressed outside", letting the activeFocus go to someone else15:39
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/more_filter_fixes/+merge/19842715:42
dandraderSaviq, but I believe that approach (focused item unfocusing itself) won't work if we are to implement the "do not unfocus if it's a content scroll" case15:42
=== alan_g|tea is now known as alan_g
dandraderbut, anyway, let's wait to see what's the desired behavior15:42
dednick_Saviq: getting a "Could not determine plugin installation dir" error building trunk15:52
tsdgeosdednick_: dist-upgrade ?15:54
tsdgeosyou need the new scopes-shell thingie15:54
tsdgeosdednick_: you're on trusty?15:54
dednick_tsdgeos: no,15:55
dednick_hm. maybe i should upgrade..15:55
tsdgeosdednick_: i'm not sure we support non trusty nowadays15:55
tsdgeosmhr3: is the unity-scopes-shell released for non trusty?15:55
mhr3tsdgeos, nope, only trusty16:00
mhr3tsdgeos, it's buildable on saucy, but you'd need to build all the dependencies as well16:00
tsdgeosdandrader: there you go, you need some trusty-ness16:00
tsdgeoserr16:00
tsdgeosdandrader: that was for dednick but he left and tab-completion decided to autocomplete to you, sorry :D16:00
dandradertsdgeos, np16:01
Saviqdednick_, ./build -c16:01
dandraderanyone has success shutting down Nexus 10 for good?16:02
dandraderfor me it just reboots when I try to shut it down16:03
Saviqdandrader, hold power until it dies - should turn it of16:03
Saviqdandrader, otherwise, hold all three16:03
Saviqdandrader, and choose POWER OFF16:03
dandraderSaviq, ok, thanks16:04
=== dandrader is now known as dandrader|lunch
dednickahhh. how do I update to 14.04?! my update manager not picking it up16:13
tsdgeosanyone knows what's broken with otto? https://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/1382/console16:14
dednicktsdgeos: how do you upgrade to 14.04?16:16
tsdgeosdednick: i did it the manual way16:16
tsdgeosreplace saucy  for trusty in /etc/aot16:16
tsdgeosapt16:16
dednicktsdgeos: ah...16:16
dednicki can see that ending in tears..16:17
tsdgeosit was pretty easy back when i did it16:22
tsdgeosbut it was just at the start of the cycle16:22
tsdgeosso there were very few changes at that time16:22
cwaynesudo do-release-upgrade -d16:22
cwaynebut disable extras first, i kept 404'ing on extras for trusty16:22
dednickcwayne: thanks!16:24
cwaynedednick: np :)16:25
Saviqdednick, do-release-upgrade -d16:36
mhall119Saviq: FYI, Kaleo has the work item to detect keyboard & mouse: https://blueprints.launchpad.net/ubuntu/+spec/appdev-1311-apps-convergence16:36
Saviq↑↑ that16:36
Saviqmhall119, yup16:36
mhall119I assume everything in the SDK is available to Unity 816:36
Saviqmhall119, yeah of course16:37
Saviqmhall119, we're pushing stuff up from unity8 into the SDK where it makes sense16:37
mhall119cool, I'll try and find out when that will be implemented then16:37
tsdgeosguys, i'm going to take a bit early EOD, not feeling too well atm16:38
tsdgeossee you tomorrow16:38
Saviqmzanetti, traceback from crashing unity-api test: http://paste.ubuntu.com/6551990/16:41
mzanettiSaviq: that's a good one :)16:44
Saviqmzanetti, yeah, but 95 steps...16:45
Saviqwe don't get as much in unity8 alone ;)16:45
Saviqwell, with 5.2 we probably do, then ;)16:46
mhr3mzanetti, you wanted more tests - https://code.launchpad.net/~mhr3/unity-scopes-shell/scopes-ng-tests/+merge/19843716:48
mhr3Saviq, ^ has your role changes too16:49
Saviqmhr3, cool16:49
mhall119Saviq: trying to load CMakeLists.txt in Ubuntu SDK, I get the following error when trying to configure cmake:16:54
mhall119CMake Error at CMakeLists.txt:64 (message): Could not determine plugin installation dir.16:54
mhall119do I need to add some Arguements to get it working?16:54
mhr3Saviq, oh and i was supposed to ask which renderer is likely to come next16:54
mhr3Saviq, as supported in the category templates16:55
=== tvoss is now known as tvoss|test
=== tvoss|test is now known as tvoss|dinner
Saviqmhr3, by renderer you mean grid/carousel etc.?17:02
mhr3Saviq, yep17:02
Saviqmhr3, carousel, probably, then v. journal17:03
mhr3thx17:04
=== dandrader|lunch is now known as dandrader
Saviqmzanetti, here's another one - seems to be racy17:10
Saviqhttp://paste.ubuntu.com/6552106/17:10
mzanettiSaviq: same test?17:10
Saviqmzanetti, yes, stripped down17:10
Saviqmzanetti, http://paste.ubuntu.com/6552115/17:11
mzanettiSaviq: hmm... it's at the end of the test when cleaning up?17:12
Saviqmzanetti, yeah17:12
nic-doffay_Saviq, currently the inverse mouse area in the PageHeader dismisses the search box and activates whatever was pressed. Is this desired behaviour?17:14
Saviqnic-doffay_, not sure, ideally it should block the tap, but allow for swiping... which is trickier somewhat17:15
nic-doffay_Saviq, yeah I def think it should block the tap.17:17
Saviqnic-doffay_, +117:17
nic-doffay_Saviq, there's still the issue of the swipe though and differentiating the touch inputs...17:18
* greyback to the airport17:18
Saviqgreyback, fly safe17:18
nic-doffay_gl greyback17:20
mhall119Saviq: any idea what I'm doing wrong in QtCreator?17:22
Saviqmhall119, not if you don't tell me what's happening :)17:22
mhall119CMake Error at CMakeLists.txt:64 (message): Could not determine plugin installation dir.17:23
mhall119pasted 30 minutes ago :-P17:23
mhall119when opening CMakeLists.txt and at the dialog to configure cmake17:23
mhall119Saviq: http://ubuntuone.com/54blhOH0aIf6WnBtt3PPBr17:24
Saviqmhall119, you don't have the build deps installed :)17:27
Saviqmhall119, ./build-s17:27
Saviqerm17:27
Saviq./build -s17:27
mhall119build -s gives:17:27
mhall119Note, selecting 'qtdeclarative5-unity-notifications-plugin' instead of 'unity-notifications-impl'17:27
mhall119E: Unable to locate package unity-plugin-scopes17:27
nic-doffay_dandrader, any thoughts on using your component to differentiate between touches and swipes?17:27
Saviqmhall119, you not on trusty are you?17:27
mhall119no, on saucy still17:27
Saviqmhall119, we only support trusty atm...17:27
Saviqmhall119, we could do a ppa with the other bits17:27
Saviqthat are unavailable on saucy17:27
mhall119if we can do a PPA, that would be best17:28
mhall119docs currently say only saucy is supported17:28
Saviqmhall119, right, outdated... sorry17:28
mhall119I'll fix the docs, how long would it take to get a PPA to let it work with saucy?17:29
Saviqmzanetti, whatever I do in the test, it has a different crash :/17:29
Saviqmhall119, it's probably just the above package alone17:29
Saviqmhall119, and it should Just Build™17:29
mzanettiSaviq: :/ it might help to start locking the dtor of the singleton.17:29
Saviqmzanetti, hmm interesting, Qt should manage the dtion of singletons17:30
mzanettiSaviq: yeah. well... seems something is accessing it while/after dtion, or maybe it's deleted twice, C++ and QML context17:31
dandradernic-doffay_, you mean PressedOutsideNotifier? I don't think it's a good idea.  This thing of differentiating between touches and swipes sounds like the kind of problem that gesture cancellation solves. like what the Flickable does when it detects a swipe and therefore sends a TouchCancel to the item inside it that received the press17:32
dandraderand grabs the touch from the child item that got the press event17:33
dandradernic-doffay_,  but I would need to dive into the problem to come up with a more solid opinion17:35
mhall119Saviq: where can I find that one package?17:35
Saviqmhall119, lp:unity-scopes-shell17:35
nic-doffay_dandrader, that's what I figured too just judging by the name of your class.17:36
nic-doffay_Saviq, your thoughts?17:36
mhall119Saviq: no binary I can install?17:36
Saviqmhall119, it's there in trusty17:37
Saviqhttp://launchpad.net/ubuntu/+source/unity-scopes-shell17:37
Saviqmhall119, nothing for saucy atm17:37
mhall119I'll try installing that one17:38
mhall119with any luck, I won't completely break everything17:38
mhall119Saviq: ok, build --setup is happy now, but I still can't get cmake working within QtCreator17:42
mhall119same error17:42
Saviqmhall119, that's probably because qtcreator tries to pick up the wrong cmakecache17:42
Saviqmhall119, make sure you point it at the right build dir17:43
Saviqi.e. unity8/builddir17:43
mhall119./builddir/ right?17:43
mhall119yeah, tried that, same error still17:43
Saviqmhall119, if you go there17:43
Saviqmhall119, in that dir17:43
Saviqand go cmake ..17:43
Saviqmhall119, does that work?17:43
mhall119FWIW, ./build gives the same error in the terminal17:44
mhall119mhall@mhall-thinkpad:~/projects/Ubuntu/unity8/builddir$ cmake ..17:44
mhall119-- Could NOT find Lcov (missing:  LCOV_EXECUTABLE GENHTML_EXECUTABLE)17:44
mhall119-- Could NOT find gcovr (missing:  GCOVR_EXECUTABLE)17:44
mhall119CMake Error at CMakeLists.txt:64 (message): Could not determine plugin installation dir.17:44
mhall119-- Configuring incomplete, errors occurred!17:44
Saviqmhall119, right, it's ./build that installs the deps17:44
Saviqmhall119, try ./build -c, it might've cached something17:44
mhall119The following packages will be REMOVED: unity8-build-deps17:44
mhall119is that okay?17:45
Saviqmhall119, yeah, that means you can't install some of the build deps17:45
Saviqmhall119, try interrupting that17:45
Saviqmhall119, and apt-get -f install to get more info17:45
mhall119-f install dosn't give me anything more, just that it's going to try and remove it17:46
Saviqmhall119, ah, new libunity-api-dev, too17:46
Saviqmhall119, https://launchpad.net/ubuntu/+source/unity-api17:46
Saviqmhall119, let me go through that tomorrow on a clean system, will put stuff into a ppa17:47
mhall119ok, let me know when you're done and I'll test it out again17:48
mhall119thanks Saviq17:48
mhall119Saviq: for now I've updated the docs to say it's 14.04 only, I'll update them again with instructions for 13.10 that uses the PPA17:52
Saviqmhall119, can you please put a unity8 branch that updates the same in CODING for ~unity-team?17:52
Saviqmhall119, so that we don't diverge17:52
mhall119Saviq: sure thing, give me a couple minutes17:53
Saviqmhall119, I'll put my changes / updates there as well17:54
mhall119Saviq: https://code.launchpad.net/~mhall119/unity8/update-CODING-to-1404/+merge/19844317:56
=== alan_g is now known as alan_g|EOD
Saviqmhall119, cheers18:01
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== jono is now known as Guest63303
=== salem_ is now known as _salem

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!