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

Saviqelopio, thanks to you, they're in silo, hopefully for release soon01:56
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== Cimi_ is now known as Cimi
=== Malsasa_ is now known as Malsasa
Cimimorning07:14
CimiSaviq, scope settings review :P07:20
CimiSaviq, what is "alt nav"?07:52
Cimibug with dash and manta08:54
Cimiopen sidestage, close08:54
Cimithen dash becomes unresponsive08:54
Cimiin reality is processing things but the texture is not updated08:55
Cimigreyback, ?08:55
greybackCimi: log a bug please08:56
SaviqCimi, yeah, need to crwbar-open the eyes first09:11
CimiSaviq, saw you worked till late09:12
SaviqCimi, till early, rather09:14
Cimihah09:15
Saviqbut!09:15
Saviqcheck this out http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-utopic/895/09:15
Cimicannot, stupid vpn09:15
Saviqno vpn for you09:16
Saviqhttps://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/895/09:16
SaviqCimi, ↑09:16
Cimi@_@09:16
Cimiwow09:16
Cimiwas used to red09:16
Saviqyikes autopilot failed on that completely, though :|09:17
* Saviq worried something got seriously wrong09:19
CimiSaviq, eyes working now?09:27
CimiSaviq, so what is Alt Navigation?09:28
SaviqCimi, https://drive.google.com/drive/#folders/0B32jwBcbaPloR1IyRF91ZWs0bFE09:28
SaviqCimi, see ebay09:28
CimiSaviq, Alt means secondary?09:35
SaviqCimi, alternative09:35
CimiSaviq, we never shortened variable names so far, btw09:35
SaviqCimi, I know09:36
Cimiwe should have called Alternative09:36
SaviqCimi, but they are all long enough already09:36
SaviqAlternativeNavigationFOo09:36
CimiSo what?09:36
Cimiis consistency...09:36
CimiI assumed it meant the Alt key09:36
Cimieither Alt key or a shortened for Alternative09:36
SaviqCimi, the Alt key means Alternative, so you assumed kinda right ;)09:36
Cimihah09:37
Cimidednick, speaking of shortening, in your indicator polishing branch, you used id: ani instead animation, as well as other shortening, can you not shorten?09:37
SaviqCimi, I understand your concern, but unless you come up with a better shorter alternative (pun intended), I think we should deal with Alt09:37
=== vrruiz_ is now known as rvr
CimiSaviq, SecondNavigation09:38
SaviqCimi, except it's the first one!09:38
Saviq;)09:38
Saviqleft-to-right that is09:39
CimiAlternateNavigation :P09:39
CimiPrimary then09:39
Cimior First09:39
dednickCimi: yeah. just add it to your comments.09:39
SaviqCimi, well, no, because it's neither first, nor primary either09:39
SaviqCimi, Alternate isn't much better09:39
Saviq2 letters09:39
CimiSaviq, anyway I commented that we need ellipsis09:40
SaviqCimi, sure, you could even review it09:40
CimiI was reviewing09:40
Cimi:)09:40
Saviqwith the exception that I redo half of it in https://code.launchpad.net/~saviq/unity8/header-customizations/+merge/23071909:40
Saviqto improve visuals and such09:40
CimiI wanted to review your small branches (so lazy I am), then I realised they are all dependent so I was starting from the first one09:41
SaviqCimi, good thinking :D09:41
CimiSaviq, they are tricky, you see they are 100 diff lines, you claim the review, and then you realise you're now in big trouble :D09:42
Cimiglad it doesn't depend on split greeter09:42
SaviqCimi, gotcha09:42
CimiSaviq, do we want to have this alt navigation in?09:43
SaviqCimi, yeah, I'll take it over09:44
CimiSaviq, then address those in your header customisation?09:44
SaviqCimi, or I can take Albert's branch over09:44
CimiSaviq, be aware that there are tags in that branch, so we need to delete them asap they got merged09:44
CimiSaviq, then you have to rebase your branches though09:44
SaviqCimi, just the first one09:45
SaviqCimi, since it's a chain09:45
SaviqCimi, I think I'll rebase and actually bring a bunch of the changes over, no point in having the branches split09:47
CimiSaviq, up to you, but would be best09:48
CimiI still have a look now09:48
SaviqCimi, I might need to re-chain them though (put alt nav last) in that case09:49
* Saviq doesn't want to :|09:49
mzanettiSaviq: you won't believe it :D09:58
mzanettiSaviq: current state of my device: touch input broken, mouse input working :D09:58
mzanettior rather :(09:58
Saviqmzanetti, sounds like you're it10:00
Saviqbug #129562310:00
ubot5bug 1295623 in Unity 8 "Sometimes input breaks and only edges are responsive" [Critical,Triaged] https://launchpad.net/bugs/129562310:00
Saviqmzanetti, I think this is the time to 'apt install qtdeclarative5-dbg qtcore5-dbg unity8-dbsym unity8-private-dbgsym`10:01
Saviqmzanetti, and step on it10:01
Saviqmzanetti, one thing10:01
mzanettiSaviq: I rebooted already10:01
Saviqmzanetti, NOOOOOOOOOOO10:01
mzanettiSaviq: and what I said is that its the other way round10:01
Saviqmzanetti, well, still10:02
mzanettiSaviq: edges were broken and apps were working10:02
mzanettiSaviq: anyways.. seems I'm getting this more often again lately10:02
mzanettiyesterday I had it twice10:02
Saviqmzanetti, so you were stuck to one app basically?10:02
mzanettiSaviq: yep, system-settings and indicators :)10:02
mzanettithat's it10:02
Saviqindicators?10:02
Saviqindicators are touch too10:02
mzanettiyeah... the panel is 2 gu's tall10:02
mzanettiin which area you can also pull it by mouse10:03
Saviqhmm can you10:03
Saviqmzanetti, no you can't10:03
Saviqmzanetti, the whole 2gu is touch10:03
mzanettiSaviq: err, yeah... the panel is more10:04
mzanettithe 2 gu is touch, the rest mouse10:04
Saviqmzanetti, ok, the whole panel is touch10:04
Saviqah wait10:04
Saviqmaybe I'm locked and that's why10:04
mzanettianyways... it was working, but no launcher and no spread10:04
Saviqmzanetti, hmm without -mousetouch I can't do anything with indicators on desktop10:04
Saviqmzanetti, and I really believe the whole panel is touch10:05
mzanettiso maybe some other issue10:05
Saviqsounds like the edge areas got stuck somewhat10:05
mzanettiI have a feeling that the lockscreen plays bad on me every once in a while10:05
Saviqor maybe went under the app10:05
Saviqmzanetti, btw https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/895/?10:05
Saviqthanks again10:06
* mzanetti gets out his pom poms10:06
mzanetti*\o/*10:06
Saviqmzanetti, there's a butt10:06
Saviqmzanetti, https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/3411/?10:06
Saviqbut I very much doubt related10:06
mzanettioh man...10:06
Saviqwaiting for some other run after this got merged10:06
mzanettiok... all fail... so there's chances its some single thing that breaks it10:06
Saviqmzanetti, well, half of 'em10:07
Saviqbut I ran them all yesterday on mako and got all OK10:07
Saviqbefore landing the silo10:07
Saviqthe other device is currently broken totally, so that failed10:07
CimiSaviq, reviewed https://code.launchpad.net/~aacid/unity8/alt_nav_support/+merge/23005410:08
laze1989Any chance that the AppIndicator's API will change with Unity 8? I see limitations that the indicators are now traditional menus (e.g., in Chrome's new Notification Icon). Windows and OS X got it right.10:10
laze1989This concerns especially the desktop part.10:10
Saviqlaze1989, we're still quite far away from desktop implementation, there's no extensive design for the next iteration of Unity on desktops10:11
Saviqlaze1989, but please file a bug against ubuntu-ux so that we don't misplace your concern10:12
CimiSaviq: 180+        // only way to convert string to color 181+        var c = Qt.lighter(arguments[k], 1.0);10:20
Cimiseriously?10:20
SaviqCimi, ideas welcome10:20
Cimithis is in C++ right?10:21
SaviqCimi, that's JS10:21
CimiSaviq, so we can write a parser10:21
SaviqCimi, no we can't10:21
CimiSaviq, I meant, Qt.lighter is C++?10:21
SaviqCimi, yes, it is10:21
Cimithe algorithm10:21
Cimiyeah10:21
Cimiso there is no benefit of writing a parser10:21
SaviqCimi, you can't even write a parser, how do you parse "white"?10:22
SaviqCimi, this needs to go through QColor10:22
SaviqCimi, this is basically a no-op10:22
Cimi]I see10:22
Cimijust ugly10:22
Cimibut probably best thing10:22
SaviqCimi, it will do QColor() and multiply its .v by 1.010:22
SaviqCimi, and return it10:22
SaviqCimi, I *could* just make this whole function C++ though10:27
CimiSaviq, it is probably faster10:27
Saviqand maybe even use .v directly, having checked that it's a good enough value for us10:27
SaviqCimi, how about a TODO then10:28
Cimican we use shaders for those color operations?10:29
Cimifeel free to offend me10:30
CimiI don't know much about shaders :)10:30
laze1989I tried to articulate myself :-) https://bugs.launchpad.net/ubuntu-ux/+bug/135681610:43
ubot5Ubuntu bug 1356816 in Ubuntu UX "[indicators] change current indicator design for the desktop" [Undecided,New]10:43
=== MacSlow is now known as MacSlow|lunch
facundobatistaHola11:13
=== FJKong is now known as FJKong_CW
SaviqCimi, ugh, I need to reorder the branches, managed to do something real weird to them11:21
SaviqI think I pushed to the wrong branch at some point11:23
Saviqyeah11:23
dednickCimi: done with review11:28
dednickfixed i mean11:28
Cimidednick, thx11:28
Cimidednick, next step is potfiles for that11:36
Cimidednick, different branch though11:36
Cimidednick, if we are going to have a horizontal listview in the indicators, we cannot stay with the sound slider issue11:37
dednickCimi: yeah11:37
Cimidednick, either add a mousearea around it eating those events11:37
Cimior follow up with sdk11:37
dednickCimi: the menus aren't interactive, only the header is11:37
Cimiit is in the examples11:37
dednickCimi: yeah, i know11:38
Cimidednick, so if is just examples - fine11:38
Cimibut if it will be in unity8, nope11:38
dednickCimi: it's just the examples11:38
Cimianyway let's make them aware11:38
mzanettiSaviq: do you know if something regarding SIM pin unlocking landed in the indicators?11:38
Saviqmzanetti, not recently no11:39
mzanettiSaviq: hmmm... the button to open the unlock dialog doesn't seem to show up any more here11:39
mzanettiwill investigate11:39
Saviqmzanetti, wfm11:39
mzanettiSaviq: hmm... maybe I blocked it11:40
mzanettiand it just fails to ask for the PUK11:40
Cimidednick, can we bump UC to 1.1?11:40
dednickCimi: why?11:41
dednickCimi: oh, you mean dependency?11:41
Cimidednick, in the files11:42
Cimidednick, I want to make sure we use the latest11:42
Cimidednick, the wobbly animation on the icon in MessageComponents only work on expansion11:46
Cimidednick, is that wanted?11:46
dednickyes11:46
Cimiok11:46
Cimidednick, also is always on for unexpandable ones11:46
dednickya11:46
Cimiok11:46
Cimiwell rest is fine11:46
Cimidednick, I'd just bump to 1.1 as said11:47
Cimifor the file we touch11:47
SaviqCimi, https://code.launchpad.net/~cimi/unity8/scope-settings/+merge/230182/comments/560879 looks like a valid fail11:48
Saviqeven though at first glance you're not touching anything there...11:48
CimiSaviq, which failure?11:48
Saviqmaybe a compare() instead of tryCompare11:49
Cimiof the two?11:49
SaviqCimi, hmm let's see another run11:49
CimiI can try locally11:49
CimiSaviq, I also merged trunk lately11:50
SaviqCimi, well, yeah, but at least after my fix-foo branch it passed, maybe something else in the recent landing11:51
CimitestDash works here11:51
dednickCimi: ok, updated to 1.111:51
CimiSaviq, QML AbstractButton: Binding loop detected for property "implicitWidth" so annoying11:52
SaviqCimi, yeah, guess where that is ;)11:52
CimiSaviq, yep :P11:52
CimiSaviq, both tests pass here11:53
* dednick is hungry!11:53
SaviqCimi, ok, let's see, there's more qmluitest runs incoming11:53
Cimidednick, http://theburgerlist.tumblr.com/post/94443927060/le-bourguinon-bun-le-bun-riverside-feast-battersea11:54
=== alan_g is now known as alan_g|lunch
=== MacSlow|lunch is now known as MacSlow
CimiSaviq, did you see my comments in the previous review?12:10
SaviqCimi, they shouldn't be lost12:11
CimiSaviq, ok12:11
Cimithey are not12:11
Saviqnot lost or lost?12:13
Saviqjeez that's a chain12:13
Saviqnow someone needs to start reviewing the beginning of it...12:13
cwayneSaviq: thanks for landing horizontal-list so quickly :)12:15
Saviqcwayne, got even more goodness in store https://drive.google.com/drive/#folders/0B32jwBcbaPloR1IyRF91ZWs0bFE12:15
Saviqcwayne, just now we've like 7 branches dependant on each other that need to be reviewed ;)12:15
cwayne:D12:16
cwaynei saw those last night, super excited about it12:16
Saviqcwayne, do you know if any of the scopes try to send preview button color? I couldn't find any12:16
davmor2Saviq: if it's this chain it's okay https://www.youtube.com/watch?v=v1iuSmZHcwY12:16
cwayneSaviq: none of ours do yet12:16
cwayneSaviq: im happy to whip one up though if its helpful12:17
Saviqcwayne, nah, is fine12:17
Saviqdavmor2, I'm not sure our chain's as nice as this one12:18
Saviqdavmor2, I fear ours is more like http://www.hoax-slayer.com/images/mother-teresa-novena-chain-1.jpg12:18
Saviqnice url...12:18
Saviqdavmor2, or well, right now it's more like http://yazminmedia.com/wp-content/uploads/2010/08/blog-broken-chains.jpg12:19
Saviq@unity I need volunteers to review https://code.launchpad.net/~unity-team/unity8/tablePreviewWidget/+merge/230787 https://code.launchpad.net/~unity-team/unity8/seeLessStuckBottom/+merge/23078612:20
Saviqthere's some free karma in:12:20
Saviqhttps://code.launchpad.net/~saviq/unity8/grey-from-palette/+merge/23060312:20
Saviqhttps://code.launchpad.net/~saviq/unity8/move-pageheader/+merge/23060912:20
CimiSaviq, I can do table preview12:20
SaviqCimi, claim it!12:21
CimiSaviq, it is automatic to mhr312:21
SaviqCimi, as long as there's no "unity team" in the reviewers list it's fine12:22
CimiSaviq, I am lost12:23
CimiSaviq, which one is the first???12:23
Cimihttps://code.launchpad.net/~saviq/unity8/grey-from-palette/+merge/230603 ?12:23
Cimiseems like12:23
SaviqCimi, yes12:24
=== _salem is now known as salem_
mzanettihttps://code.launchpad.net/~saviq/unity8/update-qmltypes/+merge/230737 and https://code.launchpad.net/~saviq/unity8/grey-from-palette/+merge/23060312:43
mzanettiSaviq: ^12:43
Saviqmzanetti, thanks, on the ScopesOverview button... it should really use SDK's Button, I think Albert didn't see the support for an outlined one12:44
Saviqah feck it, same branch12:45
* mzanetti got lost... but I'm sure you'll just fix it12:45
mzanetti:D12:45
Saviqohnoes12:47
Saviqthe stroked button does not allow for font color changes /facepalm12:47
Saviqjeezuz12:48
mzanetti:D12:49
=== alan_g|lunch is now known as alan_g
mzanettirun_on_device working for you guys?12:54
Saviqbug #135686412:58
ubot5bug 1356864 in ubuntu-ui-toolkit (Ubuntu) "Button with strokeColor does not change font colour" [Undecided,New] https://launchpad.net/bugs/135686412:58
Saviqmzanetti, it can't work too well with separate dash12:58
Saviqmzanetti, i.e. it will launch the system dash12:59
SaviqI've a branch that should've fixed that, but doesn't...12:59
mzanettiSaviq: I'm ok with that12:59
mzanettiSaviq: doesn't start unity at all here12:59
mzanettijust hangs12:59
Saviqmzanetti, status unity8?12:59
mzanettiSaviq: this is the output: http://paste.ubuntu.com/804519313:00
mzanettihangs here at the end13:00
mzanettiit says "start/running" if asking for status in another shell13:00
Saviqmzanetti, so no PID13:00
Saviqmzanetti, looks like it's in a crash loo[13:00
Saviqloop13:00
mzanettihmm13:00
mzanettiinteresting13:00
mzanettiindeed13:01
mzanetti  what():  Unknown command line options: -qmljsdebugger=port:376813:01
mzanettiwell, not in a loop, but just stuck here13:01
Saviqand bug #135686613:03
ubot5bug 1356866 in ubuntu-ui-toolkit (Ubuntu) "Button with strokeColor's border looks wrong" [Undecided,New] https://launchpad.net/bugs/135686613:03
mzanettihmm.. removing that arg makes it start...13:03
Saviqmzanetti, new mir I believe13:03
Saviqmzanetti, we need to eat that arg before it gets to mir13:03
* mzanetti still thinks libraries should not bail out on arguments they don't know13:04
mzanettiwhy would mir prevent me to add some cmdline arg13:04
mzanettidoesn't make sense to me13:04
mzanettiif it can't use it, ignore it13:04
greyback_mzanetti: Mir got a new API to allow us to stop mir bailing on args it doesn't recognise, but qtmir hasn't implemented it yet13:08
mzanettian API for that? :)13:09
greyback_yeah13:09
mzanettiwhy would anyone want to enable that?13:09
mzanettior better: not want to disable it13:09
greyback_is there a nice way a library can filter out the cmd line args it accepts?13:10
mzanettidon't think so...13:10
greyback_ultimately we don't want a mis-spelled cmd line arg being ignored13:11
mzanettihmm... I'd say yes, better that then not starting at all. but even if we agree on that, its imo the applications responsibility to do that13:11
greyback_so this is an idea we had. Pass all cmd line args to Mir, mir filters out the ones it does accept, I pass the rest to Qt/the app13:12
mzanettinot one of the libs it loads13:12
greyback_we let Qt do it. Qt is a lib13:12
mzanettiqt doesn't assert if there's an argument it doesn't know13:13
greyback_sure, that's the mir default behaviour. We can (and will) override that13:13
dandradergreyback_, do you recall why do we have "PKGCONFIG += mircommon mirclient" in the ubuntumirclient QPA? It's supposed to talk just to platform-api...13:14
greyback_dandrader: nope, I don't recall13:14
dandraderwonder if it's a leftover. couldn't find any references to mir in the code...13:14
greyback_we most've missed it so13:14
greyback_must've13:14
Saviq@unity more free karma https://code.launchpad.net/~saviq/unity8/update-deps/+merge/23080313:38
WellarkSaviq: approved13:41
Wellarkdo you need top approval?13:41
SaviqWellark, sneaky! ;)13:41
SaviqWellark, generally yes :)13:41
Cimimzanetti, I am reviewing this https://code.launchpad.net/~mzanetti/unity8/new-lockscreen-design/+merge/23047813:43
Cimimzanetti, do you want to add https://bugs.launchpad.net/unity8/+bug/1169615 ?13:44
ubot5Ubuntu bug 1169615 in Ubuntu UX "[greeter] Welcome screen AM/PM text is unnecessary big" [Medium,Triaged]13:44
ChrisTownsendI'm getting the Hello Box o' Doom (ie, the lockscreen) when trying to log into the Unity8 desktop session now.  r1150 looks suspicious to me in causing this behavior, but I'm not 100% sure.13:47
ChrisTownsendgreyback_: I saw your findings in https://bugs.launchpad.net/ubuntu/+source/unity8-desktop-session/+bug/1353041.  Thanks and sorry to blame qtmir:)13:54
ubot5Ubuntu bug 1353041 in unity8-desktop-session (Ubuntu) "Logging out of the Unity8 desktop preview session hangs" [High,Confirmed]13:54
greyback_ChrisTownsend: no worries! I'll see if I can whip up a quick patch fori it anyway13:54
ChrisTownsendgreyback_: Cool, and thanks!13:54
Saviqmzanetti, fixed all your concerns13:56
Cimido you guys use any extension for chrome to organise tabs?13:59
mzanettiCimi: I guess I can add that14:01
mzanettiCimi: have lots of conflicts now anyways after mterry's branch is merged14:01
mzanettiSaviq: I approved the grey one already14:01
mzanettiwill check the other now14:01
=== gatox is now known as gatox_lunch
paulliuSaviq: for "dash as app", is console.log() still works when do ./run.sh?14:05
CimiSaviq, good to merge https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/indicator-polishing/+merge/22908314:06
paulliuSaviq: anyway. I found the solution. Watch ~/.cache/upstart/unity8-dash.log14:08
=== charles_ is now known as charles
=== alan_g is now known as alan_g|tea
CimiSaviq, using style delegates in unity is not nice14:26
CimiSaviq, kinda kills the idea of themability14:26
SaviqCimi, no it doesn't, if the style depends on the theme14:26
SaviqCimi, which it does wherever it can right now14:27
CimiSaviq, nope because if I ship my own theme, I still need Ambiance14:27
SaviqCimi, it's there because scopes can override the default theme14:27
SaviqCimi, you do understand we have virtually no themeability in the SDK currently?14:27
CimiSaviq, why that? we have themes14:28
SaviqCimi, doesn't mean they actually work properly14:28
SaviqCimi, or are thought out14:28
Cimiit is not full, right14:28
Cimibut I can have squared buttons if I want14:29
SaviqCimi, basically our design team doesn't want to *allow* themeability to the extent we're used to it14:29
CimiSaviq, that's false14:29
SaviqCimi, talk to them14:29
CimiSaviq, what we want is to provide our theme14:29
CimiSaviq, and support that14:29
SaviqCimi, talk to them14:29
Cimias well as allowing some changes for operators14:29
SaviqCimi, aaanyway14:30
SaviqCimi, the styling component for the dash is a requirement14:30
SaviqCimi, but14:30
SaviqCimi, as you can see, it tries to base off of the theme where possible14:30
SaviqCimi, but the theme is just lacking information in a few places, or is plain incompatible with the dash design14:31
Cimidednick, standup14:31
=== alan_g|tea is now known as alan_g
CimiSaviq, how do we try qml/Hud/HudParametrizedActionsPage.qml ?14:44
Cimion https://code.launchpad.net/~saviq/unity8/move-pageheader/+merge/23060914:44
SaviqCimi, tryHud14:44
SaviqCimi, but really just make sure testHud works, if we ever bring this back we'll have to fix tons of things there anyway14:45
CimiSaviq, yes, but not good reason to regress :)14:46
Cimibut yeah don't worry about it14:46
CimiSaviq, approved, waiting CI14:56
CimiSaviq, should be fine, I grep for PageHeader all around14:57
Cimion https://code.launchpad.net/~saviq/unity8/header-customizations/+merge/230719 now14:58
CimiMacSlow, you still looking for reviews?14:58
Cimipete-woods, any update on the fix?14:59
pete-woodsCimi: it will be rolled out with our next scopes release14:59
pete-woodswhich should be "real soon now"15:00
MacSlowCimi, yes15:02
MacSlowCimi, take any of the top ten from  https://code.launchpad.net/~macslow, which isn't claimed yet...15:03
MacSlowCimi, e.g. https://code.launchpad.net/~macslow/sync-monitor/fix-135028215:04
CimiMacSlow, no thanks :D15:05
CimiMacSlow, I thought you were looking to do reviews15:06
CimiMacSlow, I have already few for today15:06
MacSlowCimi, sure... that too... but I would want to focus on Dash-related bits as Saviq suggested during the stand-up15:06
MacSlowSaviq, speaking of Dash-related MPs... any favourite ones reviewers should focus on?15:09
SaviqCimi, you're doing the customization ones and table, right?15:10
CimiSaviq, one at a time15:10
SaviqCimi, you're assigned to all of them ;)15:10
Cimihah yeah15:11
SaviqCimi, give one up to MacSlow :)15:11
cwayneCimi: any updated ETA on settings landing?15:11
Cimicwayne, talk to Saviq :P15:13
Cimicwayne, it's waiting review15:14
Cimicwayne, looks fine on my side15:14
Saviqcwayne, hopefully by Monday15:14
Cimicwayne, there is though a bug on backend side15:14
Saviqcwayne, if all else fails, I'll have everything related in silo for Monday15:14
CimiSaviq, can we have luminance in C++ ?15:16
Cimior well, a generic color util15:16
Cimiwould be nice to have in SDK though15:16
SaviqCimi, yeah, 90% of this needs to go into SDK15:17
SaviqCimi, and no, you can't atm15:17
Saviqnot worth it15:17
CimiSaviq, https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/903/?15:24
Cimistill broken on a different branch, might be trunk then15:24
SaviqCimi, right, care to find a fix?15:26
CimiSaviq, I was looking15:26
CimiSaviq, issue is that I cannot reproduce15:26
SaviqCimi, ok, I'll have a look in a moment, redoing part of ScopeStyle...15:26
Cimithey also are using tryCompare... so I don;t know15:28
=== gatox_lunch is now known as gatox
CimiSaviq, I don't understand https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/135648715:38
ubot5Ubuntu bug 1356487 in Ubuntu UX "Need to teach about right edge gestures somehow" [Undecided,New]15:38
SaviqCimi, right edge is confusing15:38
SaviqCimi, we don't have an edge demo for it15:39
Cimidescription is different15:39
Cimiwhich is https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/135648815:40
ubot5Ubuntu bug 1356487 in Ubuntu UX "duplicate for #1356488 Need to teach about right edge gestures somehow" [Undecided,New]15:40
CimiSaviq, the total flatness of the header in the new header customisation branch is wanted/design approved?16:06
SaviqCimi, wanted16:06
CimiSaviq, but we have 3d effect in every other app, and sdk16:06
SaviqCimi, https://docs.google.com/a/canonical.com/file/d/0B8I8ZVKH-8SsS01CcE93TFNUSGc/edit16:06
SaviqCimi, yeah, and?16:07
SaviqCimi, think that's the only place where we differ from SDK?16:07
CimiSaviq, there is highlight/3d effect here16:07
SaviqCimi, yeah, only in a few places16:08
SaviqCimi, but flat is really what's wanted16:08
CimiSaviq, I'd add something subtle16:08
SaviqCimi, I was told flat16:09
Cimiit's not hard to achieve, a simple rectangle on top and bottom16:09
SaviqCimi, yeah, and then you have a black divider, red header, blue background16:09
SaviqCimi, still easy to achieve?16:09
CimiSaviq, flat meaning no gradient IN the darker area16:09
SaviqCimi, no, flat meaning flat16:09
Cimibut between that and the top header, there is a little line giving 3d sensation16:09
SaviqCimi, https://docs.google.com/a/canonical.com/file/d/0B71w9p1v_aX4VjZtTkc2di15RnM/edit16:10
Saviqis there?16:10
CimiSaviq, you can apply a darkening effect to the color used in the divider16:10
SaviqCimi, a darkening effect to black?16:10
CimiSaviq, there is a whitening effect at the bottom though16:10
SaviqCimi, Joshua looked at https://drive.google.com/drive/#folders/0B32jwBcbaPloR1IyRF91ZWs0bFE, approved how the divider looks16:11
Cimidefinitely not white though16:11
Cimithis image is a mess16:11
Cimihow can you share a jpg?16:11
Cimithat compresses colors?16:11
SaviqCimi, regardless, I really did ask16:12
SaviqCimi, explicitly16:12
SaviqCimi, it's supposed to be flat16:12
Saviqif you want to challenge that design, you know where to go16:12
SaviqCimi, but please don't do it on this branch16:12
CimiSaviq, ack, will chase jouni soon16:13
SaviqCimi, don't chase Jouni, chase Esti and Josh, Jouni has nothing to do with those designs16:13
CimiSaviq, he is lead visual16:13
CimiSaviq, and he is theming sdk, which is supposed to be consistent with shell16:14
Cimiso I will ask them to agree on which style to go: flat or little highlight16:14
SaviqCimi, either case, you know where to find them16:14
CimiI personally like flat16:14
kgunn_Cimi, how about this...can you file a bug? they've been good about addressing16:14
kgunn_this way we can keep on movin' for the day16:15
Cimikgunn_, let me try sorting this out with design16:15
AlbertA2kdub: so I guess this should address the halt seen in the spinner? https://code.launchpad.net/~kdub/mir/fix-1352883/+merge/22980416:15
AlbertA2that's not in 0.6.0 right?16:15
kdubAlbertA, oh yeah, probably16:17
=== dandrader is now known as dandrader|lunch
kdublet me check where it is...16:17
kgunn_kdub, AlbertA2 nope doesn't appear to be16:18
CimiSaviq, kgunn_ we'll know tomorrow morning16:18
kgunn_on 0.616:18
CimiSaviq, kgunn_ but yes they are aware esti was doing some flat design16:19
AlbertA2ok I'll do a quick test to see if it fixes it16:19
kdubAlbertA, yeah, should catch the 0.7 train16:19
kdubif its an intermittent lock up that will continue after a second or two, its most likely the same thing16:19
kduboh, and occurs around surface reordering16:20
CimiSaviq, scopes scope in header customization has weird colors16:24
CimiSaviq, header is white on white16:24
SaviqCimi, scopes scope?16:24
CimiSaviq, in unity8-dash, maybe is a mock16:24
SaviqCimi, in unity8-dash is probably not a mock, but where?16:24
SaviqCimi, also, I just pushed a rework of that logic16:25
CimiSaviq, one of the last scopes16:25
CimiSaviq, anyway tomorrow I'll catch up with them16:25
SaviqCimi, how are you running it?16:25
CimiSaviq, from builddir16:25
Cimi./src/Dash/unity8-dash16:25
SaviqCimi, yeah, that's overview16:26
CimiSaviq, we can purge it from the mock then16:26
SaviqCimi, no, that's a real scope, there's no mocks16:26
Cimiah16:26
SaviqCimi, but you can only access it from bottom swipe IRL16:26
Saviqstart unity8-dash BINARY=$PWD/builddir/src/Dash/unity8-dash ARGS=-mousetouch16:26
CimiSaviq, I meant from this fake utility16:26
SaviqCimi, still, that should be fixed in unity-scopes-shell, but won't affect real usage16:27
CimiI know it doesn't16:27
CimiI was just saying about this issue16:27
Cimiin case is was due to a not covered case16:27
SaviqCimi, yeah, the scope doesn't send an explicit background16:27
CimiSaviq, we should not force to set both foreground/bg?16:28
SaviqCimi, we are16:28
SaviqCimi, in real life, in overview16:28
SaviqCimi, it will never show up in the dash like you see it there16:29
CimiI understood that16:29
CimiI was just wondering if we can have broken scopes16:29
Cimiwhich have same styling options of the scopes scope16:29
SaviqCimi, sure we can16:29
SaviqCimi, if they don't send a proper configuration16:30
SaviqCimi, but how's that our problem?16:30
CimiSaviq, and whether we should apply header customisation only when both bg and fg are set16:30
Cimilike, ok you developer, if you provided both fg and bg means you actually tested they work fine16:31
Cimiand might keep them independent to theme changes16:31
SaviqCimi, you are a developer, you developed a scope, means you actually tested they work fine16:31
Cimiin case one day we tweak the fg or bg colour16:31
Cimibut if one day we decide to turn the dash dark, and developers just set bg or fg (not both), scopes header might be broken16:32
SaviqCimi, that's why if they override one, they will override the other16:33
SaviqCimi, we really don't need to hand-hold them, it's enough to document16:33
Cimifine16:33
Cimianyway I was just picky now16:33
Cimirest seems fine, I quite like it flat16:33
Cimibut we need to wait tomorrow morning for jouni to catch up with esti16:34
CimiI'll review the rest16:34
SaviqCimi, you could say that about changing just the foreground color, if you don't set a background and we change it, you're screwed16:34
Cimiyeah16:35
CimiI was thinking we should allow customisation only when both are set16:35
Cimiwhen I was doing themes, 95% of times I was picking up a different bg or fg I had to change the other one, accordingly16:36
SaviqCimi, but it wasn't enforced16:36
Cimiso it will keep the contrast bg/fg balanced16:37
Cimiwith the rest of the ui16:37
Cimiwith forcing developers to set both, is like saying "you set a new bg, now please find the right fg for that"16:37
Cimiit's a more complete customisation work16:38
SaviqCimi, yes, but in your 'when I was doing themes', were you ever forced to set them both?16:38
CimiSaviq, nope, but it looked better if I did16:39
Cimiand that is entirely my point16:39
Cimiforcing devs to pick up the perfect match16:39
SaviqCimi, sure, let's do that when we can actually afford the time to have that discussion16:39
Cimiand also before criticising devs, we should look at what we do16:41
Cimieverytime I see a button in ubuntu, white text on light grey, I want to cry16:41
SaviqCimi, who said anything about criticizing devs?16:42
Cimiif you really want to do white on grey, add a subtle inset shadow to the text to increase contrast/sharpness16:42
Cimii was16:42
Cimipotential scope devs16:42
SaviqCimi, you wanna cry? check bug #1356866 out16:42
ubot5bug 1356866 in ubuntu-ui-toolkit (Ubuntu) "Button with strokeColor's border looks wrong" [Undecided,New] https://launchpad.net/bugs/135686616:42
SaviqCimi, and bug #135686416:42
ubot5bug 1356864 in ubuntu-ui-toolkit (Ubuntu) "Button with strokeColor does not change font colour" [Undecided,New] https://launchpad.net/bugs/135686416:42
CimiI'll leave them for tomorrow16:42
CimiSaviq, you were trying them for dash overview?16:43
SaviqCimi, yeah16:43
CimiSaviq, also text size changes, and border16:44
Cimiwhen you press16:44
Cimiat least trying in indicator polishing branch for settings components16:44
SaviqCimi, http://i.imgur.com/pI50Ai2.png16:44
Cimi:')16:44
CimiI need some napkins16:45
SaviqCimi, I don't think it changes size, but it does change color... from grey to white, regardless of stroke color :|16:45
CimiSaviq, it changes size if you rpess16:45
SaviqCimi, yeah maybe16:46
Saviqmy eyes are glued up whole day today16:46
SaviqCimi, the last push I made made the extreme usecase in tryDash work better16:46
SaviqCimi, started selecting the more contrasting color of the two, not just "a contrasting" one16:47
SaviqCimi, i.e. https://docs.google.com/a/canonical.com/file/d/0B32jwBcbaPloM3JlVlF2NXJDVmM/edit16:47
SaviqCimi, before it would select the grey, because it's lighter than black after all ;)16:47
CimiSaviq, what happens to the second scope? :)16:47
Cimiin trydash16:48
SaviqCimi, it's beautiful, whaddya want?16:48
Cimihah16:48
CimiSaviq, yes we should not force devs to set both fg and bg16:48
cwayneyou cant force me to do anything! ill set my fg and bg both to black if i want!16:50
mhall119Saviq: pete-woods: where does cout go to from a scope? unity8-dash.log, scope-registry.log or somewhere else?16:50
cwaynescope-registry16:50
cwaynei think16:50
pete-woodsmhall119: I think you need to do cerr. I think cout might be swallowed16:51
pete-woodsbut the log is in ~/.cache/upstart/scoperegistry.log16:51
Saviqmhall119, should be in scope-registry, yeah16:52
Saviqmhall119, unless as pete-woods says, registry swallows cout16:52
mhall119pete-woods: ok, I'll try that, thanks16:52
=== alan_g is now known as alan_g|EOD
smallfoot-Last time you updated Unity un Utopic you introduced a regression in Compiz17:04
smallfoot-https://bugs.launchpad.net/ubuntu/+source/compiz/+bug/135698117:04
ubot5Ubuntu bug 1356981 in compiz (Ubuntu) "Regression: wrong window decorator applied" [Undecided,New]17:04
CimiSaviq, commented https://code.launchpad.net/~saviq/unity8/header-customizations/+merge/230719/comments/561079 for now17:09
SaviqCimi, thanks17:10
=== dandrader|lunch is now known as dandrader
greyback__mterry: ping17:30
mterrygreyback__, heyo17:31
greyback__mterry: hey there, I was looking into bug 135304117:31
ubot5bug 1353041 in unity8-desktop-session (Ubuntu) "Logging out of the Unity8 desktop preview session hangs" [High,Confirmed] https://launchpad.net/bugs/135304117:31
greyback__mterry: which I have tracked down to inside tests/mocks/LightDM/demo/GreeterPrivate.cpp:15817:32
greyback__you are waiting for a QFuture to return a value17:32
mterryInteresting... OK17:32
greyback__that blocks shutdown on the desktop though17:32
mterrygreyback__, right, that would be us in the middle of our PAM conversation17:33
mterrygreyback__, hm17:33
greyback__I'm not very savvy with Futures, so I'm learning as I go, but I guess a way to cancel or force a future to return would do17:33
mterrygreyback__, well we use QtConcurrent::run which I don't think can be cancelled17:34
mterrygreyback__, but we can force PAM to end its converstaion17:34
mterrygreyback__, perhaps with a pam_end call or feeding it blank password answers just to get through the PAM conversation17:35
greyback__mterry: mind if I leave the bug with you? It's a sharp learning curve17:35
mterrygreyback__, OK.  What's the easiest way to test?17:37
greyback__mterry: I do what the bug report said. unity8 on desktop, log in,  then log out. It hangs on log out17:37
mterrygreyback__, I was hoping there was an easier way  :)17:38
greyback__latest unity8 I wasn't able to get past a "hello" dialog17:38
greyback__mterry: I'd recommend a secondary machine17:38
greyback__mterry: I'm happy to test anything you come up with...17:38
mterrygreyback__, I'm not made of desktops!  I am made of phones at this point though..  ;)17:38
greyback__mterry: pah, call yourself a geek17:39
greyback__:D17:39
greyback__            while (respond(QString())); // clear our local queue of QFutures17:39
greyback__calling that in a deconstructor - would that work?17:40
=== pete-woods is now known as pete-woods|away
mterrygreyback__, yeah probably17:40
greyback__trying...17:40
dandradertedg, ping17:44
cwayneman oh man am I excited for this next landing17:45
greyback__mterry: that appeared to work17:45
mterrygreyback__, yay!17:46
mterrygreyback__, it would probably be cleaner to cancel QtConcurrent::run and just call pam_end.  But we can't cancel that run.  So we'd have to use a more complicated concurrent method17:49
greyback__ChrisTownsend: with u8 on desktop, when you logged out, do you get back to the lightdm greeter?17:49
mterrygreyback__, maybe mention that as a TODO17:49
ChrisTownsendgreyback__: When it was working before, it did.17:50
greyback__ChrisTownsend: hmm okay. I have a hack to stop the hang on logout, but it logged back out to the u8 greeter. Not sure why17:51
ChrisTownsendgreyback__: Is the u8 greeter the screen with the dial looking thing in it?17:52
greyback__ChrisTownsend: yep17:52
ChrisTownsendgreyback__: Also, I think some changes when in recently that may affect this, but I'm not sure.  As of today, when I log in, it goes to the lockscreen and I can't do anything else.17:52
ChrisTownsendgreyback__: Ok17:53
greyback__according to the logs, u8 does totally shut down and something restarts it17:53
ChrisTownsendgreyback__: Hmm, the hang may just be a layer to the onion.17:53
ChrisTownsendgreyback__: Recent changes in the upstart handling may also affect this as well.17:54
ChrisTownsendgreyback__: But that couldn't be tested due to the hang.17:54
greyback__ChrisTownsend: I don't understand much of how log in/out works unfortunately17:54
ChrisTownsendgreyback__: Ok ,well sounds like one issue is knocked out.17:55
greyback__ChrisTownsend: I'll propse the fix later/tomorrow17:55
ChrisTownsendgreyback__: Cool.17:55
ChrisTownsendgreyback__: Thanks!17:55
greyback__ChrisTownsend: no worries. BTW can you log in to the most recent u8? When I started it, I got a "Hello" dialog with a text box, and had no idea what to do17:56
greyback__that was after lighdm log in17:56
ChrisTownsendgreyback__: Yep, I get that issue now too.  I filed a bug.17:56
ChrisTownsendgreyback__: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/135690217:57
ubot5Ubuntu bug 1350878 in unity8 (Ubuntu) "duplicate for #1356902 lockscreen doesn't properly unlock on desktop when a password is set" [High,Confirmed]17:57
greyback__ChrisTownsend: ok cool. I had to rewind to before 1150 to get a way to reprouce your original bug17:57
ChrisTownsendgreyback__: Somebody marked it as a dup already, so I'm guessing it's a known issue.  Although I think the dup is a bit different issue.17:58
ChrisTownsendgreyback__: I guessed 1150 causes the new issue.17:58
ChrisTownsendgreyback__: mterrry marked it as a dup, so maybe he's on top of it?17:58
ChrisTownsendErrr, mterry did.17:59
greyback__I'd say so17:59
mterryChrisTownsend, I'm not actively working on it, but I'm aware of it17:59
mterryChrisTownsend, it sounded like a dup.  But if you thought it was different, maybe my quick read of the bug was wrong18:00
ChrisTownsendmterry: Eww, can't use the desktop at all now.18:00
ChrisTownsendmterry: I think it may be related, but in my case, caused by r1150.18:00
ChrisTownsendmterry: I didn't have the issue until the latest unity8 update.18:01
mterryChrisTownsend, OK interesting18:02
mterryChrisTownsend, before 1150, did you actually see a login screen and such?18:03
ChrisTownsendmterry: Only briefly and then it would automatically go to the Dash.18:03
ChrisTownsendmterry: Now it's just the lockscreen with no way out.  Indicators don't work or anything.18:04
ChrisTownsendmterry: I call it the Hello Box o' Doom:)18:05
mterryChrisTownsend, indicators not working (visible but can't pull them down) is expected when locke18:10
ChrisTownsendmterry: Ok, that makes sense.18:10
mterryChrisTownsend, sounds like you are hitting one aspect of the bug that I marked yours as a dup-of.  It started getting worse for you from 1150 because I closed a loophole in the logic that let you not experience the other bug so bad (that's what I'm thinking anyway, without looking into it)18:11
ChrisTownsendmterry: Ok, that sounds reasonable.  I thought they might be related, but since my issue just started, I wasn't sure, so I opened the new bug.18:12
ChrisTownsendmterry: Hopefully, a resolution can be found soon-ish as the u8 desktop is unusable in this state- that is unless you know a workaround like what magic code to enter in the Hello box or some way to dismiss it or something.18:13
mterryChrisTownsend, nope!18:16
mterryChrisTownsend, I know and I'm sorry.  I've just been too busy with phone stuff18:16
ChrisTownsendmterry: Ok, that's what I thought.  And I understand you are busy with phone stuff and it has a higher prio.  Just hoping:)18:17
ChrisTownsendmterry: Thanks18:17
mterryChrisTownsend, no worries, hopefully I can get your desktop back to you before too long18:18
mterry:)18:19
ChrisTownsendmterry: I'll keep my fingers crossed18:19
=== dandrader is now known as dandrader|bbl
=== pete-woods|away is now known as pete-woods
=== salem_ is now known as _salem
=== dandrader|bbl is now known as dandrader

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