**** BEGIN LOGGING AT Wed Aug 19 02:59:58 2015 Aug 19 06:09:16 morning Aug 19 06:09:19 nizovn: ping Aug 19 06:09:28 Herrie: pong Aug 19 06:09:32 morning Aug 19 06:10:24 I found a problem with the userscript you added and want to discuss what would be the best way to deal with this Aug 19 06:10:33 Page https://www.abnamro.nl/portalserver/nl/prive/index.html?l doesn't load anymore when it's enabled Aug 19 06:11:02 https://github.com/webOS-ports/org.webosports.app.browser/blob/master/qml/WebView.qml#L115 Aug 19 06:12:58 I now have as a temporary workaround locally that I only load the js/userscript.js when it's not a https site and moved this bit to the onNavigationRequested but that's nasty imho Aug 19 06:13:36 can confirm this issue Aug 19 06:15:09 I tried to add console.log to each of the functions in userscript.js but no luck Aug 19 06:19:54 we can comment out content of userscript.js, leaving only needed touch related functions Aug 19 06:22:35 Yeah but I would expect that when I put console.log everywhere it would at least show somewhere in the logs in QTCreator and it doesn't Aug 19 06:22:46 So it seems the whole userscript.js might be a problem? Aug 19 06:27:34 i tried on stable-staging emulator image, ang that site works Aug 19 06:28:30 i mean without any modification to userscript.js Aug 19 06:32:36 nizovn: Top and bottom are OK but middle with login parts are missing here Aug 19 06:32:41 so it's maybe not caused by userscript, probably i mistyped url in first attempt Aug 19 06:32:45 Also on same emu Aug 19 06:35:57 ok, can reproduce now Aug 19 06:37:08 It's my bank's website which I use daily so for me personally quite critical to work LOL ;) Aug 19 06:37:43 It stopped working properly on legacy a while back but that was for doing transfers, the login worked fine there ;) Aug 19 06:38:50 :) Aug 19 06:39:00 it works if comment out window.onload Aug 19 06:40:36 Which we don't use right? Aug 19 06:40:42 yes Aug 19 06:42:11 Not sure what it's used for anyway. Some input hints or something? Aug 19 06:45:47 not sure too Aug 19 06:49:05 Will you do a PR or should I do it? Aug 19 06:49:24 please, do it Aug 19 06:49:42 possibly site uses window.onload, and we overwrite it Aug 19 06:50:13 This code comes from Ubuntu right? Aug 19 06:50:20 We need to add copyrights as well ;) Aug 19 06:52:01 i added this: https://github.com/webOS-ports/org.webosports.app.browser/blob/master/COPYING#L27 Aug 19 06:54:40 Who might have borrowe from https://bazaar.launchpad.net/~phablet-team/webbrowser-app/trunk/view/head:/src/Ubuntu/Web/selection02.js Aug 19 06:54:47 Not sure who invented what ;) Aug 19 06:55:05 :) Aug 19 07:36:49 Morning! Aug 19 07:46:09 Tofe: morning Aug 19 08:56:51 morning Aug 19 13:47:30 Tofe: Did a small PR for browser, nothing shocking ;) Aug 19 13:47:50 Just fixing my banking site + fixing http://issues.webos-ports.org/issues/961 Aug 19 13:48:07 Not sure this will be the final solution, but seems to do it for now Aug 19 13:49:30 Herrie: yep, just had a small comment Aug 19 13:50:37 Yeah will fix that in another PR Aug 19 13:50:47 This file needs some reformatting & Cleanup in general Aug 19 13:50:55 I removed most from this PR but seems I missed this one ;) Aug 19 13:51:09 I'll do another PR to do the formatting/cleanup later Aug 19 14:01:57 Tofe: I was looking into: http://issues.webos-ports.org/issues/901 Aug 19 14:02:13 Shouldn't we have a \ at the end of https://github.com/webOS-ports/webos-keyboard/blob/master/plugins/en/src/src.pro#l12 ? Aug 19 14:02:32 Not sure that would fix the missing dict error but wouldn't hurt to fix incorrect syntax? Aug 19 14:31:10 Herrie: that's a good idea! I didn't see this Aug 19 14:42:57 Tofe: You agree that it's missing the \ ? Aug 19 14:45:54 I agree Aug 19 14:46:27 Herrie: oh, it removed all the ";"... Well, as you wish :) Aug 19 14:47:15 I need to do a webos-keyboard PR anyway to fix http://issues.webos-ports.org/issues/959 Aug 19 14:47:26 Tofe: QtCreator doesn't seem to need/want them :P Aug 19 14:47:32 At least not by default Aug 19 14:47:35 I prefer them too actually Aug 19 14:48:04 So I merge the PR? Aug 19 14:48:38 You can ;) Aug 19 14:48:49 Unless we can tell QtCreator code reformatter to use ; Aug 19 14:48:52 I didn't find it yet Aug 19 14:49:01 Usually I use Ctrl+i Aug 19 14:49:21 It's a light reformatting, does mainly fix the alignements Aug 19 14:49:39 Ah ok Aug 19 14:56:49 Did a PR for the missing \ in 16 langs Aug 19 14:56:53 Will work on 959 now Aug 19 14:57:00 Should be easy to fix Aug 19 16:15:36 Tofe: Updated PR with fix for 959 Aug 19 16:15:43 It bit more tricky then I thought but well Aug 19 16:15:56 Fixed a few other small things too Aug 19 16:49:15 Tofe: ping Aug 19 17:39:39 Herrie|2: pong Aug 19 17:40:37 (valid until 19:45) Aug 19 17:40:48 Tofe: I tried debugging luna-systemui Enyo 1 app to see if I can fix the battery dashboards Aug 19 17:41:01 ok Aug 19 17:41:02 However it seems that the console.logs I add don't appear in journalctl Aug 19 17:41:04 Any ideas? Aug 19 17:41:05 but? Aug 19 17:41:20 ah, hmm Aug 19 17:41:27 It seems to work when connected to PC but not to charger, so wanted to add some logging to debug Aug 19 17:43:53 could be webview or webapp-launcher Aug 19 17:44:20 I'd have to look at it myself, hard to say Aug 19 17:44:23 I remember we could launch it so it would output some more debugging Aug 19 17:44:51 I already tweaked framework_config.json and put the required values in there as well but to no avail Aug 19 17:52:46 I'll have a look tonight, but I have to leave now :) cu Aug 19 17:53:38 Sure no pb ;) Aug 19 17:53:41 No hurry Aug 19 17:53:59 I'll do a new nightly build with the VKB + Browser to see if that fixes the maliit/hunspell stuff ;) **** ENDING LOGGING AT Thu Aug 20 02:59:59 2015