Project

General

Profile

Feature #979

Two little feature requests for Qt interface and playlists

Added by Andrea Antolini over 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Minor
Assignee:
-
Category:
plugins/qtui
Target version:
4.1
Start date:
April 24, 2020
Due date:
% Done:

100%

Estimated time:
Affects version:

Description

Hi, first of all thanks to your effort to develop this multimedia player (I enjoy it since 3.1 times)

then I have two litte question / feature request regarding QT interface and songs list in playlists or Library :

Could you consider increasing the height of rows or add a little bit of padding between rows?
Could the current playing song using the bold font style?

first request could be an option in the APPEARANCE setting : maybe something like compact /normal / large playlist song view

Regards
Andrea

treeview-bold-text.patch (557 Bytes) treeview-bold-text.patch Thomas Lange, June 17, 2020 18:11
treeview-bold-text-v2.patch (2.52 KB) treeview-bold-text-v2.patch Thomas Lange, June 17, 2020 20:03

History

#1 Updated by John Lindgren over 4 years ago

The row height and padding is set by your Qt style/theme, for what it's worth.

#2 Updated by Dale Powell over 4 years ago

"Could the current playing song using the bold font style?"

I would love to see the return of this feature! When I pointed out it was missing they said they wouldn't bring it back as there is a Currently Playing icon you can add :(

#3 Updated by Thomas Lange over 4 years ago

The attached patch restores the bold font style for the playing song.

@John: Do you think we can merge it as is? Or should we make this dependent on whether the playing column is hidden?

#4 Updated by John Lindgren over 4 years ago

Either way is fine with me.

#5 Updated by Thomas Lange over 4 years ago

Ok, I would then use the bold font for both cases.
Attached is another optimized patch which caches the font, similar to the approach in the playlist manager plugin.

#6 Updated by John Lindgren over 4 years ago

Looks good to me.

#8 Updated by John Lindgren almost 4 years ago

  • % Done changed from 0 to 100
  • Status changed from New to Closed

The remaining feature request (increase height/padding between rows) is a QStyle setting and can be changed by switching to a different theme, I believe. So I'll go ahead and close this.

#9 Updated by Thomas Lange almost 4 years ago

  • Target version set to 4.1

Also available in: Atom PDF