| *** frinring_ is now known as frinring | 01:40 | |
| *** zbenjamin is now known as Guest20253 | 02:46 | |
| *** zbenjamin_ is now known as zbenjamin | 02:46 | |
| r0kk3rz | oh wow, LG decided to open source some of their webos stuff | 02:49 |
|---|---|---|
| r0kk3rz | interesting | 02:49 |
| *** feodoran is now known as Guest96816 | 03:15 | |
| *** feodoran_ is now known as feodoran | 03:15 | |
| *** SpeedEvil is now known as Guest78679 | 03:26 | |
| Venemo_j | hey | 07:45 |
| dcaliste | Hello chriadam_ and jpetrell: I hope you're fine. I'm afraid I'll be 15 minutes late today. I'm still at home looking over the children. I'll leave for office in 10 minutes. | 07:52 |
| chriadam_ | dcaliste: hi. no worries :-) | 07:54 |
| dcaliste | chriadam_, thanks. Children are ready, I'm leaving. Should be online again in 15 to 20 minutes. | 07:59 |
| dcaliste | Hello again chriadam_ and jpetrell, back online. | 08:21 |
| chriadam_ | dcaliste: hi :-) I hope you had a nice holiday | 08:21 |
| dcaliste | Yes, weather was particularly great for the winter season, which was pleasant. I've seen you've pushed (or Andrew) the modification in silica, thanks. | 08:22 |
| dcaliste | As discussed on the MR in Gitlab, for the part in QtDeclarative, I'm going to propose it upstream this week and continue the discussion there. | 08:23 |
| dcaliste | I haven't time yet to further test if it's correcting the issue with lipstick-security-ui :( | 08:23 |
| chriadam_ | sounds good, and no worries | 08:24 |
| dcaliste | My main concern is that I'm not able to reproduce the bug with a simple QML test opening and closing a window that has constant colouring refresh… | 08:24 |
| chriadam_ | nor have I spent any more effort on that one, yet, unfortunately have been sidetracked onto other tasks so far | 08:24 |
| chriadam_ | dcaliste: application quit could be involved, depending on if deferred delete events are promoted in the event queue for handling before other events, etc | 08:25 |
| dcaliste | Which is strange, since the gdb trace are clear and reading the code show clearly a possible fault corrected by the patch. But well, it's a race condition anyway, so very sensitive to context… | 08:25 |
| chriadam_ | not a Qt event loop expert, but there are some complexities when dealing with application exit at least | 08:26 |
| chriadam_ | I saw that pvuorela_ provided some comments on the outstanding PRs for the email signing feature | 08:27 |
| chriadam_ | thanks for that ^ | 08:27 |
| dcaliste | Yes, I've begun to address them yesterday and will continue today. Thanks to him for the review ;) | 08:28 |
| jpetrell | hello | 08:29 |
| dcaliste | He pointed out some missing dependency issues in the packaging making the experience no so nice when some packages are missing. Need to further investigate. | 08:29 |
| dcaliste | hello jpetrell. | 08:29 |
| chriadam_ | no rush - hopefully once those review comments have been addressed, we can merge and tag everything | 08:31 |
| dcaliste | jpetrell, pvuorela, as said in a comment in Bitbucket, I've slightly changed the Loader layout in account-setting, now the crypto settings are packaged separately in a sub-package. | 08:31 |
| dcaliste | Installing this subpackage should pull the required dependencies from Sailfish Secrets and GnuPG. | 08:31 |
| jpetrell | dcaliste: cool. yeah pvuorela was talking about it, that would be cool to have one package that pulled in dependencies | 08:31 |
| jpetrell | dcaliste: yeah | 08:31 |
| dcaliste | If this sub-package is absent, there is no crypto section at all in the setting page. | 08:32 |
| dcaliste | The loader for it should fail silently not finding the source file. | 08:32 |
| dcaliste | As pointed out in the comment in Bitbucket, this sub-package is for settings only and pull in only dependencies for the account setting, which means it's not pull in the QMF crypto plugins. | 08:34 |
| dcaliste | Maybe I should do something equivalent in the jolla-email part, having a sub-package for crypto stuff that would pull in the account setting crypto sub-package and the QMF ones. | 08:35 |
| jpetrell | dcaliste: pvuorela mentioned he had issues getting the email signing to work. signing is fairly technical domain, and we still lack some enablers to make managing keys (sync, maintain, import, etc.) as easy and transparent as it would need to be for ordinary people to succeed in setting it up | 08:36 |
| jpetrell | don't know the details, but saw pekka commenting on pull requests | 08:36 |
| jpetrell | dcaliste: btw I have continued working on Documents app to get rid of the split view pattern. the design wants it to work more like Email Viewer, where document is scrolled vertically, and you back-step with simple swipe to right like with normal pages. actions (share, delete, details, etc.) are shown on top of as overlayed controls like with newer Gallery viewers | 08:38 |
| jpetrell | I've done the split view removal on other apps so can do it quite quickly also for documents | 08:38 |
| dcaliste | jpetrell: yes, managing keys is still not possible at a production quality, you have to have a working key for the email signing to work comfortably. | 08:38 |
| jpetrell | I remember you mentioned you also started similar work | 08:38 |
| chriadam_ | jpetrell: initially, I think it's fine for some manual step to be required for use of the signing feature (to allow preliminary testing by power-users, without affecting the experience of "normal" users) | 08:39 |
| jpetrell | chriadam_: very true | 08:39 |
| chriadam_ | so long as we somehow document that, and have a plan for how to go forward | 08:39 |
| dcaliste | jpetrell: yes, see https://github.com/dcaliste/sailfish-office/commits/title | 08:39 |
| jpetrell | as such its not far from experience of doing signing on desktop linux, where you often do some bits from command-line interface | 08:39 |
| jpetrell | dcaliste: thanks, will have a look | 08:40 |
| dcaliste | jpetrell: for email stuff, particularly if we put every thing behind sub-packages that power-users will to pkcon install by hand | 08:41 |
| dcaliste | jpetrell: the branch was working well and implement all from design mockups that was available at that time, except that when zoomed in the page transition was not working anymore. | 08:42 |
| dcaliste | At that time I didn't find the reason why… | 08:42 |
| dcaliste | And I put it aside because removing the drawer was very sensitive, making the link opening not working after rotation… (corrected since) | 08:43 |
| dcaliste | Before working on it again, I wanted to make the regression tests work, see https://github.com/sailfishos/sailfish-office/pull/101 | 08:43 |
| dcaliste | But the regression tests themselves have some issues ;) As discussed with pvuorela at that time. | 08:44 |
| pvuorela_ | lots of highlight here. hi! | 08:48 |
| chriadam_ | is that something that could be split up into small chunks? in recent community meetings, have seen some expressions of interest to help with coding. maybe this is a good opportunity for someone to help, e.g. fixing some regression tests? | 08:48 |
| dcaliste | pvuorela, hi! chriadam_, it's old code and I don't remember every bit, but I thing the issues with regression test were that they were too sensitive to numbers. | 08:51 |
| dcaliste | I mean, the regression test was mimicking device rotation, scrolling and we were testing if pointing at some place would trigger the link there after scroll or device rotation. | 08:51 |
| dcaliste | I don't remember exactly, but this imply some fuzzy comparison with too high bracketing for it to be reasonable. | 08:52 |
| dcaliste | The problem with the regression test was that they play the scrolling, device rotation… at device speed, and some redraw maybe skipped and it's dificult to understand what happen. | 08:53 |
| dcaliste | I remember adding lot of sleep here and there but even like that some issues remains. | 08:53 |
| dcaliste | But if people would like to try and contribute, yep that would be great. At least try to see if regression tests are always true or if they find some conditions when running the test make them false. | 08:54 |
| dcaliste | I can rebase the PR on github on master which would be a good strating point ! | 08:56 |
| chriadam_ | great, I will raise that at the next sailfish os community meeting | 08:56 |
| chriadam_ | and see if we get volunteers to help with this effort. good stepping stone, hopefully. | 08:56 |
| chriadam_ | sledges: ^ topic for next meeting ;-) | 08:57 |
| dcaliste | jpetrell: reading the commits for the 'title' branch, I remember that I try to showcase it first for the PDF document only, so the latest commit removing the drawer is doing not at DocumentPage.qml but directly at PDFDocumentPage level, which will have to be adapted for Office documents also. | 08:58 |
| dcaliste | jpetrell: but the most sensitive commit is the previous one, the one introducing a header with the document title. It is sensitive because before all assumption on Y offset was based on top of page being 0, which is not the acse anymore and can introduce a lot of bug in context menu position calculations. | 09:00 |
| dcaliste | Hopefully I did it right at that time, but that's why I wanted the regression tests to work at that time. | 09:00 |
| chriadam_ | makes sense | 09:06 |
| chriadam_ | good to have working test suite, in order to have confidence when making changes, definitely. | 09:07 |
| jpetrell | dcaliste: here is my first rough proto https://github.com/jpetrell/sailfish-office/tree/jb41071 | 09:07 |
| dcaliste | chriadam_: sure and this thing is very boring to check by hand at each modification, because you have to test at 0 zoom, zoomed in, after device rotation, after second rotation, checking that it should go back to initial position, that links open at the right place, if they are at the bottom, that they push the view a bit up… | 09:09 |
| dcaliste | jpetrell: great looking at it… | 09:09 |
| dcaliste | jpetrell: I will try it later this week. You may open a WIP PR in Github, I may comment there on code changes… | 09:14 |
| dcaliste | like import Sailfish.Silica.private 1.0 ;) But I like the DocumentOverlay unification, I'll look at it in action later. | 09:17 |
| chriadam_ | dcaliste: I had nothing else - thanks again for your time and effort | 09:45 |
| chriadam_ | if you need anything from me, or if I've forgotten something, please ping me :-) | 09:46 |
| dcaliste | chriadam_, sure, I'm testing the regression test branch after rebase now. | 09:49 |
| *** TheOnlyJoey|BNC is now known as TheOnlyBouncer | 16:43 | |
| *** svartoyg_afk is now known as svartoyg | 17:42 | |
| *** svartoyg is now known as svartoyg_afk | 19:05 | |
| *** SpeedEvil is now known as Guest94769 | 21:58 | |
| *** BitEvil is now known as SpeedEvil | 23:03 | |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!