Skip to content

Add 'Record Label' field mapped to Publisher metadata - #16799

Open
emphor wants to merge 2 commits into
mixxxdj:mainfrom
emphor:feature/record-label
Open

Add 'Record Label' field mapped to Publisher metadata#16799
emphor wants to merge 2 commits into
mixxxdj:mainfrom
emphor:feature/record-label

Conversation

@emphor

@emphor emphor commented Jul 30, 2026

Copy link
Copy Markdown

Summary

Adds support for a Record Label field throughout Mixxx, mapped to the standard Publisher metadata tag.

Changes

  • Adds a Record Label field to Track Properties.
  • Adds a Record Label column to the library.
  • Reads Publisher metadata from supported audio files.
  • Writes Record Label back to the Publisher tag.
  • Stores the value in the Mixxx library database.

Testing

  • Imported files containing Publisher tags.
  • Edited Record Label in Track Properties.
  • Verified persistence after restarting Mixxx.
  • Verified tags written using Mp3tag.
  • Confirmed existing Publisher tags are imported correctly.
Mixxx 6 5 3 Properties Mixxx 6 5 3 Mixxx columns after Mixxx Proiperties showing Record Label field MP3Tag showing Record Label using Publisher tag

@cr7pt0gr4ph7 cr7pt0gr4ph7 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code itself looks good, see below for further comments.

Comment thread src/library/dao/trackdao.cpp Outdated
@cr7pt0gr4ph7

cr7pt0gr4ph7 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

I can see the value of being able to display and manipulate the Publisher tag, and think of certain DJ workflows (esp. related to electronic music) where this would be beneficial. My only concern is that the Track Info Dialog is getting more and more crowded, and is already reaching the limit of the screen height on my laptop, even without this change.

Though this usability problem could be solved by being able to dynamically show or hide fields in the Track Info dialog. This may require some trickery to make Qt layouts play nice with this, though. I may look into doing a proof-of-concept of that.

@ronso0

ronso0 commented Aug 4, 2026

Copy link
Copy Markdown
Member

My only concern is that the Track Info Dialog is getting more and more crowded, and is already reaching the limit of the screen height on my laptop, even without this change.

yeah, this will be an issue for some users.
Note that the I have #15073 to address this in a rather 'traditional way' for small screens (768px and smaller).

I suggest to discuss the design first, before you spend time on a POC

@emphor
emphor force-pushed the feature/record-label branch from e94dcfa to 6ab3dc9 Compare August 9, 2026 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants