Lattice-Automation / seqviz

a JavaScript DNA, RNA, and protein sequence viewer
https://tools.latticeautomation.com/seqviz
MIT License
246 stars 53 forks source link

Revert "scroll to selection for managed states" #219

Closed jjti closed 1 year ago

jjti commented 1 year ago

Reverts Lattice-Automation/seqviz#214

https://github.com/Lattice-Automation/seqviz/assets/13923102/f04e998b-b1f2-415f-9be6-f25dbfaca71e

jjti commented 1 year ago

@bugzpodder heads up that I'm going to revert this. I'm seeing some unexpected behavior right now where, when clicking, it jumps to that location but also highlights a bunch of other sequence

jjti commented 1 year ago

@bugzpodder I still agree with and think it's worth addressing the original intent of the PR: the linear viewer should scroll when the managed selection range changes. I just need some more time to grok why the selection behavior here was impacted

bugzpodder commented 1 year ago

sounds good, thanks for the heads up!