/srv/irclogs.ubuntu.com/2013/06/03/#ubuntu-app-devel.txt

zsombigood morning everyone!05:45
=== 64MAAD496 is now known as aquarius
=== chriadam is now known as chriadam|away
nerochiarooSoMoN: can you please take this MR, it's been pending for a couple weeks now: https://code.launchpad.net/~amanzi-team/notes-app/notes-app-click-collapse/+merge/164926 or someone else09:28
oSoMoNnerochiaro: sure, if you take https://code.launchpad.net/~osomon/webbrowser-app/focus-fix-suggestions-list/+merge/166862 :)09:31
nerochiarooSoMoN: if i can test on desktop. i just realized i had forgotten my phone at home (at the makerspace now)09:31
oSoMoNnerochiaro: yep, there’s a bug report associated to the MR, and it’s fully reproducible on desktop09:32
nerochiarooSoMoN: i'm on it09:32
oSoMoNthx09:32
oSoMoNI’ll be on yours in a moment09:32
nerochiarooSoMoN: a couple tests failed for me on desktop after merging your MR. haven't tried before but i suspect it's not your MR fault. trying again09:45
oSoMoNnerochiaro: ok, thanks09:45
oSoMoNnerochiaro: note that CI tests passed in jenkins09:45
nerochiarooSoMoN: yeah, that's probably more reliable09:45
nerochiarooSoMoN: i also need to upgrade packages, i might be out of date on webkit or some other library09:46
nerochiarooSoMoN: it looks like the failing tests is the built in web server failing to respond for some reason09:56
oSoMoNnerochiaro: that’s weird09:57
oSoMoNnerochiaro: when you run those failing tests individually, do they fail as well?09:58
nerochiarooSoMoN: trying10:03
oSoMoNnerochiaro: I’m still seeing some random failures from time to time in the browser’s autopilot tests, I need to improve their robustness, it’s a general background task that’s been on my list for quite some time10:04
dpmbzoltan, zsombi, Kaleo_, could one of you provide some guidance on implementing the new visual theme for the clock app to nik90? We just got the visuals from design (https://code.launchpad.net/~jounihelminen/ubuntu-clock-app/designs), and while he's read the Theming API docs, he wasn't too clear on where to start.10:07
nerochiarooSoMoN: the funny thing is that now i ran them again, some random ones gave you exceptions like this http://pastebin.ubuntu.com/5728884/ but the overall result of the test run was "OK"10:08
nerochiarooSoMoN: these exceptions seem  to be totally random10:08
oSoMoNnerochiaro: those exceptions are harmless10:11
nerochiarooSoMoN: ok, then i think i'll just approve your MR, all seems good. what about mine ?10:22
oSoMoNnerochiaro: it’s the connection between the server and the client dropping for some unknown reason (maybe a bug in the server side part in python), but the connection is resumed10:22
oSoMoNnerochiaro: reviewing it now10:22
oSoMoNnerochiaro: this needs to be tested on the device, right?10:26
nerochiarooSoMoN: yes, it's a touch issue10:26
oSoMoNnerochiaro: so why was there this "pressedAtY" property in the first place, was it addressing a specific issue?10:27
nerochiarooSoMoN: it was put there because intercepting the onclicked was conflicting with a flickable IIRC, so i had to capture the press and release events instead. but now the other flickable is gone so i can just use click10:29
oSoMoNnerochiaro: ok, makes sense10:29
oSoMoNnerochiaro: the code looks good, I’ll test on the device10:29
Kaleo_dpm: don't use themeing11:00
Kaleo_dpm: it's not meant to be used by app developers11:00
Kaleo_dpm: let's see what it looks like11:01
dpmKaleo_, thanks. Here are the visual designs: lp~jounihelminen/ubuntu-clock-app/designs even just a few bullet points on how to implement them would help nik90 and the clock team enourmosly11:08
oSoMoNnerochiaro: approved your MR11:09
Kaleo_dpm: checking11:10
nerochiarooSoMoN: if there's an app that has a qml plugin (e.g. for componentizing the browser) do you know if the .mo files (for i18n) have to be installed by the qml plugin or by the app ?11:14
Kaleo_timp: can you be my reviewer for https://code.launchpad.net/~fboucault/ubuntu-ui-toolkit/smaller_xlarge_font/+merge/167020 please?11:17
Kaleo_dpm: ok, it's going to require a bit of work in the toolkit11:18
nik90Kaleo_: will those work be done by the sdk dev or do the core app devs?11:19
dpmKaleo_, is this work already tracked in a blueprint, or could we track it somewhere (e.g. bugs)?11:19
oSoMoNnerochiaro: well if some of the components in the plugin have translatable strings, then the mo files should be installed by the plugin11:19
nerochiarooSoMoN: looks like both the main app and the plugin have them, so i guess we need to build a separate set of .mo and install from both11:20
nerochiarooSoMoN: also i'm unsure about the .qmltheme file. because it should belong to the plugin but it's loaded by the main11:20
Kaleo_dpm: hang on :)11:21
Kaleo_dpm: just started thinking11:21
Kaleo_nik90: sdk dev11:22
Kaleo_nik90: it should be very simple for app devs11:22
=== MacSlow is now known as MacSlow|lunch
dpmKaleo_, how is that thinking going? ;)11:51
oSoMoNnerochiaro: the qmltheme file will go away very soon, so don’t worry about it11:55
nerochiarooSoMoN: ok great11:56
Kaleo_dpm: interrupted by a meeting :)12:14
Kaleo_dpm: still ironing some things with jouni12:15
Kaleo_dpm: I'm pretty sure it will end up being something like: app dev needs to pass 2 colors and the rest is done for him12:15
nik90Kaleo_: that is nice to know12:15
oSoMoNKaleo_: hey, could you please confirm bug #1186968 ?12:16
ubot2Launchpad bug 1186968 in Ubuntu UI Toolkit "Need a way to expose and access common colors for a theme" [Undecided,New] https://launchpad.net/bugs/118696812:16
nik90Kaleo_: I was told by Jouni to talk to you about UbuntuShape.12:16
nik90Since the new UbuntuShape allows to create a circle12:16
nik90However I cannot find this in the docs at developer.ubuntu.com12:16
Kaleo_dpm: added there though: https://blueprints.launchpad.net/ubuntu/+spec/sdk-new-widgets12:17
Kaleo_nik90: it's not ready yet12:18
Kaleo_nik90: what kind of circle do you need though?12:18
Kaleo_nik90: do you have a mockup?12:18
dpmKaleo_, excellent, and thanks for milestoning it12:18
Kaleo_oSoMoN: done12:18
nik90Kaleo_: its for drawing the clock face as shown in the mockups shared by david12:19
nik90Kaleo_: https://www.dropbox.com/sh/uzkr674j7zuglbn/jAEZkfnAAo12:20
Kaleo_nik90: ok, let's show that to loicm see if the new ubuntushape will do that12:20
Kaleo_nik90: but expect another couple of weeks before it's ready anyway12:20
nik90Kaleo_: oh ok.12:20
=== MacSlow|lunch is now known as MacSlow
GuidoPallemansfuck me that looks good12:46
GuidoPallemanswill these also be used as time pickers?12:47
Kaleo_nerochiaro: how do you run the notes app on your laptop?12:47
nerochiaroKaleo_: "notes-app"12:47
Kaleo_nerochiaro: very funny12:50
Kaleo_nerochiaro: you install it system wide??12:50
nerochiaroKaleo_: well, i do both. whe i want to run locally i run it as "qmlscene -I src NotesApp.qml"12:52
Kaleo_nerochiaro: thanks12:52
Kaleo_nerochiaro: that's what I needed12:52
timpKaleo_: ok, I'll review13:23
timpKaleo_: jenkins failed on https://code.launchpad.net/~fboucault/ubuntu-ui-toolkit/smaller_xlarge_font/+merge/16702013:23
Kaleo_timp: thanks, I pushed a fix13:29
timpKaleo_: ok, I'll wait for jenkins to approve13:31
kalikianadpm, ping13:32
dpmhi kalikiana13:32
kalikianadpm, can you check out my last comment https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/navi/+merge/165617/comments/36843413:33
dpmsure13:33
kalikianaif you have a minute I'd like to quickly discuss what to do at this point13:34
kalikianathis is blocking on non-trivial qdoc bugs now…13:34
kalikianaor we can try to "avoid" them by changing our expectations13:35
dpmkalikiana, answered13:46
dpmlet me know what you think. I'm about to enter a few hours of calls now, but we can discuss tomorrow morning or over the merge proposal13:47
oSoMoNzsombi: hey, when you have a moment could you please have a quick look at https://code.launchpad.net/~osomon/ubuntu-ui-toolkit/clear-button-0width-when-not-visible/+merge/167042, it’s a trivial one-liner MR to fix bug #118624713:53
ubot2Launchpad bug 1186247 in Ubuntu UI Toolkit "[TextField] When unfocused, text should not be anchored to the left of the area of the clear button" [Medium,In progress] https://launchpad.net/bugs/118624713:53
zsombioSoMoNÉ checking...13:55
loicmnik90: looking at the mockup, I guess you'd like to use the UbuntuShape so that you can have nice looking inner and outer shadows, so yes you'll be able to use it in order to get nice shadows on top of a circle (ie shape radius of size width/2)13:55
loicmnik90: actually you'll need 2 shapes (looking at the mockups)13:55
loicmnik90: but that's perfectly fine13:56
dpmloicm, is this already possible with trunk?13:57
loicmdpm: nope, I'm currently working on a completely procedural ubuntu shape13:59
nik90loicm: yes I need UbuntuShape mainly for shadow and styling14:00
nik90loicm: when can we expect it to be ready?14:00
loicmnik90: I'm going to propose a demo for the end of the week, then we'll need to agree on the new API and on the integration, so you might be able to test a beta version in a week, but I'm not sure it will land exactly14:02
loicm+when14:02
nik90loicm: I am not sure if you saw the video, but in the video it also shows an animation where the radius increases and then decreases. Will this also be provided by UbuntuShape?14:04
dpmthere: https://www.dropbox.com/s/zczjswik815ylxa/Clock_Motion_01.mp414:05
loicmnik90, dpm: yes it will be supported14:07
nik90loicm: that's pretty cool14:08
nik90loicm: At the end of the day, it seems like the sdk devs are doing all the heavy lifting while app dev can use them in a much easy way for their apps.14:09
loicmnik90: that's our job ;)14:10
nik90loicm: :-) thnx14:11
timpKaleo_: font-size MR approved14:14
Kaleo_timp: cheers14:18
zsombioSoMoNÉ approved14:19
zsombioSoMoN: ^ typo :)14:19
oSoMoNzsombi: I guess you meant "approuvÉ" :) thanks!14:23
zsombioSoMoN: almost :) I had HUN layout active :)14:24
guschnerochiaro oSoMoN renato_ up for a review? https://code.launchpad.net/~schwann/gallery-app/gallery-database-move/+merge/16703614:24
guschrenato_: I updated this MR https://code.launchpad.net/~schwann/gallery-app/gallery-modularity/+merge/16605414:33
oSoMoNgusch: I can put it in my list, but I can’t promise I’ll review it today14:35
renato_gusch, approved14:35
guschoSoMoN: nerochiaro renato_ would be good to have this done rather early, as my next work will depend on it14:36
guschrenato_: thx14:36
oSoMoNman, that was fast14:37
oSoMoNah, ok, not that one14:37
renato_oSoMoN, I approved the older one :D, not the new one14:37
nerochiaroKaleo_: can we have a quick meeting today to talk about where to put these components for browser and media player ?14:42
nerochiaroKaleo_: should've been a couple weeks ago but stuff got in the way ;)14:43
Kaleo_nerochiaro: how about tomorrow?14:52
nerochiaroKaleo_: bill wants it done this week, and given my schedule the sooner i know the better14:52
oSoMoNtimp, zsombi: how can I display a Tabs component with the usual header without having it inside a MainView?14:53
timpoSoMoN: the header is part of the mainview, so you cannot14:54
zsombioSoMoN: No way if I'm not mistaken - as timp said14:54
oSoMoNbleh :/14:54
oSoMoNthis is too restrictive14:54
Kaleo_nerochiaro: good, tomorrow14:54
zsombioSoMoN: you don't like MainView?14:54
Kaleo_nerochiaro: unless you are willing to do it at midnight14:55
oSoMoNzsombi: afaik, MainView doesn’t allow me to have a custom toolbar, like I do for the browser14:55
nerochiaroKaleo_: probably not, i don't have internet at my place yet or it would be ok14:55
zsombioSoMoN: how custom you need there?14:56
timpzsombi: address bar, which we don't support yet14:56
Kaleo_nerochiaro: first thing tomorrow, send an invite for my 9am :)14:56
timpwe don't support that in the standard toolbar I mean14:56
zsombitimp: so that cannot be done with th eaction hint?14:57
nerochiaroKaleo_: ok busy man14:57
oSoMoNzsombi, timp: is the header delegate available for me to reuse, somehow?14:57
timpzsombi: itemHint could work, but currently the positioning is fixed14:58
timpoSoMoN: actually, the Header is not marked as internal in the qmldir file.. so you could use it14:58
timpnot sure if that is a mistake14:58
nerochiaroKaleo_: you should have an invite now, please confirm14:59
oSoMoNtimp: cool, I’ll use it and then as it’s used by a core app you’ll have no choice but to leave it public. <insert evil laugh>15:00
Kaleo_nerochiaro: cheers15:02
om26ertmoenicke, Hey! which package to report bug for the OSK ?15:04
timpoSoMoN: uhm.. yeah. that would work for you :)15:06
zsombitimp: are you sure it's a good idea? I mean do we want to make it public?15:07
timpzsombi: no I'm not sure.15:08
timpoSoMoN: probably with that approach you will have to replicate some of the MainView functionality.15:08
timpzsombi: do you have a different solution that will work for the browser right now?15:09
zsombitimp: no, but I have the feeling that we do hacks just for someone's eyes instead of fixing the problem...15:09
oSoMoNzsombi, timp: I’m fine with not using it, provided you guys can provide me an alternative solution15:09
tmoenickeom26er: touch-preview-images15:10
zsombitimp: cannot the toolbar be made so it takes into account the itemHint's width/height?15:10
timpzsombi: I think it does. It just uses the whole item that's put in the itemhint15:11
zsombitimp: I don't get it then: browser has 3 components, 2 buttons and a TextInput...15:12
timpzsombi: it may be possible to use the itemHint for the textinput, but I haven't tried how to do it.15:12
timpzsombi: we could add a contents property to Toolbar, just like we did with Header, to allow custom contents instead of the one-button-per-action15:12
oSoMoNtimp, zsombi: there’s some additional complexity: depending on the state, some of the buttons may be shown/hidden15:13
timpoSoMoN: the automatic buttons are created from the actions of the Page, and each Action has a visible property that you can use for that.15:13
oSoMoNtimp, zsombi: also, I’d need the header to never be visible when browsing15:13
oSoMoNtimp: ok, good to know15:14
timpoSoMoN: there is a bug right now https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1177123 that messes up the automatic placing of the buttons when some are invisible, but that will be fixed15:15
ubot2Ubuntu bug 1177123 in Ubuntu UI Toolkit "Toolbar actions buttons are not placed dynamically" [High,New]15:15
timpzsombi: there is a proposal to not have itemHint in the new Action APIs (since itemHint cannot be passed over dbus)15:24
timpzsombi: in case you are interested, here is the doc https://docs.google.com/a/canonical.com/document/d/1GVw0jQfvdhG-xkL6HAb7EJdpQlb32Q2-JlBLy2slsF4/edit?disco=AAAAAF1brgI and we have a meeting in 35m15:24
oSoMoNtimp, zsombi: so I managed to use the Header component directly with Tabs: http://pastebin.ubuntu.com/5729625/, but how do I enable tab switching through the header?15:25
timpoSoMoN: assign Tabs.__headerContents to Header.contents15:27
timpsome stuff that relies on MainView (such as propagation of toolbar property) will break.15:27
timpwell you don't use the toolbar, so maybe you are lucky15:27
zsombitimp: sorry, was on hangout15:29
zsombitimp: but do we have to pass every property of Action through dbus?15:30
oSoMoNtimp: works nicely, thanks!15:30
timpzsombi: the suggestion was to allow to add the component directly in the list of actions instead15:31
timpzsombi: so tools: ToolbarActions { Action { }; Button { action: Action { } }; }15:32
timpfor example15:32
zsombiuhh...15:32
timpI gotta go now, back in 30m for Action meeting.15:33
timpzsombi: I can discuss it with you when I'm back if you are still around, otherwise tomorrow?15:33
zsombitimp: maybe tomorrow...15:33
timpok15:33
timpzsombi: I am not convinced of that change yet, so I may not be the best person to convince you ;)15:34
* timp bbl.15:34
mhall119FYI, anybody interested in beta-testing Discourse, we now have an application-development category for topics: http://test.ubuntu-discourse.org/category/application-development15:57
mhall119dpm: ^^15:57
dpmnice mhall119 :)15:57
GuidoPallemansbug: http://test.ubuntu-discourse.org/t/getting-started-with-app-development/174 contains a typing error16:05
GuidoPallemansoh, that's not actually part of discourse16:14
demonoid_mehi to all guys!17:05
demonoid_mei have samsung galaxy ace 217:05
demonoid_meit is possible to install ubuntu on this device?17:05
demonoid_mebecouse i develope apps for android phones17:08
nerochiarotimp: Kaleo_ : zsombi: do you guys know how to do i18n for libraries ?17:09
nerochiarooSoMoN: ^17:09
nerochiarotimp: Kaleo_ : zsombi: oSoMoN: i have found examples for apps, but not for libs yet17:09
Kaleo_nerochiaro: what's youro use case?17:12
nerochiaroKaleo_: browser component17:12
Kaleo_nerochiaro: and I guess you mean doing it in instead of  QML17:12
Kaleo_nerochiaro: strike that last sentence17:12
nerochiaroKaleo_: ok17:12
Kaleo_nerochiaro: ask timp torroow17:12
Kaleo_omorrow17:12
Kaleo_:)17:12
nerochiaroKaleo_: ok17:12
timpnerochiaro: or now :)17:14
Kaleo_timp: yay17:14
nerochiarotimp: how do i do i18n for a qml library ?17:14
timpI don't fufully understand the question17:15
timp*fully17:15
timpsucky internet..17:15
Kaleo_nerochiaro: (you mean how to generate the translation file and how to load it?)17:15
timpnerochiaro: for cpp it is standard gettext17:16
Kaleo_timp: and don't do cpp if you can avoid it17:18
Kaleo_(which you can :))17:18
timpif you can avoid the cpp, then I don't understand the question17:18
Kaleo_timp: :)17:19
timpnerochiaro: I had a doc once https://docs.google.com/a/canonical.com/document/d/1rLQLvWLuUJxV8LMv5s4vX3cwnWzpnM_fnkdeFSVU4JE/edit#heading=h.tiwophvmzjhk but it is outdated17:20
timpdpm: ^ do you have newer documentation ready?17:20
* dpm looks at doc17:21
timpit is old. and there are better tools than my script :)17:21
timpstill it is not fully clear what nerochiaro wants17:22
dpmtimp, nerochiaro, I created this for how the workflow should be, but Qt Creator still works a bit differently: https://docs.google.com/a/canonical.com/document/d/12fEDKGVlNAkeXQyNXUM8JcekYwfvN7brRG5m5a0a4Dk/edit#heading=h.vbkzlk3ri2wg17:23
dpmI need to run, will be back later17:23
timpdpm: great, thanks17:23
nerochiarotimp: i have a QML library (plugin) that has some strings in it. These strings need to be i18n'd17:35
nerochiarotimp: so that they can be translated. more clear ?17:35
nerochiarotimp: dpm: all documents you sent to me cover only applications, not libraries (plugins)17:36
=== greyback is now known as greyback|away
dpmnerochiaro, ah, gotcha, so this is about internationalizing a C++ plugin, right? In that case, you'll need to use gettext, as the i18n.tr() functions cannot be used from C++. See bug 1182577 and the example app link in there17:40
ubot2Launchpad bug 1182577 in Ubuntu UI Toolkit "[i18n] Cannot use the i18n plugin in C++" [Wishlist,Confirmed] https://launchpad.net/bugs/118257717:40
nerochiarodpm: it's a QML plugin17:42
nerochiarodpm: no c++17:42
dpmnerochiaro, in that case, that makes things easier:17:42
dpm- use the i18n.tr() function as you would do with an app17:43
dpm- set the i18n.domain explicitly17:43
nerochiarodpm: and where do i install the gmo files ?17:43
dpm- Generally i18n.domain is the name of the app (or plugin, in this case)17:43
nerochiarodpm: sorry, .mo files17:44
dpmnerochiaro, the same as with an app: /usr/share/locale/$LANG/LC_MESSAGES/$DOMAIN.mo17:44
nerochiarodpm: ah, sounds simple enough .i'll give it a shot and let you know (prob tomorrow). thanks17:45
dpmnerochiaro, the only thing to bear in mind is to ensure that the library loads those files. That works for apps and should work for plugins, but the later has not yet been tested afaik17:46
dpmso you've got a nice opportunity for pioneering ;)17:46
nerochiarodpm: any hints on how can i test it ?17:46
timpnerochiaro: as dpm says, the process is the same as with an app. You also set the domain, and in the app when you want to use the dictionary of the library, use i18n.dtr() to pass the domain17:46
timpapp = app that uses your qml lib17:47
nerochiarotimp: i don't get it, why do i need to call i18n.dtr() and where do i need to call it ?17:47
timphmm. maybe you don't need in most cases.17:48
timpbut in your app, if you get a string from your lib, you need to use dtr.17:48
dpmtimp, I think the i18n.dtr calls in the app using the plugin should not be needed17:48
timpdpm: yeah I just start wondering now, if the lib already returns the translated strings you don't need17:49
dpmtimp, ah, yes, in those cases perhaps17:49
timpI was thinking userString = dtr(lib.getString(), libDomainName)17:49
timpfor error messages17:49
timpbut if lib returns the translated string you don't need it. so when do you ever need dtr()?17:50
dpmtimp, when an app has several domains, for example, which is not usual17:50
timpnerochiaro: ok so forget everything I said about dtr() ;)17:52
nerochiarotimp: ok, done. forgotten. gotta go anyway18:25
nerochiarotimp: dpm: thanks for the help18:25
odraencodeHello :D18:52
=== dpm is now known as dpm-afk
=== chriadam|away is now known as chriadam

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