[01:37] Guh QML === _salem is now known as salem_ === jono is now known as Guest576 === salem_ is now known as _salem [06:57] bregma: FYI Unity7 has a blocker, failing to build on armhf https://bugs.launchpad.net/unity/+bug/1209512 [06:57] Launchpad bug 1209512 in Unity "FTBFS on armhf: ERROR: Library '/system/lib/libGLESv2.so' not found" [High,New] [08:05] Saviq, could i get you to have a rough look at https://code.launchpad.net/~mhr3/unity8/scopes-category-overriding/+merge/179015 ? [08:05] just to make sure it's going in the right direction [08:06] does the vpn work for you? [08:06] tsdgeos, +1 [08:07] i can't connect :-/ [08:07] ah connceted now [08:07] just took more than usual [08:08] mhr3_, not liking the Repeaters in DashApps.qml - would probably go for a ProxyModel [08:08] you can transform the data with ProxyModel? [08:08] lol [08:08] mhr3_, sure you can [08:08] we killed the X server in an autopilot test [08:08] good to know :) [08:09] mhr3_, and we should be able to get rid of "Available for download", no? the click scope should be providing it already? [08:09] http://paste.kde.org/~tsdgeos/p4fdedc31/ [08:09] i like the "mieq is *NOT* the cause. It is a victim." debug line [08:09] :D [08:09] Saviq, i think they're perfectly capable of removing one line once it's in :) [08:10] mhr3_, ;) [08:10] tsdgeos, :D [08:11] mhr3_, other than that - yeah, looking good [08:12] Saviq, i'm unsure whether the running apps will work, couldn't test it on the phone [08:13] mhr3_, re: reparenting - might be tricky, we probably should "undo" the override when it's destroyed, instead [08:13] mhr3_, should work, no reason why it wouldn't [08:14] Saviq, noting that, will remind you of it in a few days when you'll scream at me :P [08:15] Saviq, so i'll port also home to use the real thing [08:17] mhr3_, yup [08:18] Saviq, any pointers how to do the transform with ProxyModel? [08:18] mhr3_, just implement your own data() [08:19] you can do that in qml? [08:19] mhr3_, no [08:19] another good to know thing :P [08:19] eeek [08:19] then i don't like that [08:19] mhr3_, I don't like the Repeater more ;) [08:19] Saviq, come on, it's mocked data [08:20] mhr3_, not for running apps i isn't [08:20] t [08:20] mhr3_, when they get reordered [08:20] mhr3_, or added / removed [08:20] mhr3_, the Repeater approach will just bite you in the a$$ [08:20] Saviq, it's not using the repeater for the running apps [08:20] just the mocked freq and downloadable [08:21] mhr3_, ah, so we're targetting the "real scope will already be there" case? [08:21] well not exactly, that's why i said i'm unsure if the running apps will work :P [08:22] mhr3_, they would, with a Proxy ;) [08:23] then i'll need to update also the renderer, right? [08:23] meaning "Apps/RunningApplicationsGrid.qml" [08:24] mhr3_, yeah, to use the real scope roles [08:24] mhr3_, but that would need to happen for the "real running apps" scope anyway [08:24] mhr3_, so a good thing [08:24] right, see, i almost broke running apps :P [08:25] mhr3_, I'd really go for a c++ ProxyModel or two, they would need to be maybe 50 lines each [08:25] mhr3_, well, for the mock data [08:26] mhr3_, just make the mocks have the "real scope roles" straight away [08:26] mhr3_, no need for a Proxy or the Repeater - they're just ListModels with some ListElements inside them [08:26] also true, sounds easier [08:26] mhr3_, so as long as they have the same roles, you just pass it straight in [08:26] Saviq: in https://code.launchpad.net/~saviq/unity8/reenable-battery-drag-test/+merge/178911 when you say "make sure it works on devices", you mean running the autopilot test on the Nexus4? [08:27] also the Nexus10 or one should be enough? [08:28] tsdgeos, would be good for N10 as well [08:28] ok, me tries in both places [08:28] tsdgeos, should be just a case of "grab the output.zip; phablet-run-test -n unity8" [08:28] installing the stuff from the .zip on the device first, of course [08:29] i probably need to flash the Nexus10 first [08:29] it's ooold [08:35] hi, i think a recent kernel update was incompatible with my graphics card or something, someone Dr_willis suggested trying to create a new user and testing whether the problem persisted, i tried that and the problem persisted, i am not sure how to restore unity [08:40] Saviq: obviously our autopilot packages miss some deps [08:40] tsdgeos, apt-get -f install [08:40] i'm getting: RuntimeError: Unable to instantiate any backends [08:40] tsdgeos, but actually [08:40] Saviq: did that already [08:40] tsdgeos, phablet-test-run -i [08:41] and after that i had to manually install python-autopilot [08:41] tsdgeos, that will install all that's needed on the device for autopilot [08:41] and now i'm getting that [08:41] ahhh [08:41] tsdgeos, yeah, phablet-test-run -i will do stuff [08:41] tsdgeos, need ssh first, though [08:41] got that [08:41] bug #1205244 [08:41] bug 1205244 in Phablet Tools "phablet-test-run should install ssh if it requires it" [Undecided,Confirmed] https://launchpad.net/bugs/1205244 [09:02] I've MRed the themeiconprovider to the sdk, is there a way to make another module depend on that? [09:02] I don't want to duplicate the code between unitymenumodel and the sdk [09:02] and I'm sure it might be problematic when two modules add icon providers with the same name [09:06] larsu, why two with the same name? [09:07] Saviq: if I were gto leave the provider in unitymenumodel [09:07] tsdgeos, it should've been 12 tests with the reenabled test [09:07] and somebody links against that and the sdk [09:08] Saviq: :S [09:09] tsdgeos, or waity [09:09] tsdgeos, no, 11 [09:09] tsdgeos, it was skipped before, and now it passes, that's fine :) [09:09] phew [09:10] Saviq: ping. we may have run into a bit of a snag with this glib event loop issue i was having with indicators. [09:11] dednick, :/ [09:11] dednick, how bad? [09:11] Saviq: https://bugreports.qt-project.org/browse/QTBUG-32859 [09:11] Saviq: "Do we even want to support calling glib directly?" [09:11] Saviq: "I'd say that running GMainLoop directly is out-of-scope." [09:12] dednick: please don't approve ted's indicator-signals merge yet. I've had an unresolved discussion with him and we've decided to continue this discussion later and focus on other work right now. [09:12] larsu: ok [09:12] larsu: can you comment that on the MR itself ? i was roaming on that MR and almost approve it :D [09:12] thanks [09:12] larsu: think it needs some work anyway [09:13] Saviq: i do have a potential solution, but it involves work for every instance we have a glib callback calling back into qt. [09:13] dednick: if you ask me, it needs to die :) [09:13] Saviq: essentially need to send a custom QEvent every time [09:13] Saviq: which is how qt handles window events and such. [09:14] tsdgeos: it was quite a heated discussion, I wanted a low-profile way to stall this a bit until I have time to talk to him again [09:14] ok [09:16] dednick, re: the comments, I think we need to restate that the g_ thing is just the simplest way to reproduce, and not how we're trying to use it [09:17] Saviq: fyi, this problem is valid for anywhere else we use glib. which means probaly means everything from unitycore::scopes. [09:17] dednick, yeah, I understand [09:17] dednick, hmm but wait, the problem is that we actually *do* use glib directly in there? [09:17] hmmm [09:17] Saviq: yes [09:17] Nexus 10 is not flipped? [09:18] tsdgeos, yes it is [09:18] Saviq: we're connecting directly to gmenumodel [09:18] ah wait phablet-flash failed [09:18] dednick, right [09:19] well it' a bit lame they claim glib event loop integration [09:19] if the event loop integration doesnt' work :D [09:19] Saviq: I think their point is that we should be sending events back through the qt event system. [09:19] MacSlow, I reproduced the autopilot failure when running the tests on device [09:21] Saviq, I'll look into it later today. [09:21] MacSlow, thanks [09:22] Saviq, I already switch the branch back to "wip" to avoid load on the jenkins-metal [09:22] MacSlow, yeah, saw that [09:23] ha, i'm 0% of battery, that's why flashing fails :D [09:23] Saviq, currently back on the UI-side of things regarding the "extended snap-decisions" [09:23] MacSlow, yup [09:24] larsu, btw, did you have time to write the menumodel vs. notifications email? [09:24] Saviq: typing right now :) [09:24] larsu, thanks [09:25] larsu, looking forward to it [09:27] dednick, the QEvent thing would be a workaround, then, while incrementing the loop level would be a solution, right? [09:31] Saviq: I think we would need to increment the loop level in our code (in the glib callbacks). I'm not sure there's a way around it. Problem is that the both routes (to glib handler, and to delete) pass through the same code. [09:32] Because all the event handling goes through the glib event dispatcher. [09:33] Saviq: unfortunately the QEvent is the actual solution in this case it would seem. I think that's how it was intended to work. Unless i can find a better option. [09:34] dednick, that sounds like a substantial amount of work, too? [09:34] Saviq: i'm looking into it now, but most likely quite a bit. [09:34] custom event types for each different callback. [09:35] nic-doffay, hey there... [09:36] MacSlow, what's up? [09:36] nic-doffay, I'm trying to get an expanded OptionSelector into a flickable (with a scrollbar) ... without any success sofar... it that possible at all? [09:36] Saviq: the only other solution i can potentially think of is to pull all the custom glib stuff onto a different loop. But I'm not sure how possible that would be, or how much work. [09:37] MacSlow, where did you find "OptionSelector"? [09:37] * seb128 needs it but though that didn't land yet [09:37] MacSlow, I haven't tried a scrollbar but I've put it in a flickable without issue. [09:37] seb128, yeah, not landed yet [09:38] seb128, it's in a not-yet-merged branch... I'm trying to get a feel for this... as it'll be needed in the wifi-ap selection dialog/snap-decision [09:38] nic-doffay, ok... I keep trying [09:39] MacSlow, I'm going to pastebin you some of my filters code. [09:39] nic-doffay, ok... thanks [09:39] It has scrolling too. [09:40] https://pastebin.canonical.com/95620/ [09:40] MacSlow, ^ [09:40] dednick, so if I understand correctly, we're fooked regardless which route we'll go, and it's not just something that Qt guys say they don't care about [09:40] dednick, so there's no "easy" fix for Qt itself [09:40] MacSlow, you'll see there there are three OptionSelectors in a column which is in a flickable. [09:40] seb128, if you want to give it a try... lp:~nicolas-doffay/ubuntu-ui-toolkit/list-item-option-selector [09:40] nic-doffay, great.. thanks! [09:40] Saviq: yes, from my understanding of the issue. [09:41] MacSlow, np! [09:41] MacSlow, thanks [09:42] MacSlow, nic-doffay: when is that likely to land? [09:42] dednick, :| [09:42] * seb128 wants to start using it in system settings [09:42] our current hacked valueselector suck [09:42] Saviq: i may have a very dodgy workaround for the cpu problem that we can use for the moment while i work on the issue. Will only work while we have the proxy to qmenumodel though. [09:43] seb128, when you use it... (without installing it) be sure to use qmlscene -I path/to/modules ... and "import Ubuntu.Components.ListItems 0.1" so you can use OptionSelector in your QML-code. [09:44] seb128, got feedback from design yesterday. [09:44] seb128, with "path/to/modules" being where ever you put nic-doffay's list-item-option-selector/modules [09:44] Basically I need to ensure there's a pause between switching the chevron and tick then it's landable. [09:44] MacSlow, thanks [09:44] seb128, I'm unsure when it will be reviewed though, obviously I'll push for asap. [09:44] nic-doffay, great, soon then [09:50] dednick, yeah, would be good to have something temporary, even [09:55] the tablet has regressed [09:55] can't seem to be able to run the sidestage apps if there are no mainstage apps open [09:57] well, you can run it [09:57] it just doesn't paint [09:57] Saviq: want me to open a bug? [09:57] tsdgeos, please do [09:59] what was the file that has the build number? [09:59] * tsdgeos always forgets [10:02] nic-doffay, I can't get your example to work fully... I can expand option-selectors, but I can't scroll through them [10:02] MacSlow, you mean with the flickable? [10:02] nic-doffay, yes [10:03] MacSlow, weird, I didn't have any issues. [10:03] You can check my whole branch if you want. [10:03] MacSlow, it could be because it's disabled. [10:03] MacSlow, interactive: false [10:03] ? [10:03] nic-doffay, I changed those of course [10:04] MacSlow, then I have no idea, sounds weird. [10:05] I tried to add it as an element in my dialog... but there it just didn't show up... I 'm currently using it stand-alone (also set a fixed width and no longer parent.width) [10:06] nic-doffay, also disabled (commented out) the anchors to be able to use it stand-alone. [10:23] Saviq, I've modified the CrossFadeImage a fair amount, think we'll have to get Kaleo in on this review now too. [10:23] It might be best just to put all that functionality in the OptionSelector and not use CrossFadeImage, or create a new SDK component. [10:29] nic-doffay, so I went from this http://ubuntuone.com/3plX5yStNdhHqFS4yLqtO4 to this http://ubuntuone.com/1BZOL12QuErfCfwjq5gIBa which is already a bit closer to that mpt has in mind... but scrolling I could not get going yet... also the icons per entry are not supported in the option-selector as far as I can tell [10:29] nic-doffay, that scribble is all the visual-spec I have atm [10:29] * mpt applauds [10:30] MacSlow, nope it doesn't support the icon. [10:30] This is what will need to be amended. [10:30] mpt, well there's much much left to be done (on the UI-side alone) [10:30] sure :-) [10:30] MacSlow, I think it's probably going to change a hell of a lot. [10:30] nic-doffay, yeah... I remember you mentioned there are still discussions gonig on with Design about details (such as this I assume) [10:30] I'm wondering if it's worth even using an OptionSelector. [10:31] Or at least just copying and pasting the relevant code. [10:31] MacSlow, like the selection stuff etc. [10:32] nic-doffay, at least I can actually select elements in the list now... that didn't work with the first iteration http://ubuntuone.com/3plX5yStNdhHqFS4yLqtO4 [10:33] MacSlow, it looks awesome! [10:33] thx [10:49] ouch, my X suddenly restarted out of nowhere :-/ [10:51] valgrind :-) [11:20] Saviq: https://code.launchpad.net/~nick-dedekind/unity8/indicators-cpuburn.QTBUG-32859/+merge/179161 === MacSlow is now known as MacSlow|lunch [11:40] dednick, +1 on greyback's comment, and FWIW I don't think that's a huge hack [11:40] nic-doffay, you should make the CrossFadeImage mods a separate merge request, though [11:41] nic-doffay, and it's the sdk guys that need to review anyway, I'm only looking at it from a visual / functional perspective [11:41] Saviq: yeah, but it's kind of a sidestep just because we're using a proxy. the problem will re-appear when we change to unitymenumodel if not fixed. [11:41] dednick, right [11:44] Saviq: comment added. [11:45] cheers [11:52] dednick, so that's bug #1206991 / bug #1183065 that should be helped with with that? [11:53] bug 1206991 in Unity 8 "Frequent CPU hogging" [Critical,Incomplete] https://launchpad.net/bugs/1206991 [11:53] bug 1183065 in touch-preview-images "Occassional severe battery drain" [High,Confirmed] https://launchpad.net/bugs/1183065 [11:53] Saviq: yeah === alan_g is now known as alan_g|lunch [11:54] dednick, awesome [11:55] dednick, less awesome that it's not a proper fix, but still awesome, and great investigation, thanks [11:57] Saviq: my pleasure. i have a much more detailed understanding of the qt event loop now. For the next few days until i forget it all at least. [11:57] :) === MacSlow|lunch is now known as MacSlow [12:42] dednick, we seem to be getting more and more of https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-saucy/975/console [12:42] dednick, which gets stuck in test_rightEdgeDrag() [12:42] dednick, you were looking at Daniel's edge drags / velocity calculator [12:43] dednick, could you look at the test and see if you can reproduce it locally, and ideally - fix? [12:43] dednick, I've seen that in autopilot tests as well [12:44] where the unlocking drag got stuck at ~ 1/3 of the screen from the left [12:45] there's a suspicion that due to the VM's speed the gesture isn't completed properly, 'cause velocity drops down or similar === alan_g|lunch is now known as alan_g === _salem is now known as salem_ [13:11] and we're back to 695$ :-) [13:15] tsdgeos, waouh, quite a change [13:15] yep :) [13:19] wow, and refunds [13:19] nice [13:20] bregma: so the previous Unity release blocker was resolved, but what remains is the amount of failing autopilot tests (varying on each run) that makes it hard to release. do you see anything at http://pastebin.ubuntu.com/5962552/ not addressed by other bugs you're already working on? [13:22] Mirc, the pinyin problems are in the new ibus, we're investigating those, a couple of the menu problems are because of a new indicators library upstream being broken (bug filed, no progress from them) [13:23] the rest are vaguaries of the test system [13:24] for example the previous Unity release blocker was fixed by fixing the build environment because libhybris broke x.org (nothing to do with Uniy or Compiz) [13:24] dednick, Checking in on the indicator work. Are we at the point that we can have old/new indicators in Unity? [13:26] tedg: yes, but not in trunk [13:26] tedg: need unitymenumodel to land, then my branch. [13:27] larsu, When is unitymenumodel landing? [13:27] Is it up for review somewhere? [13:27] tedg: https://code.launchpad.net/~larsu/qmenumodel/add-unitymenumodel/+merge/176857 [13:27] tedg: there's still a bit missing which larsu is looking at [13:30] bregma: yes, I know many are long known flakiness in the tests, just pointing out the ones that keep popping up now and make it difficult to release. http://pastebin.ubuntu.com/5962573/ contains a couple more from two earlier runs from today. if there's anything you can do to reduce the flakiness of those tests (the ones in the latter pastebin link didn't show up now in the latest run), it'd help [13:31] greyback_, standup? [13:31] not sure if there'd be an option to mark some tests as false alarms, but then the tests don't do much good in the first place if they just randomly fail and they'd be even ignored [13:32] Saviq: coming [13:34] tsdgeos: are you doing notes then? [13:34] greyback_: i thought i was, but if you do, all yours :D [13:38] dednick, So talking with larsu he knows of one more bug for that branch. [13:38] dednick, After that's fixed, he thinks we could probably merge it. [13:38] dednick, Is that your assessment as well? [13:39] There'll be additional bugs, but this starts unblocking things. [13:39] tedg: yep [13:39] dednick, Cool. [13:39] larsu, Is it fixed yet? :-) [13:40] tedg: no. I'm still have the occasional discussion ähere at guadec. I'll try to get it done today thouhg, as I'll be on a train tomorrow [13:41] larsu, Great, that'd be good. [13:41] dednick, ^ [13:42] tedg: got it [13:43] tedg: do you know how could review https://code.launchpad.net/~joe-yasi/libdbusmenu-qt/fix-x-timestamp/+merge/174573 a fix against libdbusmenu-qt from community and it looks sane (optional dependencies and fallbacks) [13:43] sil2100 seems to be away. [13:43] xnox, Hmm, have you ping'd agateau ? [13:43] xnox, He'd be my first person to ask, but not sure if he'd do it. [13:44] tedg: nope. agateau is not here either? [13:44] xnox, Hmm, he doesn't seem to be in any channels on freenode. [13:45] oh ffs, how do i use Regex::escape in qml? [13:46] dednick, ping [13:46] Saviq: pong === greyback_ is now known as greyback|latelun [14:07] tsdgeos, looking at http://paste.ubuntu.com/5962716/ (and the console output) [14:08] tsdgeos, then, reading http://qt-project.org/doc/qt-5.0/qtqml/qml-size.html [14:08] tsdgeos, would you consider the error a bug? [14:08] you get a qsize vs qsizef thing? [14:10] Saviq: problem is 0x0 creases a qsize and then you can't assign it to qsizef [14:10] i can "see" why it happens [14:10] but yeah i don't think you should get a warning [14:10] anyone seen SortFilterProxyModel provide one extra empty result? [14:10] any ideas why it does that? [14:10] tsdgeos, yeah, it should get converted automagically, right? [14:11] Saviq: yeah [14:11] mhr3: shouldn't, can you reproduce it? [14:12] always [14:12] therefore i guess i'm doing something stupid [14:13] mhr3: not necessarily [14:13] may be exposing a bug in the code [14:13] tsdgeos, this does it [14:13] http://paste.ubuntu.com/5962736/ [14:14] interesting [14:14] let me see if i can repro too [14:15] tsdgeos, lp:~mhr3/unity8/scopes-category-overriding [14:15] apply there [14:20] mhr3: would it be the 7th item in "Recently used Apps"? [14:21] tsdgeos, yea, also in "More suggestions" [14:21] tsdgeos, also, works fine it you revert the pastebin [14:24] dednick, Did you notice? https://code.launchpad.net/~ted/unity8/indicator-signals/+merge/179028 [14:24] the DestkopFileModel.qml is weird :D [14:24] tedg: lars told me not to look at it. he said he wanted to discuss with you [14:24] using a repeater to fill a list feels weird [14:25] tsdgeos, works :P [14:26] mhr3: but you use more memory than you need [14:26] patches welcome :) [14:26] i mean that repeater is there forever [14:26] dednick, Could you look to make sure t is correct though. I'm not as familiar with the code, not sure I put things in the right places. === alan_g is now known as alan_g|tea [14:30] mhr3: there's something really fishy [14:30] oh really? :) [14:30] tedg: i was thinking we should have a upstarter object in the utils/unity library rather than embedding the libupstart code in indicators. If we're going to be signalling things from in process, we will problably have do it more than once. [14:30] mhr3: if you add "onCountChanged: console.log("SearchableResultModel count", count)" inside id: frequentlyUsedAppsModel [14:30] it works :D [14:31] ehm [14:31] yeah that [14:31] dednick, Perahps, but many of those are through libupstart-app-launcher. And libdbus consolidates the connection. So there's not too much benefit to abstracting away libupstart. [14:31] mhr3: but it's totally reproducible :S [14:31] comment the line, 7 items [14:31] uncomment it again 6 items [14:33] tedg: except class encapsulation [14:34] tedg: this is c++ after all :) [14:35] dednick, Yes, classes are definitely too cheap in C++. [14:35] Too bad tvoss is off today. He needs to see how bad C++ is :-) [14:36] tedg: better to keep it readable (unlike C ;). anyway. the code looks to be in the right place. [14:38] dednick, readable_is_only_a_few_underscores_away -- thanks for looking! [14:39] andHereIWasThinkingCamelCaseWasBest [14:40] opinionHuNoBtr === alan_g|tea is now known as alan_g [15:11] tsdgeos, do you guys still have a list of tasks for unity8-mir integration somewhere? [15:12] Saviq: http://studio.sketchpad.cc/UFFAX8fxc2 ? [15:14] tsdgeos, greyback|latelun I added implementing QDesktopServices:openUrl so that we don't misplace that task [15:14] ooka [15:14] mhr3: something seems to be wrong at the QSortFilterProxyModel level (or we are breaking it somehow) [15:14] because i do stuff like [15:14] give me your number of rows and the number of the rows of your source model [15:15] and get [15:15] 7, 6 [15:15] and why would the proxy have more rows than the source? [15:15] mhr3: i'm going back to finish my other dash task, but this defenitely needs someone investigating [15:16] Saviq: ↑↑↑ [15:16] tsdgeos, i tried adding a qDebug to the sortfilterproxymodel::data, but it's not really called much [15:17] tsdgeos, where were you debugging this? [15:17] tsdgeos, mhr3, interesting, did you manage to build a minimal reproducible test case? [15:17] Saviq: nope [15:17] mhr3: http://paste.kde.org/~tsdgeos/pe86c3eff/ [15:18] http://paste.kde.org/~tsdgeos/pe86c3eff/raw/ [15:18] includes your changes [15:18] and a few qdebus [15:18] +g [15:19] tsdgeos, yea, so why doesn't it call data() on that object, it's pretty weird [15:19] on the model? [15:19] it does [15:19] the data there is not the *right* one [15:20] virtual QVariant [15:20] data(const QModelIndex & index, int role = Qt::DisplayRole) const [15:20] is the one you want to reimplement if you want to hijack them [15:20] the one there is for "direct use from qml" [15:22] ah, maybe we could use clearer names next time [15:26] nic-doffay, can the divider in an OptionSelector be disabled completely? [15:27] nic-doffay, just curious... not that important [15:27] MacSlow, yeah [15:27] nic-doffay, didn't find the property yet for this [15:27] MacSlow, sorry the components have support for this the OptionSelector doesn't. [15:27] nic-doffay, only the internal "showDivider" flag [15:27] MacSlow, worth adding? [15:28] If so I'll throw it in now. [15:28] nic-doffay, *shrugg* [15:28] MacSlow, might as well :) [15:28] showDividers bool it is then. [15:29] cool === greyback|latelun is now known as greyback [15:31] MacSlow, nic-doffay I think this should be theme-driven, no? [15:31] Saviq, yeah... maybe even that [15:31] MacSlow, nic-doffay, I don't think we should be adding a property to disable them [15:31] MacSlow, and in any case... I don't think there is real visual design for them... [15:32] MacSlow, if you look at http://design.ubuntu.com/wp-content/uploads/option_selector_01.png for example [15:32] MacSlow, there are dividers [15:32] Saviq, true... was just looking through the code of OptionSelector and was wondering as it is still work-in-progress [15:33] MacSlow, when you're wondering such things, you need to have a red light light up in your brain - does that even make sense? ;) [15:33] MacSlow, don't take visuals for granted - push back when you see something inconsistent [15:35] Saviq, totally forgot about themes. [15:38] MacSlow, even for https://lh4.googleusercontent.com/goPxF-p_6nxA8PM8vdawkie6HVW4ubTJ-hi_ajOHKUs2IQH34uvqIFLofNPP37GUj7enLg_f9kgaT8kegrKvWa3WyISjMmhGs4kRb8Uw_JWFt0v9Af0fC26f there are dividers, where did you see them not being there? [15:41] Saviq, one of mpt's scribbles... but again I was just curious and not really pushing for anything -> https://wiki.ubuntu.com/Networking?action=AttachFile&do=get&target=phone-wifi-prompted.png [15:41] MacSlow, right, don't consider those anything close to visual design :D [15:42] MacSlow, but in any case - changes like this should be theme-driven [15:42] Saviq, of course === om26er is now known as om26er|away === Guest79504 is now known as amithkk === greyback is now known as greyback|dinner === alan_g is now known as alan_g|EOD === om26er|away is now known as om26er [18:14] mterry, if you have a minute: https://code.launchpad.net/~saviq/unity8/prepare-cross-build/+merge/179250 [18:15] Saviq, looking [18:28] mterry, yeah, I thought about the FATAL_ERROR [18:28] mterry, can make it non-fatal === greyback|dinner is now known as greyback [20:39] Hey guys... is any of you going to be at FOSSCON this weekend by any chance? :) [20:39] We've got this bug that we'd really like to be fixed in Saucy Salamander and not quite sure whether it's our toolkit's fault or Unity's fault... [20:39] Said bug being @ http://ecere.com/mantis/view.php?id=850 === jbicha is now known as Guest66961 === salem_ is now known as _salem === jhodapp is now known as jhodapp|afk