Tom Moor
Tom Moor
I think the best thing here is to put some protections in at the API level that prevent removing the last admin from a collection, in the same way we...
Gave this a test, apart from the icon comment above I noticed that if you have positional sorting enabled there is a flash of comments in chronological order when the...
Interesting, the problem makes sense but adding more callbacks and state is not ideal. I wonder if we could just have `getComments` return `undefined` until the editor is loaded and...
Unfortunately this last fix seems to have create a flash when opening the comment sidebar, it gets almost the entire way through the animation before the comments are rendered. Are...
Wrapping in another div prevents scrolling unfortunately, but you're right that it's sometimes visible in production – will consider separately.
That last commit broke the sorting entirely I'm afraid
I did see that and removed it, but the order still was no longer changed by switching the UI. Perhaps removing the computed getter is the culprit
Okay, don't worry – I'll check it out and try again. Perhaps I messed up somehow
> I also noticed that document restore doesn't create a documents.restore event which sometimes messes up the history If the history is entirely driven from `revisions` then this should become...
Well not sure I want to _remove_ them, I guess I'm just not sure how not having `documents.restore` is an issue here as we'll have an associated `revision`