.47 Bad pattern scrolling when clicking

Saga Musix · May 13, 2007, 12:07:29

Ensuring that cursor is visible works, but i found a new strange behaviour.
I think that's the way it happens.
1) Set cursor position to 1st channel.
2) Scroll to the last channel (e.g. channel 32) without moving the cursor
3) click a channel that is visible
4) now the pattern scrolls, although it's not necessary.

Edit by rewbs: topic should describe the bug.

rewbs · May 13, 2007, 13:05:54

I don't understand step 3.
Edit: sorry I get it now - reproduced and bug description clarified.

rewbs · May 13, 2007, 13:44:28

The problem is the separate consecutive calls to SetCurrentRow() and SetCurrentColumn() that occur when the pattern editor is clicked.
If the cursor is off-screen when the click occurs, SetCurrentRow() will result in scrolling back to the cursor's original column. Then, the call to SetCurrentColumn() causes another scroll to restore the visibility of the new column position.

pelya · May 13, 2007, 15:48:36

Since that was my bugfix :oops: I've posted patch against that new bug introduced.

--- Yay, my first patch is committed! I'll set some avatar picture to celebrate it

.

Saga Musix · May 16, 2007, 17:43:50

okay, there's another strange behaviour which seems pretty the same: if i set the cursor to channel 1 and scroll again to the last channel, it automaticly scrolls back to channel 1. that's annoying if you just want to see "what' going on over there" - at the other channels.

rewbs · May 16, 2007, 20:30:22

Jojo, I'm guessing you have Follow Song on?

Saga Musix · May 16, 2007, 23:18:17

yes. i need it.

and this new behavious is quite annoying. one should be able to turn it off.

rewbs · May 27, 2007, 20:33:04

Please try v1.17.02.48.

Saga Musix · May 27, 2007, 21:11:27

i'm sorry but the site looks kinda broken to me - coz I can't see any download links anymore for the relaeses - just the changelogs! also, the table header says "OpenMPT dev build v ()" i think a version is missing here. seems like something went wrong over there.

EDIT: I got the file using the RSS feed now, let's see if it works!

Saga Musix · May 27, 2007, 21:28:58

okay, it works in .48 - my day is saved! it was a pain to do large tracks (like Grinimal) with this bug - again, thanks for the good work!