From e195db267c8754c2014d6afebb0f631eccb09063 2017-07-04 07:23:22
From: Alexandre Leroux <aleroux@itlink.fr>
Date: 2017-07-04 07:23:22
Subject: [PATCH] Adds icon for 'Plot' menu

---

diff --git a/gui/include/Visualization/operations/MenuBuilder.h b/gui/include/Visualization/operations/MenuBuilder.h
index d5ee390..cb78fb7 100644
--- a/gui/include/Visualization/operations/MenuBuilder.h
+++ b/gui/include/Visualization/operations/MenuBuilder.h
@@ -27,8 +27,9 @@ public:
     /**
      * Adds a new menu to the current menu
      * @param name the name of the menu
+     * @param icon the icon of the menu (can be null)
      */
-    void addMenu(const QString &name);
+    void addMenu(const QString &name, const QIcon &icon = {});
 
     /// Adds a separator to the current menu. The separator is added only if the menu already
     /// contains entries
diff --git a/gui/resources/icones/plot.png b/gui/resources/icones/plot.png
new file mode 100644
index 0000000000000000000000000000000000000000..bd783f0b1aa15db3d7b0fc906377baf9eb7b8ff1
GIT binary patch
literal 2319
zc$@(b3GnubP)<h;3K|Lk000e1NJLTq004jh004jp1^@s6!#-il00004b3#c}2nYxW
zd<bNS00009a7bBm000XU000XU0RWnu7ytkWu}MThRCt{2om+@jRTze!QOVIH)5=We
zLOTj!M(2Y`M=AtKFz7;~=%O;qZmNqw3W^9qI!MQhsBVJJXNbrkJCmKui_|Pr0~Lyy
zXw;eM;+v?(fB&<-y-s`Y|9fEFG`r=y)_V8;*YR6m7=~dOhGBFZUAYA)nF$;N%mQWr
zhXMx!-M~0t0O$vLfnC53U<<Gj_;L8}#($`?0S*Kf0P}$}ffIo#9{KeH-vgfj9{_8C
zUp+M%ftdy@2i`#%Y$$U28n_EMAw}Vg){F%f0dD~Nh~MZ%=^J1reqR|w!9=9OH<Pc?
zTcuv$A*7#SL|{B{Gw>Vv8@pZF13U)w#HohPWHHhs&)vA1)Cb%FObA~cokI`s8u=U3
zC;bSVAG%^Xfu+Fj<Zr-GX$|%@aAG3x4EY-@M*0qz6RxV7$82Pbox7o8razEAO|6MB
z6L<#jDsWifeEWdUfG?0Ya1%0<w{!S!dyzSX3CLLf2xKTX2bedkYkFW!L%^-TeS!0>
zEf)iS67DVg0hy^g3z_Tpp$j+~Sb@wm4g}P>+EZI2SVWlV+g{*d;3VR%RZK_T-c12C
z4*`!x)>~;V#=r4*1GkfYDt0VzIbr%N4<y!J2~NlFv)>Qg3rtQed<0{VA?sHBOt31o
z@SM!XZ%TI^)f-MtLf+&-%9m?da@84_Ncqp^UgY<8Y;x7^oD2Lyx$^x5oRVB|V#-eP
zY(skWb>J{$#89T3TY#zQC8uF2<tpVX>bJT!4(Z{SDensz^hT$La&!CZ$luW`OBZk_
z<xFs82E9=!v#Pfa>0H&FRp`sU3)$+{2o_WBvtQRYe<WSVte>_Hp2{IMG~?0Fnr)+g
zzxksv26zcwJq!Y;<q#W^o6!||FZJo|YdHwmfUZ8?%ONxr6Ols}ngZXJMQJ+bp`ZS~
zFpJUvtVG}GtMVjIC~iepFKhG2^kXc#t=s*io!)JwVM(HYdmOO7KtoYFIgz$tEJ9bn
zdlT!+X#kL;Ynu9bGNHC0yn%b+c9VX{m)igUFBa%%mHMdBuy`7}<65_8&{zcn%q`GS
zl&(pvuQbbXFH|pi&ec`H006HSXla$+&Y-Upnw7AJGij`f0WK=g(kksk;u=(%1JR9h
zPs$^+G6oofZnSiH295hMrp@0L00%hcvjN$z-N_6B&nlk3D3%Zn?F=wqk<SZ=t2>?N
z70+twHyZ5>aHb-kH+_XPgfEaik}k}l0>LQS8Q?_6Jo<o7188XigSapHbWwrOG(`rO
ziMu;LLwdD_@PXplxzSWOQe=Q*6nS)d=WirmDxN($nhHmX3@}TP$9DmAGzm?8%#NnQ
zks<@kP~@>GgpdZY8`(7L!pxXLTqrWYp^7{#*y8?db$srSs45&NGJs~9a;K*T%_2&x
z)Qaak^Yoobbgh%h4g<RtdGv)4(lA`>WPDT=4ip(c6O3VR2q6t)kK=QdHGn3nb|>up
z)8hI9WshE7WPpA}9u_ife_Ve;|B9-@fg-2&D)N|uM9eCS@3QD~E&khT;3JPbKlb>4
z!M%6yQ*S{iGQciH9!DTYq06FKL38D!Ncp<r8jWmsR22>s{RHe#<S{FNjwaEKyc8~M
zk1523A_HjN=&FTy$<1+!XMc{S!jU2aY*geiFNBZ=p;_Y9)N>Tt8NhWUa0VS5AFl<P
zjWpR8(uxeQ54m8;h3UXiVdOM|1&U`iOUF@Yw<G8iMLw7K3TX&E!10dH?x4bb0MPAh
zxy)BcL(tU2htX6hXlH=6ihRx^d?fBVa)sjAccZCL(9QtAB3G%o=F_-nSW1os&Q?7C
zRxBYJ+8F>O?!PP7(b)?_%N5UlPfd1Goq%p5MULZ6RX7?p6<I%Vt*2TAl>P?yf_=^C
zUzN~Q!2ov`=xCJ&soAhu8y{$CVtu&{FazD&ujyP@f+^?<xRta~L3116sR9jE`kD0R
zA#{aId2_~+7AaM5$_vod%VzxVKFY!@WS^VnckbK_I#V(k_z9f>Zpa}t6px`R>@Vof
z7u1O^Bo=_CK7OVGn?QZ&LAE<<3jGc>{p5{;Hf-(?aCH`?fw=>H!5^Vn%f$=P-#NIR
z^uyyQOh9hY)|%kX9AYCdAN^Nw0J&MMF=%_f|D;va`%7>(<!(w@lS6E5)=<s_tEuOw
za3T6Pe+U^lHFA}70<xW{j0s*s;_cUgE~LZfbI<VmG~>FQ{g{IsY$#JH8>s$f(#hzD
zdL3ti#gxBcvmc4KU&UeK1?Yl4cC-m@!LMT0Q<0c?$imLkgl+haFhQA{v__^E$rm+i
zDzc=jS@bd!jG*l8nj_QeNEp==qlIIE`$^OAjW<CTvI-cgk~RbPAXlt}G(!)t0=d^H
zWute53838Vnxmy1NRRwR;2hjz>vSWdjb%tU`)?y?tF;MY&CD$Wo*?e(*YC*Ylby&U
z%^qY#(T$wXpN{(}-JGlgE=3oUp%$EqoHEY+menC;{}RMvD*uL(Z%Wc^j4=}ob0rdW
zBzNQaqygYT;Bcg|%mhlNAsZ;m&K%}UpY#7E-AR}UN^=?#Q$Kg3x}+_@RkR+4nh89)
zkm?#?Gjh~xv{8(iAOI&L*Qu6SZx5LUkwY6x(4BEK69i&1a--(k=x!_ZP2VDuS3bwJ
z%mjfs7&$@nIC7FUWXEwkGBmvbI5ME-TAH9Pd-SFtCza<S*R0G&E?YQc_;2Hf(*P2F
z^$#Qn<91{}{zha6)0e~z%Q6W$_a9?<yRkEdO{11y0wXMD0>eyTm<bHb1O{dT12chv
znZUqIU|=RNFcTP<2@K2x24(_-xF&cxVL1#BVwqrD%0d`^#4y42NlRdO6Ve1L6KgjD
z5yAx4_%+AJ1UF^SYeb-o39RuOhQ<UpX3}Ux#>E8I_zkC>39RuugjOb4kyVkAoJqhd
pz;D2{nY9>(VHk#C7=}?R{ssq@VDHBZ4Ql`Z002ovPDHLkV1nbSE))O&

literal 0
Hc$@<O00001

diff --git a/gui/resources/sqpguiresources.qrc b/gui/resources/sqpguiresources.qrc
index ff211a8..6d9479d 100644
--- a/gui/resources/sqpguiresources.qrc
+++ b/gui/resources/sqpguiresources.qrc
@@ -3,6 +3,7 @@
 		<file>icones/delete.png</file>
         <file>icones/openInspector.png</file>
         <file>icones/next.png</file>
+        <file>icones/plot.png</file>
         <file>icones/previous.png</file>
     </qresource>
 </RCC>
diff --git a/gui/src/Visualization/operations/GenerateVariableMenuOperation.cpp b/gui/src/Visualization/operations/GenerateVariableMenuOperation.cpp
index dec20d8..4e93dae 100644
--- a/gui/src/Visualization/operations/GenerateVariableMenuOperation.cpp
+++ b/gui/src/Visualization/operations/GenerateVariableMenuOperation.cpp
@@ -21,7 +21,7 @@ struct GenerateVariableMenuOperation::GenerateVariableMenuOperationPrivate {
     void visitRootEnter()
     {
         // Creates the root menu
-        m_MenuBuilder.addMenu(QObject::tr("Plot"));
+        m_MenuBuilder.addMenu(QObject::tr("Plot"), QIcon{":/icones/plot.png"});
     }
 
     void visitRootLeave()
diff --git a/gui/src/Visualization/operations/MenuBuilder.cpp b/gui/src/Visualization/operations/MenuBuilder.cpp
index 23b8e01..3d21803 100644
--- a/gui/src/Visualization/operations/MenuBuilder.cpp
+++ b/gui/src/Visualization/operations/MenuBuilder.cpp
@@ -12,10 +12,10 @@ MenuBuilder::MenuBuilder(QMenu *menu)
     }
 }
 
-void MenuBuilder::addMenu(const QString &name)
+void MenuBuilder::addMenu(const QString &name, const QIcon &icon)
 {
     if (auto currMenu = currentMenu()) {
-        m_Menus.push(currMenu->addMenu(name));
+        m_Menus.push(currMenu->addMenu(icon, name));
     }
     else {
         qCCritical(LOG_MenuBuilder()) << QObject::tr("No current menu to attach the new menu");