Scroll position is lost when refreshing the preview

Perhaps we could use localStorage to store the scroll position?

Used localStorage to remember the last scroll position and scroll back to the same position refresh.
Please review and suggest if we can use a better approach.

@omkar-pd feedback added. This is moving in the right direction.

@cosmicdreams
I have addressed the given feedback. Please have a look.

And in other work, we could hide view mode behind a permissions

back to needs work. Great progress, please keep going.

Made all the changes. Changing status back to Needs review.

looking good, back to Brian for review

Updated based on most recent commits to the 2.0.x branch. Also, to update the target scroll position before refresh.

I updated the MR to merge in the latest commits from the 2.0.x branch, and to have the scrollPosition state variable updated just before a refresh. I updated this issue to target the 2.0.x branch but it won't let me update the target for the MR.

Good call on updating the target for the MR. I'll jump in and review.

MR21 still appears to target 1.0.x. That seems to no longer be the intention for this work's inclusion. I'll see if I can switch it to 2.0.x but we might be at a point where we have to close the MR and open a new one.

Pushed up a fix to reset the scroll position when changing view modes, and also made the code cleanup tweaks I mentioned.

Needs a last look from someone else, but good to go from my perspective.

I made one additional (very minor) tweak, so maybe I shouldn't be the one to RTBC this, but this is looking really solid to me.

Merged, THANK YOU!!

Automatically closed - issue fixed for 2 weeks with no activity.