**** BEGIN LOGGING AT Sat Sep 24 02:59:58 2016 **** BEGIN LOGGING AT Sat Sep 24 03:36:57 2016 **** BEGIN LOGGING AT Sat Sep 24 03:47:33 2016 Sep 24 03:54:08 * DougReeder waves to Tofe and _morphis **** BEGIN LOGGING AT Sat Sep 24 04:18:25 2016 Sep 24 04:38:11 morning Sep 24 04:38:25 * DougReeder waves Sep 24 04:45:05 DougReeder: I'm just flashing latest nightly now :P Sep 24 04:45:18 * DougReeder nods Sep 24 04:45:41 To test Just Type & https://github.com/webOS-ports/luna-appmanager/commit/f11d34ea8ccb3c33c0159c43284367e83f947561 Sep 24 04:45:53 Ah! Sep 24 04:46:43 I’m not sure I have a good way to test the second one, any more. Sep 24 04:47:20 I updated Zap to work around that, I think Sep 24 04:47:56 DougReeder: Well I can just do luna-send :P Sep 24 04:49:24 * DougReeder nods Sep 24 04:50:20 Just type seems to behave :) Sep 24 04:50:31 My ADB is acting up (as usual) Sep 24 05:00:51 Do you need me to download and test? Sep 24 05:04:21 I tested it before :P Sep 24 05:04:26 I know it works Sep 24 08:03:36 Morning Sep 24 08:04:48 Tofe: morning Sep 24 08:21:16 Herrie: nothing against PR merging for the cryptofs workaround? Sep 24 08:23:11 Tofe: If C++ syntax is correct like this not ;) Sep 24 09:24:13 Tofe: I see Sep 20 08:20:27 mako luna-next[1242]: WARNING: 08:20:27.849: QObject::connect: No such slot NotificationManager::removeUserRemovableNotifications() in the log Sep 24 09:24:59 Can we simply remove https://github.com/webOS-ports/luna-next/blob/cd038a7555b0886485177e79e8e67ab3333f29a1/plugins/shell/notifications/notificationlistmodel.cpp#L26 to "fix" this? Sep 24 09:25:25 Or... ? Sep 24 09:28:58 We should also check that the clearRequested signal is indeed never used Sep 24 09:35:14 Tofe: These are the only references it seems: https://github.com/search?q=org%3AwebOS-ports+clearRequested&type=Code Sep 24 09:35:29 I'm scanning the image to see if we have anything else (in a forked repo for example) Sep 24 09:38:36 looks good; I don't think we use it, but it's better to check before than after :) Sep 24 09:50:17 Seems not.. So we drop both the signal in the header & the bits in cpp ? Sep 24 10:14:56 Tofe, Herrie: i'm going to work mostly on fixing small UI bugs, to make LuneOS feel solid. also i will try to mimic legacy touchpad gui as far as possible Sep 24 10:20:00 Tofe: btw do you know simple way to get current keyboard height in LNC? it would be better to avoid showing tap ripple on top of it. Sep 24 10:28:10 Herrie: yes Sep 24 10:29:40 nizovn: Qt.inputMethod.keyboardRectangle Sep 24 10:30:16 I think you should test Qt.inputMethod.visible too Sep 24 10:30:53 ok, thanks, will try Sep 24 10:32:08 nizovn: also, even though touchpad gui is quite a good one, we have to keep in mind that we also have target phones; this means we sometimes don't mimic touchpad gui Sep 24 10:32:39 yes, i will do that in separate branch, maybe even non-offician Sep 24 10:33:11 the best way would to create several gui types like phone and tablet Sep 24 10:33:48 e.g. notificationarea is very different on those Sep 24 10:34:06 we try to keep a unified look when possible, it simplifies the code; but sometimes it diverges a bit yes Sep 24 10:35:41 We have the Settings.tabletUi boolean to know if we are on a tablet target or not Sep 24 10:36:01 ah, good Sep 24 10:36:04 So far it's not very used in our code, maybe just once or twice Sep 24 10:36:28 (for the keyboard I think we use it) Sep 24 10:39:39 nizovn: whenever there would be a need to switch between two UIs, we should use a Loader; this will isolate these parts, both in the code and in the runtime Sep 24 10:40:29 Currently I see mainly a need for the alerts and notifications; I don't know if you saw other places Sep 24 10:42:51 ok, agree Sep 24 10:45:32 can't imagine other places for now Sep 24 11:44:23 Tofe: Qt.inputMethod.visible is always false and Qt.inputMethod.keyboardRectangle is QRectF(0, 0, 0, 0) for me Sep 24 13:12:03 nizovn: I did a little for the UI, but it was only a few lines of code, so go ahead. Biggest things imho are: Sep 24 13:15:11 nizovn: Color of status bar + dividers + notification on top for tabletui Sep 24 13:15:38 Just type is too wide (should be 57%), should be in italic as well Sep 24 13:16:46 ok Sep 24 13:17:02 I think launcher background on bottom might need to be a bit darker Sep 24 13:17:19 i downloaded new vbox image, found some bugs in firstuse, so maybe i'll start with them Sep 24 13:17:37 nizovn: Like what? Sep 24 13:18:05 e.g. time settings are incorrect Sep 24 13:19:54 nizovn: I think that's an issue on C++ side Sep 24 13:19:58 I couldn't get that fixed Sep 24 13:20:21 ok Sep 24 13:24:08 I think it's luna-sysservice or lunasysmgr check log Sep 24 13:28:55 nizovn: well, we use that for luna-webappamanger, so somehow it is working :) Sep 24 13:29:22 https://github.com/webOS-ports/luna-webappmanager/blob/master/src/qml/ApplicationContainer.qml#L109 Sep 24 13:30:27 Tofe: maybe it's available only in app, not in compositor (as it can't have open keyboard)? Sep 24 17:13:11 nizovn: FirstUse works OK on desktop Sep 24 17:13:25 I think it's some timing or restart in luna-sysservice or luna-sysmgr Sep 24 17:13:31 I recall seeing something like that in the logs Sep 24 17:18:11 ka6sox: ping Sep 24 17:20:44 Herrie, pong? Sep 24 17:20:57 ka6sox: How's the work on Kaylee going? Sep 24 17:21:29 is done for now...novaldex will be in touch to let you know the next timing Sep 24 17:21:39 OK Sep 24 17:21:56 He's taking up some things that Scout was doing before? Sep 24 17:36:51 Tofe: Your PR should fix the issue that nizovn reported? Sep 24 17:44:30 Herrie: it should Sep 24 17:59:42 Tofe: OK seems it does Sep 24 18:00:32 Will test it a bit more Sep 24 18:04:15 Yeah seems to do the trick I guess Sep 24 18:04:27 I'll try it on my lastop in vbox in a bit and will merge then **** ENDING LOGGING AT Sun Sep 25 02:59:59 2016