/srv/irclogs.ubuntu.com/2014/01/02/#ubuntu-unity.txt

Akiva-Mobileis there a terminal lens programmed yet?04:47
Akiva-Mobileor scope*04:47
Akiva-Mobileand if its not, (i'll think of programming one), is unity in a state right now that it won't be rewritten as soon as I am done developing it?04:47
=== duflu_ is now known as duflu
xnoxAkiva-Mobile: what do you mean by terminal lens? one can execute commands with alt-f2 on the desktop (unity 7) and new style scopes have like manpages references etc (unity 8)06:02
Akiva-Mobilexnox: namely to be able to observe the results06:02
xnoxAkiva-Mobile: there are terminals to observe results =)))) (both on touch UI and desktop UI) not sure how security wise that will work, given that it's essentially arbitrary code-execution.06:03
Akiva-Mobilexnox: well, I'd rather have a terminal built into the dash06:04
Akiva-Mobilexnox: Which mind you, is very hard to program06:04
xnoxAkiva-Mobile: the way I use dash / terminals, i prefer improvements into the terminal apps over "terminal scope". And i treat dash more of a single query/output, instead of persistent "backscroll" + multiple commands/outputs.06:05
Akiva-Mobilexnox: okay. I want a dash with an embedded terminal.06:06
Akiva-Mobileits sort of like a plugin layer06:06
Akiva-Mobilebut more native.06:06
xnoxAkiva-Mobile: i'd discuss it on the mailing lists first. Cause indeed it sounds non-trivial to implement and it's not clear how-to implement it best.06:06
Akiva-Mobilexnox: oh yah; I found out hard it is to do this sort of thing by trying to program a syntax highlighter06:06
xnoxAkiva-Mobile: yeah, like e.g. Super+A starts apps lence, one could have Super+T to get a terminal lences. But lences don't have free-form UI to code which is needed for something like a terminal.06:07
Akiva-Mobilexnox: although I found a cute work around to do it by exploiting a record terminal command, and then doing an active sync with the text file it outputted.06:07
Akiva-Mobilexnox: I heard that, although I am not sure what it means06:08
Akiva-Mobilexnox: is it programmed in QT yet? I wonder if you could just somehow toss a qtextbrowser in there :P06:09
xnoxAkiva-Mobile: no idea, i don't hack unity/dash/lenses =)06:09
Akiva-Mobilexnox: heh06:10
Akiva-Mobilexnox: I really like unity06:10
Akiva-MobileI think a terminal lens, especially if you could do it if you could embed tty1 in there or something06:10
Akiva-Mobilewould be freaking godly06:11
tvoss_good morning :)06:22
Akiva-Mobiletvoss_: morning~06:29
tvoss_Akiva-Mobile, hey there and a happy new year06:36
Akiva-Mobiletvoss_: ITS ROSH HASHANA ALREADY?!06:36
tvoss_Akiva-Mobile, was just about to add: if you happen to have celebrated the new year :)06:37
Akiva-Mobiletvoss_: you lie06:37
Akiva-Mobile>:/06:37
tvoss_nope06:38
tsdgeoshi ho folks08:00
Saviqmoornin'08:06
tvoss_Saviq, morning :)08:17
Saviqtvoss_, o/08:18
tvoss_Saviq, happy arbitrary date change08:18
Saviq:)08:18
tsdgeosSaviq: answered in https://code.launchpad.net/~aacid/unity8/more_logs/+merge/19893308:38
Saviqtsdgeos, ok that's fine, thought you lost hope :)08:39
tsdgeosnah08:39
jalcine/b 1309:20
mhr3mzanetti, ping?09:31
mhr3mzanetti, anything still missing @ https://code.launchpad.net/~mhr3/unity-scopes-shell/json-overrides/+merge/199508 ?09:32
tsdgeoslarsu: you aware of the gsettings-qt tests not passing in qt5.2?09:34
larsutsdgeos: nope, thanks for letting me know (and happy new year)09:47
tsdgeoslarsu: happy new year09:48
tsdgeoslarsu: it's getting an assert inside qtdeclarative :-/09:48
larsutsdgeos: the the issue that mardy fixed get merged?09:49
larsumaybe that's still the one09:49
larsubut that wasn't an assert...09:49
tsdgeoslarsu: not sure, when i tried with qt5.2 from 2 weeks ago i had a different error09:49
tsdgeoslarsu: the error i had as the one with bindings not working09:49
tsdgeosthen i pulled and the binding thing worked but now i get an assert09:49
larsutsdgeos: ya, that other one is marked as resolved: https://bugreports.qt-project.org/browse/QTBUG-3523309:51
larsusigh09:51
tsdgeosthink i can create a simple test case09:54
tsdgeoslet me see09:54
tsdgeos_larsu: https://bugreports.qt-project.org/browse/QTBUG-3590610:21
larsutsdgeos_: thanks a lot!10:22
tsdgeos_no worries10:22
tsdgeos_that's a pretty bad bug tbh10:22
larsuis it?10:23
* larsu is not very familiar with qt's internals10:23
dandradertsdgeos_, will start reviewing your lp:~aacid/unity8/horizontalJournal branch now11:01
tsdgeos_dandrader: cool11:01
tsdgeos_checking what's wrong with the uitests11:02
tsdgeos_meh11:02
tsdgeos_that expand_expand is creeping everywhere11:02
tsdgeos_need to add some logging to the more_logs11:02
tsdgeos_so it stops failing and then can't repro anymore ^_P11:02
Saviq;)11:10
Cimiany code review I can do?11:20
dandradertsdgeos_, any thoughts on having HorizontalJournal & VerticalJournal vs Journal with a direction (horizontal | vertical) property?11:24
SaviqCimi, https://code.launchpad.net/~aacid/unity8/search_history_pointer_correct_pointer_position/+merge/19742311:24
CimiSaviq, yeah let's go for it, I forgot to update after the chat here11:25
tsdgeos_dandrader: good question11:25
tsdgeos_i guess the property is better from the "qml user" part11:26
tsdgeos_but it may end up as a bit more convoluted code11:26
tsdgeos_Saviq: ↑↑↑ ?11:26
SaviqCimi, you could take over https://code.launchpad.net/~macslow/unity8/fix-1257312/+merge/197740 and add the test as requested by tsdgeos_11:26
dandradertsdgeos_, right, but more work to implement11:26
Cimisure11:27
Saviqtsdgeos_, dandrader, not sure, tsdgeos_ can you assess how difficult / convoluted it would be?11:28
Saviqit does feel more consistent with QML views11:28
SaviqCimi, or https://code.launchpad.net/~unity-team/unity8/unity-scope-tool/+merge/199831 could use a review11:28
tsdgeos_Saviq: well i guess i can just have two inner classes that do all the work11:28
tsdgeos_and just create/delete them as needed11:28
tsdgeos_and have the Journal just be a front-facing class11:29
tsdgeos_that just has a pointer to the existing ones11:29
tsdgeos_that should not be much work11:29
Saviqtsdgeos_, how is it for code duplication atm?11:29
tsdgeos_Saviq: there's none11:29
tsdgeos_we have a shared class with the "duplicated" code :D11:29
Saviqtsdgeos_, right, good11:29
=== boiko_ is now known as boiko
dandraderI would say, the less API the better.  Thus one class with an extra property is better than two separate classes11:30
SaviqI agree11:30
tsdgeos_dandrader: Saviq: the problem introducing the "Journal" and then adding horizontal/vertical is that we need to find a generic enough name for stuff like rowHeight/columnWidth11:30
dandraderunless VerticalJournal and HorizontalJournal have very distinct implementations11:30
tsdgeos_since you can only set one of the other depending if you're horizontal/vertical11:31
Saviqtsdgeos_, well, you can set both, but one is ignored11:31
tsdgeos_Saviq: don't like that tbh11:31
tsdgeos_but it's a possibility11:31
dandraderhmm11:31
Saviqtsdgeos_, I think it makes more sense than trying to come up with a common name, really11:31
Saviqtsdgeos_, at least it's clear - and the docs just need to say only one is taken into account depending on direction11:32
tsdgeos_yeah11:32
Saviqtsdgeos_, otherwise we'll try and go for cellDimension or such, which will require even more documentation to be understandable11:32
tsdgeos_ok, let me finish adding all the national holidays to canonical admin, test the bugfix for the qt assert i just foind and i'll do that11:33
Saviqcheers11:33
SaviqCimi, here's another review you might like https://code.launchpad.net/~nick-dedekind/unity8/indicator.highlight-updates/+merge/19947111:33
dandrader-1 on the "cellDimension" approach11:33
SaviqCimi, so there's plenty to choose from11:33
Cimiok enough now :)11:33
Saviq;)11:34
SaviqCimi, just claim the review when you've decided which one to go for first :)11:34
CimiSaviq, I'll start with adding the test11:34
Cimisome 2014 coding11:34
SaviqCimi, cool11:34
dandradertsdgeos_, Saviq: Now I'm starting to think that having HorizontalJournal and VerticalJournal is indeed better than Journal + direction property. It will translate to less application code written. With the former you write 2 things: the component name and spacing prop (rowHeight or columnWidth). With the latter you write 3 things: component name, direction and spacing prop11:40
Saviqdandrader, it's just more consistent with QML Views, it's easier to switch, probably easier to find docs11:41
SaviqI don't think it's a huge matter, though, can go with either11:42
dandraderthe problem of a property (direction) having a side effect on another one (rowHeight or columnWidth) is not nice. properties should ideally be orthogonal to each other11:42
tsdgeos_dandrader: Saviq: i guess the question is that if you'd ever programatically want to switch from one to the other11:42
dandraderSaviq, e.g. in qml you have Row and Column11:43
Saviqtsdgeos_, yeah, that's what I was thinking, but then you can use a Loader11:43
Saviqdandrader, right, that's a counter-example indeed11:43
Saviqok, let's go for separate11:43
Saviqtsdgeos_, ↑11:44
tsdgeos_okidoki11:44
tsdgeos_Saviq: did you start a run of https://code.launchpad.net/~aacid/unity-mir/application_manager_tests/+merge/180898 ?11:50
Saviqtsdgeos_, yes11:51
Saviqtsdgeos_, looks pretty stale ;)11:51
tsdgeos_Saviq: it is11:51
tsdgeos_Saviq: i asked gerry about it11:51
tsdgeos_and he said11:51
tsdgeos_asdasgdafgsgaf11:51
tsdgeos_kind of :D11:51
Saviqtesting in unity-mir is kind of like that ;)11:52
tsdgeos_for me we can kill it11:52
tsdgeos_but there may be something of value11:52
tsdgeos_it's just sad noone cared the small number of days it did compile and pass11:52
tsdgeos_there's also sadness in noone caring about https://code.launchpad.net/~aacid/platform-api/papi.rules.typo/+merge/18235411:53
* greyback hears he's being talked about12:02
davmor2greyback: yeah that about not to ;)  Just accept the blame/praise and carry on :D12:04
greybackdavmor2: it wasn't a direct "he should do something" comment, so yeah I went back into my cave :)12:05
Saviqtsdgeos_, I'll try and push the papi fix through today12:09
tsdgeos_cool12:10
tsdgeos_greyback: it was about https://code.launchpad.net/~aacid/unity-mir/application_manager_tests/+merge/180898 where you tell me if you want me to spend time on making it compile again or we just discard it and forget about it12:10
greybacktsdgeos_: was the main blocker the fact we couldn't figure out how to get the tests to run on Jenkins?12:11
tsdgeos_greyback: ah right12:11
greybackthat's a problem we've to figure out at some stage12:11
greybacktsdgeos_: I would love if you could make another attempt12:12
greybackeven just to get 1 test in there, so the whole test framework is there, and unblocks that jenkins run problem12:13
=== dandrader is now known as dandrader|afk
tsdgeos_greyback: ok, i'll see what i can do/who can i poke12:19
tsdgeos_dandrader|afk: answered the implicitheight comment12:20
davmor2anyone any idea why trustys lightdm password entry won't accept input from the keyboard?12:29
=== dandrader|afk is now known as dandrader
dandraderdavmor2, If I'm not mistaken that's a known but. restarting lightdm in a VT works around it12:37
dandraders/but/bug12:37
davmor2dandrader: ah thanks12:37
dandraderdavmor2, I think mterry knows the details12:38
davmor2and logged in :)12:38
dandradertsdgeos_, was reading the scope toolkit UX doc regarding journals12:42
dandradertsdgeos_, so both the horizontal and vertical journals scroll vertically?12:43
tsdgeos_yep12:43
tsdgeos_it's just how the delegates get positioned that changes12:43
tsdgeos_Saviq: right? ↑↑↑12:43
dandraderI found that a bit surprising, considering their names12:43
Saviqdandrader, tsdgeos_correct12:44
tsdgeos_dandrader: well the naming is about the layouting12:44
tsdgeos_dandrader: remember the journal doesn't really scroll12:44
tsdgeos_you need to put it in a listview if you want to make it scroll12:44
dandradertsdgeos_, but what about that "maximum two rows of cards" constraint on the horizontal journal?12:45
tsdgeos_dandrader: that's when collapsed12:45
tsdgeos_i.e. like the current grids have 2 rows vs n rows when collapsed/uncollapsed12:46
dandradertsdgeos_, so that's implemented by the container holding that journal, not by the journal itself, right?12:46
tsdgeos_that's my idea yes12:46
tsdgeos_we just feed it a different model12:47
tsdgeos_like we do in the grid case12:47
dandradercool, makes sense12:47
* dandrader plays with "make tryHorizontalJournal"12:49
dandradertsdgeos_, so HorizontalJournal works just like the layout of words in a paragraph?12:50
tsdgeos_yep12:50
tsdgeos_left to right12:50
tsdgeos_and leave space on right if next doesn't fit12:51
dandradertsdgeos_, like when word-wrapping is disabled :)12:51
dandradertsdgeos_, so in a VerticalJournal, if items in a row have all the same height it will behave just like a HorizontalJournal13:07
dandraderfilling out the next row, in a left-to-right manner13:07
Saviqdandrader, yes13:13
Saviqdandrader, but in a Vertical all items have the same width - not so in a Horizontal13:13
Saviqdandrader, so they'd behave exactly the same if the items are square, for example13:14
Saviqdandrader, or at least constant width + height13:14
dandraderyes13:14
Cimido you have both ubuntu sdk and qt 5.2?13:18
Cimiqtcreator is not installed here13:18
Saviqdandrader, one thing that might still happen in the horizontal case is that we'll stretch the delegates to fill horizontally, either all of them or just the last one13:20
Saviqdandrader, although TBH there's nothing in visual design currently that actually uses a horizontal journal...13:21
=== greyback is now known as greyback|lunch
=== greyback|lunch is now known as greyback
tsdgeos_lol14:35
tsdgeos_i just realized this in mumble14:35
tsdgeos_warning: The VAD has been replaced by a hack pending a complete rewrite14:35
tsdgeos_Sad panda14:35
Saviqwth is VAD? ;D14:36
tsdgeos_no clue, but it's making the panda sad it seems14:36
=== dandrader is now known as dandrader|lunch
mhr3Saviq, mzanetti is still holidaying?14:56
mhr3Saviq, so maybe you want to give final ack to14:56
mhr3https://code.launchpad.net/~mhr3/unity-scopes-shell/json-overrides/+merge/199508 ?14:56
mzanettiSaviq: mhr3: o/ This was looking good. the only reason I didn't top-approve before xmas is because jenkins was complaining and it looked like something real14:58
* mzanetti -> back to slacking off14:58
mhr3ah, no, it was the crappy qmlplugindump thing, i workarounded that in another branch14:59
mhr3which is already merge14:59
mhr3d14:59
Saviqmhr3, mzanetti ack, will look it through15:05
=== boiko_ is now known as boiko
=== ricmm is now known as ricmm|sick
* tsdgeos_ wondering what took CI that much time to run16:11
tsdgeos_basically the fact that i had not pushed the stuff :D16:11
=== dandrader|lunch is now known as dandrader
Saviqtsdgeos_, oups, I think you over-did VerticalJournal...16:37
tsdgeos_did i?16:38
Saviqtsdgeos_, I think it was always meant to fill left-to-right16:38
Saviqtsdgeos_, not start at the highest possible spot...16:38
tsdgeos_errr?16:38
tsdgeos_that's not what the numbers said16:38
tsdgeos_on the screenie16:39
tsdgeos_it had numbers16:39
Saviqtsdgeos_, really? /me looks16:39
tsdgeos_and they were like i did16:39
Saviqtsdgeos_, indeed16:39
Saviqtsdgeos_, ok then, ignore me16:39
tsdgeos_for one time i read the specs... :D16:39
Saviqtsdgeos_, yeah, the numbers weren't there before and I didn't pay enough attention apparently :)16:40
Saviqtsdgeos_, got a dash_scope_on_load failure in more_logs!17:02
tsdgeos_yeay, let's see if they give anything intersting :D17:02
tsdgeos_Saviq: it seems like the scope it's not getting loaded17:08
tsdgeos_i.e. it is waiting for MockScope5 to get loaded17:08
tsdgeos_but only home.scope and MockScope117:08
tsdgeos_get loaded17:08
tsdgeos_which is weird17:08
tsdgeos_since in the previous test17:09
tsdgeos_MockScope1, home.scope  and applications.scope17:09
tsdgeos_so i'd expect at least applications.scope to load in this one too :-D17:09
mhr3perhaps the mock is just trying too hard to simulate real scopes :)17:12
Saviqlol17:14
Saviqtsdgeos_, we don't have timestamps do we?17:14
tsdgeos_Saviq: nope17:15
tsdgeos_i'll add those17:15
Saviqtsdgeos_, maybe it's just timing again? how long are we waiting there?17:15
tsdgeos_5 sec17:15
tsdgeos_but cimi did a branch with 10 sec17:15
tsdgeos_and we got it to fail to17:15
tsdgeos_o17:15
Saviqhmm ok17:15
tsdgeos_but yeah17:16
tsdgeos_should add timestamps anyway17:16
tsdgeos_ok, some more logs added17:25
tsdgeos_let's see what tomorrow yields17:25
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
cwaynemhr3, ping18:37
cwayneSaviq, mhall119 do we have a roadmap for the new unity scope api?18:43
Saviqcwayne, nothing specific that I know of, no - but the overall goal is to implement as much as possible for 14.04 (on unity8, not unity7)19:36
cwayneSaviq, so there's no documentation on for example which order scopes will be ported in?19:38
Saviqcwayne, I only know about the UI and the shell-facing API, thostr and lucio would know about the other parts19:41
cwayneSaviq, so do we expect the switchover to using the new api's by default to happen for 14.04?19:43
Saviqcwayne, that's the plan, yes19:43
cwayneSaviq, what about the renderers, do we have any idea when they'll be implemented?  (IIRC grid is done, and we're expecting Carousel soon?)19:45
Saviqcwayne, they're being done as we speak19:45
Saviqcwayne, well, maybe not right now, it's EOD after all ;)19:46
cwayneSaviq, :)  so as in, this/next week?19:47
Saviqcwayne, some should happen next week, more late Jan/early Feb19:48
Saviqcwayne, unless I delegate, at which point more should happen in between (there's the sprint in Cape Town mid-Jan)19:48
cwayneSaviq, and at that point will the scopes themselves start to be ported over?19:49
Saviqcwayne, it will happen in parallel19:49
Saviqcwayne, but again, more details on that you'll get from thostr and lucio19:50
cwayneSaviq, right, thanks.  is there any wiki or doc that you know of (even if just for the UI side?)19:51
cwaynedetailing the schedule/roadmap19:51
Saviqcwayne, that was your first question, wasn't it ;)19:51
cwayneoops, it was wasn't it19:52
cwaynesorry :P19:52
=== kdub is now known as kdub^lunch
=== salem_ is now known as _salem
=== kdub^lunch is now known as kdub

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