=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
dholbachgood morning07:03
=== chriadam is now known as chriadam|away
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== davidcalle_ is now known as davidcalle
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
nerochiarooSoMoN: https://bugs.launchpad.net/webbrowser-app/+bug/1459564 can you please confirm this ?08:33
ubot5Launchpad bug 1459564 in webbrowser-app "Only one "new tab" page should be open at any time" [Undecided,New]08:33
oSoMoNnerochiaro, confirmed08:33
nerochiarooSoMoN: also just wanted to confirm that you are aware of the 2 MRs that i have pending, find in page which should need only to verify your comments were fixed and keyboard navigation which needs full review08:34
oSoMoNnerochiaro, they’re on my TO-DO list08:34
nerochiarooSoMoN: thank you08:34
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== MacSlow is now known as MacSlow|lunch
=== chihchun is now known as chihchun_afk
kalikianazsombi: t1mp FYI https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/145963211:37
ubot5Launchpad bug 1459632 in ubuntu-ui-toolkit (Ubuntu) "Failures on staging in ubuntuuitoolkit.tests.components.test_textinput" [Critical,In progress]11:37
t1mpkalikiana: thanks for looking at it11:38
zsombikalikiana: kthx11:40
t1mpzsombi: how should I access the theme inside a style?11:41
t1mpzsombi: styledItem.theme11:41
t1mpor just theme?11:41
zsombit1mp: yes11:41
zsombit1mp: styledItem.theme11:41
zsombit1mp: why would you need that?11:41
t1mpI was using 'theme' only and that seems to work11:41
t1mpzsombi:     property color sectionColor: styledItem.theme.palette.selected.backgroundText11:42
zsombit1mp: aaah... well, theme.palette.selected.backgroundText is also enough11:42
zsombit1mp: as each style component is created inside the StyledItem's context11:43
zsombit1mp: fyi, StyleHints started to work :)11:44
t1mpzsombi: cool :)11:44
t1mpzsombi: I was just wondering what's the cleanest way to refer to theme11:44
t1mpstyledItem.theme may be more clear, but we already use theme in many places (without styledItem prefix)11:44
zsombit1mp: theme is enough, no need to use the styledItem11:44
t1mpunit tests are done.11:45
t1mpa new component takes two days to write, then 1 day unit tests and 1 day autopilot tests...11:45
zsombit1mp: I'm adding unit tests as well :) and docs ;P11:46
t1mpyes I hope you do :)11:46
DS-McGuiredholbach, Sorry I missed you yesterday12:03
DS-McGuireI like what that guy had to say12:03
DS-McGuireI had thought about the search bar going into the top header however I didn't see it in the Ubuntu HTML5 api like it is with the QML so I guess I could make it, can't be that hard. Also the back button being missing is a known problem12:04
DS-McGuirewell not a problem, I didn't add it yet12:05
dholbachyou can always ask the guys in #ubuntu-webapps like dbarth or alex-abreu12:08
=== mzanetti is now known as mzanetti|run
=== MacSlow|lunch is now known as MacSlow
=== _salem is now known as salem_
=== greyback__ is now known as greyback
kalikianamzanetti|run: I can't reproduce your problem https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1413801 mind you the snippet was not a complete app, so if you did anything special I wouldn't see it12:57
ubot5Launchpad bug 1413801 in Ubuntu UX "TextField's placeHolderText invisible inside a dialog" [High,Triaged]12:57
=== mzanetti|run is now known as mzanetti
mzanettikalikiana, odd... I thought I had pasted a complete standalone example. I'll prepare it for you. sorry for this13:13
=== zulucloud is now known as mixu
DS-McGuiredholbach, Sorry I didn't see the notification. I will ask them now, are they the guys who help with the HTML5 API?13:28
DS-McGuireAwesome, thanks.13:29
kalikianamzanetti: actually I found it. and now this is really odd. as soon as I set a backgroundColor on the MainView I see the issue - if I don't all is fine13:42
kalikianawhat color doesn't matter even13:43
mzanettikalikiana, right, yes, it's about a non-standard background color13:45
mzanettistill odd that my example only contains half of it :(13:45
kalikianamzanetti: I realized afterwards you mentioned the dark background - which I mistook for the black overlay from the dialog13:47
kalikianabut thart has nothing to do with this bug13:48
kalikianamzanetti: I assume your dialog also has modal: false originally13:49
mzanettikalikiana, no, I don't think I ever used "modal: false"13:49
* mzanetti wasn't aware of that13:49
kalikianamzanetti: if you don't you would have a black overlay hiding the mainview13:50
kalikianaand that is true even for rtm13:50
mzanettiyes, what's the problem with that?13:50
kalikianamzanetti: the problem is that I mistook the "dark background" for that overlay :-)13:50
mzanettinow I get it13:51
mzanettiyeah... again, sorry for being unclear :(13:51
mzanettiwill do better next time13:51
kalikianashit happens, at least I got it in the end - now the real fun begins in tracking the bug down :-D13:52
mzanettiheh :)13:52
mzanettigood luck :)13:52
seb128does anyone has an example of app that got migrated from Ubuntu.Components.ListItems to the new ListItem?14:10
sverzegnassiseb128, Ubuntu Touch Tweak Tool uses that. https://launchpad.net/ubuntu-touch-tweak-tool14:11
seb128sverzegnassi, thanks14:13
seb128sverzegnassi, doesn't seem to14:14
seb128sverzegnassi, http://bazaar.launchpad.net/~ubuntu-touch-tweak-tool-devs/ubuntu-touch-tweak-tool/trunk/view/head:/app/components/ListItems/SectionDivider.qml14:14
seb128    ListItem.ThinDivider {14:14
seb128that's the old one14:14
seb128"import Ubuntu.Components.ListItems 1.0 as ListItem"14:15
sverzegnassiseb128, that one comes from dekko. it's just the divider. the other list item in that folder derives from the UITK1.2 component14:19
seb128sverzegnassi, right, I'm trying to figure out how to do a divider with the new component14:20
sverzegnassiseb128, here there's also an example of the drag&drop feature :)14:21
sverzegnassiI'd go for a Rectangle14:21
seb128that creates a white line14:23
seb128or I need to set a color but it doesn't look the same14:24
nerochiarooSoMoN: quick question, in the webbrowser AuthenticationDialog.qml, where is the "model" property supposed to come from ? I can't seem to find it declared or documented anywhere14:24
DanChapmanseb128: doesnt the new ListItem have a divider attached property. Something like divider: {14:25
DanChapmancolor fom: colorTo: }14:25
seb128DanChapman, it does but I don't have a listitem in my column14:26
seb128I've a grid with icons14:27
seb128then I need a line14:27
seb128I guess I can use a rectangle tweaked then14:27
seb128with a height of 114:27
oSoMoNnerochiaro, model is the contextModel provided by oxide, see e.g. http://bazaar.launchpad.net/~oxide-developers/oxide/oxide.trunk/view/head:/qt/quick/oxide_qquick_javascript_dialog.cc#L5814:28
oSoMoNnerochiaro, the AuthenticationDialog component was being instantiated by QtWebKit, so it was setting a model property on the item to allow interacting with it. We have a similar mechanism in oxide for various dialogs and for the context menu, so you might want to implement something similar for the authentication14:30
oSoMoN(if having oxide instantiate a component for the authentication makes sense, maybe it doesn’t and a simple signal is good enough of an API)14:30
nerochiarooSoMoN: more specifically from what i see on that code you linked it was setting a context property, not a property on the item. but yes, as you say in my case I have a signal to request authentication and two methods,one to send credentials and one to cancel14:31
nerochiarooSoMoN: so I will alter that dialog to call the methods and show it via PopupUtils.open14:32
nerochiarooSoMoN: unless you don't agree with the API i just mentioned14:32
oSoMoNnerochiaro, can you show me the code for the oxide API?14:33
nerochiarosignal void basicAuthenticationRequested(); slot void cancelBasicAuthenticationRequest(); slot void sendBasicAuthenticationCredentials(const QString& username, const QString& password);14:35
nerochiarooSoMoN: ^14:35
nerochiarooSoMoN: but if you prefer I am happy having a property QQmlComponent* alertDialog14:36
nerochiarooSoMoN: er, authenticationDialog14:36
oSoMoNnerochiaro, can’t the signal carry a BasicAuthenticationRequest object as a parameter that would be used to cancel/acknowledge the request? that would avoid cluttering the WebView API with two extra slots14:36
nerochiarooSoMoN: i can do that, i need to add a parameter that has the realm and the host requesting authentication anyway, so i might as well add these too14:37
oSoMoNnerochiaro, see how Chris did it for the WebView.mediaAccessPermissionRequested API14:37
nerochiarooSoMoN: ah, good tip. will look. thanks14:38
oSoMoNI think we want something similar14:38
balloonsdholbach, so anything else for https://code.launchpad.net/~nskaggs/help-app/functional-test-template/+merge/260206?15:00
dholbachballoons, I'm working with fgimenez to get selenium in15:01
dholbachI really wouldn't like to ask people to add yet another PPA15:01
dholbachapart from that it looks great to me15:01
balloonsdholbach, ok, so you'll hold on selenium in the archive?15:01
balloonswould you consider holding on making it a part of the build, but landing the template?15:02
dholbachI'm not sure I understand15:02
balloonsie, atm these tests won't run as part of the build. Jenkins doesn't need selenium, nor does anyone wanting to work on the app (there's no functional tests)15:03
dholbachok, now I see15:03
balloonsonce selenium lands, we can add the build-dep, add it to make, etc15:03
dholbachI just brought it up because it was mentioned in the HACKING doc15:03
balloonsright, I would leave that for now, and remove it as part of incorporating things15:03
=== chihchun_afk is now known as chihchun
dholbachok, wfm15:12
dholbachballoons, I'll merge https://code.launchpad.net/~nskaggs/help-app/functional-test-template/+merge/260206 and add a small comment to the HACKING doc afterwards15:27
dholbachdpm, sorry... I found an issue15:35
dholbacherr, sorry.15:35
dholbachI meant balloons15:35
dholbachadded it to the bottom of https://code.launchpad.net/~nskaggs/help-app/functional-test-template/+merge/260206?15:35
mzanettipopey, meeting time15:36
balloonsohh I see, first you bait me, then :-)15:36
popeymzanetti: ooh15:36
balloonsdholbach, on the web version, that's a good point.15:37
balloonsdholbach, so I guess we would need to choose which one to launch. In theory either should work, and we can launch either one15:48
dholbachlet me take a closer look15:49
balloonswe launch via ubuntu-html5-app-launcher15:49
balloonsit does expect and index.html; i'm not sure if it's possible to override that or not. The web version is all localized, so there's no index.html15:50
dholbachmaybe we could add a target for it in the Makefile15:50
dholbachlike "make functional-web-test" or "make functional-app-test"15:50
dholbachand make each depend on the related other make target15:51
balloonsthe path is hardcoded, but not reason it couldn't be a variable. That said, I guess it would really be two test suites eh?15:52
dholbachit could figure out which path exists and take it from there15:52
balloonsand if both? you think the testsuites would be the same?15:55
balloonswe could also simply make launchers for both and run all tests as a scenario with each launcher feeding into it. In effect, we can run every test that is the same on both as needed within one testsuite15:56
balloonsif there's a specific test that needs only one or the other we can make that as well. It would require both app and web to be built in order to run the full suite ofc15:57
balloonsperhaps I can just modify the mp so it's a little clearer what I mean. I would make 2 classes; one for each testsuite15:58
dholbachif both, it could run both, yes :)15:58
=== chihchun is now known as chihchun_afk
=== oSoMoN_ is now known as oSoMoN
=== boiko_ is now known as boiko
mhall119kenvandine: I haven't gotten an actual title for your presentation yet, do you have one?18:46
mhall119kenvandine: unless "Application Security in Ubuntu" is the title18:48
kenvandinethat was it18:54
kenvandineunless you have ideas to make it snazzy :)18:54
kenvandine"Application Security in Ubuntu, why you should care"18:55
kenvandinemhall119, maybe :)18:55
kenvandinethe key is i want it to be focused on why users should care about our application confinement18:55
mhall119kenvandine: that sounds good18:58
balloonsmhall119, so is the json for the python scope harness a-ok for you>20:40
=== salem_ is now known as _salem
mhall119balloons: haven't run it against the importer yet, sorry, have been trying to finish ubucon stuff21:34
balloonsmhall119, ack, let me know21:46

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