/srv/irclogs.ubuntu.com/2014/08/12/#ubuntu-unity.txt

=== Trevinho is now known as Trevinho|holiday
=== _salem is now known as salem_
=== salem_ is now known as _salem
Saviqmo'nin06:58
Cimimorning07:38
CimiSaviq, remove 0.1.16 from trunk or I'll do :)07:43
SaviqCimi, check again07:43
CimiSaviq, I just merged and I got it07:43
Cimiweird07:43
Cimimaybe was last night merge07:44
SaviqCimi, well, you got it because you have it07:44
SaviqCimi, not because it's in trunk ;P07:44
CimiI am infected!07:44
Cimi@unity is canonical irc down for you?08:06
dednickCimi: seems like it08:06
Cimiok08:06
dednickCimi: unless they've done something with dns again08:07
mzanettio/08:37
SaviqRIP Robin Williams :|08:41
CimiSaviq, do we have scope settings in the mock or I have to c++ them?08:53
SaviqCimi, in "the mock"?08:53
SaviqCimi, you mean in unity-api's mock?08:53
CimiSaviq, in Unity 0.2 mock08:54
SaviqCimi, right, no08:54
SaviqCimi, btw, found a bug with your recent changes08:56
SaviqCimi, there's no right margin in overlay either08:56
SaviqCimi, where it should be08:56
CimiSaviq, added to my todo08:58
mzanettiSaviq: flashing mako. bar goes from right to left :)09:00
mzanettiSaviq: logo rotating clockwise09:00
Saviqmzanetti, so it's just wrapped in the middle or something ;)09:01
mzanettiso its either mirrored vertically09:01
Saviqor yeah, mirrored most probably09:01
mzanettior yeah, shifted09:01
Saviqwell, no, if it was mirrored09:01
Saviqwouldn't it rotate counter clockwise?09:01
mzanettitrue :D09:01
mzanettiso its a different piece of animation.09:01
mzanettiI don't think its shifted either09:02
mzanettibecause spacing between bar and logo wouldn't match09:02
Saviqso what probably really happens09:02
mzanetti"probably really" yes. :)09:02
Saviqis coordinates are topleft vs. bottomright09:02
Saviqor no09:03
Saviqprobably really not09:03
mzanetti:)09:03
Saviqbecause that'd have failed in a more weird way09:03
mzanettiI'd say its just implemented the other way round. that's it09:03
Saviqmhm09:03
CimiSaviq, so in fake_scope.cpp it returns a null when you look for settings, is there any other mock code I could use there?09:05
Cimimaybe from unity api09:05
SaviqCimi, you need to write a mock settings model09:05
CimiSaviq, yup, but maybe they wrote it already no?09:05
Cimiotherwise how did they test it?09:05
SaviqCimi, not likely09:05
SaviqCimi, with the real thing09:06
SaviqCimi, pete-woods will know09:06
Cimipete-woods, I need to write a scope settings mock, where is the real thing? :)09:06
pete-woodsCimi: the settings model in unity-api gives a reasonably complete mock09:08
pete-woodsit has bool, string, number and list in it09:08
Cimipete-woods, found it09:10
pete-woods:)09:10
=== vrruiz_ is now known as rvr
mzanettiSaviq: when you have some time, can you please verify this grooveshark thingie?10:22
mzanettihttps://bugs.launchpad.net/unity8/+bug/135535610:22
ubot5Ubuntu bug 1355356 in unity-webapps-grooveshark (Ubuntu) "App spread should not kill media" [Undecided,New]10:22
mzanettiSaviq: works fine with youtube and others here10:22
mzanettiand I'm not really in the position to test with grooveshark10:22
Saviqmzanetti, will do in a mo10:23
Saviqmzanetti, critical one for you bug #135572910:34
ubot5bug 1355729 in Unity 8 "Network caching doesn't work anymore after introducing Dash as an app" [Undecided,New] https://launchpad.net/bugs/135572910:34
Saviqmzanetti, we didn't bring over caching from shell to dash10:35
mzanettilike if that ever worked :D10:35
Saviqmzanetti, yes it did10:36
Saviqmzanetti, check out your ~/.cache/unity8/network10:36
mzanettiSaviq: then the cache is the same speed as the network connection :D10:36
Saviqmzanetti, we're not caching for speed, we're caching for lower data usage10:37
mzanettiSaviq: we should cache for speed too :D10:38
mzanettiSaviq: jokes aside... how come then the cached content still takes seconds to load?10:38
Saviqmzanetti, that assumes that network is the bottleneck, which we know it is not10:38
Saviqmzanetti, bug #133843010:38
ubot5bug 1338430 in Unity 8 "Slow loading/showing of images in the dash" [Critical,Triaged] https://launchpad.net/bugs/133843010:38
dednickSaviq: hi. Have a bit of a conundrum about qml singletons. Can't really use the objects given to qml safely outside of qml, since they're managed by the engine (can be deleted before c++ objects which may use them on destruction). You ever come across that problem before?10:59
Saviqdednick, nope, but sounds legit11:00
Saviqdednick, why would you need to use the singleton in some other object's d'tor?11:00
dednickSaviq: yeah, but the qml documentation says not to use singlton factory.11:01
dednickSaviq: because there's some regegistration when objects are destroyed11:01
CimiSaviq, spot any mistake? http://paste.ubuntu.com/8025733/11:01
dednickde-registration11:01
SaviqCimi, is that a review or debugging?11:02
CimiSaviq, QMetaProperty::read: Unable to handle unregistered datatype 'unity::shell::scopes::SettingsModelInterface*' for property 'Scope::settings' in tryDash11:02
dednickSaviq: http://bazaar.launchpad.net/~unity-team/unity8/trunk/view/head:/tests/mocks/Unity/Application/ApplicationInfo.cpp#L5511:02
CimiSaviq, my C++ still lacks...11:03
dednickSaviq: although i'm not saying that's very good design11:03
SaviqCimi, and where are you registering that type in your mock plugin that you expect it to work?11:03
CimiSaviq, right11:04
CimiSaviq, adding it11:04
SaviqCimi, that message was so clear it couldn't be more so ;)11:04
CimiSaviq, it is clear if you understand qml and c++11:05
SaviqCimi, afternoon read http://qt-project.org/doc/qt-5/qtqml-cppintegration-topic.html11:05
Saviqdednick, so... AppMan is being destroyed at the same time SurfMan is, correct?11:05
dednickSaviq: surface manager has already been destroyed11:06
Saviqdednick, well, yeah, that's what I mean11:06
Saviqdednick, why doesn't SurfMan connect to the surface's destroy signal? does AppMan really need to tell SurfMan that's happening?11:07
=== pete-woods is now known as pete-woods|away
dednickSaviq: well, i guess there are ways to mitigate the problem, but still need to use SurfMan from App/Surface at some point. Just wondering if you knew a better way.11:11
Saviqdednick, nothing that would save us this issue, engine can destroy the singleton at will11:11
Saviqdednick, I don't think there's a way to "get" it from the engine11:12
dednickSaviq: right. I've done a bit of a crazy solution decorating a QSharedPointer<SurfMan> and passing that as the singleton, then storing the shared pointer where-ever it's used. Works, but it's pretty messy and quite a pain to maintain.11:14
facundobatistaHolas11:14
Saviqdednick, and then mixing Qt's QML/parent-child ownership and smart pointers is somewhat messy, too11:14
CimiSaviq, ok of this http://pastebin.com/8zbBKHjC how do I correctly set implicitHeight?11:15
dednickSaviq: the qml singleton isn't a smart pointer, the object it uses just holds a ref to one. but it's pretty crap11:15
SaviqCimi, spam detection, please use paste.ubuntu.comn11:16
dednicki'll just try sort it out with connections11:16
CimiSaviq, I used qtcreator plugin for the first time in my life :D11:16
CimiSaviq, http://paste.ubuntu.com/8025836/11:16
CimiSaviq, ScopeSetting is Item with list of props11:16
SaviqCimi, why would you set implicitHeight when in this case this widget only really makes sense in the height of the ItemSelector's?11:17
CimiSaviq, because they appear all on top of each other11:17
SaviqCimi, aaand?11:17
CimiSaviq, this and others11:17
SaviqCimi, you need to bind ScopeSettings's height to ItemSelector's11:17
SaviqCimi, but you should be using Ubuntu{ListView,Column} here and an Expandable11:17
CimiSaviq, http://paste.ubuntu.com/8025853/11:17
SaviqCimi, in which case there's more that needs to happen11:18
CimiSaviq, that one as well, no implicitHeight they are all on top as delegates11:18
SaviqCimi, your component is 0-height11:18
Cimiyeah I need to set implicitHeight of the children11:18
SaviqNO11:18
Saviqyou need to set height11:18
Saviqnot implicit height, because that component doesn't deal well with height other than the one of its children11:19
SaviqCimi, if you have a ListItem.Empty in every ScopeSetting11:20
SaviqCimi, why isn't ScopeSetting a ListItem.Empty already11:20
CimiSaviq, because List is not using ListItem.Empty11:20
CimiSaviq, I did that originally, but then I needed to change ScopeSetting11:20
SaviqCimi, what's List?11:20
CimiSaviq, list of words11:20
Cimihold on11:20
SaviqCimi, I really don't understand your problem11:21
CimiSaviq, something like the language setting I think11:21
SaviqScopeSetting has no height11:21
SaviqListItem.Empty has height11:21
CimiSaviq, yes11:21
SaviqListItem.Empty is in ScopeSetting11:21
Saviqso what height is ScopeSetting supposed to have?11:21
CimiString, Number, Boolean use Empty11:21
CimiList might use ItemSelector11:22
SaviqAAND?11:22
Saviqdoesn't ItemSelector have a height?11:22
Cimiyes, so I set height of that ScopeSetting as height of the children?11:22
Ciminope11:23
Cimibecause it expands..11:23
Cimithis is not correct http://paste.ubuntu.com/8025908/11:23
Cimi:/11:26
SaviqCimi, why isn't that correct?11:27
CimiSaviq, btw someone linked me this http://theburgerlist.tumblr.com/post/94443927060/le-bourguinon-bun-le-bun-riverside-feast-battersea11:27
CimiSaviq, guess could be a treat next week :)11:27
SaviqCimi, only there until Wed afternoon, though11:28
Cimiok11:28
CimiSaviq, so if I set that, when I click on the item it should expand, but is clipped instead to that height11:28
SaviqCimi, sounds like a bug in the component11:31
SaviqCimi, except there's http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/modules/Ubuntu/Components/ListItems/ItemSelector.qml#L15311:32
SaviqCimi, I'm afraid you can't do ScopeSetting if you want to use ExpandablesColumn (which you should)11:33
Saviqah damn with a *Factory you can't even do that either11:34
Saviqmzanetti, do you have an idea about using Expandables and a Loader as delegate?11:34
mzanettiSaviq: hmm... you don't know the loader's content size in advance, do you?11:35
mzanettiSaviq: dunno. would need to try11:35
mzanettimight work, might not...11:35
mzanettiSaviq: so you can put inside an Expandable whatever you want, also loaders11:36
Saviqmzanetti, no, I was wondering if we can reparent Loader.item to the Expandable{ListView,Column}11:36
mzanettino11:36
Saviqmzanetti, right, so delegate Expandable { Loader { } }11:36
mzanettiyes11:36
mzanettiSaviq: and:11:36
Saviqwe'd have to fwd everything things from item to the parent expandable11:36
mzanettiExpandable { expandedHeight: loader.height ... }11:37
CimiSaviq, shall I use expanded ones?11:42
CimiSaviq, I'll use expanded: true in the meanwhile11:46
SaviqCimi, what do you consider a "meanwhile"?11:47
CimiSaviq, hah, until we realise what's wrong with it11:48
Cimiwith the height11:48
SaviqCimi, we did already11:48
CimiSaviq, so shall I always use expanded?11:51
SaviqCimi, no, why?11:52
Cimiso how can I make it work?11:52
CimiI am confused11:52
SaviqCimi, http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/examples/ubuntu-ui-toolkit-gallery/ListItems.qml#L10611:53
SaviqCimi, this works in the gallery11:53
CimiSaviq, this is a column11:53
CimiSaviq, I already checked11:53
SaviqCimi, so? a column takes the item's dimensions11:53
CimiSaviq, maybe because is not embedded in one item?11:54
Cimiwrapped11:54
Ciminope11:55
Cimijust tried removing the item on top of listitem.itemselector11:56
Cimistill issues11:56
SaviqCimi, http://paste.ubuntu.com/8026115/11:56
SaviqCimi, btw, ItemSelector has its own header, use it11:56
CimiSaviq, I am already using it11:57
Cimihttp://paste.ubuntu.com/8025836/11:57
CimiSaviq, you might want to try this http://paste.ubuntu.com/8026141/11:59
SaviqCimi, yeah, and why would that ever work?11:59
SaviqCimi, when you set the Item's height to 100?12:00
CimiSaviq, it doesn't12:00
SaviqCimi, well, how can it?12:00
CimiSaviq, I need to set the item height to be exactly of the container12:00
CimiSaviq, because if the item is collapsed it should not eat more height12:00
Cimithan it needs12:00
SaviqCimi, does not compute12:01
Ciminot container12:01
Cimisorry children12:01
CimiItem height, where you set 600, should be the height of the itemselector12:01
Cimiand the added rectangle12:01
Cimibecause when you use the item in a listview, you want to allocate width and height12:02
SaviqCimi, http://paste.ubuntu.com/8026161/12:02
Cimithius works?12:03
Cimiit is basically http://paste.ubuntu.com/8025908/12:03
SaviqCimi, why would it *not* work?12:04
Cimithat code I pasted doesn't work12:04
Saviqwhat do you mean "doesn't work"?12:04
CimiSaviq, when I click on the itemselector to expand it12:05
CimiSaviq, the height doesn't grow12:05
SaviqCimi, but you have a Loader in the middle, how does the Loader's height look?12:05
CimiSaviq, the height doesn't change12:08
Cimineither the inner one12:08
Cimiof the item I mean12:08
SaviqCimi, you need to push the code somewhere if you still want to do IRC debugging, which you know I resent12:08
CimiSaviq, call it help...12:09
SaviqCimi, what I say happens: you're overriding the height in the delegate12:09
SaviqCimi, which means whatever you do *inside* the component doesn't matter12:09
CimiSaviq, pushed12:10
Cimi~cimi/unity8/scope-settings12:10
Cimimake tryDash12:10
=== MacSlow is now known as MacSlow|lunch
SaviqCimi, ok confirmed, this looks real weird indeed12:33
CimiSaviq, I am working on a workaround12:34
SaviqCimi, talk to SDK peeps, too12:34
SaviqCimi, TBH the ListItem.Empty is complex enough that something there must be getting awry12:35
CimiSaviq, http://paste.ubuntu.com/8026405/12:37
CimiSaviq, ETOOUGLY12:37
SaviqCimi, indeed12:37
Cimiworks btw12:37
SaviqCimi, the same seems to happen with OptionSelector12:38
SaviqHUH12:38
SaviqCimi, height: listItem.height + 112:40
CimiSaviq, can I commit that workaround with a comment?12:40
SaviqCimi, talk to SDK first12:40
CimiSaviq, that works too12:40
Cimibut without animation12:40
SaviqCimi, well, there is animation if you look at the console output12:41
SaviqCimi, it just doesn't get "up" correctly12:42
SaviqCimi, but in any case that suggests something's awfully broken in the ListItems' height12:42
CimiSaviq, can you join me in #sdk and probably explain better the issue?12:43
CimiSaviq, also wondering how containerHeight is not causing binding loops12:48
CimiSaviq, having headache trying to understand how the properties are nested12:49
=== _salem is now known as salem_
=== MacSlow|lunch is now known as MacSlow
CimiSaviq, can I also do http://paste.ubuntu.com/8026544/ to make tryGenericScopeView work?12:56
WellarkSaviq: FYI: https://bugs.launchpad.net/unity8/+bug/131853312:57
ubot5Ubuntu bug 1318533 in indicator-network (Ubuntu) "provide unity-notifications-service virtual package" [Wishlist,Triaged]12:57
ChrisTownsendgreyback: Hey, I have a question about the transition to qtmir wrt to https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1353041.  Whenever you have some time, could we chat about it for a bit?13:08
ubot5Ubuntu bug 1353041 in unity8-desktop-session (Ubuntu) "Logging out of the Unity8 desktop preview session hangs" [Undecided,Confirmed]13:09
greybackChrisTownsend: hey. Sure, happy to talk.13:10
ChrisTownsendgreyback: Cool.  So I've done some investigating and made some notes in the bug, but in a nutshell the QGuiApplication dtor is blocking waiting on some pthread_cond_wait.  I have no clue what it's waiting on though.13:11
ChrisTownsendgreyback: So teardown gets stuck at this point and the log out hangs.  I'm thinking it might have to do with the transition to qtmir, but wanted to see your opinion.13:12
cwaynehas the category header links changed? ours in our aggregating scopes aren't working atm13:13
greybackChrisTownsend: hmm yeah, I've skimmed your comments so far. It possible qtmir waiting for something and thus blocking shutdown, but I see nothing that obvious aside from one QMutexLocker, but I'd be surprised if that the culprit13:14
greybackChrisTownsend: I can certainly investigate it further13:14
ChrisTownsendgreyback: Awesome, that would be great!13:14
greybackChrisTownsend: please add qtmir as an affected project, and assign to me13:14
greybackI'll give it a look today13:15
mzanettiSaviq: ok... I guess this is it now: https://code.launchpad.net/~mzanetti/unity8/new-lockscreen-design/+merge/23047813:15
ChrisTownsendgreyback: Ok, will do.  And thanks!13:15
greybacknp13:15
Saviqmzanetti, kk13:15
WellarkSaviq: any flaws in my reasoning at bug #1318533 ?13:27
ubot5bug 1318533 in indicator-network (Ubuntu) "provide unity-notifications-service virtual package" [Wishlist,Triaged] https://launchpad.net/bugs/131853313:27
SaviqWellark, nope13:30
SaviqWellark, all good13:30
WellarkSaviq: hate to reject a community contribution, though..13:31
SaviqWellark, well, it might be better at some point soon13:37
SaviqWellark, when we drop the whole UnityMenuModel concept from notifications13:37
SaviqWellark, at least in the shape they're there now13:37
SaviqWellark, one idea was to use trusted sessions for this (so i-n would actually *draw* the UI it wants, not only direct it)13:38
mzanettiSaviq: hmmm... do we still need the caching in unity8?13:39
Saviqmzanetti, we don't *need* it, but it can't hurt13:39
mzanettiI guess it doesn't hurt...13:40
mzanettiack13:40
Saviqmzanetti, we can get remote images for avatars in notifications, for example13:40
Saviqand in messaging menu13:40
mzanettiI guess it *can* hurt... but lets hope all goes well :D13:40
Saviq;)13:40
mzanettipstolowski: https://code.launchpad.net/~mzanetti/unity8/dash-caching/+merge/23048713:52
pstolowskimzanetti, awesome, thanks!13:53
dandraderSaviq, did we change dash so that it no longer goes back to the apps scope when you switch back to it?13:54
Saviqdandrader, depends on how you switch to it13:54
Saviqdandrader, it should still happen when you left-swipe or press the button13:54
pstolowskimzanetti, is it enough to register it in main? no qml changes?13:55
mzanettiSaviq: except when you're already at the dash13:55
Saviqmzanetti, well, button still works13:55
dandraderSaviq, right13:55
Saviqmzanetti, left swipe doesn't13:55
Saviqand overview broke it13:55
mzanettipstolowski: yep. it registers a QNetworkAccessManagerFactory13:55
Saviqdandrader, but yeah, the bug you were looking at13:55
Saviqdandrader, swipe dash to a scope13:55
Saviq(different than apps)13:56
mzanettipstolowski: so if QML wants something it'll create a new QNAM and will get it from that factory13:56
Saviqdandrader, launch an app from the launcher13:56
Saviqdandrader, long-left-swipe to dash13:56
Saviqdandrader, you see the old scope, until you touch it13:56
Saviqdandrader, I wonder, though, ig the dash getting suspended is causing this13:56
Saviqif13:57
mzanettiSaviq: yes, I think so13:57
Saviqbut unlikely13:57
Saviqmzanetti, nah, happens without even waiting for the other app to launch13:57
pstolowskimzanetti, k, thanks for explaining. +113:57
Saviqmzanetti, so within the 5 or so seconds before it gets suspended13:57
mzanettiSaviq: not here13:58
mzanettioh. yes... this time yes13:59
mzanettiSaviq: not if you're really quick13:59
mzanettiSaviq: its 3 secs btw13:59
mzanettistrage... it does not happen all the time anyways14:00
MacSlowwhat (additional) branch is needed to get current lp:unity8 trunk to to work with the run-script on the desktop?14:00
mzanettibut not related to the 3 secs thing14:00
mzanettiMacSlow: forget about the run script :D14:00
seb128Saviq, when you reassign bugs to settings, please do it against the package, we don't use the project for bugs14:01
MacSlowmzanetti, but I should now use instead.... ?14:01
mzanettiMacSlow: make tryShell for instance14:01
mzanettiMacSlow: depends a bit on what you need14:01
SaviqMacSlow, lp:~saviq/unity8/tweak-runscript14:01
Saviqit's a start14:01
mzanettior that :)14:01
tedgSaviq, Is there a way to know when the dash has settled?14:05
tedgSaviq, i.e. when it's done loading icons, etc.14:06
tedgSaviq, Trying to make the app startup results more reliable, avoiding having the dash stealing CPU.14:06
Saviqtedg, right now it's never "done loading icons", because it won't start loading them until you look at the dash14:06
tedgSaviq, So I look at the dash by unlocking Unity before starting the app.14:07
Saviqtedg, just start the app, it should unlock itself14:07
SaviqI *think*14:07
MacSlowmzanetti, Saviq: with lp:~saviq/unity8/tweak-runscript I still only get a tiny black square window... and I don't have any Makefile (and therefore no make-targets)14:07
mzanettiMacSlow: not following...14:08
MacSlowmzanetti, you said something about "make tryShell" a few lines back14:08
tedgSaviq, Hmm, that seems like a bad thing, no?14:08
mzanettiMacSlow: anyways... IMO you're better off using make tryShell14:08
SaviqMacSlow, the makefile's in builddir14:08
mzanettiMacSlow: unless you really require unity to be started by upstart14:09
Saviqtedg, or even better14:10
Saviqtedg, just stop unity8-dash14:10
tedgSaviq, Ah, that's a good idea.14:10
tedgSaviq, Though, if that makes apps start faster you might get pressure to do that on all devices ;-)14:12
dandraderSaviq, mzanetti, btw, unity8-dash is never suspended because "ApplicationController::primaryPidForAppId FAILED to get PID for appId= "unity8-dash""14:13
Saviqtedg, is FINE14:13
Saviqdandrader, right, it doesn't have an appid14:13
dandraderSaviq, which actually helps me with debugging :D14:13
MacSlowmzanetti, Saviq: how do I get unity8 (via "make tryShell") to use/load the notifiation-plugin?14:14
SaviqMacSlow, you don't, tryShell will load the mock one14:15
SaviqMacSlow, ./run.sh should still launch the correct shell (and with my branch the correct dash, too)14:16
SaviqMacSlow, if it doesn't work for you, do you get any interesting output on the console?14:16
SaviqMacSlow, does it launch if you just go ./builddir/src/unity8 /14:16
Saviq?14:16
MacSlowSaviq, "./builddir/src/unity8 /" works14:17
MacSlowSaviq, what's the pin/password? :)14:18
mzanettioh... mterry collecting abstain votes again :D14:18
mterrymzanetti, gotta have 'em all!14:18
MacSlowSaviq, my local one?!14:19
SaviqMacSlow, yes14:21
SaviqMacSlow, with ./run.sh you won't need it14:21
SaviqMacSlow, yeah the / was meant to be ?, no need for that ;)14:21
MacSlowSaviq, yeah... working now14:22
Saviqmterry, oh I missed your query about wrong-pass review14:23
Saviqmterry, and I actually thought I asked mzanetti for it already https://code.launchpad.net/~mterry/unity8/wrong-password-handling/+merge/22918414:23
* Saviq greps logs14:23
mzanettiyeah... might well be14:23
mterrymzanetti, well regardless.  If you have time today, it would be swell14:24
Saviqmzanetti, I did, back on the 8th! ;)14:24
mzanettiyeah... I had a feeling14:24
SaviqTHAT TONIGHT'S GONNA BE A GOOD NIGHT14:24
mzanettihmm... not really, no14:25
mzanettimterry: btw... https://code.launchpad.net/~mzanetti/unity8/new-lockscreen-design/+merge/23047814:27
mterrymzanetti, bro...14:28
cwayneSaviq: chances horizontal-list gets landed for next week?14:28
mzanettimterry: ?14:28
Saviqcwayne, *for* next week for sure14:29
mterrymzanetti, I was expressing frustration at all the points of conflict14:29
mterry:)14:29
mterrymzanetti, but yours is only an hour old!14:29
mzanettimterry: yeah...14:29
cwaynecool beans14:29
mzanettiwhat's horizontal-list?14:32
cwaynemzanetti: a new renderer for unity that joe wanted14:40
mzanettiah. a card layout thingie14:40
mzanettiok. thanks14:41
greybackhey, can I get a volunteer to review these branches:14:43
greybackhttps://code.launchpad.net/~gerboland/unity-api/surfaceSizerCallback/+merge/23027014:43
greybackhttps://code.launchpad.net/~gerboland/unity8/initialSurfaceGeometry/+merge/23049014:43
greybackhttps://code.launchpad.net/~gerboland/qtmir/initialSurfaceGeometry/+merge/23049114:43
Saviqgreyback, don't touch UNITY_API_MICRO14:44
greybackSaviq: why not?14:44
Saviqgreyback, because you're not touching the library14:44
Saviqgreyback, *do* touch VERSION in include/unity/applications/CMakeLists.txt14:44
greybackSaviq: understood14:44
Saviqgreyback, also add -0ubuntu1 so that dch doesn't complain about native vs. non-native versions14:45
Saviqgreyback, doesn't *really* matter as train will overwrite anyway, but just for correctness :)14:45
greybackyeah I just expect train to fix that14:45
Saviqgreyback, wonder, could this not ba Q_PROPERTY?14:46
Saviqhmm probably not14:46
Saviqor maybe?14:46
greybackhmm, that's a thought14:47
=== gatox is now known as gatox_lunch
Wellarkdednick: https://bugs.launchpad.net/ubuntu/+source/indicator-network/+bug/132249015:12
ubot5Ubuntu bug 1322490 in indicator-network (Ubuntu) "Clicking on a connected network's checkbox disconnects & re-connects" [High,Triaged]15:12
Wellarkwe need to change the access point items to resemple checklist15:12
Wellark*radiolist15:13
dednickWellark: the deisgn has already changed. there is no more checkbox, only different color for active connection.15:14
dednickWellark: when i user taps an item, i just send activate.15:16
Wellarkdednick: ah, right. well anyway we then need to modify the item15:16
Wellarkto support the coloring at least and remove the checkbox15:16
Wellarkthat will also change the system-settings then15:17
dednickhttps://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/indicator-polishing15:17
dednickhttps://code.launchpad.net/~nick-dedekind/unity8/indicator-polishing/+merge/22870015:17
dednickWellark: ^15:17
Wellarkdednick: also, are we able to give the submenus titles properly?15:18
WellarkI don't think any indicator is doing that ATM15:18
dednickWellark: submenus? you mean current & available?15:19
dednickWellark: there is a submenu item around somewhere.15:20
dednickWellark: "com.canonical.indicator.section" i think15:20
Wellarkdednick: yes, but does it support specifying the labels such as "Current Network:", "Available Networks:"15:21
Wellarkthat .section should map directly to GMenuModel sections15:21
dednickWellark: not sure about that15:21
dednickWellark: would need to ask lars i think15:22
dednickor look at the qmenumodel code15:22
dednickor unitymenumodel rather15:23
=== dandrader is now known as dandrader|afk
Wellarkdednick: actually it comes down to the Factory in unit8 side15:26
Wellarkwill check from there15:26
Wellarkdednick: you can't land that polish branch without indicator-network being updated as well if you modify th accesspoint item15:27
Wellarkor are you setting the color based on the checked property?15:27
Wellarkif so, then the connected item should be on the top of the list with green color15:28
dednickWellark: the color is based on whether it's checked.15:28
WellarkI can add the sections in indicator-network side then separately15:28
dednickWellark: well, that's up to the backend to order15:28
dednickWellark: cool15:29
Wellarkdednick: sure, it is, but I was just thinking what happens if that polish branch lands without indicator-network being modified15:29
Wellarkand it would still work okay'ish15:29
dednickWellark: i don't understand. there don't need to be any changes to indicator-network for the polish. It''s just that they'll be colored instead of checked. Clicking another one will still activate it.15:38
dednick"checked" is just a state in this instance; not a ui element15:38
CimiSaviq, which one is clickscope?15:39
SaviqCimi, apps15:39
CimiSaviq, ok15:39
CimiSaviq, ap tests are failing here15:39
CimiStateNotFoundError: Object not found with name 'QQuickLoader' and properties {'scopeId': 'clickscope'}15:40
Cimidid you ever found it?15:40
Cimi(trying PYTHONPATH=../tests/autopilot autopilot run unity8.shell.tests.test_emulators.DashEmulatorTestCase.test_search )15:40
CimiSaviq, ?15:46
SaviqCimi, do you have unity-scope-click installed?15:47
CimiSaviq, why is this not deps of unity8-autopilot?15:47
SaviqCimi, good question, it's a recommends of unity815:48
SaviqCimi, so not sure how you ended up without it15:48
CimiSaviq, because recommends is not dep?15:48
Cimishould be dep for unity8-autopilot15:48
SaviqCimi, yeah, but everything installs recommends by default15:50
SaviqCimi, but sure, should be a dep of unity8-ap15:50
CimiSaviq, not if you dpkg15:50
CimiSaviq, only if you install through apt afaics15:50
Cimiwith out the flag to skip recommends15:50
SaviqCimi, yeah, and why would you dpkg?15:50
CimiSaviq, when I install local deps?15:51
CimiSaviq, not everything is in the archive15:51
Saviqif you just went "apt install unity8-autopilot", you'd get it15:51
CimiSaviq, but I am installing my local build of unity8-autopilot15:51
SaviqCimi, why are you doing that?15:51
SaviqCimi, when you then run them from the builddir?15:52
CimiSaviq, I did install unity8 build15:55
CimiSaviq, with scope settings, mocks etc15:55
SaviqCimi, which is not needed15:56
SaviqCimi, just ./build.sh, make -C builddir install15:56
SaviqCimi, and autopilot from tests/autopilot will use the unity8 from builddir/install15:56
CimiSaviq, anyway, how can I select the search button in the header?15:57
Cimitried http://paste.ubuntu.com/8027597/15:57
=== gatox_lunch is now known as gatox
SaviqCimi, what ap test are you adding btw?15:58
Cimiunfortunately we cannot assume anymore the search button is topmost right15:58
Saviqmzanetti, pstolowski, https://code.launchpad.net/~mzanetti/unity8/dash-caching/+merge/230487/comments/55994415:58
CimiSaviq, that I can open and exiting the scopes15:58
SaviqCimi, you need to select it by name15:58
Cimis/scopes/settings15:58
SaviqCimi, and why isn't that a QML test?15:58
CimiSaviq, I might do it with qml in the end15:59
SaviqCimi, yes, do that15:59
CimiSaviq, but for the moment I need to fix the search test15:59
CimiSaviq, which assumed the search button was on the right edge of the container15:59
SaviqCimi, lemme have a look15:59
Cimiline 142 dash.py16:00
Saviqjeez we have like 50 OverflowPanels :|16:01
SaviqCimi, objectName = search_header_button16:06
CimiSaviq, where does this come from?16:06
SaviqCimi, from the SDK16:07
CimiSaviq, ouch16:07
CimiSaviq, so for setting action would be?16:07
SaviqCimi, "search_header_button"16:07
Cimiright16:07
Cimiweird16:08
Cimiarg16:09
Cimithis gets all of them16:09
mzanettiSaviq: https://code.launchpad.net/~mzanetti/unity8/dash-caching/+merge/230487/comments/55994716:10
Saviqmzanetti, in unity9816:10
Saviqunity98!16:10
Cimiwhich doesn't make sense because should only be one16:10
mzanettiSaviq: still different than unity98-dash, no?16:10
Saviqmzanetti, yes, but where did you set the dir to *be* unity8-dash?16:10
Cimiok nevermind16:10
Saviqmzanetti, or does it use standard cache dir?16:10
mzanettiSaviq: CacheLocation is patched on ubuntu16:10
mzanettiyes16:10
Saviqmzanetti, ok, it was hardcoded to unity8 before16:11
mzanettiSaviq: ~/.cache/<appid>/...16:11
Saviqmzanetti, yeah ok good16:11
mzanettiwas it?16:11
Saviqgrrr16:11
mzanettiSaviq: ok... I just verified16:12
mzanettiSaviq: seems we're good16:12
mzanettiSaviq: also my test dis save things to ~/.cache/unity8-dash/16:12
Saviqmzanetti, yeah, sorry for the noise16:15
mzanettinw16:24
mzanettibetter checking twice than having a nasty bug with corrupted files16:24
=== dandrader|afk is now known as dandrader
josharensonAnybody have suggestions for power management use cases? Preferably something that can be automated for consistencies sake? I'm picking up a power monitor today and I'd be cool to get some numbers today/tomorrow.16:53
greybackjosharenson: /me most interested in idle power consumption. unity8-dash does appears to use a steady 3% of CPU on manta at the moment16:57
greybackif you could start a timer on the clock, would be interested to see how power-usage heavey QtComp is in rendering that16:58
josharensongreyback, any specific interval seem interesting?16:58
josharensonlong term / short term16:58
greybackjosharenson: I'd have to leave that to you, to see what duration gives figures that are reliable. I've no idea really.16:59
josharensongreyback, ack. I'll try a lot of things17:01
=== alan_g is now known as alan_g|EOD
CimiSaviq, finished and pushed17:17
CimiSaviq, feel free to need fix tomorrow (not too much :))17:17
dandradergreyback, https://code.launchpad.net/~dandrader/qmir/dashRedraw_lp1353374/+merge/23052518:39
greybackdandrader: wrogn project dude ;)18:40
greybackqtmir18:40
dandraderoh18:40
dandradergreyback, https://code.launchpad.net/~dandrader/qtmir/dashRedraw_lp1353374/+merge/23052718:43
Saviqcool18:43
Saviqdandrader, I'll have to ask you to try and dig your teeth into bug #129562318:44
ubot5bug 1295623 in Unity 8 "Sometimes input breaks and only edges are responsive" [Critical,Triaged] https://launchpad.net/bugs/129562318:44
greybackdandrader: think you can you test it?18:44
Saviqdandrader, people have been seeing it more often recently18:44
Saviqdandrader, and you know your way around input best of us all18:44
dandradergreyback, you mean writing a test for that lp1353374 fix?18:44
dandraderSaviq, ok18:49
dandraderits description is quite outdated though18:50
=== dandrader is now known as dandrader|afk
greybackdandrader|afk: if possible yeah. Don't stress if it's tough19:10
cwaynewould it be possible to have a scope that *only* showed up in an aggregator scope?19:10
Saviqcwayne, no19:11
Saviqcwayne, we totally dropped the concept of scope visibility afaict19:12
cwaynedamn, ok19:13
Saviqcwayne, hmm or maybe I'm lying19:13
Saviqcwayne, looking at https://code.launchpad.net/~unity-team/unity-scopes-shell/overview/+merge/227745 around line 70119:14
Saviqcwayne, it goes continue if invisible19:14
Saviqcwayne, OTOH there's a FIXME: filter invisible scopes? just after that19:14
cwaynehm yeah19:15
Saviqcwayne, sounds like you should check :)19:15
cwaynewould that be set in the ini file?19:16
Saviqcwayne, yes19:16
Saviqcwayne, but the docs don't mention it...19:17
cwaynetrying it now19:17
cwaynethat could in theory solve at least one of my current problems though, so here's hopin'19:18
Saviqcwayne, if I can read at all http://bazaar.launchpad.net/~unity-team/unity-scopes-api/trunk/view/head:/src/scopes/internal/ScopeMetadataImpl.cpp#L45519:19
cwayneSaviq: seemed to work \o/19:19
cwaynedoesn't show up in overview, but does in aggregator19:19
cwaynecool, thanks19:19
Saviqcwayne, looks like both a) doc and b) FIXME need fixing ;)19:20
Saviqcwayne, because it does filter out invisibles already19:20
cwayneSaviq: btw emblems landed today, we've just tried them and they look a bit stretched, what aspect ratio does an emblem expect?19:20
Saviqcwayne, and I don't think we should filter out if they're favorited (which it doesn't do)19:20
Saviqcwayne, square19:20
cwayneSaviq: any size in particular?19:20
Saviqcwayne, 2GU19:20
Saviqcwayne, so 60px square usually19:21
cwaynessweeny: ^19:21
ssweenycwayne, Saviq thanks19:21
Saviqcwayne, can I see them somewhere?19:21
cwaynessweeny: ^ got a screenshot?19:22
Saviqssweeny, ideally using colour #80808019:22
Saviqssweeny, this way they'll be colourized to foreground color (where it makes sense of course)19:22
Saviqsame applies to attributes actually19:22
ssweenySaviq, 808080 over transparent bg?19:23
Saviqssweeny, yup19:23
ssweenyok19:23
Saviqwe might actually need to add keyColor to the scope customization options19:23
Saviqbecause if somewhere someone actually wants #808080, that will be replaced with foreground19:24
Saviqkarni, you'll be pleased to know that we broke network caching in the dash... https://code.launchpad.net/~mzanetti/unity8/dash-caching/+merge/23048719:24
Saviqkarni, so it'll get much better soon19:24
Saviqgreyback, if around, was there a bug about apps not being refocused when opening urls? (when they are running already)19:46
greybackSaviq: news to me19:47
Saviqgreyback, TBH I think this predates qtcomp19:47
Saviqtedg, does url dispatcher request running app focus when sending a URL to it?19:49
greybackSaviq: it does.19:50
Saviqgreyback, hmm hmm19:51
Saviqgreyback, wonder why it doesn't close indicators then19:53
Saviqhuh now it did19:53
Saviqhah19:53
Saviqit doesn't close if already focused?19:53
Saviqyeah19:53
Saviqgreyback, when app is in front, does shell even get the focus request?19:55
greybackSaviq: if ual asks for focus, shell gets that request19:56
Saviqgreyback, ok, so it just doesn't act if nothing changes, should be easy to fix19:57
greybackSaviq: yep. Is unity8 bug so19:57
=== dandrader|afk is now known as dandrader
=== salem_ is now known as _salem

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