timeline

Hi to all, I'm developing an audio editor that at the moment has basic functionalities as draw the sound waveform, play the sound.
I would implement a time line ( a moving vertical line ) that give to the user an idea of which part of the sound is playing.
In the paintEvent I have to draw the waveform ( cached in a QImage ) and the time line.

This is my paintEvent

Code:

---

void WaveWidget::paintEvent( QPaintEvent* pe )
{
	// Update the ev->rect area with the wave from the rendered QPixmap here
 
	if( m_waveCachePixmap.isNull() )
	{
		updateWave();
	}
 
	QPainter p( this );
	QPen pen = QPen(Qt::blue, 1);
	p.setRenderHint( QPainter::Antialiasing, true );
	p.drawImage( 0, 0, m_waveCachePixmap );
 
	QRect& rect = QRect(pe->rect());
	p.setPen(pen);
 
	p.drawLine( rect.x(), 0, rect.x(), height()  );
}

---

I also have to delete the previous line before to draw the new? Or the final result is a growing rectangle. How can I do?

Best,
Franco

What exactly is the problem?

Quote:

---

Originally Posted by wysota

What exactly is the problem?

---

Instead of drawing a horizzontally moving line it draw a rectangle that grow.

The rect() you get in the paint event can be an arbitrary rectangle from within your widget's coordinates. I don't think you want to paint arbitrary rectangles on your widget...

Quote:

---

Originally Posted by wysota

The rect() you get in the paint event can be an arbitrary rectangle from within your widget's coordinates. I don't think you want to paint arbitrary rectangles on your widget...

---

Sorry but I didn't get you.
So what must I do? Where my code is wrong?

Quote:

---

Originally Posted by franco.amato

So what must I do?

---

Draw a line in coordinates you want it to be drawn at, not the ones passed to you by the event.

Quote:

---

Where my code is wrong?

---

Here:

Code:

---

QRect& rect = QRect(pe->rect());
p.setPen(pen);
 
p.drawLine( rect.x(), 0, rect.x(), height()  );

---

Quote:

---

Originally Posted by wysota

Draw a line in coordinates you want it to be drawn at, not the ones passed to you by the event.



Here:

Code:

---

QRect& rect = QRect(pe->rect());
p.setPen(pen);
 
p.drawLine( rect.x(), 0, rect.x(), height()  );

---

---

Sorry but I don't understand you.
How can I draw I line where I want whitout considering the rect()? How can I pass the coordinates of the line to the paintEvent?

Can you explain to me?

Best

I solved it.
The problem was the rect I passed to the paintEvent. It must completely contain the 2 lines ( old and new ). Something like this

Code:

---

QRect r = QRect( old_line - 1, 0, new_line + 1, height() )

---

and pass r to the paintevent

Code:

---

update( r )

---

Regards,
Franco

No, that's wrong as well. Take a window from another application and start moving it around over your widget. You will see it's getting messy.

Quote:

---

Originally Posted by wysota

No, that's wrong as well. Take a window from another application and start moving it around over your widget. You will see it's getting messy.

---

So please let me know how must I do.


Thank you

Quote:

---

Originally Posted by franco.amato

So please let me know how must I do.

---

I already told you - draw the line in a place where you want it drawn, not in a random place. The line has some logic behind it, right? Are you able to tell exactly where the line should be drawn at a specified arbitrary moment based on the state of the application?

Quote:

---

Originally Posted by wysota

I already told you - draw the line in a place where you want it drawn, not in a random place. The line has some logic behind it, right? Are you able to tell exactly where the line should be drawn at a specified arbitrary moment based on the state of the application?

---

Yes Is exactly what I did

Quote:

---

Originally Posted by franco.amato

Yes Is exactly what I did

---

You mean now or when? Because none of the code you pasted does what I said.

Quote:

---

Originally Posted by wysota

You mean now or when? Because none of the code you pasted does what I said.

---

Here my last code:

Code:

---

void WaveWidget::paintEvent( QPaintEvent* pe )
{
	if( m_waveCachePixmap.isNull() )
	{
		updateWave();
	}
 
	QPainter p( this );
	QPen pen = QPen(Qt::blue, 1);
	p.setRenderHint( QPainter::Antialiasing, true );
        //wave
	p.drawPixmap( 0, 0, m_waveCachePixmap );
 
	//timeline
	p.setPen(pen);    
	p.drawLine( m_CurrentTimePosition, 0, m_CurrentTimePosition, height() );
}

---

and the setCurrentTimePosition (as in my last post):

Code:

---

void WaveWidget::setCurrentTimePosition()
{
	uint newTimeValue = getCurrentPos();
 
	/* trasform time coord in widget coord */
	qreal samplesPerPixel = m_wave->getSamples() / width();
	qreal x_pos = newTimeValue / samplesPerPixel;
 
    if( x_pos == m_CurrentTimePosition )
		return; //no change
 
	QRect r = QRect( m_CurrentTimePosition - 1, 0, x_pos, height() );
	update( r );
 
	/* update current pos */
	m_CurrentTimePosition = x_pos;
}

---

Is what did you suggested to me?

Yes, it's much better now. The only thing I could suggest is to move updateWave() to setCurrentTimePosition() method. Your paintEvent() would then execute a bit faster which might be beneficial if complexity of your application grows.

Quote:

---

Originally Posted by wysota

Yes, it's much better now. The only thing I could suggest is to move updateWave() to setCurrentTimePosition() method. Your paintEvent() would then execute a bit faster which might be beneficial if complexity of your application grows.

---

Hi. Imagine that scenario:
1) I load an audio file -> the waveform is generated.
2) I resize the window WITHOUT PLAY THE FILE
in this case I have to update the wave so I can not move the updatewave inside the setCurrentTimePosition

Did you get me?

Best

Quote:

---

Originally Posted by franco.amato

2) I resize the window WITHOUT PLAY THE FILE
in this case I have to update the wave so I can not move the updatewave inside the setCurrentTimePosition

---

Call updateWave() from the resizeEvent() as well. It's still worth it to write another method to have the paint event not have to execute the if statement every time your widget is redrawn.

Quote:

---

Originally Posted by wysota

Call updateWave() from the resizeEvent() as well. It's still worth it to write another method to have the paint event not have to execute the if statement every time your widget is redrawn.

---

I moved the updateWave inside the resize event but the resizing process now seems slower.

What do you think about?

During every resize your wave is being updated. It would be wise to update it only when the user stops resizing but then it might not be worth the effort. Maybe you should stick with generating the pixmap during paint event for now until you reach a point where it becomes a bottleneck.