From 9ab597c0e93d92bafaf9beb842ab37f9b1a63b78 Mon Sep 17 00:00:00 2001
From: Bruno Herbelin <bruno.herbelin@gmail.com>
Date: Thu, 8 Jun 2023 00:19:21 +0200
Subject: [PATCH] Icons to show enabled and disabled Hardware decoding

Updated the left panel for media source to show indication of hardware decoding on/off
---
 rsc/images/icons.dds | Bin 1638528 -> 1638528 bytes
 src/ImGuiVisitor.cpp |  24 +++++++++++-------------
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/rsc/images/icons.dds b/rsc/images/icons.dds
index 585a7e0c60cbeb61a9e59101b89a69027b53a9fa..8c393e07ec4e1a5fd614ac438d8abdf0fc01187e 100644
GIT binary patch
delta 2589
zcmb7GT}&KR6uvXV?jn?(A7Cji+TBeh6k@xp4FRFaQdT1|*6|5_D6Wa&#ilGFhV;c9
zz@+hk$T&7PJhf;H#6B!rG}Dx%jm0-(h-qW`;D*M4_~fQGX<=FRdhW1yW?2#F8NP&b
z=bZDMbN0-=R~!3CgTK$X@A)4l6$vd6kncbc$OA&~<zDV0H);6#lKZ~BZjN1$f8xw<
zb-r5iNIA7|jm+cO9RInL;<7GMMXFh3U5l3*XpPJ4O^Un2bs+&UUDsYUC%)ldR<6wR
zGEV1s4F}&6hb&FcbAxi0S&}m=73Yx);rEQ1_f6g8|B+AjqQ9+3RypI-PpGXWw7lsG
z;<g@Gp?Vts+Fzp@>5*vCPj;T-o9J|$(1(iU*+PaarKlrxniAPbL*Jf_nWU|U#m5i(
zYkG`Fbpi2W){{g>?ZCSBQb<GV%x<`4bEr^EgyvPu!yNsYuTsL8{(i{z&5@;y6moRp
z&bUIHqyK;TaiZ?_Ol7)DJumGL*-YqJnfEF`P^^p9O0JUUQ+%6E!s%nptvq|H9+aXv
zyut?+tSpN4RRZ&JO1|<VUyt9oDN49~Q4C|>q9~V}Nz0^rx#Dd*0bU<FUK>VtS+Hp1
zJ%1j?26x;PU!-AL&8f%yBiR>38JLY*%x;a)pDd5@W4;aF$ni<s(<B7i2MC>H`#Vt3
zFW_|DW<&c$x$CQrcn7lXPi=D-O-@2OBMofLkWEl<N0V^Ew>OtL>Jb{1;A%D!E1jEy
zJI0slPScvpK9hJhu-o%otQI`+Y=^fsaG0pCc)uWxS2>on$B6iwRh6mIMuMzc&D;43
z={gfS{l#zAPuHlvPdcxeGkzf}>r)jkL}IJ<7DCI6@dY=F;FJpUQQH)V#dn+57S1tK
zT|0hIsMl*k-aDji*&aVX?rGqfr7(+u*3|9$WZPJ;Yo&a3d-3794sWM9b4qw9`)<L6
z{EmXmJrDmB$XfYe(UHoM0cHgw1*4_dlY;h%B9awy9(HaJ+R!t`@E^^R#8k8t-nv0@
zczr<Z!)lWt`}U?XM|<g9y*o(T>+Cm!!fxGZc0<`*g^QlX+-OJ3fni8Gn@*-Ham#O9
zr)Ngm)W`-g=0^JVi)v2w3s2=%Y?p?l&{8HxRjYD@LR`iLhec6$axFh95O}JYI5l)U
zyGhE?Y1NmU_U|;|(9A0A*i=}&FH=D&f&Nqrt8~SpEWi<_wwLwrka?M-9}94F5ID|y
z9TFP(2^pVe3_Is0oSz_cFr0SKy{!Ih)Ul){L(?tHk_J9ql0FRMf0>rYJMmxlj+_Y4
zLP3}_ilY1BytUz4wA)Z<++sX-0Hg8A>@oAJq|j7)jTjL%DS*C~a@H<sMckq{j0C-5
zM9ncnIHgq1lWRmRNI5(Q_ZYF7RApT$dxO%o4<!jD&{JfM3T?%Pbw}|O4e^_+a+f(V
zE1Xo61tgUgL&yU>F~ldllclR&u;nEVX}S>cf54ej7i68g9Jwt3DgZ8k8{h#{0-gg@
i0jdE<05t$FKm<qt8K3}a0d;_SfDh1MMs6Q{=j^}MmZUoX

delta 4002
zcma)9Yitx%6h1pM?Y5=0Gur|s8oON#6hYa7yhLSz26+`E5~xN<8?dNIwCRsc6Lq_w
z7EO?D_QzOa#0o~Oi48Bu1f;2~fk*>_gkO-2;iZzcX|$zjw;j)!+cPs=3^2Dh%{TX+
zbMN`?x!=xdh;D7bzxO>yrDJ=HzBL7(Nk+AiwccZV6_duPv8Ix$t?Cdp6Q5Rmlu%9w
zHO6ycze<hqRwb34%1Pxg4lO&>MvXCSRa4X$%lhJaITCB~SF4TcFf|^@sLj4cjq!3d
z;-|*AQ5nvn#<;ff!hULu)yCimHO7iwmdKnE_ZH?A?3^~(&ac6GBqMx4THwQd=)qTf
z$_@<qZ=N9n&OXFg1pJ8|GKHJUxt-19BH)(K8H<3`+l;e;S{5*dc<-UT`foKE-gUq7
zI}y6~Fk=z=oA#O_^s|{fE<$g4hp`CVe^!4@(u!x;oI#5^_@y~Wv#is^e@Sz5GJ|mj
z$VQzZslM2*#XHnjkI{Q;BFqA)sj;AiCI47F8Ci6X&yvPsKTdFu-{nvLW201{6vFp%
zylBf?`mHlk-lkCFpUws}fZe;D4S3Wvj2m!|<Mc6DO|u8Rqdm9aiBDIUtH>#(HXQa3
zrIFKPRygEr(yJ%<rWlQ9r9mwg4XQ7|A?SmCHXn)i7LM0{m@8N1erqSO`K&JODtIo|
zROWpfuE8A`h<~%{QP0Qm)4gjJwuPT(<F+GD4;?kFTjm^0MRE?0SAc>6dF-5tHIx|9
zq@?*4soK%w)*eI*plQdyGke$$`k`Oym+ejBakp6wCs52%ELs)04jt6k2CYRKouh!F
zDo2$*9tT?EIH6JAK!f98+Iz!|T8}ec61aTzj%$7(aP=qL2|l!k5*UC14>o>*8{BKP
z(FvYdbXk>?!+1=G@20Aptt*a{cO`m;Z3-liN~Ft@O#uKdf0btQ_5*MgOe6e{l;fM1
zw)car0IHVg<jJl3DEiJ5j3wlk7}j^+$gB@pG~R>A0~VDDHpY}AXO45dwkW^2v@Q%#
zHv3jLPYZ7Ym_57u9AgjCMTF^R{`IbP^bigQhmM$*u)0%7VY7-3<sx8XoPNww9WqrW
z-Q@8wZUdKfwt>{lhVHauHa_&qIlj_<Gu}*0v9a0H{1FAmSVoSBDT}Ab{dnqAb`eu-
zY?{sJ+;L_CLQFayA3ZZ0-!Zq;0F(rH+)UPNpi^g@Maf1S5`&9o1o=Ju{67q?J}>h)
zUP68Hfb`^)erXG36~8yl8eGruZvsLH=wYwC&(I!`BU)*qQ!vCPm_a?wm`jZ5juJx;
zmPZMtPl+qGF=Y@j#LhbP+O^)|(p%kkOv)vG-F?zbQZ6wj7KjI8Vdmow3#WpqQ|WoM
z`n;1zdi7tq!C;z5NH?W^${==5c|#dwW4(51ezDOsQ|{H~jUE_}2?;yc%2-HP>psRp
z!bUIIeUr!1D&+th3$1E(oLgGeYHut(d4yKgh1p(cRd^d?uh1%wZqz7#HJvqm>WPe%
zdf9;FpJIuQRL6Egbi$5b5*-K|;id8<U#U=;u{usU!q%+g4|ab@urWt$V_Rid@igeX
zr(l!(xCPsipCtR6dY536Tu031JOQi}8YVJcCNct#p{}?Un01BsW9f<!-YBa+ed@yO
zN%KiP=wXjSJ)9p~>o?5W5W4^<)XsJ4rA7G_t6iFkKWrDYn^#8IQM&3`N&~XJ5Zxp9
z!p*U0@kWvp_e+u!H@4JdH!R^r_WrUwb;-0o0$7UAo72y~$m7r8f`LGnetL(zGISBH
z%^K{YAu(56tR!PgYQVWzBsK1}Yhuwt<Ad#TioQuEUWXP4I<%3KVRL7f{$o_GSCe=~
z55McNJ;$OOf+0qb2&H~c-}AzDJEj>ow~X8{uf!YoyrH(4n|rcW*qnLvb=S3!5sS#h
z_yw&+@LSQn!}4-O>zxvUu#9B{uS|FjmJx2eUQXHH(=;BM<CGAi{_GN>{3-N@U({^x
z!H=2^5qDt2X@}8nPH&fZ`g9e)Rq<gQ2;gPJTEva17I9;$MckNb5jU=_3^;{|aOe~w
zgcb0DUK!7;G@9bF->y55C5I<bcQ`zWy2If~)E!Q#CF%}`r_s72rO&UIU-w?}4~It7
zA-%F8V?Grd>T*$IIDv^8!wF2(7*1fK#&F^iHAZkmFHK~=WJJG~gFd`0R6k{GckxMK
zxcQ|p+?c{}V+zBKDGWEJ;>nF;4eQjn^~OHJX|d?cYOE#4+Fi@K=(daa-}AdMhU<0X
z3;R3vx^ZyC-{r<$-Lqia7e?=zf-0%8?4*oc!JdIV6T26C7WQoHIoQWx&&8gH{UPl6
c*j4Nrb|3Zv?1k9JW1oP1qEU9TXzh~!0M0--lmGw#

diff --git a/src/ImGuiVisitor.cpp b/src/ImGuiVisitor.cpp
index 6c1ea29..042993a 100644
--- a/src/ImGuiVisitor.cpp
+++ b/src/ImGuiVisitor.cpp
@@ -666,9 +666,8 @@ void ImGuiVisitor::visit (Source& s)
 
 void ImGuiVisitor::visit (MediaSource& s)
 {
-    ImVec2 _top = ImGui::GetCursorPos();
-    _top.x = 0.5f * ImGui::GetFrameHeight() + ImGui::GetContentRegionAvail().x IMGUI_RIGHT_ALIGN;
-    ImVec2 top = _top;
+    ImVec2 top = ImGui::GetCursorPos();
+    top.x = 0.5f * ImGui::GetFrameHeight() + ImGui::GetContentRegionAvail().x IMGUI_RIGHT_ALIGN;
 
     // Media info
     ImGui::PushTextWrapPos(ImGui::GetCursorPos().x + ImGui::GetContentRegionAvail().x IMGUI_RIGHT_ALIGN);
@@ -684,6 +683,15 @@ void ImGuiVisitor::visit (MediaSource& s)
     if ( !s.failed() ) {
         // icon (>) to open player
         if ( s.playable() ) {
+            ImGui::SetCursorPos(top);
+            std::string decoder = s.mediaplayer()->decoderName();
+            if ( decoder.compare("software") != 0) {
+                decoder = "Hardware decoder\n" + decoder;
+                ImGuiToolkit::Indication(decoder.c_str(), 13, 2);
+            }
+            else
+                ImGuiToolkit::Indication("Software decoder", 14, 2);
+            top.x += ImGui::GetFrameHeight();
             ImGui::SetCursorPos(top);
             std::string msg = s.playing() ? "Open Player\n(source is playing)" : "Open Player\n(source is paused)";
             if (ImGuiToolkit::IconButton( s.playing() ? ICON_FA_PLAY_CIRCLE : ICON_FA_PAUSE_CIRCLE, msg.c_str()))
@@ -695,16 +703,6 @@ void ImGuiVisitor::visit (MediaSource& s)
         if (ImGuiToolkit::IconButton(ICON_FA_FOLDER_OPEN, "Show in finder"))
             SystemToolkit::open(SystemToolkit::path_filename(s.path()));
 
-        // Icon to inform hardware decoding
-        std::string decoder = s.mediaplayer()->decoderName();
-        if ( decoder.compare("software") != 0) {
-            top = _top;
-            top.y += ImGui::GetFrameHeight();
-            ImGui::SetCursorPos(top);
-            decoder = "Hardware decoder\n" + decoder;
-            ImGuiToolkit::Indication(decoder.c_str(), ICON_FA_MICROCHIP);
-        }
-
         ImGui::SetCursorPos(botom);
     }
     else {