2021-12-23 17:50:57

by Nicolas Fella

[permalink] [raw]
Subject: [PATCH BlueZ] Use audio-card-bluetooth icon

PulseAudio uses this icon for this kind of device

Let's be consistent

Users will gracefully fall back to audio-card if audio-card-bluetooth
is not found
---
src/dbus-common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/dbus-common.c b/src/dbus-common.c
index 5e2c83d52..3611cb013 100644
--- a/src/dbus-common.c
+++ b/src/dbus-common.c
@@ -80,7 +80,7 @@ const char *class_to_icon(uint32_t class)
case 0x0d: /* Camcorder */
return "camera-video";
default:
- return "audio-card"; /* Other audio device */
+ return "audio-card-bluetooth"; /* Other audio device */
}
break;
case 0x05:
--
2.34.1



2022-01-05 19:07:34

by Luiz Augusto von Dentz

[permalink] [raw]
Subject: Re: [PATCH BlueZ] Use audio-card-bluetooth icon

Hi Nicolas,

On Fri, Dec 24, 2021 at 5:06 PM Nicolas Fella <[email protected]> wrote:
>
> PulseAudio uses this icon for this kind of device
>
> Let's be consistent
>
> Users will gracefully fall back to audio-card if audio-card-bluetooth
> is not found
> ---
> src/dbus-common.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/dbus-common.c b/src/dbus-common.c
> index 5e2c83d52..3611cb013 100644
> --- a/src/dbus-common.c
> +++ b/src/dbus-common.c
> @@ -80,7 +80,7 @@ const char *class_to_icon(uint32_t class)
> case 0x0d: /* Camcorder */
> return "camera-video";
> default:
> - return "audio-card"; /* Other audio device */
> + return "audio-card-bluetooth"; /* Other audio device */
> }
> break;
> case 0x05:
> --
> 2.34.1

It doesn't seem to be part of
https://specifications.freedesktop.org/icon-naming-spec/latest/ar01s04.html,
does the icon themes really have such icon?


--
Luiz Augusto von Dentz