Keep scroll position in flatprofile view on repaint

[DanielRuf]

Closes #266

[NoiseByNorthwest]

@DanielRuf LGTM, thanks 👍

Side note: the text is still unselected on repaint, but things are way better with your patch.

[DanielRuf]

Side note: the text is still unselected on repaint

I will check this.

[DanielRuf]

When I click in the timeline view, the selected text on the left side is still selected:

@NoiseByNorthwest is this logic correct or should it automatically clear the selection on click in the timeline view at the top?

A simple "remove text selection" logic would be window.getSelection().removeAllRanges().

[NoiseByNorthwest]

When I click in the timeline view, the selected text on the left side is still selected:

I don't reproduce it, where do you click ? If I click on a span it makes that span selected (thus clearing the previously selected span info), if I click elsewhere (not on a span) it clears the overlay with the focus/selected span information...

[DanielRuf]

I mean visually selected, the text itself changes here in Microsoft Edge.

Maybe creating a separate issue with the right steps to reproduce might be helpful. Otherwise if this is not a big issue and will be solved with the UI rewrite, it might be not relevant at the moment.

[NoiseByNorthwest]

Maybe creating a separate issue with the right steps to reproduce might be helpful.

Yes

Otherwise if this is not a big issue and will be solved with the UI rrwrite, it might be not relevant at the moment.

Maybe, I let you decide