fix alternate shading for changing time signature (#3559)
* fix alternate shading for changing time signature * fix alternade shading for automation editor on changing time signature * code cleanup
This commit is contained in:
committed by
GitHub
parent
c0b910ec57
commit
6970c88e28
@@ -1233,18 +1233,20 @@ void AutomationEditor::paintEvent(QPaintEvent * pe )
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
// alternating shades for better contrast
|
||||
// count the bars which disappear on left by scrolling
|
||||
|
||||
float timeSignature = static_cast<float>( Engine::getSong()->getTimeSigModel().getNumerator() )
|
||||
/ static_cast<float>( Engine::getSong()->getTimeSigModel().getDenominator() );
|
||||
float zoomFactor = m_zoomXLevels[m_zoomingXModel.value()];
|
||||
int barCount = m_currentPosition / MidiTime::ticksPerTact();
|
||||
int leftBars = m_currentPosition * zoomFactor / m_ppt;
|
||||
//the bars which disappears at the left side by scrolling
|
||||
int leftBars = m_currentPosition * zoomFactor / MidiTime::ticksPerTact();
|
||||
|
||||
for( int x = VALUES_WIDTH; x < width() + m_currentPosition * zoomFactor; x += m_ppt, ++barCount )
|
||||
//iterates the visible bars and draw the shading on uneven bars
|
||||
for( int x = VALUES_WIDTH, barCount = leftBars; x < width() + m_currentPosition * zoomFactor / timeSignature; x += m_ppt, ++barCount )
|
||||
{
|
||||
if( ( barCount + leftBars ) % 2 != 0 )
|
||||
{
|
||||
p.fillRect( x - m_currentPosition * zoomFactor, TOP_MARGIN, m_ppt,
|
||||
p.fillRect( x - m_currentPosition * zoomFactor / timeSignature, TOP_MARGIN, m_ppt,
|
||||
height() - ( SCROLLBAR_SIZE + TOP_MARGIN ), backgroundShade() );
|
||||
}
|
||||
}
|
||||
|
||||
@@ -2887,18 +2887,20 @@ void PianoRoll::paintEvent(QPaintEvent * pe )
|
||||
++key;
|
||||
}
|
||||
|
||||
|
||||
// Draw alternating shades on bars
|
||||
// count the bars which disappear on left by scrolling
|
||||
|
||||
float timeSignature = static_cast<float>( Engine::getSong()->getTimeSigModel().getNumerator() )
|
||||
/ static_cast<float>( Engine::getSong()->getTimeSigModel().getDenominator() );
|
||||
float zoomFactor = m_zoomLevels[m_zoomingModel.value()];
|
||||
int barCount = m_currentPosition / MidiTime::ticksPerTact();
|
||||
int leftBars = m_currentPosition * zoomFactor / m_ppt;
|
||||
//the bars which disappears at the left side by scrolling
|
||||
int leftBars = m_currentPosition * zoomFactor / MidiTime::ticksPerTact();
|
||||
|
||||
for( int x = WHITE_KEY_WIDTH; x < width() + m_currentPosition * zoomFactor; x += m_ppt, ++barCount )
|
||||
//iterates the visible bars and draw the shading on uneven bars
|
||||
for( int x = WHITE_KEY_WIDTH, barCount = leftBars; x < width() + m_currentPosition * zoomFactor / timeSignature; x += m_ppt, ++barCount )
|
||||
{
|
||||
if( ( barCount + leftBars ) % 2 != 0 )
|
||||
{
|
||||
p.fillRect( x - m_currentPosition * zoomFactor, PR_TOP_MARGIN, m_ppt,
|
||||
p.fillRect( x - m_currentPosition * zoomFactor / timeSignature, PR_TOP_MARGIN, m_ppt,
|
||||
height() - ( PR_BOTTOM_MARGIN + PR_TOP_MARGIN ), backgroundShade() );
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user