Three column layout (rebased/squashed)

[juliushaertl]

Rebased and squashed version of #842 to have an easier way to resolve the conflicts

So far pushed another commit additionally to bring back the actions menu in the list after my rebase and some minor styling fixes

[theatischbein]

I will test the function tomorrow, but here some comments on the code

[theatischbein]

Why you catch for > 40 but slice to 0-30 ?

[juliushaertl]

Something to consider separately I'd say as it is not introduced here but originating from https://github.com/nextcloud/notes/pull/424/files#diff-84c733529b3b3fd6edfc369f85fe95b5cfb8d1f0a74d39712293aa3b037a0848R113-R117 s

[theatischbein]

May move 1000 to const variable

[juliushaertl]

I'd still keep this as this is a pattern that js developers likely know well and a constant (or timestampToMilliseconds helper that I was thinking about) would be easy to be overseen in new code introduced and then we'd have two ways of doing that.

[juliushaertl]

Thanks for the review @jonnytischbein, very much appreciated. I addressed most of them, two I'd keep out of this PR.

[juliushaertl]

Let's get this in so we can have a first beta for more testing.

[theatischbein]

Hey,
I can load any note on the current master branch. The JS console shows:

Uncaught TypeError: this.note is null
    refreshNote NotePlain.vue:206
    refreshTimer NotePlain.vue:201
    setTimeout handler*startRefreshTimer NotePlain.vue:199
    fetchData NotePlain.vue:129
    promise callback*fetchData NotePlain.vue:123
    created NotePlain.vue:94
    VueJS 28

Do I have to do any migration steps ?
This UI looks great though! :)

[juliushaertl]

Hm, I cannot think of any. Can you test with a new user account to see if it might be related to the account settings with which editor is being used as I see NotePlain in the trace?