Add widget that displays the duration of a mix being recorded so far

Bug #1529908 reported by ronso0
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Wishlist
ronso0

Bug Description

It would be nice to have an small indicator (like clock widget) which displays how long the mix is I'm currently recording.

My scenario: I do mixes for friends which should fit on a 70/80min audio CD, so it would be nice to
1) see duration of recording so far (to exhaust that 80 minutes)
2) have a pushbutton like "Start recording"

Tags: easy recording
ronso0 (ronso0)
summary: - Display duration of mix being recorded so far
+ Display duration of mix being recorded so far - Feature request
Revision history for this message
Daniel Schürmann (daschuer) wrote : Re: Display duration of mix being recorded so far - Feature request

This is an ideal starter bug and to learn the Mixxx control system.
@ Ronny, do you have fun and time to dive into the Mixxx source?
If yes, I can give you some additional hints.

tags: added: easy
Changed in mixxx:
status: New → Confirmed
importance: Undecided → Wishlist
Revision history for this message
ronso0 (ronso0) wrote :

@Daniel: yes, I could do so. I'm back to this bug https://bugs.launchpad.net/mixxx/+bug/1462350 from this summer and for the PR I'm learning about git right now. If you think is good for a starter, go ahead with your hints...

Revision history for this message
Daniel Schürmann (daschuer) wrote :

I am not sure if you know already:
http://www.mixxx.org/wiki/doku.php/bugfix_workflow

For this bug you need a new ControlObject e.g. [Recording], duration in seconds.
This can be instantiated somewhere here: https://github.com/mixxxdj/mixxx/blob/8f70bdcb277ed67e6bc53993c1bc97e8c160e141/src/recording/recordingmanager.cpp#L25
(don't be confused by ControlObjectThread below that line. It is legacy code and is not required here any-more)

In a first version you can update your new CO in RecordingManager::slotDurationRecorded() by parsing the sting.

Than you may move QString EngineRecord::getRecordedDurationStr() RecordingManager and path a double instead of the formated via the durationRecorded signal.

EngineRecord::m_recordedDuration should become a double.

Hope that helps.

ronso0 (ronso0)
description: updated
Revision history for this message
ronso0 (ronso0) wrote :

@Daniel: my apologies for the long delay.

I dramatically over-estimated my spare time..
I forgot to mention I'm a C/C++/Cwhatever newbie.

Right now I'm getting familiar with Qt-Creator, trying to understand the mechanics of COs, Widgets etc.
I'll be back when I have something to talk about.

Thanks for your help!
Cheers!

Revision history for this message
jus (jus) wrote :

In the Recordings view of the library, there is the `´Start recording``button.
If you start a recording, the filename, current file size, and elapsed time is printed next to the recording button.

I understand the features you are proposing kind of exist already.

Revision history for this message
ronso0 (ronso0) wrote :

@Jus: Yes, I know, but to see these features requires to change from Crates/Playlists/History view to Recording view, which is inconvenient while recording a session.
My idea was to have just 2 easy-to-use widgets, i.e. <RecDuration> and <RecButton> which could be placed anywhere in the skin.

I'm sorry, this C stuff is way to abstract for me and I can't assist in porting the available features to a widget. Instead I'll rather focus on building a controller-focused skin.

Revision history for this message
jus (jus) wrote : Re: [Bug 1529908] Re: Display duration of mix being recorded so far - Feature request

Thanks for the update.

Recording button feature already exists, i did add demo in the Deere skin, see https://github.com/mixxxdj/mixxx/blob/master/res/skins/Deere/tool_bar.xml#L16 <https://github.com/mixxxdj/mixxx/blob/master/res/skins/Deere/tool_bar.xml#L16>.
Just uncomment the bit in the xml and reload the skin.

Hope someone picks up the ``[Recording],duration`` widget, sounds useful.

> On 06.06.2016, at 21:12, Ronny Sandig <email address hidden> wrote:
>
> @Jus: Yes, I know, but to see these features requires to change from Crates/Playlists/History view to Recording view, which is inconvenient while recording a session.
> My idea was to have just 2 easy-to-use widgets, i.e. <RecDuration> and <RecButton> which could be placed anywhere in the skin.
>
> I'm sorry, this C stuff is way to abstract for me and I can't assist in
> porting the available features to a widget. Instead I'll rather focus on
> building a controller-focused skin.
>
> --
> You received this bug notification because you are a member of Mixxx
> Development Team, which is subscribed to Mixxx.
> https://bugs.launchpad.net/bugs/1529908
>
> Title:
> Display duration of mix being recorded so far - Feature request
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/mixxx/+bug/1529908/+subscriptions

summary: - Display duration of mix being recorded so far - Feature request
+ Add widget that displays the duration of a mix being recorded so far
Revision history for this message
ronso0 (ronso0) wrote :

Thanks for pointing there! This visual feedback for Rec status makes me happy already.

Revision history for this message
Ishan (puzzlemeso) wrote :

Hi,

I was wondering if this is an entry level bug / wishlist for somebody new. If yes, I would like to work on this and contribute. Can somebody give me starting pointers on how to be on this way?

Thank you,
Ishan

Revision history for this message
Daniel Schürmann (daschuer) wrote :

Welcome Ishan, thank you for offering you contributions.

https://bugs.launchpad.net/mixxx/+bug/1529908/comments/3 contains some useful hints.

I have also just updated this side:
http://www.mixxx.org/wiki/doku.php/developer_guide_control

What else do you need?

Revision history for this message
ronso0 (ronso0) wrote :

@Ishan are you working on this?
I made a naive try: https://github.com/mixxxdj/mixxx/pull/1220
Any help is appreciated!

ronso0 (ronso0)
Changed in mixxx:
assignee: nobody → ronso0 (medontknow)
status: Confirmed → Fix Committed
Revision history for this message
ronso0 (ronso0) wrote :

Added widget in https://github.com/mixxxdj/mixxx/pull/1221 and info to the wiki http://www.mixxx.org/wiki/doku.php/creating_skins#reording_duration

I can imagine to push this two steps further and create a combination of this label and Recording button where label text for different states can be defined. This would allow a compact widget for skins, without having to stack buttons and labels.

jus (jus)
Changed in mixxx:
milestone: none → 2.1.0
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/8393

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.