From 09630614fb6ba1faab302dba3448e4847788b3ff Mon Sep 17 00:00:00 2001
From: Lars Vogel <Lars.Vogel@vogella.com>
Date: Thu, 18 Jun 2026 15:28:46 +0200
Subject: [PATCH] Unify JFace popup dialog menu icon with workbench view menu

The PopupDialog menu button used the old blue gradient chevron while
workbench view menus already use the three vertical dots glyph. Replace
the JFace popup_menu artwork (enabled and disabled variants) with the
view_menu three-dots design so both controls match.

Fixes https://github.com/eclipse-platform/eclipse.platform.ui/issues/4089
---
 .../jface/dialogs/images/popup_menu.png       | Bin 164 -> 128 bytes
 .../jface/dialogs/images/popup_menu.svg       |  82 ++++++++----------
 .../jface/dialogs/images/popup_menu@2x.png    | Bin 248 -> 212 bytes
 .../dialogs/images/popup_menu_disabled.png    | Bin 168 -> 262 bytes
 .../dialogs/images/popup_menu_disabled.svg    |  82 ++++++++----------
 .../dialogs/images/popup_menu_disabled@2x.png | Bin 241 -> 422 bytes
 6 files changed, 70 insertions(+), 94 deletions(-)

diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu.png b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu.png
index 3e01269324f3178dd3e10ba2de39885fb1722618..b025b28d1815b0c9a12df5529651352760e69107 100644
GIT binary patch
delta 99
zcmZ3&*uXeJCC1&;#WAEJE;(U=sDX(|Pe)hRE|x<IX1i=01!QGwU)qZ_+<5Vch4Cm`
zz#G;s#-n-*EO<1`8j|z_cr?lwk0v^7WE81xj9_5US(yG)(Eh0_0}yz+`njxgN@xNA
D(5xZm

delta 136
zcmV;30C)d@0i*$tB!6p3L_t(|USnVw1%nZUbo4U>cl;-S^U<}T0-zznNfZC)uG#y)
zV(TF=PF=S1KX*(o5t?DZptfJwwf%qY%02&67jFO0ZQJyp!Jvjn%`m{1-!H6L^PgM3
q{6B+09*LS^0B8vLnnwXq008)CHhV@(*wz35002ovPDHLkU;%<x=09=(

diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu.svg b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu.svg
index f77c1fdfa22..83758e542ee 100644
--- a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu.svg
+++ b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu.svg
@@ -5,71 +5,59 @@
    width="16"
    height="16"
    viewBox="0 0 16 16"
-   id="svg4136"
+   id="svg2"
    version="1.1"
-   xmlns:xlink="http://www.w3.org/1999/xlink"
    xmlns="http://www.w3.org/2000/svg"
    xmlns:svg="http://www.w3.org/2000/svg"
    xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
    xmlns:cc="http://creativecommons.org/ns#"
    xmlns:dc="http://purl.org/dc/elements/1.1/">
   <defs
-     id="defs4138">
-    <linearGradient
-       id="linearGradient5954">
-      <stop
-         style="stop-color:#005388;stop-opacity:1"
-         offset="0"
-         id="stop5956" />
-      <stop
-         style="stop-color:#000f6f;stop-opacity:1"
-         offset="1"
-         id="stop5958" />
-    </linearGradient>
-    <linearGradient
-       xlink:href="#linearGradient5954"
-       id="linearGradient5960"
-       x1="9"
-       y1="1043.3621"
-       x2="9"
-       y2="1045.3621"
-       gradientUnits="userSpaceOnUse" />
-  </defs>
+     id="defs4" />
   <metadata
-     id="metadata4141">
+     id="metadata7">
     <rdf:RDF>
       <cc:Work
          rdf:about="">
         <dc:format>image/svg+xml</dc:format>
+        <dc:title />
         <dc:type
            rdf:resource="http://purl.org/dc/dcmitype/StillImage" />
-        <dc:title />
+        <dc:creator>
+          <cc:Agent>
+            <dc:title>Remain BV, W.S. Jongman</dc:title>
+          </cc:Agent>
+        </dc:creator>
+        <dc:publisher>
+          <cc:Agent>
+            <dc:title>Eclipse Foundation</dc:title>
+          </cc:Agent>
+        </dc:publisher>
+        <cc:license
+           rdf:resource="https://www.eclipse.org/legal/epl-2.0/" />
       </cc:Work>
     </rdf:RDF>
   </metadata>
   <g
      id="layer1"
-     transform="translate(0,-1036.3621)">
-    <g
-       id="g4745"
-       transform="translate(-2,0)">
-      <path
-         id="path4686"
-         d="m 6,1042.3621 10,0 -4,4 -1,1 -1,-1 z"
-         style="fill:url(#linearGradient5960);fill-opacity:1;fill-rule:evenodd;stroke:none;stroke-width:1px;stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1" />
-      <path
-         id="path4686-3"
-         d="m 8.5,1043.3621 5,0 -2,2 0,0 -0.5,0.5 -0.5,-0.5 z"
-         style="fill:#78b5c2;fill-opacity:1;fill-rule:evenodd;stroke:none;stroke-width:1px;stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1" />
-      <image
-         y="1036.3621"
-         x="18"
-         id="image5945"
-         xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABHNCSVQICAgIfAhkiAAAAIlJREFU OI1j/P//PwMlgIki3aMGMDAwMDCwoAswChT8Z3CRxK1jz3OG/x8mMMLVY4tGRoGC/3FNjgxSKkJw sWd33jEsqtuPohmnATBDbFNNGQRl+RneP/7IcHj2aQzNDAwMDAz////HiRn48//rZ87/z8Cf/x+n GnwGwAzBJ4/TC8SCgU8HA28AAOgegGgTr8SsAAAAAElFTkSuQmCC "
-         style="image-rendering:optimizeSpeed"
-         preserveAspectRatio="none"
-         height="16"
-         width="16" />
-    </g>
+     transform="translate(0,-1036.3622)">
+    <circle
+       style="opacity:1;fill:#ffffff;fill-opacity:1;fill-rule:nonzero;stroke:#000000;stroke-width:0.5;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4790"
+       cx="8"
+       cy="1039.3622"
+       r="1.75" />
+    <circle
+       style="opacity:1;fill:#ffffff;fill-opacity:1;fill-rule:nonzero;stroke:#000000;stroke-width:0.5;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4790-4"
+       cx="8"
+       cy="1044.3622"
+       r="1.75" />
+    <circle
+       style="opacity:1;fill:#ffffff;fill-opacity:1;fill-rule:nonzero;stroke:#000000;stroke-width:0.5;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4790-4-7"
+       cx="8"
+       cy="1049.3622"
+       r="1.75" />
   </g>
 </svg>
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu@2x.png b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu@2x.png
index e944d0a9ccaec6370f2237843d8b95909751fbb2..7fb11a9c494def015abad65c6905eea4176f22a1 100644
GIT binary patch
delta 184
zcmV;p07w7$0n`DIB!8PpL_t(|UhR~z3dA4~M7P#P`~0r{&)q+X<)u&I-GJ4W!SLYl
zfgrmJSu!)4ZSHZTlwL8$Q%85z{LoVzA%wG(a`oPaj_#_d=ZBY*bAH!aui;QlJwLyk
z-!K2P*8KN}YU=s<<=_k85*Tg52^zzVKqX*Jh{q6ri0}n)2{DW|;RKE0MxYX~Cd6Zi
mKScNfxCBO<aDvA0zxV>+6O$OrQeJ=n0000<MNUMnLSTaFfnD|h

delta 220
zcmcb@_=9nRO8tIM7srr_xa0&0*2M`TJxzfITpW!P`s&&q^!*Px^z+3rVfGb|IgE7M
z8CBdZ%Gdsh-}qG^E+=UFgsy3a8-j!F1^%-qK7W=fG3}79{!H#L=40p1hHR2y<H`Ch
z$&>xBc|-BpB%jlp(;J^2xUrz^6i`o4q<wSJ&3b_{y~vIYHldkT5j;JjUYRMDJRhEZ
zH~#P(WZs?1|GtORn{OuQOmI&=oyc%f@XK}?wh6M1<v`Ap4#v$j5SO-DF)&R3d-2Pe
SYtLpe0D-5gpUXO@geCw8#$%@d

diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled.png b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled.png
index fca23252b1415bf752790601c96c1dc300378ee9..cc26b2487478fb7d761422eb8d40610017fbb1e0 100644
GIT binary patch
delta 234
zcmZ3%*v2$LrJl3EBeIx*f$uN~Gak=hk;1^hAX(xXQ4*Y=R#Ki=l*&+EUaps!mtCBk
zSdglhUz9%kosASwksL^oV`)i7ei1`ZYHng?o`O@DjzYLzu%3cfeqMTRV%~X{<;Fmj
znVv3=Ar`&KfByfsXI5=soqpufRq4eMGiPc^CQC?2)cyN+SxQ=N2h(gTwHUSy8x6ue
zx<0%H;erhY;SYn_FEfZ7IMH!=zQx`TkFUSq_B`J{)X>1d;Lo4G%Vn3}T-<0O)$xPJ
dfw9?2fbovFw}sXL|1zMh44$rjF6*2UngHGQTI~P;

delta 140
zcmV;70CWF_0;mCyB!6#7L_t(|USnVw1%nVAI(V=ji2vim2M-?1N7sh}z=j+;^nd@c
z<Nx;`KLNo&Hb{)1W*At%X<PrMtvmkjJ$mf_o+HQpZ``uu|N4zviPH=Nvlp%GU$kP~
u|M|<;{-3jW6=|Ab0AvW+nnwX~008dgT2Oa4+XMgr002ovP6b4+LSTXu_CcZm

diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled.svg b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled.svg
index 2d2c343c07e..183f875934b 100644
--- a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled.svg
+++ b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled.svg
@@ -5,71 +5,59 @@
    width="16"
    height="16"
    viewBox="0 0 16 16"
-   id="svg4136"
+   id="svg2"
    version="1.1"
-   xmlns:xlink="http://www.w3.org/1999/xlink"
    xmlns="http://www.w3.org/2000/svg"
    xmlns:svg="http://www.w3.org/2000/svg"
    xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
    xmlns:cc="http://creativecommons.org/ns#"
    xmlns:dc="http://purl.org/dc/elements/1.1/">
   <defs
-     id="defs4138">
-    <linearGradient
-       id="linearGradient5954">
-      <stop
-         style="stop-color:#c2c1c1;stop-opacity:1"
-         offset="0"
-         id="stop5956" />
-      <stop
-         style="stop-color:#9ca3aa;stop-opacity:1"
-         offset="1"
-         id="stop5958" />
-    </linearGradient>
-    <linearGradient
-       xlink:href="#linearGradient5954"
-       id="linearGradient5960"
-       x1="9"
-       y1="1043.3621"
-       x2="9"
-       y2="1045.3621"
-       gradientUnits="userSpaceOnUse" />
-  </defs>
+     id="defs4" />
   <metadata
-     id="metadata4141">
+     id="metadata7">
     <rdf:RDF>
       <cc:Work
          rdf:about="">
         <dc:format>image/svg+xml</dc:format>
+        <dc:title />
         <dc:type
            rdf:resource="http://purl.org/dc/dcmitype/StillImage" />
-        <dc:title />
+        <dc:creator>
+          <cc:Agent>
+            <dc:title>Remain BV, W.S. Jongman</dc:title>
+          </cc:Agent>
+        </dc:creator>
+        <dc:publisher>
+          <cc:Agent>
+            <dc:title>Eclipse Foundation</dc:title>
+          </cc:Agent>
+        </dc:publisher>
+        <cc:license
+           rdf:resource="https://www.eclipse.org/legal/epl-2.0/" />
       </cc:Work>
     </rdf:RDF>
   </metadata>
   <g
      id="layer1"
-     transform="translate(0,-1036.3621)">
-    <g
-       id="g4745"
-       transform="translate(-2,0)">
-      <path
-         id="path4686"
-         d="m 6,1042.3621 10,0 -4,4 -1,1 -1,-1 z"
-         style="fill:url(#linearGradient5960);fill-opacity:1;fill-rule:evenodd;stroke:none;stroke-width:1px;stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1" />
-      <path
-         id="path4686-3"
-         d="m 8.5,1043.3621 5,0 -2,2 0,0 -0.5,0.5 -0.5,-0.5 z"
-         style="fill:#bfc7c8;fill-opacity:1;fill-rule:evenodd;stroke:none;stroke-width:1px;stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1" />
-      <image
-         y="1036.3621"
-         x="18"
-         id="image6040"
-         xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABHNCSVQICAgIfAhkiAAAAIFJREFU OI1j/P//PwMlgIki3aMGMDAwMDCwoAscPnSIYLTY2tkx4nQBTPLD998Mf9nY4fjD998YmhkYGBgY caWDw4cO/X/44h0DFw8Xw7cv3xjkJYQwNOM1AGbIrUcvGdTkxLFqZmBgYGD4//8/Xnzo4MH/+OTx uoAYMPDpYOANAABVPlt9ucAHFQAAAABJRU5ErkJggg== "
-         style="image-rendering:optimizeSpeed"
-         preserveAspectRatio="none"
-         height="16"
-         width="16" />
-    </g>
+     transform="translate(0,-1036.3622)">
+    <circle
+       style="opacity:1;fill:#c2c1c1;fill-opacity:1;fill-rule:nonzero;stroke:#9ca3aa;stroke-width:0.5;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4790"
+       cx="8"
+       cy="1039.3622"
+       r="1.75" />
+    <circle
+       style="opacity:1;fill:#c2c1c1;fill-opacity:1;fill-rule:nonzero;stroke:#9ca3aa;stroke-width:0.5;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4790-4"
+       cx="8"
+       cy="1044.3622"
+       r="1.75" />
+    <circle
+       style="opacity:1;fill:#c2c1c1;fill-opacity:1;fill-rule:nonzero;stroke:#9ca3aa;stroke-width:0.5;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4790-4-7"
+       cx="8"
+       cy="1049.3622"
+       r="1.75" />
   </g>
 </svg>
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled@2x.png b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/images/popup_menu_disabled@2x.png
index 618b10096f236e8fb9a38394b7a964e5bcf7eacf..2e330bee8e7984ede6e2e7973ea965dd5c261c40 100644
GIT binary patch
delta 396
zcmey!xQuy%N<C+RM`SSrgKRqpGxk6In99JwAX(xXQ4*Y=R#Ki=l*&+EUaps!mtCBk
zSdglhUz9%kosASwksL^oV`)i7ei1`ZYHng?o`O@DjzYLzu%3cfeqMTRV%~X{<;DyQ
zjGUe>jv*1PZ>Jb~IXeop{lB^WZ1S0=W0&q6(5`o}X%0Q$wZVH`vX<i14_fahJmHXY
zye(c(YB1rhEcaTG?%El9(w582sIU8xZWk2#C34yIw+uPc7rI{VneayDfP|M<%hp+U
zzoZ#ep1Zo}4u_bp@uigcezKn%vbi<(<T$@}oiyw7?boZP%k2HbF=;M8`<df?YuEDR
zgg!j7QP%1iSN%)9@O7tod+rAQ)wVjmHT1<(MIMG9+GbwW<_-&{PS5>u^ZR@GMvl{a
zr2{t4-WHegShYn$FzL{<P^aH&LRZfeeciO5f#IFJ-1&WceR}(Hb(PQPd}p?j{&8MQ
hMF#`EncFg-X_tt`iN*W-fdR<i>FUSovd$@?2>{W?rsx0w

delta 213
zcmV;`04o2c1MvZnB!9R`L_t(|UhR(W3W6{cM}6ZSWe-rFiiGq-u0N*RGVfCAt(F_!
z5QU1EMsN=tkZpK=K|l}$K@e}y0IbpY5CCTV|9^z`FyiYtqB|qXx;B^F{iRS<{^CfU
zOO^{|tV&-8QB;=T-dKfDe&Wb>m$F7<&5CjmPgUXWoc!oFj!uLzXJMQe>p`BMah?fx
zW?e^ooH#OHZrEZK8EcYB6Yo#%yLH?+(j75*9O;diHjcC-eiJ7Mg7|wLAe2*KX2BYo
P00000NkvXXu0mjfjJ#lm