You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is created to track the leftover issue of #6254. Basically, since we are not restoring the instrIndex when we navigate to a history entry in the linear view, it gets its default value (-1). While this behavior is much better than the original behavior (the value is read before initialization and can return anything), it is still not the most desired result.
We cannot simply save the instrIndex into the HistoryEntry object -- since the instrIndex could change in the future. Also, this issue is unlikely to make substantial difference to the user -- since if they randomly click somewhere in the linear view, these things will all be populated correctly. In other words, the instrIndex is only missing when the user does not click anywhere else. In this sense, if we do not find out a good way to restore it, I think it is tolerable to leave it as-is. That said, I am still creating this issue to track the progress and latest status on it
The text was updated successfully, but these errors were encountered:
This is created to track the leftover issue of #6254. Basically, since we are not restoring the instrIndex when we navigate to a history entry in the linear view, it gets its default value (-1). While this behavior is much better than the original behavior (the value is read before initialization and can return anything), it is still not the most desired result.
We cannot simply save the instrIndex into the HistoryEntry object -- since the instrIndex could change in the future. Also, this issue is unlikely to make substantial difference to the user -- since if they randomly click somewhere in the linear view, these things will all be populated correctly. In other words, the
instrIndex
is only missing when the user does not click anywhere else. In this sense, if we do not find out a good way to restore it, I think it is tolerable to leave it as-is. That said, I am still creating this issue to track the progress and latest status on itThe text was updated successfully, but these errors were encountered: