From 628c78409c0a93675fa976ce92c53244e93a86ff Mon Sep 17 00:00:00 2001
From: David Mudrak <david.mudrak@gmail.com>
Date: Fri, 12 Nov 2010 08:52:55 +0000
Subject: [PATCH] New icon for portfolio export and improving its position

The old icon was considered "temporary" (according to Penny) and it was
not very clear. Hopefully this one is a bit better.
---
 lib/portfoliolib.php          |   6 +++---
 pix/t/portfolio.gif           | Bin 642 -> 0 bytes
 pix/t/portfolioadd.gif        | Bin 0 -> 140 bytes
 pix/t/portfoliono.gif         | Bin 618 -> 0 bytes
 theme/standard/style/core.css |   1 +
 5 files changed, 4 insertions(+), 3 deletions(-)
 delete mode 100644 pix/t/portfolio.gif
 create mode 100644 pix/t/portfolioadd.gif
 delete mode 100644 pix/t/portfoliono.gif

diff --git a/lib/portfoliolib.php b/lib/portfoliolib.php
index c92242990b9..19e00cd2acb 100644
--- a/lib/portfoliolib.php
+++ b/lib/portfoliolib.php
@@ -334,7 +334,7 @@ class portfolio_add_button {
 
         $formoutput = '<form method="post" action="' . $CFG->wwwroot . '/portfolio/add.php" id="portfolio-add-button">' . "\n";
         $formoutput .= html_writer::input_hidden_params($url);
-        $linkoutput = '<a title="'.$addstr.'" href="' . $url->out();
+        $linkoutput = '<a class="portfolio-add-link" title="'.$addstr.'" href="' . $url->out();
 
         switch ($format) {
             case PORTFOLIO_ADD_FULL_FORM:
@@ -344,11 +344,11 @@ class portfolio_add_button {
             break;
             case PORTFOLIO_ADD_ICON_FORM:
                 $formoutput .= $selectoutput;
-                $formoutput .= "\n" . '<input type="image" src="' . $OUTPUT->pix_url('t/portfolio') . '" alt=' . $addstr .'" />';
+                $formoutput .= "\n" . '<input class="portfolio-add-icon" type="image" src="' . $OUTPUT->pix_url('t/portfolioadd') . '" alt=' . $addstr .'" />';
                 $formoutput .= "\n" . '</form>';
             break;
             case PORTFOLIO_ADD_ICON_LINK:
-                $linkoutput .= '"><img src="' . $OUTPUT->pix_url('t/portfolio') . '" alt="' . $addstr .'" /></a>';
+                $linkoutput .= '"><img class="portfolio-add-icon" src="' . $OUTPUT->pix_url('t/portfolioadd') . '" alt="' . $addstr .'" /></a>';
             break;
             case PORTFOLIO_ADD_TEXT_LINK:
                 $linkoutput .= '">' . $addstr .'</a>';
diff --git a/pix/t/portfolio.gif b/pix/t/portfolio.gif
deleted file mode 100644
index 0a69acb94551cee570026322fc46b3567a133deb..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 642
zcmZ?wbhEHb6krfw_*TZiz#zoHD#ySk&%mL`z@fyzt;)co#=xV&zz0N{EFy+X5=#8i
z(t<KFGKvgxN(_ps41&t8${LJHdS=RI!K!A#%1#XOZVX0t-l9R=rq1!kuJO*EGR~gb
zmLAd89ucPAxi$gCegO{75h_kW`Od-l4xwfGSq#Rx3=wI$kp<eJnXL&0J+Vd2u|+*e
zMZHPI{mI4s8dDfFrZH$vWAN<d@a*SIt(=}&ImKrJx91cF)42>`lO;l@h~+lSv|hkq
zvw*>2F@w_*2G=DF9?KYfmoc<;<tNNnt?6Bvx==HDiQ0t8$&qV0{5LX8nqD<!X2z5`
zSySfZ`y61%-=dSYPiXFv;>^7g%U0E9o#I}(rfKDx=2dH()^2EByP<vU#@6*4+c#|P
z+_0r>^R}LCJA1b5oVI7*q&@p4?>jJM-$5Xnw(r2ggNK(KJU0E{v4w|^%|CK{*5Tvx
zj~!or?BtxbSIQG!Sf4$&9vH-UfZ{(v=c3falGGH1^30M91$R&100zaMEUa7%WehqX
zji88fU_aGR*3{h6+Qg}7s3d9W=wvCGSSr-a)Txx3mY$KB>88Zf%&cgj<S*%J;v=qU
z$1T~!tfr@wSFk-r&fJ5mnN3kgDLg<@GB7GWi?5koQBuRx%i2EJU0rfY6N8|Lu&j)j
zWJGaE*oJ0NX$hq_Z=_UI8K*8$%E>P(%J-AE*xSUS7}xRPkgE4Iy@VrcDh)Uo_i9Pz
zTsg5J)klV-Szzml3&|Zlk}ee~8I#ZYH3}$qr7~=CZa>8~)nSdmMi<vMwV)L%Y|=e0
LoO5hsV6X-N|AN=L

diff --git a/pix/t/portfolioadd.gif b/pix/t/portfolioadd.gif
new file mode 100644
index 0000000000000000000000000000000000000000..3fdabc32a41f3832bc68d4625ee2988d3622a926
GIT binary patch
literal 140
zcmZ?wbhEHb6krfwIK;{T1V$;wcIkFH*R*18C6s(BfA{VkkW~E1!pOkD&Y%NQ0aDAr
z99Xe|=c3O{%flQcOrFe|EN;0D<~2u5i^Lk2DP^)8ioCP3Vd9sc4|g6{OxSq0pxHaY
pL*-V&+0>PvLNYBjuR5Wyc+r}e9B!6(Y5Sv#x9n`%A<fBP4FJf;H3|R#

literal 0
HcmV?d00001

diff --git a/pix/t/portfoliono.gif b/pix/t/portfoliono.gif
deleted file mode 100644
index d53cf983c3e6b372b4532f405ce5dc664708ccd2..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 618
zcmZ?wbhEHb6krfw_*Txqz#zoHD#ySk&%mL`z@fyzt;)co#=xV&zz0N{EFy+X5=#8i
z(t<KFGKvgxN(_ps41&t8${LJHdS=RI!K!A#{}~vRofv*FGyP;?m3L#f!_Q-6=Peq<
z{Xk5{)H&YRHQw1%#@SQb(j(g1Bf``>*CwFYFTlYmDBn3a-yyV2KZ_wEEjO}2J2bO3
zwx~I_s3)noKe@PHV+w=DGzQIS44%Clp8cGumD4jTr}#|Z_MF0CI+r1AvP9?<vD}84
z)(aSH7BDz0W^h`<;JSptV;O_*GKRLU{Dk?cHN7iS7iuOiQJXM1IdUzB|3-#M)2pV;
z%$PDKYs#E_p92i}TXfR)3C&$noVizG*{a&CQ`{@pG_73IylQRJ+6}F1H?*(a*t&jW
z`-aV(8@9A<-qy2iXU~?M)AsC}v}fPseFvuOI|xM6_8nMw@bHp@$EF`Vw(#(=`A3e=
zI(&TovE$2+ot)G5N_oNy>$B(9149`XPy~h=7ehIN4oCqgP8isaH<ULux3o5kD#)mJ
zcgt&NGBz`78K~zM&PY|Z@Z@S{QZrP~EhsK7@K>>%*R;S$JuN*WGb_toou`>aO-w94
zHYzGEF*%37nQf0)M4+r}P;^2zUo(fAq(qpkxtVNCN|8V_r<$C!td*0qm26U(&|@`y
zSvwbF6IWZ=ybn$6r@W?pU~pnR*~Rzwok+kl9gcq4b5<r3L|(B>;d9F}xp?5w@g}~t
pUOOxfE^%d74_@WMQpxVX!X@m|vg3jx_gt&+2q~#8MF9#7)&PBj&)Wb1

diff --git a/theme/standard/style/core.css b/theme/standard/style/core.css
index 498d8d61dbd..cbc6375aa43 100644
--- a/theme/standard/style/core.css
+++ b/theme/standard/style/core.css
@@ -442,3 +442,4 @@ table#tag-management-list {margin: 10px auto;width: 80%;}
 
 /* Portfolio */
 .path-portfolio .exportsummary {margin:0px auto 1em auto;}
+.portfolio-add-icon {margin-left:5px;}
-- 
2.36.1