Shift-clicking anywhere will move the cursor to the point, and the "number
area" of the ruler is now clickable.
And I got to bump up notationview.cpp rev number to 1.200 :-).
Well, I did some more testing on collapsing rests. Looks like if you
created a two-bar measure and tried to use the eraser the first rest,
you get some bizzare results... this is because the combiner code
doesn't care if it has to combine across bars.
This should definitely not be allowed. Here's a patch, which includes
my previous patch also, so you don't need the other one if you apply
There are a bunch of endpoint conditions in the collapse code so please
give it a careful review.
There is still an endpoint condition bug in
Track::findContiguousPrevious where it will not find the first event in
a track as a previous event, but I'm too tired to fix it today.
From: Chris Cannam <cannam@al...> - 2001-11-14 11:20:34
Ron Kuris wrote:
> [...] the combiner code
> doesn't care if it has to combine across bars.
True enough. Your fix looks good to me, so I've committed it.
(One possible reason the collapse code didn't deal with barlines,
btw, is that it was written before the code that locates barlines...
Likewise, note the confusion over barlines in the comments in
TrackNotationHelper::insertNote and insertRest, although in those
methods I think the "correct" behaviour is less obvious.)
On Wednesday 14 November 2001 12:12, Chris Cannam wrote:
> Guillaume Laurent wrote:
> > And I got to bump up notationview.cpp rev number to 1.200 :-).
> Yeah, and you left an unfortunate typo in the 1.200 revision comment...
Ooops. Unfortunate indeed :-).