/srv/irclogs.ubuntu.com/2016/11/17/#ubuntu-unity.txt

Trevinhodmj_s76: hey, sorry... its quite late here... So I can't now, and andyrock is in holidays, but we'll take care of it.02:18
vigomorning07:44
cimimorning pstolowski, how are we supposed to proceed implementing what we discusses yesterday? you start working on the API then when you have it we add on the dash or I already add that to the dash? also, do we want to document this somewhere?09:22
pstolowskicimi, morning09:23
pstolowskicimi, there is no new api, you just need to support it via category renderer attribute, and then we fix scopes09:23
pstolowskicimi, and yeah, it needs to be documented in the scopes api doc09:24
cimipstolowski, when will you start that? can I add this to the dash today?09:29
pstolowskicimi, ah, I'll probably need to add it to CATEGORY_JSON_DEFAULTS in shell plugin, but that should be enough.09:31
pstolowskicimi, let me know when you have a branch and then i'll do it in the plugin and music scope, should be quick09:31
cimipstolowski, oki09:31
cimitsdgeos, still building but I hope that something like this will work http://paste.ubuntu.com/23489853/10:28
tsdgeosi guess10:29
tsdgeosyou'll want some expandable ones in the fake plugin thoguh10:29
tsdgeosso you can still see/less more10:29
cimiSaviq, interesting, if we set cardtool template collapsed-rows to 0 we were basically hiding see more/less10:30
cimiSaviq, maybe this is what you remembered?10:30
Saviqcimi, with how many rows?10:30
cimimmm10:31
cimiyeah 0 doesnt make much sense10:31
cimitsdgeos, yeah I will tweak just for the first index10:32
tsdgeoscimi: it means "show everything"10:32
tsdgeoswhich is not what we want here10:33
tsdgeoswe want "show 2 rows without see more/less"10:33
cimitsdgeos, need to test it10:33
tsdgeoswhat's the name of the channel with the bileto status reports for silos?10:34
* tsdgeos forgot10:34
tsdgeosSaviq: ↑↑10:34
Saviqtsdgeos, #ubuntu-ci-eng10:34
cimitsdgeos, yeah you're right10:36
cimitsdgeos, works http://paste.ubuntu.com/23489903/10:45
tsdgeosyou don't need the expanded one, no?10:46
tsdgeosafter all you're already changing expandable10:46
cimimaybe should rename to notExpandable10:46
cimitsdgeos, if I don't change readonly property bool expanded: baseItem.expanded || !baseItem.expandable10:47
cimitsdgeos, then height: expanded ? item.expandedHeight : item.collapsedHeight10:47
cimiand height will be of the expanded10:47
cimiso I need to force the collapsed height10:47
cimidoes this make sense?10:47
tsdgeoscimi: yes/no maybe10:48
cimi:D10:49
tsdgeosi guess it depends where you want to make this count10:49
tsdgeosand if you want item.exapndedHeight know that it will always be just 2 rows or not10:49
tsdgeosor you want to force that at the GSV level10:49
tsdgeosboth are ok approaches i guess10:50
tsdgeosbut yeah we need better names for those variables10:50
tsdgeoswas a bit of a headache reading10:50
tsdgeosexpanded exandanble and noExpandable10:50
tsdgeos:D10:50
cimitsdgeos, better name for template["expandable"] == false?10:54
cimilike, the property noExpandable10:55
cimithat could be notExpandable10:55
cimior forceNotExpandable10:55
cimior some other crap10:55
tsdgeosi like the force10:58
cimitsdgeos, problem is, if expandable is not set, the property forceNotExpandable is true11:21
cimitsdgeos, shall I check for the property to be defined, or I expect pawel to always define expandable to true?11:21
tsdgeosno11:23
tsdgeoscheck for it to be defined11:23
cimipstolowski, oki11:49
cimipstolowski, https://code.launchpad.net/~cimi/unity8/dash-forceNotExpandable/+merge/31113411:49
cimipstolowski, building on silo 2202 right now11:50
pstolowskicimi, cool, i'll create my part today too11:56
cimitsdgeos, https://code.launchpad.net/~cimi/unity8/dash-forceNotExpandable/+merge/31113412:01
=== JanC_ is now known as JanC
=== jhodapp_ is now known as jhodapp
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
cimipstolowski, managed to get something working?16:00
pstolowskicimi, i haven't started yet, about to16:01
=== JanC is now known as Guest90855
=== JanC_ is now known as JanC
=== alan_g is now known as alan_g|tst
=== dandrader is now known as dandrader|afk
=== alan_g|tst is now known as alan_g
pstolowskicimi, are we targeting vivid with this fix?16:47
cimipstolowski, I guess too16:51
cimipstolowski, for clicks? ask Saviq too16:52
pstolowski?16:54
pstolowskicimi, we've been told to land only critical fixes that are targeted for OTA16:54
pstolowskicimi, my stuff is in silo 2217; will try to test tomorrow morning (not sure how exactly since my phone and tablet are vivid)17:07
pstolowskithe scope will probably need a few iterations to make sure it looks good in all circumstances17:08
=== dandrader|afk is now known as dandrader
cimipstolowski, I only have vivid devices too17:09
pstolowskianyway.. i'll figure this out tomorrow. eod, cu!17:11
cimicu!17:11
mterryjosharenson: hey btw I am aware of your session-icon MP but have been too busy with snappy work17:18
mterryjosharenson: I will point out that depending on session-lightdm is a sure way to not land for a long time anyway17:18
mterryjosharenson: I think I need to pull out the mock refactoring in that branch17:18
josharensonmterry: haha ok17:18
josharensonmterry: 90% of the branch depends on the new mock...17:19
josharensonmterry: but I can backport if you think its a better idea17:19
mterryjosharenson: yeah.  I can't live without that mock refactor17:19
mterryThat's why I have like 6 branches depending on session-lightdm17:19
mterryBut I need to pull it out or they'll never land17:19
mterryWhen the u8 snap calms down...17:19
josharensonmterry: sure thing17:20
=== boiko_ is now known as boiko

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