/srv/irclogs.ubuntu.com/2014/07/03/#ubuntu-unity.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
CimiSaviq, you know what is blocking input events under the osk in unity?07:47
Cimiotherwise I'll try to dig07:47
SaviqCimi, the InputArea in OSK07:48
Saviqin OSKController that is07:48
CimiSaviq, so why doesn't work in wizard?07:48
SaviqCimi, did you even look in OSKController? did you check what doesn't work?07:50
Saviqmzanetti, please keep an eye on © dates, in new files07:51
Saviqthere's 2012,2013 in the lights plugin.h07:51
mzanettimeh...07:52
CimiSaviq, I will try to look for bugs07:52
mzanettiI told hom twice07:52
CimiSaviq, anyway in Shell.qml we have deprecated code then07:52
CimiSaviq, the MouseArea responding to ApplicationManager.keyboardVisible can be removed07:52
SaviqCimi, no it can't07:52
SaviqCimi, that one blocks input *within* the shell07:53
SaviqCimi, so now that you mention this07:53
CimiSaviq, ApplicationManager.keyboardVisible does not exist07:53
SaviqCimi, ah ok07:53
SaviqCimi, OSKC took that over07:53
SaviqCimi, what do you mean "you'll try to look for bugs"?07:54
CimiSaviq, in the OSKController07:55
Saviqk07:55
Cimiis it possible that in the wizard, the surface does not change to MirSurface.Maximized?07:57
SaviqCimi, you should close your IRC client when working07:57
CimiSaviq, it's a valid question because I don't know mir07:58
Cimineither OSK07:58
SaviqCimi, it's easy to check, too07:59
SaviqCimi, it's just one console.debug line in that file07:59
SaviqCimi, which you can even edit on device07:59
CimiSaviq, well the answer is yes, don't have to debug that07:59
CimiSaviq, my question was more why this happens07:59
Cimijust on the wizard07:59
SaviqCimi, you typed more to ask the question than to check yourself07:59
Saviqmhr3, something dawned on me... how do we deal with dark/light foreground color (on cards with background)...08:00
Cimistart reading mir code? :)08:00
SaviqCimi, no, check what happens in OSKC08:01
SaviqCimi, whether everything there behaves as expected08:01
SaviqCimi, if not, you'll at least know which part is responsible08:01
CimiI checked, this is the only thing that enables the inputarea08:01
SaviqCimi, how about its size08:02
mhr3Saviq, hm, scope can specify the overlay background as well, can§t it_ so it§s problem08:02
mhr3eh08:02
Saviqmhr3, not overlay08:02
Saviqmhr3, background08:02
Saviqmhr3, like in scopes08:02
Cimithat could be as well indeed08:02
Saviqmhr3, and well, same for overlay colour, once we allow different overlay colours, we'll need to make the "dark or light" decision on the colour as well08:03
Saviqmhr3, and this decision assumes that foreground colour is dark, and goes to white if background is dark08:04
Saviqmhr3, but if foreground colour is light (because scope background is dark)?08:04
Saviqwe *could* check the foreground colour and fall back to white/grey accordingly08:05
Saviqjust wondering if we should supply both dark and light foreground colours08:05
Saviq(or scope should)08:05
Saviqmhr3, it's probably not even a question to you ;)08:06
mhr3Saviq, eh, give me a moment, you love pinging me during our morning meeting :)08:06
Saviqmhr3, lol08:06
mzanettiSaviq: re:  Text field itself, back and clear buttons should be exempt from tap-to-dismiss behaviour of the popup and activate instantly.08:50
mzanettiSaviq: how would that work? The Popover has the InverseMouseArea integrated08:50
mhr3Saviq, so seems you know what has to be done ultimately :)08:53
mhr3Saviq, glad to be your debugging duck ;)08:54
Saviqmhr3, you mean "to ask design"? ;)08:54
Saviqmzanetti, yeah, but I thought you could tweak it08:54
Saviqmzanetti, we did that before08:54
mzanettihmm... I just copied the Popover as it was in the previous code08:55
Saviqmzanetti, hmm seems like we lost the code08:55
Saviqmzanetti, because the behaviour change08:55
Saviqd08:55
Saviqmzanetti, please check with timp whether that's possible08:56
Saviqmzanetti, I think what we had is we had our own IMA covering more than just the popover... and maybe disabled the popover IMA?08:56
Saviqmzanetti, don't remember now, but with the new behaviour (history as type-ahead search), we'll need that badly08:57
dednickSaviq: could you do a an update for silo 15? lp:~nick-dedekind/unity-mir/trusted-sessions08:59
Saviqdednick, ah you just need a rebuild?09:03
dednickSaviq: if that pulls down the latest from the branch.09:04
Saviqdednick, yup, it does, kicked09:04
dednickSaviq: thanks09:04
=== vila_ is now known as vila
mzanettiSaviq: you really wanna get rid of the clear history button?09:10
Saviqmzanetti, that's not a "clear history" button, just a "close history" one09:10
mzanettioh right...09:11
mzanettidropping09:11
=== shiznix_ is now known as shiznix
mhr3mzanetti, you had the uitk pkgs with the necessary header changes?09:24
mzanettimhr3: one sec09:24
mzanetti[15:38] <mzanetti> https://chinstrap.canonical.com/~mzanetti/qtdeclarative5-ubuntu-ui-toolkit-plugin_0.1.47+14.10.20140618-0ubuntu1_amd64.deb09:24
mzanetti[15:38] <mzanetti> https://chinstrap.canonical.com/~mzanetti/ubuntu-ui-toolkit-theme_0.1.47+14.10.20140618-0ubuntu1_amd64.deb09:24
mzanetti[15:38] * mzanetti hopes you use amd6409:24
mzanettimhr3: ^09:24
mhr3thx09:25
* mhr3 just wgetting some html :P09:26
CimiSaviq, so UbuntuKeyboardInfo does not connect to the socket09:43
SaviqCimi, do you see error messages on its console?09:43
Cimi"QLocalSocket::connectToServer: Connection refused"09:43
SaviqCimi, and is the keyboard launched along with the wizard or somewhen later?09:44
CimiSaviq, I see this after creating the mir surface09:44
Cimiso I'd say at wizard startup09:45
SaviqCimi, no, you see this after instantiating OSKController09:45
Cimiat wizard startup09:45
SaviqCimi, which uses UbuntuKeyboardInfo09:45
Cimisince it's in main.qml09:45
SaviqCimi, but that doesn't answer my question09:45
SaviqCimi, it's the keyboard that creates the socket09:45
Cimiit is launched in main.qml09:45
Cimilike, OSKController is in the main.qml09:45
SaviqCimi, maliit is launched in main.qml?09:45
Cimino in the upstart09:46
Cimilet me see when09:46
SaviqCimi, that was my question09:46
CimiSaviq, in post-start09:46
Cimimight move to pre-start09:46
SaviqCimi, no you can't09:47
Cimiok09:47
SaviqCimi, as it will die 'cause it couldn't connect to mir09:47
SaviqCimi, you need to check if the keyboard creates the socket09:48
* Cimi builds unity-mir09:50
Saviqwhy?09:51
SaviqCimi, it should be in /run/user/32011/09:51
Saviq/run/user/32011/ubuntu-keyboard-info09:51
Saviqif it's not there, but is in /tmp/ubuntu-keyboard-info09:51
Saviqmeans maliit doesn't get XDG_RUNTIME_DIR set in time09:52
CimiSaviq, to debug09:52
CimiSaviq, it's in run/user09:52
SaviqCimi, I'm not getting the keyboard error messages in ~/.cache/upstart/ubuntu-system-settings-wizard.log09:53
CimiSaviq, grep -rn UbuntuKeyboardInfo /home/phablet/.cache/upstart/09:54
Ciminothing?09:55
SaviqCimi, no09:55
Cimiugh09:55
CimiSaviq, so you don't have this? https://bugs.launchpad.net/ubuntu-welcome-wizard/+bug/133442909:55
ubot5Ubuntu bug 1334429 in Ubuntu Welcome Wizard "touch event allowed through OSK to what is underneath" [High,New]09:55
SaviqCimi, wizard just crashed for me when I tried this09:56
CimiSaviq, that was another bug09:56
CimiSaviq, https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/133420309:57
ubot5Ubuntu bug 1334203 in ubuntu-system-settings (Ubuntu) "ubuntu-system-settings-wizard crashes/freezes when using the back button" [High,New]09:57
SaviqCimi, yeah, so seems I do have the bug, if the back button is causing it09:58
CimiSaviq, which means the events pass through09:58
SaviqCimi, yes, but the cause looks different than what you found09:58
CimiSaviq, in my case, UbuntuKeyboardInfo width and height is 009:59
SaviqCimi, I got one 'UbuntuKeyboardInfo - socket error: "QLocalSocket::connectToServer: Connection refused"' now10:02
SaviqCimi, but it retries10:02
SaviqCimi, and connects fine again10:02
CimiSaviq, if you want to add console.log("oskSurface state changed", ubuntuKeyboardInfo.height, ubuntuKeyboardInfo.width);10:02
Cimiin /usr/lib/arm-linux-gnueabihf/unity8/qml/Unity/Application/OSKController.qml10:02
Cimiat the last connection near end of file10:02
SaviqCimi, you'd have more of those warnings, and then an ultimate "failed to connect" if it really failed to connect10:03
Cimiah ok10:03
Cimiso different indeed here10:03
SaviqCimi, btw, wizard-has-run is created too early IMO10:10
SaviqCimi, it should only be created after the wizard completed10:10
CimiSaviq, it is10:10
CimiSaviq, post-stop10:10
SaviqCimi, does it check the exit code?10:10
Cimino10:10
SaviqCimi, it should10:11
Cimiit's in upstart10:11
CimiSaviq, what about wizard keeps crashing and you cannot boot?10:11
CimiI understand both ways10:11
SaviqCimi, a bit extreme, but maybe10:11
CimiSaviq, we can echo the exit code in the wizard-has-run10:12
SaviqCimi, oh and also, SIM page should probably be optional on devices where there isn't a modem at all10:13
CimiSaviq, and like redo the wizard at boot if contains just 1 invalid error code10:13
CimiSaviq, I know this10:13
Saviqk10:13
CimiSaviq, but need to hack on ofono for that10:13
Cimi(and add api to expose number of modems)10:14
mhr3Saviq, once 004 finishes migrating, am i green to rebuild and land 005? or do you have more landings and would like to combine them?10:15
Saviqmhr3, maybe we should indeed10:16
mhr3Saviq, fine with me, it's nicely isolated change10:16
SaviqCimi, I added some qml debugs and everything works as expected10:17
Saviqin OSKController at least10:18
mhr3Saviq, so, feel free to add more mps to 005... or if you want to merge it with like 009, then perhaps the other way around10:19
CimiSaviq, UbuntuKeyboardInfo is still 0 0 width height10:19
SaviqCimi, here's my OSKController10:19
Saviqhttp://paste.ubuntu.com/7741264/10:19
CimiSaviq, and what does it print for you?10:20
SaviqCimi, http://pastebin.ubuntu.com/7741273/10:21
Saviqdednick, btw the package built in silo 1510:21
dednickSaviq: thanks!10:21
=== zequence_ is now known as zequence
Saviqmzanetti, .length will complain if imageSource is not a string10:30
Saviqmzanetti, result the same, but less warnings10:31
mzanettimhm... so given that imageSource is defined as property string... is there a difference?10:31
Saviqhmm right10:31
Saviqmzanetti, probably no diff, maybe better readable10:32
mzanettino problem anyways10:32
mzanettiyeah. imo the .length is the more readable one...10:32
mzanettibut really no strong opinion10:32
mzanettiwas just curious if I really miss a technical detail10:32
Saviqmzanetti, I probably come from python, where you rely on boolean infers a lot10:32
mzanettiyeah, I don't like that at all... but again, no problem at all. I can deal with both10:33
CimiSaviq, it is not positioned correctly10:33
CimiSaviq, it seems like it has space at the bottom10:33
SaviqCimi, panel height maybe?10:33
Cimimore10:33
CimiI think10:33
Cimilet me see in unity10:33
CimiSaviq, I put a nice Rectangle on the input area :)10:33
SaviqCimi, yup10:34
CimiSaviq, and I disccovered a bug in unity :)10:36
CimiSaviq, the very bottom edge of the keyboard passes through10:36
Cimiin unity as well10:36
SaviqCimi, you can pat yourself on the back, well done10:36
Saviq;P10:37
Cimiit is just not as bas as the wizard10:37
CimiSaviq, I tried moving a dash app at the bottom of the screen, like gallery app10:37
CimiSaviq, open a search10:37
SaviqCimi, you don't have to explain the bug to me10:37
Cimiand tap between the spacebar and the edge10:37
Cimiok10:37
SaviqCimi, either file it as a bug or, better yet, fix it!10:38
CimiSaviq, I need some thoughts on this10:38
Cimiso let me ask10:38
Cimithe input area should be taller than the keyboard, otherwise we cannot dismiss by swiping down, right?10:39
mzanettigosh... we have a very bad mix of onClicked, onTriggered, onPressed10:42
mzanettiall doing the same thing10:42
SaviqCimi, not sure, I think the keyboard only has something like 1gu near the top10:43
SaviqCimi, doesn't actually go out of the keyboard rectangle10:43
CimiI see those info coming from ubuntu-keyboard10:44
Cimion portrait, it is misplaced of 1 panel height up10:44
Cimi(it is roughly 1 panel height)10:44
Cimion landscape, is fine10:44
Cimibefore digging, I need to understand one thing10:45
Cimi- is this filter area supposed to extend also to the dismiss area on top of the keyboard?10:45
Cimi(I think so, but currently it is not)10:46
Cimibecause uses UbuntuKeyboardInfo that has keyboard dimensions10:46
Cimiubuntu-keyboard provides the area, now there could be a bug there or in mir itself10:47
mzanettiSaviq: look at that :D10:49
Saviqmzanetti, k, one minor thing in new-header10:49
mzanettiif (searchTextField.text) {10:49
mzanettithis doesn't work ^^10:49
mzanettiwhile this does: if (searchTextField.text.length == 0) {10:49
Saviqmzanetti, you're missing a !10:49
mzanettilol10:49
Saviqif you're comparing to 010:49
mzanettiI guess I need a break10:50
mzanettibtw. still fixing the non-inline comments10:50
Saviqmhm10:50
Saviqmzanetti, the dash failures are weird, they seem to come and go on jenkins10:50
mzanettiyeah, will look into that10:51
SaviqCimi, I don't think there's a dismiss area "on top" of the keyboard10:53
SaviqCimi, there's only the 1GU padding you have above the keys10:53
CimiSaviq, ok right indeed10:54
CimiSaviq, so the y in the socket is wrong in portrait10:54
SaviqCimi, here's the MP that dealt with that10:55
Saviqhttps://code.launchpad.net/~dandrader/ubuntu-keyboard/improve_kbd_info_ipc/+merge/19041810:55
CimiI'm looking at ubuntu-keyboard10:55
SaviqCimi, not sure I can confirm your findings, though - I added a rectangle here, too, and it covers the whole bottom of the screen correctly10:56
CimiSaviq, in portrait?10:56
SaviqCimi, yes10:56
Cimion mako>10:56
Ciminot here10:56
SaviqCimi, on flo actually, let me try on mako10:56
SaviqCimi, looks fine on mako, too10:58
SaviqCimi, maybe you're seeing http://bazaar.launchpad.net/~mir-team/unity-mir/trunk/revision/238 ?10:59
SaviqCimi, or well I meant https://code.launchpad.net/~afrantzis/unity-mir/fix-1332624-input-area/+merge/22476810:59
SaviqCimi, that was released yesterday11:00
SaviqCimi, https://bugs.launchpad.net/unity-mir/+bug/133262411:00
ubot5Ubuntu bug 1332624 in unity-mir "cant' swipe the keyboard away anymore" [Critical,Fix released]11:00
Cimilet me upgrade11:00
SaviqCimi, hah11:02
SaviqCimi, it moved after typing11:02
SaviqCimi, and yeah, it's panel-height11:04
Cimiit was moving immediately for me11:05
Cimianyway11:05
Cimidoesn't matter11:05
Cimiin wizard is like double panel height11:05
SaviqCimi, was one panel height for me11:05
Cimithat's why it is possible to tap on back button underneath11:05
CimiSaviq, weird - with the today image I have no bug11:06
Cimilet me see in wizard11:06
CimiSaviq, did the rectangle move after typing in unity8?11:06
SaviqCimi, no11:06
CimiSaviq, unity8 is fine for me with 111 image11:07
SaviqCimi, yeah, so that but above11:07
CimiSaviq, wizard is 1 panel height shifted11:07
SaviqCimi, while in wizard panel isn't taken into account11:08
Cimibut what is setting the panel11:08
Saviqunity-mir11:08
Saviqbut OSK surface should be fullscreen11:08
Saviqpanel shouldn't matter11:08
Cimistatus is maximised11:09
Ciminot fullscreen11:09
Saviqyeah11:09
SaviqCimi,11:09
Saviqthere's an explicit topMargin in OSKController in Shell.qml11:09
SaviqCimi, that should've been a hint11:10
CimiSaviq, ok yeah sounds like it11:11
CimiSaviq, it is hardcoded or what?11:11
SaviqCimi, yes it is11:11
Cimiin unity mir?11:11
CimiI could get the panel height from ApplicationManager in Unity.Application11:11
SaviqCimi, there, and in Panel.qml11:11
Cimi(maybe)11:12
Cimibut wizard should not look into unity811:12
SaviqCimi, panelHeight: units.gu(3) + units.dp(2)11:12
SaviqCimi, with a FIXME that this shouldn't be hardcoded11:12
CimiI can copy that as topMargin? ;\11:12
SaviqCimi, yeah11:12
SaviqCimi, I agree OSK should communicate global coords11:13
SaviqCimi, and that's worth a bug11:13
SaviqCimi, but a quick fix is just to copy that11:13
=== MacSlow is now known as MacSlow|lunch
CimiSaviq, thanks11:13
mhr3Saviq, wrt SettingsModelInterface, shouldn't it have a isEmpty / rowCount prop? so you can do showSettingsButton: !scope.settingsModel.isEmpty11:17
Saviqmhr3, yeah, count, forgot that's not Q_PROPERTY by default...11:18
mhr3Saviq, or do you consider scope.settingsModel being null enough?11:18
Saviqmhr3, I'd probably like both11:18
CimiSaviq, so where is the bug then? unity mir or ubuntu keyboard?>11:19
SaviqCimi, keyboard11:19
SaviqCimi, and unity811:19
Cimiunity8??11:19
facundobatistaHolas11:24
Saviqo/11:26
SaviqCimi, unity8 should not have the margin applied there11:26
Cimiah sure11:26
Cimiyes11:26
Saviq/food11:26
* mzanetti thanks init for cleaning up his window mess11:27
=== alan_g is now known as alan_g|lunch
=== MacSlow|lunch is now known as MacSlow
=== _salem is now known as salem_
Saviqmzanetti, remember we'll need to bump the UITK dependency12:53
Saviqmzanetti, but we need to wait until silo 4 lands & merges finally12:54
mhr3Saviq, are there plans to resurrect grid-see-more branch? i'd kinda need that so we can have the new section header queries12:58
Saviqmhr3, yeah, there are13:00
Saviqmhr3, the string needs to come from scope, though13:00
Saviqmhr3, and we need to add a sticky footer13:00
mhr3Saviq, what string?13:00
Saviqmhr3, "show more" / "show all"13:00
mhr3oh?13:00
mhr3why?13:00
greyback_l13:01
greyback_f*cking multi-monitor13:01
Saviqmhr3, because it's meant to be "show more" or "show all", depending on what it will actually do13:01
Saviqgreyback_, ah so that's why we know your password?13:01
mhr3Saviq, it will be just all for now, no need for more imo13:01
mhr3Saviq, unless you decide to chunk it13:01
Saviqmhr3, tell Mike that13:01
greyback_Saviq: you know an old password...13:01
Saviqmhr3, no no, it's more because it can't display all13:01
Saviqgreyback_, I know I know13:01
Saviqmhr3, not because pressing it will display less than show all would13:02
greyback_gnome-terminal's cursor flashes even when its window does not have focus. So annoying13:02
Saviqgreyback_, hmm?13:02
Saviqgreyback_, not here it doesn't13:02
greyback_does here.13:02
Saviqgreyback_, I get a hollow rectangle when unfocused13:02
* greyback_ digs into settings13:03
Saviqgreyback_, I'm using byobu + bash, maybe zsh doesn't deal with that well?13:03
greyback_Saviq: maybe13:03
mhr3Saviq, afaik there are no plans to be able to do offsetted queries, so i'm not sure now13:03
Saviqgreyback_, same behaviour without byobu13:03
Saviqmhr3, again, it's not about what we can display in dash13:04
Saviqmhr3, it's about what we can't13:04
greyback_Saviq: for me same bad behaviour with bash13:04
Saviqmhr3, if the max you can display in dash is not "all" (because of, say, API), then "show more" is meant to be shown13:04
Saviqmhr3, let's confirm with Mike13:05
mhr3Saviq, yea, that's the first time i hear this tbh13:05
mhr3Saviq, in spec?13:05
Saviqmhr3, yeah... no13:05
Saviqmhr3, well, in multiple specs13:05
mhr3sigh13:05
Saviqmhr3, some say "more", some say "all" ;)13:05
=== alan_g|lunch is now known as alan_g
SaviqNOW MIGRATE ALREADY!13:14
* Saviq can die now13:22
Saviqdednick, please merge trunk into lp:~nick-dedekind/unity8/move-indicator-qml13:36
dednickSaviq: doing13:39
Saviqdednick, thanks13:39
dednickSaviq: done13:48
Saviqdednick, tx13:49
mhr3Saviq, is u8 trunk not merged with ubuntu?13:52
Saviqmhr3, it's not13:53
Saviqmhr3, I'll prep an MP shortly13:53
Saviqmhr3, why?13:53
mhr3landing bot complaining13:53
mhr3https://ci-train.ubuntu.com/job/landing-013-1-build/55/console13:53
Saviqmhr3, ah you're landing that separately?13:54
Saviqmhr3, I added it to my silo13:54
Saviqmhr3, lemme prep the mP13:54
mhr3Saviq, imo should just push the fix to trunk, we said the two should be always in sync13:57
Saviqmhr3, yeah ok, doing13:57
mhr3Saviq, when was the last time you just pushed to trunk anyway, doesn't it sound exciting? :)13:58
=== pete-woods is now known as pete-woods-lunch
Saviqmhr3, done13:59
mhr3ty13:59
greyback_Cimi: hey, where does the welcome wizard print its logs? I can't find it in .cache/upstart or /var/log/upstart14:05
Cimigreyback_, ~/.cache/upstart/ubuntu-system-settings-wizard.log14:14
greyback_Cimi: ah, forgot the ubuntu- prefix. thanks14:15
greyback_l14:15
Cimigreyback_, what are you looking?14:16
greyback_Cimi: welcome wiaard needs some changes to make compatible with QtCOmp14:16
greyback_Cimi: I've them done, just wanted to check things are ok14:16
greyback_Cimi: I'm not seeing any wifi networks in the list, any idea how I could debug?14:17
Cimigreyback_, we had this issue once, was due to initialisation errors of the indicators14:18
Cimimaybe mterry knows14:18
mterryCimi, I don't recall *why* it was happening -- I believe it went away for us by starting indicators differently -- instead of just the network-indicator, we did the general indicator startup14:18
Cimigreyback_, do you hae wireless in Ireland?14:19
Saviqlol14:21
mhr3Saviq, could you jump into the meeting 5 minutes earlier? got few Qs14:27
mhr3the one in 30ish minutes14:27
Saviqmhr3, prolly yes14:31
=== salem_ is now known as _salem
=== alan_g is now known as alan_g|tea
kgunngreyback_: did the wifi list show up after you touched it ?14:40
greyback_kgunn: reboot fixed it14:40
kgunni saw the same bug last week...any interaction populated the list14:40
kgunnah14:40
greyback_Cimi: mterry: ever seen this crash? http://pastebin.ubuntu.com/7742229/ - I reproduce reliably with my code, when I go to list of networks (one already selected), then hit back14:42
mterrygreyback_, someone filed a bug with that, yea14:42
greyback_mterry: ok14:42
mterrygreyback_, haven't looked at it yet14:43
seb128somebody needs to do a landing with https://code.launchpad.net/~nick-dedekind/qmenumodel/unitymenuaction.lp1334203/+merge/22506314:43
seb128mterry, ^ I think that might be the fix for that issue14:43
seb128https://bugs.launchpad.net/qmenumodel/+bug/133420314:43
ubot5Ubuntu bug 1334203 in ubuntu-system-settings (Ubuntu) "ubuntu-system-settings-wizard crashes/freezes when using the back button" [High,New]14:43
greyback_mterry: no worries, just checking it wasn't consequence of my changes14:43
mterryah indeed14:43
mterryseb128, thanks14:43
seb128who is lander for qmenumodel?14:43
mterrydunno14:43
seb128thostr_, ?14:44
=== alan_g|tea is now known as alan_g
thostr_seb128: yeah, let us have a look at it...14:46
seb128thostr_, thanks, https://code.launchpad.net/~charlesk/qmenumodel/lp-1282282-dangling-UnityMenuAction-pointer/+merge/211211 is also approved/would be good to land14:47
Cimigreyback_, yes that is the fix14:53
Cimihttps://code.launchpad.net/~nick-dedekind/qmenumodel/unitymenuaction.lp1334203/+merge/22506314:53
greyback_Cimi: good to know14:54
Cimigreyback_, I am not aware of any issue atm14:54
Saviqmhr3, ready when you are14:54
mhall119I don't like the Apps scope changes, why did we split the "available" functionality to a different screen?15:03
Saviqmikenagle, ↑15:03
=== _salem is now known as salem_
greyback_Saviq: suru icons landed?15:12
=== greyback_ is now known as greyback|post
Saviqgreyback|post, yes15:14
=== pete-woods-lunch is now known as pete-woods
pete-woodsSaviq, Cimi: hey guys, just checking what the status on the infographics is?15:18
pete-woodsI tested it (to success) last week, but wasn't sure if there were more changes coming15:18
Cimipete-woods, we're waiting a release of the sdk15:19
pete-woodsCimi: okay, cool, thanks for the infos! :)15:19
pete-woodspresumably that's for your fixed cross fade image thingy15:19
Cimiy15:20
Cimiactually branching right now15:21
Cimiargh got infected by tags15:32
Cimifixing15:32
CimiSaviq, I updated the infographics branch with merge from trunk and removed crossfadeimage15:35
SaviqCimi, thanks, I'll rebuild the silo after the unity8 landing15:36
CimiSaviq, I am on holiday from tuesday afternoon, so need fix until monday evening pls :P15:37
SaviqCimi, why is there only 4 dots at the bottom of wizard when there's 5 pages?15:37
SaviqCimi, yeah, I plan to land this asap15:37
Cimiwill have a look at the wizard in a bit15:38
sil2100Saviq, rsalveti, seb128: what do you think of such output? https://ci-train.ubuntu.com/job/landing-000-1-build/7/console15:39
sil2100I had problems with finding the right wording, so I invented the whole 'twin package' thingy15:39
dandraderSaviq, mzanetti: trying out latest trunk. I'm unable to drag in the launcher (left-edge drag). do you guys experience the same?15:39
seb128sil2100, looks good to me15:39
sil2100You can override this of course15:40
=== gatox is now known as gatox_lunch
seb128sil2100, you can directly use ""their -gles counter-parts" instead if you want to avoid new vocabulary there though15:40
rsalvetisil2100: sounds good15:40
rsalvetiyeah15:40
dandraderSaviq, mzanetti right edge also broken for me15:40
sil2100Yeah, I wanted to get ready for the future, in case we have new cases of such duplicated packages15:41
sil2100Anyway, we can change the wording as we want even later15:41
Saviqdandrader, sounds like your edges got stuck15:41
Saviqdandrader, works fine here15:41
* dandrader reboots his N415:41
dandraderhmm, I though "sudo restart lightdm" would be enough to get rid of any bogus state. let's see if a full reboot does it15:42
Saviqmhall119, if it's gonna help you, the installed category is not going to be collapsible soon15:43
Saviqmhall119, so you'll basically have a page of your installed apps, and then a button to get more15:43
dandraderyeah. rebooting solved it. maybe a had a rogue usc running....15:44
=== greyback|post is now known as greyback
dandrader*I had15:44
mzanettidandrader: hmm... haven't tried that in the last few hours15:44
dandradermzanetti, anyway. false alarm. rebooting did solve it. :/15:45
mzanettiah ok15:45
Saviqsil2100, I wonder if this check could only be ran on publish, and not on build?15:45
mhall119Saviq: I'm not sure that's what I wanted15:46
mhall119but, I'll wait and see15:46
Saviqmhall119, it was a design decision to split it out15:46
mhall119I know it's not your decision15:46
sil2100Saviq: we could do that, my original plan was that as well, but then I thought that actually the earlier the better15:46
mhall119it was probably going to stop being useful when we have more apps anyway, it was just a nice feature15:46
mhall119even having it just show results in response to a search would be nice15:47
Saviqsil2100, yeah, but that means you'll need to keep the second package updated as you rebuild15:50
Saviqsil2100, your call really15:51
Saviqsil2100, maybe you're right, all should be tested together15:51
sil2100Saviq: I'm not entirely sure myself, it's easy to move it one way or another, so I'm open to propositions anyway ;)15:51
Saviqmzanetti, hmm PageHeader.qml missing Themes 1.1?15:52
mzanettiSaviq: ?15:57
Saviqmzanetti, most recent autopilot failure in new-header branch15:59
mzanettiSaviq: did uitk land?15:59
Saviqmzanetti, yes, but maybe that run didn't get it yet16:00
* Saviq looks16:00
Saviqmzanetti, ah stupid16:00
Saviqmzanetti, no, the new uitk is in silo16:00
mzanettiah ok16:01
mzanettiphew :)16:01
mhr3Saviq, so how do we do the sign in button-cards?16:18
mhr3Saviq, feels like it should special, like new category-layout or something16:18
Saviqmhr3, it could be a completely custom category16:19
Saviqmhr3, like you say category-layout: "sign-in"16:19
Saviqmhr3, and the shell takes over16:19
mhr3Saviq, do we just ignore the component mapping completely then?16:19
mhr3or pretend that it's a card with summary and title or something?16:20
Saviqmhr3, depends, do we want to put multiple buttons in one category, or category per button?16:20
mhr3designs have the latter16:21
mhr3generalization would suggest the former :)16:21
Saviqmhr3, well, the designs can be achieved either way16:21
Saviqmhr3, we'd just maybe need an account type id to be able to display the logo and colours or something16:21
Saviqmhr3, and open the right account page straight away when possible16:22
mhr3Saviq, *plus* we could misuse it for "This category requires internet connectivity [REFRESH]" :)16:22
Saviqmhr3, no we couldn't16:22
Saviqmhr3, that thing needs to be above departments16:22
mzanettinow that's interesting... the screen is off but tapping it gives me haptics feedback in some places16:22
Saviqmhr3, in header16:22
=== alecu is now known as alecu_
mhr3Saviq, it *should*, api doesn't support such hints though16:22
Saviqmhr3, yet ;)16:23
mhr3just saying, would be easy fix and make it more reusable16:23
Saviqmhr3, you mean multiple buttons per category?16:24
mhr3i mean thinking about the no-internet case16:24
mhr3will generalize the sign-in card16:25
Saviqah so sign-in to the internets?16:25
Saviqmhr3, so we'll have category-layout: special16:26
mhr3dejavu16:26
Saviq;)16:26
Saviqmhr3, at least it'd be sourced by the scope still16:28
mhr3Saviq, but yea, what i'm suggesting - special category-layout, and mentioning in docs that in that case of using it, you can only map summary and title?16:28
mhr3s/?//16:28
Saviqmhr3, so card-layout: {sign-in,no-internets}16:29
Saviqmhr3, and then a custom set of components16:29
Saviqmhr3, do you have the component names hardcoded, or if we said that "account-type" is a component, would that work?16:29
mhr3Saviq, it's pretty much hardcoded for the json merging16:30
Saviqmhr3, I wouldn't like to abuse current component names for this16:31
mhr3Saviq, well, it's also because of the model, it's roles aren't dynamic16:32
mhr3its*16:32
Saviqmhr3, ah right, model16:32
Saviqmhr3, well, they could be ;)16:32
mhr3doesn't mean they should :)16:32
Saviqmhr3, well, otherwise we'll end up saying subtitle is account type or some such :|16:33
mhr3Saviq, honestly, i don't find it an issue, it's still a card16:34
mhr3action card, but still a card16:34
mhr3and yes, it does have subtitle16:34
Saviqmhr3, no but I mean16:35
mhr3doesn't matter that the subtitle will be used for account type16:35
Saviqmhr3, that we'd end up with a account type id (like facebook, xmpp etc., for accounts consumption)16:35
Saviqmhr3, in some random component, just because it's not used16:35
=== gatox_lunch is now known as gatox
mhr3ah, right, what does uoa actually need?16:36
Saviqmhr3, not sure really16:36
mhr3Saviq, but.. we've been already discussing that this actionable card could be maybe used even for "Location unavailable, please allow access in scope settings [Settings]"16:37
Saviqmhr3, but I imagine account type id is a must16:37
Saviqmhr3, yeah, well, would that be a different type of special card though?16:37
mhr3i'd say ideally no16:37
SaviqI'd say yes, TBH16:38
Saviqbecause they will differ in design for sure16:38
mhr3hm, and i'd also open doors for more misuse16:38
mhr3so maybe you're right16:38
Saviqyeah16:38
Saviqthey need to be as locked down as possible16:38
=== dandrader is now known as dandrader|lunch
Saviqmhr3, I'd think the location one would be closer to how the internet one looks16:40
mhr3eh, more and different api :/16:40
Saviqmhr3, what's more, it needs to be dismissable somehow, if you forbade the scope to get access to network or location16:40
mhr3forbidding access to network?16:41
mhr3hope that's not something for rtm :)16:42
Saviqmhr3, just thinking ahead16:43
mhr3Saviq, stop scaring me!16:43
Saviqmhr3, just means that shell needs to know what type of card that is16:44
Saviqs/card/banner/16:44
Saviqwhatnot16:44
mhr3right16:44
=== alan_g is now known as alan_g|EOD
=== dandrader|lunch is now known as dandrader
=== salem_ is now known as _salem
=== _salem is now known as salem_
Saviqdandrader, btw, you know we have inline comments in launchpad reviews now?18:24
dandraderSaviq, yes, but I don't like them as you cannot select a stretch of code to comment, only a specific line18:26
dandraderSaviq, and sometimes it18:26
dandraderit's clearer to comment on the resulting code, not the diff18:26
Saviqdandrader, mhm, was just asking18:26
Saviqdandrader, and I agree with the first point, wonder how the second could be solved18:27
dandraderSaviq, it's sad that lp doesn't properly display unformatted text (i.e. code) though18:27
dandraderSaviq, I recall I've used a review system where you could select a stretch of code and comment on it18:28
Saviqdandrader, I think that's just a missing feature, not something that couldn't be done18:29
dandraderah, for the second, I think sticking with copy-paste is fine as long as the comments could display pasted code properly18:29
dandraderinline diff comments is still in beta, right? so I also don't trust it :D18:30
dandradermy precious comments could get lost! who knows! :)18:30
Saviqdandrader, you'd be surprised18:37
Saviqdandrader, they survive closed tabs, closed browsers etc.18:37
Saviqdandrader, they are saved in LocalStorage AFAICT18:37
=== dandrader is now known as dandrader|afk
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
greybacktedg: hey, is there a nice way to start a click app under gdb?20:14
Saviqgreyback, zbenjamin was working on a launcher that would allow that, but afaict for now20:15
Saviqgreyback, just read up the Exec line in the .desktop file20:15
greybackSaviq: I know /how/ to do it manually. But I wondered if a nicer way existed20:15
Saviqgreyback, you probably need to do it in Path from the .desktop file as well20:15
Saviqgreyback, k, so yeah, nothing there yet, but zbenjamin's working on it20:16
Saviqoh mhr3 got r1000 on unity820:19
Saviqnasty20:19
cwayneoh cool, we'll get a blue led now when there's a message?20:22
Saviqcwayne, yup :)20:24
cwaynewill different notifications have different color led flashes?20:24
=== robru_ is now known as robru
Saviqfoook tags20:45
Saviqgreyback, if you merged trunk, you need to strip tags in your branches20:47
=== _salem is now known as salem_
Saviqmzanetti, you're potentially infected, too ↑20:48
greybackSaviq: dammit20:48
Saviqgreyback, indeed, my bad for not checking :|20:48
mzanettihehe20:49
* mzanetti uses a modded commit hook to print tags on every commit :)20:50
mzanettishould I propose that for merging?20:50
Saviqmzanetti, that would give us false sense of security ;)20:51
Saviqmzanetti, assuming people would actually run it20:51
mzanettiSaviq: how about a test that simply does bzr tags | grep "?" or similar?20:52
Saviqmzanetti, same thing20:52
mzanettijenkins would fail on it20:52
Saviqmzanetti, and actually we have one ? tag20:52
Saviqmzanetti, jenkins doesn't deal with branches20:52
mzanettiwell, I'm sure there would be a regexp that would fit20:52
Saviqmzanetti, it just exports a tarball20:52
=== salem_ is now known as _salem
Saviqmzanetti, I even have a sh snippet to drop branches to that print out the number of tags...20:53
Saviqand I should use it... but forget :|20:53
Saviqmzanetti, we already have a "did you check..." in the checklist... but apparently people don't actually read the question :|20:54
mzanettilol, yeah... the checklist20:54
mzanettialso the design question...20:54
mzanettithere *never* is ux change20:55
Saviqyeah, I'm inclined to drop the checklist, it's not really working20:56
mzanettiwell, tbh the tag question is working for me20:56
Saviqdoes for me, too20:56
Saviqthat's how I found out we have them back again20:57
mzanettiI constantly find myself to alt-tab back to the terminal and recheck the output from the commit hook in the history when I get to that question20:57
Saviqbut design and core dev review...20:57
mzanettiyeah...20:57
SaviqMPs are too granular for that20:57
mzanettiyes20:57
Saviqwe get core dev review when publishing anyway20:57
Saviqand we ask design for review here and there, too20:57
Saviqit's gonna be better with airline I think20:58
Saviqwhere we'll be able to ask for design review on a whole silo (aka ticket), not per-MP20:58
Saviqand it'll be easier for everyone to pick a silo up20:59
mzanettiyep20:59
cwaynethe new icons in system-settings are bitchin'21:04
Saviqheh :|, rewrote the strip-tags script in python... down to under a second locally, but didn't really help for the remote branch21:31
Saviqat least it's not trying to remove tags that are not tehre21:31
=== dandrader|afk is now known as dandrader
=== _salem is now known as salem_
=== salem_ is now known as _salem
dandraderbug 1332598 was fixed by unity8!? I don't get it21:52
ubot5bug 1332598 in Unity 8 "make testShell crashes" [Critical,Fix released] https://launchpad.net/bugs/133259821:52
dandraderSaviq, ^21:52
Saviqdandrader, https://code.launchpad.net/~aacid/unity8/no_model_reset_from_destructor/+merge/22512721:53
Saviqdandrader, there were three components that were involved in that (or well, were found to be broken during investigation for that bug)21:53
dandraderSaviq, including that Qt 5.3 fix?21:54
Saviqdandrader, yes, there were qtbase, qtdeclarative and unity8 components for that bug21:54
dandraderok21:54
Saviqdandrader, the branches are linked (but bug didn't actually have the relevant affects: set)21:55
=== mterry is now known as 18VAAOQDS
Saviqoh well, 3:13 vs. 20:44... significant improvement after all...21:59
greybackdandrader: hey, before you go, please don't forget to push your key sending branch, so we can continue working on it22:00
dandradergreyback, sure. don't worry about it22:01
dandradergreyback, I'm back at it now. who knows I get lucky and get it finished before my EOD?22:02
greybackdandrader: you have holidays, go enjoy them!22:02
dandradersays the guy that's still online at 23:03 local time :)22:04
greybackSaviq: unity8's changelog has a rather verbose entry from Albert. Think it could be edited down22:04
greybackdandrader: I don't have holidays. If I had, I wouldn't be online!22:04
Saviqgreyback, yeah... that's what he put in the commit message (not the first time)22:04
Saviqgreyback, we could, if you MP, I'll ACK, but can't be bothered really ;)22:05
greybackNor I22:05
greybackeasier to moan22:05
Saviq@unity: rewrote the strip tags script in py, cuts down the time to 1s locally, 3mins remotely (from a minute, 20 minutes, respectively): http://people.canonical.com/~msawicz/unity8/strip-u8-tags.py22:09
18VAAOQDSoh nice22:09
18VAAOQDSwoah22:10
=== 18VAAOQDS is now known as mterry
mterryweird22:10
Saviqlol22:10
greybackbregma: hey, I've hit the old problem of Mir GLES and Qt's desktop OpenGL22:14
bregmagreyback, is that Qt 5.3?22:15
greybackbregma: your solution was to split qtubuntu into 2 separate packages, where in one you call eglBindAPI ...22:15
greybackbregma: yes22:15
greybackbregma: note: qtubuntu still works fine. I have a separate project which has hit the same problem22:15
bregmaI though they were doing proper runtime autodetection in Qt 5.3, but I admit I haven't been paying close attention22:17
greybackI'm just curious if you considered other options like checking the GL string for "Mesa" - or doing a test eglBindAPI call to see if desktop GL would be accepted by the driver22:17
greybackbregma: maybe they have. I'm only learning about this now. Should I ping rsalveti?22:17
bregmathe problem was that _both_ EGL and GL were accepted, but Qt did not accept multiple values so it defaulted to EGL22:18
bregmaer, GLES22:18
greybackaccepted by what? the driver? Where does it specify that?22:18
bregmaI'm forgetting all my EGL now, but as I recall there's a query function and it reported both GL and GLES were supported, but Qt itself uses an enumeration22:19
greybackhmm ok, that gives me somewhere to start22:20
bregmait was after an eglChooseConfig() call, but for the life of me I can't remember where22:28
greybackbregma: platformsupport/eglconvenience/qeglconvenience.cpp ?22:32
bregmagreyback, yep, that's the one22:49
greybackbregma: dynamic gl/gles switching seems to have landed in 5.3 anyway22:59
greybackbut not turned on in our packaging23:01

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