Monica Cellio
Monica Cellio
Does `enabled inline actions on comments (editing, deletion)` also address (or enable addressing) https://github.com/codidact/qpixel/issues/544 ?
The latest commit might address https://meta.codidact.com/posts/282342/282348#answer-282348 and https://meta.codidact.com/posts/282342/282382#answer-282382 (check in testing).
Testing notes: - Tested the own-comments limit for someone without Participate Everywhere and worked as advertised: upon reaching the limit buttons are grayed out with suitable message, and buttons on...
> Approved from a code POV - haven't run locally to look at the UI/UX but looks like @cellio has covered that. Yup. I haven't looked at the code, just...
> * preserve scroll state after opening tools (might be better suited for Co-Design); Or if that's too hard (or sends us into Co-Design), if the refresh went to the...
> After successfully replying to a thread, we now redirect back to the post (unless the reply is made from the full view), but also ensure that the thread is...
> Probably won't move it from the draft status for a couple more days (want to improve coverage after the latest changes first), but I believe it's ready otherwise. Made...
Testing notes based on checklist: - autofocus the reply textarea when the reply to thread button is clicked - I'm not getting auto-focus using Chrome or Firefox - I don't...
That fixed it, thanks! (Tested Chrome and Firefox, though it doesn't sound like this was browser-dependent anyway.)
@ArtOfCode- could you review this request? Are there downsides we should consider, or should we go ahead and add labels and add it to the queue?