/srv/irclogs.ubuntu.com/2015/09/16/#ubuntu-app-devel.txt

=== daker_ is now known as daker
=== chriadam|away is now known as chriadam
ahoneybunsince everytime I install the SDK it breaks my machine I've decided to install it in a VM01:35
ahoneybunlol01:35
xtalmathis it possible to make my application generate a prompt dialog, that is system wide? so it can block me from using the browser?01:55
xtalmathwhat call is used for a system wide prompt?01:55
rschrollxtalmath: I suspect not, at least for an app with standard permissions02:17
rschrollNot sure, though02:17
xtalmathperhaps its just a matter of going fullscreen and not forwarding keyboard input, handling it all?02:19
=== kalikiana_ is now known as kalikiana
dholbachgood morning06:21
=== chriadam is now known as chriadam|away
davmor2popey, mzanetti: Hey guys so did you get my pings from last night :(  So the accounts issue I think is definitely an apparmor issue based off of the name I would assume.08:22
mzanettidavmor2, right. I saw it, but didn't look into it any more yday08:34
mzanettidavmor2, can't reproduce this reliably08:53
mzanettiI've seen it once, but not any more08:53
popeydholbach: click-review complains when a click package contains two desktop files. Yet we do allow two desktop files in a click (as I understand it from conversations between aq and ted last week). Should we lift that check, or maybe turn it into a warning?09:30
=== zbenjamin_ is now known as zbenjamin
dholbachpopey, sounds like it should be a warning, let me ask jdstrand10:49
dholbachjdstrand, should or do we allow two desktop files in a click?10:49
popeydholbach: thanks.10:52
=== faenil_ is now known as faenil
=== faenil is now known as faenil_
=== faenil_ is now known as faenil
karniIs there any way I can get my hands on the new qmake app template that supports desktop file translation?11:46
karniI hear one exists alerady, but may not have been released yet.11:46
karniI've tried on 15.04 with ppa installed, new project pro file does not yet include the desktop file for translation.11:47
kalikianapopey: ping, wrt issues with the date (or time?) picker in some app(s) you were seeing11:48
popeyyo11:49
popeykalikiana: davmor2 saw it first, in reminders (notes) app on rc-proposed. http://people.canonical.com/~alan/reminders/com.ubuntu.reminders_0.5.491_multi.click  is a good test, create a local note then press the clock icon in the note to make a reminder11:49
kalikianaa bird told me you saw some issues with 1.3 versus 1.2 in an app using a date or time picker11:49
kalikianapopey: okay, will give it a try11:50
kalikianais that a special branch?11:50
kalikianaof reminders11:50
popeykalikiana: no, its trunk11:53
popeykalikiana: we wanted to push to the store, but this issue came up in qa11:53
K1773Rcan i get GPS coordinates from CLI?11:54
kalikianapopey: just checking to get an idea where the relevant changes are. ie if this is a very recent uitk change or potentially with the code in the app (or both)11:55
popeykalikiana: version in store is bzr rev 472, this one is 491 (obviously from the filename) :D - mostly bug fixes.11:56
popeykalikiana: some of us can't reproduce this issue, I have only seen it on rc-proposed11:56
popeyso between ota-6 and now11:57
kalikianahmmm on stable it takes ages to startup, but it looks fine, I'll switch to proposed and try there12:08
kalikianapopey: I see the border and the highlight of where the selection would  be but no number, I assume that's the bug12:21
popeykalikiana: correct12:21
kalikianahmm and the last change in trunk moved everything to 1.312:22
kalikianaI'll investigate that a bit12:22
=== faenil is now known as faenil_
popeykalikiana: it happens with r490 (before we moved everything), r491 was a (failed) attempt to fix this issue (or at least be consistent) - r490 click can be found in same place12:39
jdstranddholbach: re 2 desktop files> there is no security or policy reason to not allow it. I'm not sure the apps scope will handle it properly (for adding or deleting)13:00
dholbachpopey, ^13:00
dholbachpopey, do you have the example click somewhere?13:00
karnidpm: kyleN: I need to support plurals in C++, is ngettext the way to go?13:05
kyleNkarni, for plural translations yes13:06
karniDon't we have a unified way for translating both from C++ and QML? (C++ equivalent of i18n.tr )13:06
kyleNin c++ we use gettext13:07
kyleNat least in our projects13:07
nik90karni: We use gettext for clock app as well. Example http://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/trunk/view/head:/backend/modules/WorldClock/statictimezonemodel.cpp13:08
karnikyleN: I know, and I think in general the story is kinda poor. Compare that to translation experience on Android, where you drop translated strings in a directory named by language code, and you're basically set.13:08
karninik90: thanks, looking13:08
nik90karni: make sure to set the locale properly, otherwise it won't work13:08
karninik90: right, that is what we're doing. I just happen to have a more interesting case of a required plural form :) thanks for reference though13:09
nik90ah ok13:09
popeydholbach: i can get one13:10
karninik90: inteersting, you're using dgettext13:10
karninik90: if you bind the translation domain, I suppose plain gettext has the same behavior as dgettext (?)13:11
nik90karni: I believe I looked at the app scope if I vaguely remember. It is the only c++ code we were translating, so used a reference implementation.13:11
karni:D13:11
kyleNkarni yes. dgettext allows you to specify the domain for that gettext call13:12
kalikianapopey: ah, thanks, I don't need to biset it then (was in a meeting so didn't get far yet, investigating more in a moment)13:16
popeykalikiana: thanks for taking the time to look at it, appreciate it13:16
popeydholbach: jdstrand http://people.canonical.com/~alan/multi-desktop-click.sil_0.1_all.click13:26
=== JMulholland_ is now known as JMulholland
jdstrandpopey: dholbach asked me about that (I think) earlier. I see this:13:28
jdstrand - lint_hooks_multiple_apps13:28
jdstrandmore than one desktop app specified in hooks13:28
jdstrand/tmp/multi-desktop-click.sil_0.1_all.click: FAIL13:28
popeyjdstrand: yeah, i asked dholbach, he asked you, then he asked for a click :)13:29
popeyjdstrand: I think it should be a warning, not an error13:29
jdstrandbut, I think that is only there because of (old?) limitations on the device13:29
popeyif num_desktops != 1 then warning13:29
jdstrandsure13:29
popeydo you want me to file a bug to track this?13:29
jdstrandI'm happy to change that, but someone should confirm that will work correctly13:30
popeyok, i can do that13:30
jdstrandI can't recall, possibly b euno's team? there might've been a store component, but I think that might have been fixed? so, the store and the scope (install and remove) I think are what were the issues13:30
popeyahhh, good point, will test that too13:31
=== JMulholland_ is now known as JMulholland
=== javiercrowsoft1 is now known as javiercrowsoft
davmor2popey: weather app spotted an niggle, open it, allow location, let it load, scroll to the last day, and tap on it.  If it is the first one you click on you have no idea that the days details are off the bottom of the screen, would be nice if we could make it scroll up slightly13:36
popeydavmor2: will file a bug for that, thanks, agree, a niggle13:36
davmor2popey: should we be able to still access the 10 day forecast also?13:37
popeynot with TWC13:37
popeyTWC limits us to 5 days, OWM API allows 10 days13:37
popeywe deliberately didn't add links to the TWC site13:37
davmor2ah that's fine then13:38
davmor2popey: and another, swipe up to add a location, tap on the plus, add a location, it moves back and shows you the location only now the back arrow is surrounded by black13:42
popeydavmor2: sounds like a toolkit bug13:42
popeyyou on rc-proposed?13:42
nik90davmor2: that is a recent SDK bug13:42
popey(I have seen this in another app)13:42
* nik90 was just passing by ;)13:43
davmor2only happens on that one page which is odd13:43
popeyyeah, i saw it in reminders only once13:43
popeyjdstrand: filed https://bugs.launchpad.net/click-reviewers-tools/+bug/1496402 to track it13:47
ubot5Ubuntu bug 1496402 in Canonical Click Reviewers tools "Number of .desktop files check should issue warning, not error" [Undecided,New]13:47
popeyJamesTait: ^ see that bug, do you forsee any issues with having clicks containing multiple .desktop files from the store point of view? (click scope and dash seem to cope fine)13:47
* JamesTait looks13:48
popeyta13:48
JamesTaitpopey, off the top of my head, the only scanning we do of the .desktop file is in click-reviewers-tools on the server. So I don't think it'll cause a problem.13:50
jdstrandpopey: ack, commented with a question that might simply be a paste of JamesTait's answer13:51
popeyheh13:53
JamesTaitOn a less technical level, it might be an idea to have such packages trigger a manual review just because, IIUC, that would allow the two apps in the package to share data.13:53
jdstrandJamesTait: yes, but that was always intended in the security design13:54
popeyyeah, and in this instance the developer wants two apps to share data13:54
popeyi think its a relatively rare use case13:54
jdstranddesktop apps should be able to share user data-- the user is driving that interaction. the sharing becomes an issues with behind the scenes stuff, like push helpers13:55
jdstrandissue*13:56
jdstrandanyhoo, ok, I'll fix this in the tools13:56
popeythanks13:56
JamesTaitRight. But the alternative to a manual review might be something like a warning on the upload UI saying "are you sure you want these apps to share data?" and Joe Q Nastyhacker isn't going to say "Oops, no." which puts the onus on the end user.13:56
JamesTaitWhich is fine, and is how we deal with other permissions, as long as it's clear to the end user what it actually means.13:57
=== faenil_ is now known as faenil
=== faenil is now known as faenil_
davmor2popey: Ensure icon looks like a cloud with a 12 spiked sun:- http://people.canonical.com/~alan/screenshots/device-2015-03-05-131345.png I assume that is the wrong screenshot right14:17
popeycorrect14:17
popeywill fix14:17
popeyfixed davmor2 http://people.canonical.com/~alan/screenshots/device-2015-09-11-142604.png14:18
davmor2popey: thanks I figured you had the image there somewhere :)14:19
davmor2popey: question how do I know what the wind speed is for now?14:20
popeydavmor2: go outside and put a finger in the air?14:21
popeydavmor2: we discussed that but I dont think we ever got a design for it, will make a note14:22
davmor2popey: Today gives me, haze, current of 15°c, minimum 7°c and max 15°c and that is it, if I tap on tomorrow I actually get way more info than I do for today which seems a little backwards14:23
popeyyeah14:23
popeydavmor2: there's nothing in the design for it, I'll raise it as a design bug. I know we talked about it in the past, I think we just haven't had the design, thats all.14:25
davmor2popey: also not a fan of the scattered cloud icon, it would be too easy to mistake for rain for a new user I think14:25
davmor2But I love the rest :)14:26
popeyI dont mind it14:26
popeyjust scrolled forward and see lightning forecast for me next week14:26
popeyoh, not next week, friday14:27
davmor2popey: you threw me then I'm thinking how is he scrolling to next week it only goes to sunday :)14:29
popeyheh14:31
davmor2popey: I think that's it.  So 1 design bug, 1 niggle, and one sdk bug https://launchpad.net/bugs/1495408  do you want me to write up bugs for those or did you say you were?14:32
ubot5Ubuntu bug 1495408 in ubuntu-ui-toolkit (Ubuntu) "Black square in header" [Critical,In progress]14:32
popeyI haven't if you could, that would be super14:32
davmor2popey: will do, do you want to let it in as is and fix it up, or fix the design issue first and then land it?14:33
popeydavmor2: I'll speak to the devs, if you +1 it, I'll hold it back till I speak to them and you again14:34
popeyokay?14:34
davmor2popey: that's fine,  I'm happy for it to go in as is, but then I would provisionally put a block on ota7 on the design bug.  I would always expect more info today than any other day :)14:36
popeyyeah, agreed14:36
popeythanks davmor214:36
t1mpkalikiana, zsombi: what would be the best way to set the z-value of an Item to be the maximum value for z?14:41
t1mp^include float.h and set the value to FLT_MAX?14:41
t1mp(for a qml component written in cpp)14:42
t1mpthey use FLT_MAX in qquickitem.cpp for widths and heights14:42
t1mploicm: ^14:43
kalikianat1mp: there is Number.MAX_VALUE but afaik nowhere in QML is it documented what the maximum value for z is or what behavior to expect - in practise there is no error regardless of how high the value is14:47
kalikianatry it and see what doesn't happen14:48
t1mpkalikiana: my code is in cpp14:49
kalikianat1mp: even then you might want to know what happens if QML in the app does use a higher z value14:50
kalikianabecause whatever you expect might turn out to be false14:50
kalikianafor that matter, what is your goal?14:50
t1mpkalikiana: I meant in CPP I don't have the JS Number14:50
t1mpkalikiana: I want the Header to go on top of other components by default14:51
kalikianat1mp: so in CPP you have a max value, but in QML you don't. seems like you'd want to at least verify what happens in practise14:51
davmor2popey: https://bugs.launchpad.net/canonical-devices-system-image/+bug/1496422 https://bugs.launchpad.net/ubuntu-weather-app/+bug/149642514:51
ubot5Ubuntu bug 1496422 in Ubuntu Weather App "Info for today is not full enough" [Undecided,New]14:51
ubot5Ubuntu bug 1496425 in Ubuntu Weather App "On the main page if you scroll to the last day in the list and tap on it there is no indication there is more info" [Undecided,New]14:51
popeythanks davmor214:52
kalikianat1mp: undefined behavior and things like that... I'd assume the worst if you can't find a guaranteed bheavior14:53
t1mpkalikiana: how come you don't have a max value in QML? There is a maximum value for a real14:53
kalikianat1mp: okay, so there is a maximum for real. yet QML does not throw any errors. so... you're free to make a guess what the end result is. maybe you have 2 items one of which by chance is on top? or maybe the order of declaration matters?14:54
t1mpif they have the same z-value, then the order of declaration matters14:55
kalikianat1mp: and note that the text handlers need to go higher than the header, you can't be obscuring them14:56
t1mpohh14:56
t1mpkalikiana: alternatively, I could keep the z-value at 0 and document that the Header should be declared last or its z-value must be set14:57
kalikianat1mp: how come this hasn't been a problem until now?14:58
t1mpkalikiana: AppHeader is declared last inside the MainView. App developers couldn't add their own Header component14:58
t1mpkalikiana: maybe we can do the same when we add Header to Page, but I'm not sure yet if that would be a Component (with a Loader), or the Item instance15:00
t1mpkalikiana: still, if you have Item { Header { }; Rectangle { } }, would you expect Header to go on top of the Rectangle?15:01
kalikianat1mp: hmm that is an odd example. if you just declare the header like that I'd expect it to do whatever the rectangle does, depending on anchoring15:03
kalikianabut I assume even if it goes to the Page it will be assigned in a place like header: Header15:04
kalikianaand then you'd expect it to behave like a header15:04
t1mpkalikiana: why is it odd? I expect it can be used like this http://bazaar.launchpad.net/~tpeeters/ubuntu-ui-toolkit/cppHeader/view/head:/tests/unit_x11/tst_components/tst_header.qml (ignore the UbuntuTestCase)15:05
t1mpkalikiana: right, we'll probably do something 'special' for putting the Header in a Page, but still we will also have a Header component that can be used anywhere15:06
t1mpkalikiana: it is an option of course that we don't deal with the z-value at all in the Header, only in the Page15:06
t1mpso then the z-value is the same as for any Item15:07
t1mpah.. in my example, the flickable contents goes OVER the header, instead of UNDER it15:08
t1mpI didn't even realize that because I set opacity of the Rectangle to 0.515:08
kalikianat1mp: well, yeah, I said odd because we're discussing special properties of the header and the example to me says completely up to the developer, no magic15:10
kalikianaI would not expect anything about its z if it's just declared like that15:11
t1mpkalikiana: I could do something like header.z: header.flickable.z15:11
t1mpkalikiana: ^but that approach results in different behavior for z when you set a flickable or when you don't set a flickable (locked header)15:11
t1mpkalikiana: so in my example, the flickable contents scrolls on top of the header, unless if I set header.z: 115:11
=== JMulholland_ is now known as JMulholland
kalikianat1mp: I don't see anything in the example defining the size or position of the header15:12
kalikianaI have a feeling you're assuming something that I'm not taking for granted15:12
t1mpkalikiana: right. for vertical position, the default value of y=0 works fine15:15
t1mpkalikiana: but implicitWidth of the Header is parent.width15:15
t1mp(as is the case with the ListItem as well)15:15
kalikianat1mp: lemme re-word. if the Header is simply in an Item or maybe lower in the hierarchy. what I do expect it to do? attach itself to the root item? or the parent?15:16
t1mpkalikiana: of course you can anchor the Header somewhere else15:16
t1mpkalikiana: to the parent15:16
kalikianahmmm15:16
t1mpkalikiana: it is a subclass of QQuickItem15:16
t1mpbut I do set its implicitWidth to parent.width15:17
kalikianaokay15:17
t1mpand implicitHeight units.gu(6)15:17
kalikianaso setting the z would only make sense if there is a flickable - otherwise you have no idea what's in there15:17
t1mpyou can have an Item in which you place a Header, even without a Flickable15:17
t1mphmm.. but if there is no Flickable, probably you will anchor the rest of the contents to the bottom of the Header15:18
t1mpotherwise stuff gets occluded (assuming header.z >0)15:18
kalikianat1mp: yes, so you either never set z or you have a special use case that can't be automatic15:22
t1mpsetZ(FLT_MAX) can be the default15:22
kalikianathe only case where it makes sense to have a default is if flickable is set15:22
kalikianawell15:22
kalikianathat would make the header on top of potential unrelated things, though15:23
kalikiana*put15:23
kalikianasay I want my app to have a kittycat border in which there is a header, enclosed by the border like every nother part of the UI15:23
kalikianaand maybe the border slightly blends over with transluceny15:24
kalikianaand maybe there is a panel coming in from the left side15:24
t1mpright15:24
kalikianaI'm making stuff up, but it seems like there's a number of things you would expect to have a higher z than the header15:24
t1mpkalikiana: the z-order only matters for siblings inside the same Item15:26
t1mpso you can still do Item { id: root; Item { Header{}; Rectangle{} }; KittyCatBorder{} }15:27
t1mpon the other hand,15:27
t1mpif I don't touch z, and apps have contents going on top of the header, the appdevs easily detect and fix that issue15:27
kalikianat1mp: the example still has to assume I'm setting something like anchors.top: header.bottom on the Rectangle, otherwise it's just partially hidden15:30
kalikianain that case it's obvious anyway regardless of z being special15:31
t1mpok, then: Item { id: root; Item { Header{}; Flickable{} }; KittyCatBorder{} }15:32
t1mpthat doesn't need additional anchors to the header (alhthough Flickable should fill its parent)15:32
t1mp-h15:32
kalikianayeah, that one works out of the box15:33
kalikiana(almost)15:33
kalikianaand if you wanted to "lock" the header you might still set the flickable to anchor to the header's bottom15:34
t1mpright15:39
aquariusrschroll, heya. On the "you don't get icons" front, do you get an icon if you try to addtodash https://jakearchibald.github.io/svgomg/ ?15:43
rschrollaquarius: That works.  Let me try to find one that doesn't.16:10
aquariusrschroll, cool. I haven't done any comprehensive tests on lots of sites yet, so I'm expecting there will be bugs, certainly :)16:14
rschrollaquarious: en.wikipedia.org has a favicon set, but this isn't picked up16:24
rschroll(Should it be?)16:24
rschrollAlso an apple-touch-icon16:25
rschrollaquarius: Twitter has a favicon, a mask-icon, and a <meta name="msapplication-TileImage">, none of which are found16:31
aquariusrschroll, cool -- thank you!16:31
rschrollaqarius: Washingtonpost.com just has a favicon at /favicon.ico (old school!), but also have an open graph image, which we may or may not consider using16:36
rschrollThe wonderful thing about standards is, there are so many to choose from.16:37
aquariusfixing bugs16:47
rschrollFor the favicon selector, use rel~=icon.  (In case you hadn't figured that one out yet_16:49
rschrollaquarius: This may be useful in hunting down the various places icons might be hiding: https://css-tricks.com/favicon-quiz/16:59
aquariushttps://mathiasbynens.be/notes/touch-icons too16:59
aquariusrschroll, hrm. In two minds what to do about tile images17:25
aquariusrschroll, the MS tile images provide an image (almost always white on transparent background) and a tile colour (the background). But we need just an image... so we could use just the tile image, or we could create an image by compositing but that's work we'd have to do, or we can ignore it.17:26
aquariusI'll go with just using the tile image for now, but it's not ideal.17:26
rschrollQt Creator has forgotten the run configuration for my app.  (It's offering only "custom executable".)  How do I restore the old value, so it launches the app on my device.19:49
rschrollI've deleted the associated .user file, to no avail19:50
=== faenil_ is now known as faenil
aquariusrschroll, is that for addtodash? I think it gets confused because we have multiple desktop files :(20:03
rschrollaquarius: Yeah.  I disabled the other desktop files, to work on other things, and Qt Creator is still confused.20:32
aquariusbah20:32
rschrollI have a vague memory of this happening before, and needing to delete some obscure Qt Creator file to get it to reset.20:33
rschrollBecause storing this info in the .user file would make way too much sense.20:33
kivipopey, ping?21:01
popeyhi kivi21:01
kivipopey, is the contest still on?21:01
popeykivi: https://developer.ubuntu.com/en/blog/2015/07/23/announcing-the-ubucontest-2015/21:03
popeysept 18th apparently :)21:03
kivity21:03
kiviah nice21:03
mcphailWasn't sturmflut involved in the contest? Is he OK? Haven't seen him around here for ages21:08
=== faenil is now known as faenil_
ogra_mcphail, he is very busy at work i think21:37
mcphailogra_: aah. I'm missing his insights21:45
popeyhe is on holiday i think21:53
popeyoh, maybe not21:54
popeyyeah, just busy21:54
JanCI think he was on holiday, but supposedly back from that, so maybe busy with other things?22:31

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