=== jono is now known as Guest94995 === duflu_ is now known as duflu [08:53] morning [08:54] tsdgeos, question: now that we have delegateCreationBegin/End, do we actually gain much by filtering the actual model instead of just clipping it? [08:54] "it" being the view [08:57] Saviq: you save having to do the math of how tall two rows are [08:57] and also delegateCreationBegin/End still do buffering outside the delegateCreationBegin/End area [08:57] so you'd end up with a row more [08:58] tsdgeos, but I do have do to the math for how tall the full view is going to be ;) [08:58] but for the VerticalJournal we'll need clip [08:58] since they want "half" items there [08:58] tsdgeos, and then do the proxying with the LimitProxy [08:59] yeah haven't thought about it but it's a possibility yeah [08:59] tsdgeos, yeah, for VJournal we'll need to clip - and we'd need to calculate how many items to show anyway, which would be pretty tricky, too [08:59] tsdgeos, if we wanted to filter, that is [09:00] trueth [09:00] Saviq: can we get a test for making sure no like is longer than 120 chars? that'd way i'd hate the rule and not the messenger [09:00] s/like/line [09:00] tsdgeos, you mean dandrader? ;D [09:02] i mean the messenger :) [09:04] tsdgeos, I'm not sure - we could, of course, but did we ever actually agree to impose that limit... [09:06] tsdgeos, would you have time to review https://code.launchpad.net/~unity-team/unity8/unity-scope-tool/+merge/199831 ? [09:09] Saviq: sure [09:10] tsdgeos, thanks [09:23] tsdgeos, does the fact that PathView is always looped feel weird to you, too? do you know if there's any reason (like it being difficult otherwise) to not allow disabling it via a property or something? [09:27] Saviq: yeah i see it's a bit weird, but haven't looked at it in detail so no idea if it'd be hard to fix or not [09:28] tsdgeos, ok thanks [09:28] * Saviq just looked at Carousel.qml and longed for using a PathView there ;P [09:29] yeah :D === dandrader is now known as dandrader|afk [10:01] tsdgeos, is GenericName required at all? [10:01] Saviq: nope [10:01] http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html [10:01] tsdgeos, dropping, it doesn't make sense [10:02] tsdgeos, done [10:04] Saviq: do we care for i18n of the unity-scope-tool ? [10:04] tsdgeos, hmm we should - domain not set, is it? [10:05] tsdgeos, or no i18n at all is there... [10:05] Saviq: we are not calling i18n at all no [10:06] that's what i was asking [10:06] should we [10:06] and if we should [10:06] should we extract to the same .pot or to a different one [10:06] my vote is for yes and different [10:06] tsdgeos, yeah, that's what I was thinking, which means we need a separate qml dir for it [10:06] but that's just me, i can understand if we say "dev tool, learn english" [10:07] tsdgeos, I think let's go for english now, as it's not trivial to have a separate .po* [10:07] and wouldn't like to "pollute" the unity8 domain [10:09] and we can always add it later [10:14] ok [10:14] the time test fails here btw [10:14] is it locale dependent? [10:14] Actual (formatter.timeString()): 03-01-2014 11:05 [10:14] Expected (time.toString("dd-MM-yyyy hh:mmAP")): 03-01-2014 11:05AM [10:16] tsdgeos, yes it is [10:16] tsdgeos, should pass under LANG=en_US.UTF-8 [10:17] can we force that in the make test step? [10:17] tsdgeos, we should, yes [10:22] when should (if it) input fall through? https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1257312 [10:22] Ubuntu bug 1257312 in unity8 (Ubuntu) "Input going through the wifi connection dialog when not in any input field" [Medium,In progress] [10:23] Cimi, never [10:23] Cimi, the bubble should be input-opaque, basically [10:23] Saviq: i'd go with http://paste.ubuntu.com/6683685/ but if you don't agree that's fine :D [10:24] Saviq, and on desktop? [10:25] Cimi, we don't really support that use case yet (it doesn't blur / go transparent) [10:25] tsdgeos, that's copy/paste ;) [10:25] tsdgeos, from main.cpp [10:25] tsdgeos, but yeah, let's [10:26] tsdgeos, fixed [10:27] Saviq: i'm getting lots of ": SyntaxError: Unexpected token u" [10:28] when running the tool [10:28] any idea where that comes from [10:28] ? [10:28] tsdgeos, JSON parsing [10:30] so which json is wrong? [10:30] tsdgeos, none should be wrong, maybe the overridden one? [10:31] do i have a too old unity-scopes-shell? [10:31] tsdgeos, possibly [10:31] tsdgeos, try running under UNITY_FORCE_NEW_SCOPES=1 [10:32] i was getting that error too [10:32] always wondered where is it from :) [10:32] it gets very unhappy then [10:32] https://code.launchpad.net/~cimi/unity8/fix-1257312/+merge/200385 [10:32] Saviq: http://paste.ubuntu.com/6683708/ [10:32] tsdgeos, yeah, you don't have any scope [10:33] tsdgeos, you need the onlinemusic scope from ppa:ubuntu-unity/demo-stuff [10:33] tsdgeos, sorry about missing that detail :) [10:33] you can also use the awesome --scope-dir param [10:34] mhr3, assuming you build it somewhere locally, that is? [10:34] right [10:34] a built checkout of lp:unity-scopes-api would do [10:35] mhr3, is "overrideCategoryJson" available anywhere already? [10:35] mhr3, as in, in a package? [10:35] Saviq: mhr3: so i need ppa:ubuntu-unity/demo-stuff or lp:unity-scopes-api or both? [10:35] Saviq, you approved it yesterday, so should be in demo-stuff [10:35] tsdgeos, just the ppa should be enough, /me checks [10:36] yep, ppa only should suffice [10:36] somehow I didn't get the package :/ [10:36] is ppa:ubuntu-unity/demo-stuf the correct name? [10:37] add-apt-repository complaiined [10:37] Saviq, oh, i did some renames, it's now considered older than what was there [10:37] Saviq, so just force reinstall [10:37] mhr3, yup, see that [10:37] tsdgeos, +f [10:37] tsdgeos, but unity-team [10:37] i had the f there [10:37] :D [10:37] tsdgeos, ppa:unity-team/demo-stuff, sorry [10:38] that worked [10:38] hmmmm [10:38] Saviq: apt-get tells me i already had unity-scope-onlinemusic installed [10:39] tsdgeos, the new one? [10:39] tsdgeos, yes, different version [10:39] so how do i install the new one? [10:40] tsdgeos, apt-cache policy unity-scope-onlinemusic? [10:41] tsdgeos, 0.1+14.04.20131129.1~bzr59-0ubuntu1~ubuntu14.04.1 is there in the ppa, is higher version than in archive [10:41] i guess i needed an apt-get update... [10:41] :D [10:41] lol [10:41] tsdgeos, that helps, yes [10:41] mhr3, why do we need the popup for overriding the category? [10:42] Saviq, cause i wanted "apply" button :) [10:42] mhr3, we can have the apply button without the popup ;) [10:42] Saviq, also, you get more real estate :) [10:42] mhr3, yeah, but then should we maybe drop the one from the side bar altogether [10:43] mhr3, as it's confusing [10:43] Saviq, maybe, i considered that a view-only [10:43] mhr3, should've made it so ;D [10:43] though couldn't find editable property [10:43] not that i looked for it too much [10:44] mhr3, activeFocusOnPress: false should work [10:45] mhr3, or well... file:///usr/share/ubuntu-ui-toolkit/doc/html/qml-ubuntu-components0-textarea.html#readOnly-prop [10:45] yea, i imagined that it'll be very accurate naming :P [10:47] uh, still displays the cursor [10:50] Saviq: that didn't help at all [10:50] Saviq: http://paste.ubuntu.com/6683774/ [10:51] tsdgeos, `status scope-registry`? [10:51] Saviq, btw why are the card titles so broken? the antialiasing seems to be majorly broken [10:51] tsdgeos, `restart scope-registry` [10:52] mhr3, because it's not using cards, but the old UI [10:52] mhr3, merge into new-scopes and try there? [10:52] Saviq, nope, it's using cards for me [10:52] it's just terrible on the white background [10:53] oh wait [10:53] no you're right [10:53] wrong branch [10:54] Saviq: ok, the restart worked [10:54] Saviq, still bad on category titles though [10:54] mhr3, yeah, probably a shadow or some such [10:56] mhr3, style: Text.Raised [10:56] styleColor: "black" [10:57] tsdgeos, mhr3, bug #1265766 [10:57] bug 1265766 in ubuntu-ui-toolkit (Ubuntu) "Read-only TextArea gets focus and displays the text cursor" [Undecided,New] https://launchpad.net/bugs/1265766 [10:57] please confirm [10:58] done, thx [10:59] tsdgeos, you still won't be able to override the categories [10:59] tbh i don't think that's a bug [10:59] tsdgeos, not unless you rebase onto lp:~unity-team/unity8/new-scopes [10:59] if you don't want to get the focus, disable it :D [11:00] tsdgeos, should default to false if read only, IMO [11:03] Saviq, i guess the "black" should be a global const [11:03] it's on too many places [11:03] mhr3, true [11:03] mhr3, should be part of the palette [11:04] didn't know there's such a thing, but sure :) [11:05] mhr3, file:///usr/share/ubuntu-ui-toolkit/doc/html/qml-ubuntu-components-themes0-palettevalues.html [11:05] mhr3, used like so file:///usr/share/ubuntu-ui-toolkit/doc/html/qml-ubuntu-components-themes0-palette.html [11:06] * Saviq finds it funny that he can link to a file on his own file drive ;) [11:06] s/file/hard/ [11:06] Saviq, though shell palette and apps palette should be different, no? [11:06] mhr3, it is [11:07] mhr3, they're per-theme [11:07] ah, ok [11:08] heh, i noticed just now that these were file:/// urls [11:08] the repeater thing for the textarea is a bit weird [11:08] weird, but works :) [11:09] and it wasn't easy to get it into that state :P [11:14] mhr3, h4x0red you ;D [11:15] Saviq, grrr, i knew unity8 is just a trojan! [11:15] mhr3, it's unity7 that spies on us all the time! [11:20] ah, so you're saying we just took it further? :) [11:20] more "features" :) [11:20] Saviq: mhr3: ok, added a final comment and my approval, if you want i can top approve too [11:20] tsdgeos, thanks! [11:22] tsdgeos, mhr3, +1 on Q_INVOKABLE data() instead of the repeater [11:24] i'd prefer QVariantMap get(int index) then [11:31] mhr3, that's something that we try not to do [11:31] mhr3, to not encourage getting the whole dataset unnecessarily [11:32] Cimi, you should've re-submitted Mirco's merge proposal, not do a new one === dandrader is now known as dandrader|lunch [11:32] Cimi, /me does [11:34] Cimi, and next time please just branch from the branch in question and commit on top of it, to not lose the history [11:49] Saviq, but it's just one row of the dataset [11:49] mhr3, that's still more than just one role [11:50] one role is ENOTENOUGH :P [11:50] Saviq, I redid the branch [11:50] Cimi, thanks [11:50] Saviq, I did not use his [11:50] Cimi, yes I know, that's what I said - you should have used his, merge trunk, do your changes [11:50] Saviq, there were conflicts [11:50] Cimi, so that his commit is still there in the history [11:51] Cimi, not important, just in general we should try and maintain the history [12:13] Saviq, https://code.launchpad.net/~unity-team/unity8/carousel-shader/+merge/196942 ? === _salem is now known as salem_ [12:13] Cimi, animation still missing? [12:14] oh ok [12:36] dandrader|lunch: i think i've addressed all your concerns [12:36] dandrader|lunch: can you please recheck? === alex-abreu is now known as alex-abreu|off === dandrader|lunch is now known as dandrader === greyback_ is now known as greyback|lunch [14:04] mhr3, back to data() vs. get() etc., maybe it'd be enough if picked the value from the option selector? [14:04] mhr3, by adding a property on the selector delegates? [14:04] or properties, for that matter [14:06] mhr3, if the selector doesn't give you access to the selected item, just go 'onSelectedChanged: categorySelector.selectedItem = categoryDelegate' or so [14:07] what's to do to get a package from "proposed" to "release" in trusty? [14:14] Zhenech: i'd say probably #ubuntu-devel is a better channel to ask that question [14:14] ups, i thoght this is actually -motu === greyback|lunch is now known as greyback [14:33] Saviq: standup? [14:33] Cimi: ↑ [14:33] tsdgeos, rigt [14:38] mhr3, tsdgeos I just pushed a simplified "get data from the model" approach to the scope tool [14:40] cool [14:40] let me see [14:46] Saviq, aaah, clever [14:47] mhr3, ;) [14:47] mhr3, offtopic, got a SA power adapter? [14:48] if not, it's about time [14:48] Saviq, that reminds me, what SA power adapters look like? [14:48] nothing universal will work [14:48] mhr3, https://www.google.com/search?q=south+africa+plug&client=ubuntu&hs=A0m&channel=fs&tbm=isch&source=iu&imgil=ROKN542cw-aGfM%253A%253Bhttps%253A%252F%252Fencrypted-tbn1.gstatic.com%252Fimages%253Fq%253Dtbn%253AANd9GcQrjgzIyU1wIrARhw2YsoPCvNW8ktNhNOVJfn_MfQvpFxUuxkf4%253B346%253B346%253Bho-Yz_gQZBODQM%253Bhttp%25253A%25252F%25252Fwww.power-cord.org%25252Fsouth-africa-plug.html&sa=X&ei=rs3GUsW4O-m04ASQ0IGIBQ&ved=0CDIQ9QEwAg&biw=1219&bih=781#facrc=_&i [14:48] mgdii=_&imgrc=ROKN542cw-aGfM%3A%3Bho-Yz_gQZBODQM%3Bhttp%253A%252F%252Fwww.power-cord.org%252Fuploadfile%252Fsouth-africa-sabs-israel-power-cords-n02-1439.jpg%3Bhttp%253A%252F%252Fwww.power-cord.org%252Fsouth-africa-plug.html%3B346%3B346 [14:48] yikes.. [14:48] eh [14:48] mhr3, it's a huge 3-prong one [14:48] Saviq, think dednick will have one, will try to borrow from him [14:48] mhr3, right, makes sense [15:09] larsu: hi! Do you have a minute for a question about indicator-messages? [15:11] Saviq: that looks more reasonable :-) [15:11] mhr3: Saviq: so want me to top-approve? [15:11] tsdgeos, fine by me [15:12] tedg: or do you (see 4 lines above ^)? [15:14] charles: or you ^ :-) [15:15] mardy, What's up? [15:16] tedg: I'm having a look at bug 1130084 [15:16] bug 1130084 in unity (Ubuntu) "restore IM presence at startup" [Medium,Triaged] https://launchpad.net/bugs/1130084 [15:16] tedg: I'm trying to understand how things work [15:17] tedg: so, it seems that indicator-messages fires a DBus signal (StatusChanged), but there's no one listening to it [15:17] mardy, For the most part we just try to reflect what telepathy has as it's state. [15:17] mardy, It seems to me that telepathy should be saving that state and restoring it. [15:17] tedg: because telepathy-indicator.desktop has X-GNOME-Autostart-Delay=15 [15:18] tedg: is there a way for telepathy-indicator to get the current status from messages-indicator? I tried inspecting the latter with d-feet, but I couldn't find any DBus properties [15:20] mardy, It could just look at the status of the action. But, really, indicator-messages shouldn't be driving the state. It should be representing the state that telepathy has. [15:21] mardy, So telepathy-indicator should look to telepathy to find out what the state is. And request and update based on that. [15:21] tedg: ah, then I understood it the other way round [15:23] mardy, For debugging you can get the state with: $ gdbus call --session --dest com.canonical.indicator.messages --object-path /com/canonical/indicator/messages --method org.gtk.Actions.Describe status [15:23] tedg: so, when the user changes the presence status from the indicator, a signal is emitted, and any presence-capable app should try to match that status and save it in its config? [15:24] mardy, Correct, we don't require a save, but that makes sense to me. [15:25] tedg: I see... I'll try to fix telepathy-indicator then. [15:25] tedg: and do you know the reason of that 15 seconds delay? Isn't that a bit too much? (or is msecs?) [15:25] mardy, Why not fix telepathy? [15:26] Seems this should be a mission-control feature more than a bridge feature. [15:26] mardy, Not sure, I'm guessing it was a boot optimization thing. kenvandine might know. [15:27] tedg: I don't think so, mission-control probably doesn't even start by default. [15:27] mardy, We should really switching it an upstart job that is "start on started indicator-messages" [15:27] it was for boot speed [15:27] we delayed a bunch of things like that [15:27] to let stuff settle more [15:28] tedg: I disagree; mission-control doesn't save its presence status, and starting it by default would be annoying for people who don't use it [15:28] mardy, Hmm, seems you're right. IMHO, kinda silly, but eh, okay. [15:28] mardy, It shuts down if there are no accounts. So really it'd just check to see if it's configured and die. [15:28] mardy, I don't think that'd be too annoying. [15:29] tedg: I'd actually like if instead of reacting to a StatusChanged signal these bridges could register itself as DBus activatable, and indicator messages invoke a method on them when needed [15:30] mardy, If they're connected they need to be running, so I'm not sure dbus activatable is the right choice. [15:30] mardy, For instance you wouldn't want to start up XChat just to see if it wanted to change the status. [15:30] mardy, When it's only a bridge, that makes sense. But I'd rather have bridge be the minority case. [15:30] tedg: exactly, I would like them *not* to be running by default, but started on demand, if needed [15:31] mardy, How would we know if they're needed? [15:32] tedg: we always invoke SetStatus(...) on them, when the user changes the status from the menu; if they are running, they'll react to it (and maybe quit if the status is "offline"), if they aren't they'll be started [15:32] tedg: compare it to the current situation, where the *need* to be running [15:33] all the time [15:33] s/the/they/ [15:33] mardy, If they're connected to an IM network, they need to be running to keep the socket to the network open. [15:33] tedg: yes. But if the status is "offline", it would be nice if they quit [15:34] mardy, And you want us to restart everything possible if you set to "Available" ? [15:34] tedg: D-Bus doesn't restart already running services [15:35] mardy, I mean, I have IM clients installed that I'm not using. They shouldn't start. [15:36] tedg: mmm... I'm starting to see your point [15:36] tedg: I was assuming that this indicator-messages was written especially for things like telepathy, but now I realize that maybe telepathy is the weird one. Right? [15:36] Interestingly someone could set up an upstart job to start when it sees the DBus signal. [15:37] I think that telepathy kinda is. Though, frankly it's our biggest user. [15:37] tedg: I'm thinking about the telepathy-indicator bridge; my point is that if the status is "offline", it should not be running [15:38] And perhaps that's the special case. We should start only those special cases when going to an active status. [15:39] tedg: what about this: we keep everything as it is, but we add a way for presence-aware apps to register themselves, and get their SetStatus() DBus method called at the same time as StatusChanged is emitted [15:39] I wonder if the real solution here is just to bless telepathy is the ruler of everything status, and tell others to integrate there. [15:40] tedg: and we make telepathy be one (the only one?) of the implementors of this interface [15:40] I'd say that we destroy the interface and call telepathy directly. [15:41] tedg: +1 [15:42] I think that I need to think about the implications there a bit... but, it doesn't seem crazy right now :-) [15:43] tedg: on the practical side, I think I can quickly fix bug 1130084 in telepathy-indicator, without much effort; for the rest, maybe I can file a wishlist bug, and let's see who gets the time to do it? [15:43] bug 1130084 in unity (Ubuntu) "restore IM presence at startup" [Medium,Triaged] https://launchpad.net/bugs/1130084 [15:49] mardy, Makes sense. I think charles is the one scheduled to open up indicator-messages next. [15:49] Though that's for greeter work. [15:51] tedg: OK, thanks [15:51] mardy, how do you plan to store the last presence? [15:52] it's annoyed me for ages, i had considered storing that in gsettings but someone convinced me not to [15:52] kenvandine: not sure. Just an .ini file, unless telepathy-indicator is already using other stuff (DConf?) [15:52] * kenvandine wonders if accountsservice has anything for that [15:52] kenvandine, sure, you can story any property you want in accountsservice [15:52] i really want a global presence [15:53] kenvandine: if telepathy-indicator is the one who stores the settings and reads it, I think it can be a file private to it [15:53] we weren't really using accountsservice when i was looking into this [15:53] I think that you want to store the user selected status, not generally the status in GSettings. [15:54] The reason is that you don't want to store when the screensaver sets you to away. [15:54] good point [15:54] Where in account service you'd want to have the current status. [15:55] what does gnome-shell and kde do? [17:09] Saviq, re-run carousel shader, but I only see mocks on the desktop, no icons or so... what change? [18:26] Hello, anybody around for a review? [18:26] https://code.launchpad.net/~elopio/unity8/open_scope/+merge/200426 [18:33] Cimi: ping? ^ [19:42] mardy: sorry I was out before. Are you still around? [19:43] larsu: hi! No problem, I already solved the issue with tedg [19:43] mardy: awesome :) And happy new year! [19:44] larsu: thanks, same to you! === salem_ is now known as _salem