Skip to content

Commit

Permalink
Revert "Merge pull request mixxxdj#4804 from ronso0/waveform-tiny-ran…
Browse files Browse the repository at this point in the history
…ge-fix"

This reverts commit 8facfb6, reversing
changes made to 690e776.
  • Loading branch information
napaalm committed Mar 23, 2023
1 parent 7b25ebe commit 3eaf6a5
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions src/waveform/renderers/waveformrendermarkrange.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,6 @@ void WaveformRenderMarkRange::draw(QPainter *painter, QPaintEvent * /*event*/) {

double startPosition = m_waveformRenderer->transformSamplePositionInRendererWorld(startSample);
double endPosition = m_waveformRenderer->transformSamplePositionInRendererWorld(endSample);
// The painter would extend the rectangle to the entire width while
// span is < 1 AND span != 0.25 * [1;3], i.e. works with span = [.25; 0.75] px
// TODO(xxx) Figure what's wrong with QPainter / the renderer
const double span = std::max(endPosition - startPosition, 1.0);

//range not in the current display
if (startPosition > m_waveformRenderer->getLength() || endPosition < 0) {
Expand All @@ -82,9 +78,9 @@ void WaveformRenderMarkRange::draw(QPainter *painter, QPaintEvent * /*event*/) {
// this shouldn't involve *any* scaling it should be fast even in software mode
QRectF rect;
if (m_waveformRenderer->getOrientation() == Qt::Horizontal) {
rect.setRect(startPosition, 0, span, m_waveformRenderer->getHeight());
rect.setRect(startPosition, 0, endPosition - startPosition, m_waveformRenderer->getHeight());
} else {
rect.setRect(0, startPosition, m_waveformRenderer->getWidth(), span);
rect.setRect(0, startPosition, m_waveformRenderer->getWidth(), endPosition - startPosition);
}
painter->drawImage(rect, *selectedImage, rect);
}
Expand Down

0 comments on commit 3eaf6a5

Please sign in to comment.