Clicks between spans will mess up router with escape key

[lforst]

I clicked between spans and then pressed ESC and it moved me between the spans again. Ideally, we close the drawer if esc is pressed.

Screen.Recording.2023-12-18.at.16.41.31.mov

[Shubhdeep12]

Hi @lforst
the reason why clicking the escape spandetails sidepanel is not closing is that it's a route (/traces/:traceId/:spanId)and in the spotlight when escape is clicked, we navigate to the previous route in history.

the solution which we can implement here is to remove the spanId from the route(not treating the sidepanel as a page) and open the sidepanel based on the active span state.

Also, with this commit - 14eadb9
we added a UX of treating sidepanel as a dialogue box in which on clicking outside of panel close the sidepanel