Doing this lets someone quickly delete a stray note since the last inserted note would be selected automatically, but comes with the con of moving the selection index around automatically.
I have no idea if this is desired behavior or not, but in case it's acceptable behavior I'll leave this PR here.
Doing this lets someone quickly delete a stray note since the last inserted note would be selected automatically, but comes with the con of moving the selection index around automatically.
I have no idea if this is desired behavior or not, but in case it's acceptable behavior I'll leave this PR here.
Since all notes are reordered by measure when they are added, this does not select the most recently added note, but the physically last note. I don't believe this is intended behavior?
Since all notes are reordered by measure when they are added, this does not select the most recently added note, but the physically last note. I don't believe this is intended behavior?
@veroxzik No idea how I missed that, should be fixed with the new commit.
I guess it's yet another O(n) call, but it's probably fine for charts in this game especially since it's just looking for a reference ¯\_(ツ)_/¯
@veroxzik No idea how I missed that, should be fixed with the new commit.
I guess it's yet another O(n) call, but it's probably fine for charts in this game especially since it's just looking for a reference ¯\\_(ツ)\_/¯
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Doing this lets someone quickly delete a stray note since the last inserted note would be selected automatically, but comes with the con of moving the selection index around automatically.
I have no idea if this is desired behavior or not, but in case it's acceptable behavior I'll leave this PR here.
I personally like this idea, as it can be rather confusing to think your last placed note is the selected note when it turns out to not be.
Since all notes are reordered by measure when they are added, this does not select the most recently added note, but the physically last note. I don't believe this is intended behavior?
@veroxzik No idea how I missed that, should be fixed with the new commit.
I guess it's yet another O(n) call, but it's probably fine for charts in this game especially since it's just looking for a reference ¯\_(ツ)_/¯
Fix looks good to me.