Bug 518199: revise styling of bookmark editing panel [r=gavin.sharp]
--- a/mobile/chrome/content/browser-ui.js
+++ b/mobile/chrome/content/browser-ui.js
@@ -886,16 +886,20 @@ var BookmarkHelper = {
let self = this;
setTimeout(function() {
self._editor.init(itemId);
self._editor.startEditing();
}, 0);
},
+ save: function BH_save() {
+ this._editor.stopEditing(true);
+ },
+
close: function BH_close() {
BrowserUI.updateStar();
// Note: the _editor will have already saved the data, if needed, by the time
// this method is called, since this method is called via the "close" event.
this._editor.parentNode.removeChild(this._editor);
this._editor = null;
--- a/mobile/chrome/content/browser.css
+++ b/mobile/chrome/content/browser.css
@@ -57,16 +57,17 @@ autocompleteresult[value=""]:first-child
}
autocompleteresult[value=""]:not(:first-child) {
display: none;
}
placeitem {
-moz-binding: url("chrome://browser/content/bindings.xml#place-item");
+ background-color: transparent;
}
placeitem[type="folder"] {
-moz-binding: url("chrome://browser/content/bindings.xml#place-folder");
}
placelist {
-moz-binding: url("chrome://browser/content/bindings.xml#place-list");
--- a/mobile/chrome/content/browser.xul
+++ b/mobile/chrome/content/browser.xul
@@ -310,17 +310,20 @@
<vbox>
<button id="bookmark-popup-edit" label="&bookmarkEdit.label;" oncommand="BookmarkHelper.edit(); BrowserUI.hideControls();"/>
<button id="bookmark-popup-remove" label="&bookmarkRemove.label;" oncommand="removeBookmarksForURI(getBrowser().currentURI);"/>
</vbox>
</vbox>
<vbox id="bookmark-container" hidden="true" class="panel-dark window-width">
<hbox id="bookmark-form" align="start">
- <image id="bookmark-image" src="chrome://browser/skin/images/starred-default-64.png"/>
+ <image id="bookmark-image"/>
+ </hbox>
+ <hbox pack="center">
+ <button class="button-dark" label="&editBookmarkDone.label;" oncommand="BookmarkHelper.save();"/>
</hbox>
</vbox>
<box id="panel-container" hidden="true" class="panel-dark">
<box id="panel-controls" oncommand="BrowserUI.switchPane(event.target.getAttribute('linkedpanel'));">
<toolbarbutton id="tool-panel-close" class="page-button button-image" command="cmd_panel"/>
<toolbarbutton id="tool-addons" type="radio" group="1" class="panel-button button-image" linkedpanel="addons-container"/>
<toolbarbutton id="tool-downloads" type="radio" group="1" class="panel-button button-image" linkedpanel="downloads-container"/>
--- a/mobile/themes/hildon/browser.css
+++ b/mobile/themes/hildon/browser.css
@@ -709,27 +709,31 @@ box[type="documenttab"]:only-child .docu
#newtab-button:hover:active {
list-style-image: url("images/newtab-active-64.png");
}
/* bookmark editor ------------------------------------------------------- */
#bookmark-container {
padding: 1mm; /* core spacing */
+ -moz-box-shadow: black 0 0.25mm 0.25mm;
}
#bookmark-form {
- background-color: #fff;
padding: 1mm; /* core spacing */
}
-#bookmark-form button.bookmark-remove {
+#bookmark-form .bookmark-controls {
display: none;
}
+#bookmark-image {
+ list-style-image: url("images/star-40.png");
+}
+
/* Identity popup -------------------------------------------------------- */
#identity-container {
border-bottom: 0.1mm solid grey;
}
#identity-popup-container {
padding: 1mm; /* core spacing */
}
new file mode 100644
index 0000000000000000000000000000000000000000..a597628043aab72421a09b115f818e78b30f3f1a
GIT binary patch
literal 2203
zc$@*72xRw(P)<h;3K|Lk000e1NJLTq001Ze001Zm1^@s6jQ+T70000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBU)JxN4CRCwCNnQd%aR~g68xz~>4*sh)B
zJ#qXNH|f&C5J&}Obj=2gGN?LDn#LE<q*ZFC6<Rj7eV{^2nnWpGGa-!+pdisssQXe_
zr$M()n;OtcT2XCfTh=UZYt|%P^U@@?W8b&K^E~I+buy9$oPecA_xj$Odw%{uulI(t
z|MN!(vEq4cjn}JDwq_q!sW-6g1={MmxAnQ_e(_!0f8vR!PuO<4AM`2K>NESdQV&18
zR6xg(fSTt0Mk+P^5-#rdS3LxPXO;siKBj4b^>#5wxRz(0`RSLuURR~J+*SGT!{1Np
zy03P54fy{q$Q>t(n|tQwCdW-<T$sjHF*4HM3&*yCiVy-RW%J7Bl+BW%n^&@u71Xry
z>SVH|%I7PuckF3qThpw$xnV0Di-L;PCEtGyM2x-m+Mo9eCj)Ax!RD9)%AG#_%6>ez
za*0`gz^aCZ`e-~Jz0<L0V8I+%7F4{-;_=9xcuv>-RWd1-v-Jgx_Vm03>MekB8CcII
z!x%#)dwR|+qoPpituNqhp-?E&(h}ch+f$^TT+m?xTx}U6ERl$93x$Hsx*oV~as68z
zT&aMu^XK1I>ZM^ls{qJ^LCNBP_5S^=R1vAj9C=+~e|-S4#x+5vcG{4+j7p6y4O}dk
z1#8bS(->gMWV{ov)m_2bf^I^rTESbjx*(WeStVGRI>tW!c<>-*u5jjIDFY0^Kt;h!
zSlfjC1yFGU;_*5g8Gip@EWY_!-4((=#nuBCu?a8(41OU?3qdYewxr{)t5XneunOv_
zkMB?3@r64Y6NyAqEEc`p>n-<JR#qnSe{6w54kU!kUV}0?pD;pX$nBb5$tnUn3(93O
znd^pO%=h(O?wy^TOTGDKPv5CifB9r!a2VZSDMR=x06mL<wa<4Qezf!MZC?(D!);|{
zWj?>pU$>Z_%h*z-LBPIkUIld~m0D(@#2_HmM3JFnL1Elw30CgA3$9i7<7JHVG8JLG
zu#9C{S<|#qgM&jCjvYI3=IGHEeut;@aJcdQ{{DA=>+zIV$}}?wgP|HMM=|C&DN=+I
zg6HGNQ8A|gm%_o{I0Pw=Bdp8a2$N4Wb9>qu*ETTSRS(@LNWfh467<BxwM$LmO?yGX
zb9sqcG1Eqd&;Gozq3(;8@e#OgiXb`1Fx6qJ-~7drP?i)79dd7uK*+rSCa9~VBqRXC
zNC5y9a33#Nxm1vpbkTR0I}FgDXJg}&7vn8E_QB0Z0C2o`tf<GrJJ5gnSIv=ztpsUJ
zNtFmrr$yjyg+3)HYt2RRCq!DR2FW7<0Z#LQN(fg$DoH>O+{;D4sH&^1osEu8oKLoX
z>xY<w05|dP<{nIzX#a<&o{vVu9hT7tnRrdb-%|Ail@oNkgv(N5Naf~9IlP8zehJ8Z
zMGAVPdhp%lLbz6_3S)0?Z2RT`1^4MngBeaBu@65u*%gmBebvmpN4*tY$<^f)pQN6p
z)XS+jOBVofaU4cD8b<|v5>6|JRm%a`%XD`GSdtA7kN&M~)4e}ZZk=4s<XDA}AH4Tc
zS0WMKX6F6@Sv$J8ol)!+XK#kwi?>852?y$NKkWOJdI}C;F*$U1oDB_K`D@#zujS#U
z{*%!bA*1hKdhuvWOVf5Edk*T}psH`GC?l{^kK9Ra^pk07D<LZi?nO9FuLHOU8yviH
zcGK;hk1MxM{Ws4f*yLlE-swKFabx4%x%BG**)L-aT?=u?xuyVo(s)26MIO#qP!Eq|
ziP7CL*8kD)TkY+gKUT_3m-5hrwK4Y2#TO1I6WZ6asXswXj!~e?n5tL;m{(8k%dE`<
z1^}*^UG5)yW6PH9Pbs*WQilb+!Bk)0P@kq(so)0)9pX&i02WjhqP8ZtU|L9?hhj{L
zqEtl}Uz?a3gL}@y^UTazJr@V!iS`I)E$411iyKH4nVCwtf;)?j@#30?h1o-zG&V(>
z@t&)3;A#P@67kk8j&qTcK~svz@)0O<r~jtA3Cyc9S9T1>-Llx$)+@lZd$SM>1wAdf
zg><1c<?9<)<wPq0%D7|%Sk%&>6UmOmS!F1clw;f4DM+=BY8)w~pa{~hzRTLjRHxv|
zr5s1dMSUFGr2rLc-I`XWL6SJ7CQ0zXfjtSqprlai$_KlGg2bSr>xzIy!NPE?8qNjP
zkMR_MIk}iqt$+HdCpW8FTe5-;4?cK*2Zu{>tfI0(nGA`7HBE0|8uU|ynVg)x%(%&F
zs%zV1OSDzzRwKT+Ia79Yd}WIQyk4q;f%>{ovv6)u|CQYU$_Xgta<{=Y!fxC!2KMZI
z^0B)5w%v7g$=!SQ?0;<LMrttM+VIEm!csOpBE%EXRVlcd&pk6!y;m3JkBijIvm!Is
zEsTX8VH%f(ksB}%9s0>ZxCX{$7IEl#7^fH(!DIXOJ^F*`>4_o7SrCprFKp|&u*^}|
zj*DD&Y96lHajSPTtNH9SvKK@){ffw@UKU37BG^BjW4}Lg1kQaMY%?rOui+dA+n-5W
zn!KTxgKa;4y!!|MI*vUHa1&r-V3WA|&QbwA+wbc59x9ewI16Cs#2;Qf)CI@C1NJ$3
zyJgQRID9O~Dabr1e78|3x_0M@6Wv|5^(iT6S)Yi#dw1jH94G~_=9h<mb}*NDU0mqx
zK81_q!EU~OOVgOY7;>ql)Zi;Y|9G<d*OoaXUVZiCZ{T~p6hQB;T{|ABsR=!xZtnl=
z`@z%_$b}-<?BZkakPl!4T3cK7z^+~QJ%Vej1h|@M1|PL@YVKdl52m$8$YFB*-chgj
dzajo5zyQE`2upQ1nP30_002ovPDHLkV1lv6Ju3hJ
deleted file mode 100644
index 0935f835c458c0b2e68fa87408b164f5a8504dfc..0000000000000000000000000000000000000000
GIT binary patch
literal 0
Hc$@<O00001
--- a/mobile/themes/hildon/jar.mn
+++ b/mobile/themes/hildon/jar.mn
@@ -16,24 +16,24 @@ chrome.jar:
images/weave.png (images/weave.png)
images/aboutBackground.jpg (images/aboutBackground.jpg)
images/arrowright-16.png (images/arrowright-16.png)
images/arrowup-16.png (images/arrowup-16.png)
images/arrowleftdark-16.png (images/arrowleftdark-16.png)
images/arrowrightdark-16.png (images/arrowrightdark-16.png)
images/ratings-18.png (images/ratings-18.png)
images/favicon-default-30.png (images/favicon-default-30.png)
- images/starred-64.png (images/starred-64.png)
+ images/star-40.png (images/star-40.png)
+ images/star-24.png (images/star-24.png)
images/throbber.png (images/throbber.png)
images/navigation-magnifier-30.png (images/navigation-magnifier-30.png)
images/folder-30.png (images/folder-30.png)
images/stop-30.png (images/stop-30.png)
images/reload-30.png (images/reload-30.png)
images/go-30.png (images/go-30.png)
- images/star-24.png (images/star-24.png)
images/addons-default-64.png (images/addons-default-64.png)
images/addons-active-64.png (images/addons-active-64.png)
images/back-default-64.png (images/back-default-64.png)
images/back-active-64.png (images/back-active-64.png)
images/back-disabled-64.png (images/back-disabled-64.png)
images/bookmark-default-64.png (images/bookmark-default-64.png)
images/bookmark-active-64.png (images/bookmark-active-64.png)
images/bookmarks-30.png (images/bookmarks-30.png)
--- a/mobile/themes/wince/browser-high.css
+++ b/mobile/themes/wince/browser-high.css
@@ -281,16 +281,21 @@ box[type="documenttab"] .documenttab-clo
#newtab-button {
list-style-image: url("images/newtab-default-64.png");
}
#newtab-button:hover:active {
list-style-image: url("images/newtab-active-64.png");
}
+/* bookmark editor ------------------------------------------------------- */
+#bookmark-image {
+ list-style-image: url("images/star-40.png");
+}
+
/* Popup Icons */
#identity-popup-icon {
padding: 0;
list-style-image: url("chrome://browser/skin/images/identity-64.png");
}
#identity-container[mode="verifiedDomain"] > hbox > #identity-popup-icon {
list-style-image: url("chrome://browser/skin/images/identitySSL-64.png");
--- a/mobile/themes/wince/browser-low.css
+++ b/mobile/themes/wince/browser-low.css
@@ -281,16 +281,21 @@ box[type="documenttab"] .documenttab-clo
#newtab-button {
list-style-image: url("images/newtab-default-36.png");
}
#newtab-button:hover:active {
list-style-image: url("images/newtab-active-36.png");
}
+/* bookmark editor ------------------------------------------------------- */
+#bookmark-image {
+ list-style-image: url("images/star-24.png");
+}
+
/* Popup Icons */
#identity-popup-icon {
padding: 0;
list-style-image: url("chrome://browser/skin/images/identity-36.png");
}
#identity-container[mode="verifiedDomain"] > hbox > #identity-popup-icon {
list-style-image: url("chrome://browser/skin/images/identitySSL-36.png");
--- a/mobile/themes/wince/browser.css
+++ b/mobile/themes/wince/browser.css
@@ -441,25 +441,25 @@ box[type="documenttab"][selected="true"]
}
box[type="documenttab"]:only-child .documenttab-close {
display: none;
}
/* bookmark editor ------------------------------------------------------- */
#bookmark-container {
- padding: 1.1mm; /* core spacing */
+ padding: 1mm; /* core spacing */
+ -moz-box-shadow: black 0 0.25mm 0.25mm;
}
#bookmark-form {
- background-color: #fff;
- padding: 1.1mm; /* core spacing */
+ padding: 1mm; /* core spacing */
}
-#bookmark-form button.bookmark-remove {
+#bookmark-form .bookmark-controls {
display: none;
}
/* Identity popup -------------------------------------------------------- */
#identity-container {
border-bottom: 0.1mm solid grey;
}
new file mode 100644
index 0000000000000000000000000000000000000000..a597628043aab72421a09b115f818e78b30f3f1a
GIT binary patch
literal 2203
zc$@*72xRw(P)<h;3K|Lk000e1NJLTq001Ze001Zm1^@s6jQ+T70000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBU)JxN4CRCwCNnQd%aR~g68xz~>4*sh)B
zJ#qXNH|f&C5J&}Obj=2gGN?LDn#LE<q*ZFC6<Rj7eV{^2nnWpGGa-!+pdisssQXe_
zr$M()n;OtcT2XCfTh=UZYt|%P^U@@?W8b&K^E~I+buy9$oPecA_xj$Odw%{uulI(t
z|MN!(vEq4cjn}JDwq_q!sW-6g1={MmxAnQ_e(_!0f8vR!PuO<4AM`2K>NESdQV&18
zR6xg(fSTt0Mk+P^5-#rdS3LxPXO;siKBj4b^>#5wxRz(0`RSLuURR~J+*SGT!{1Np
zy03P54fy{q$Q>t(n|tQwCdW-<T$sjHF*4HM3&*yCiVy-RW%J7Bl+BW%n^&@u71Xry
z>SVH|%I7PuckF3qThpw$xnV0Di-L;PCEtGyM2x-m+Mo9eCj)Ax!RD9)%AG#_%6>ez
za*0`gz^aCZ`e-~Jz0<L0V8I+%7F4{-;_=9xcuv>-RWd1-v-Jgx_Vm03>MekB8CcII
z!x%#)dwR|+qoPpituNqhp-?E&(h}ch+f$^TT+m?xTx}U6ERl$93x$Hsx*oV~as68z
zT&aMu^XK1I>ZM^ls{qJ^LCNBP_5S^=R1vAj9C=+~e|-S4#x+5vcG{4+j7p6y4O}dk
z1#8bS(->gMWV{ov)m_2bf^I^rTESbjx*(WeStVGRI>tW!c<>-*u5jjIDFY0^Kt;h!
zSlfjC1yFGU;_*5g8Gip@EWY_!-4((=#nuBCu?a8(41OU?3qdYewxr{)t5XneunOv_
zkMB?3@r64Y6NyAqEEc`p>n-<JR#qnSe{6w54kU!kUV}0?pD;pX$nBb5$tnUn3(93O
znd^pO%=h(O?wy^TOTGDKPv5CifB9r!a2VZSDMR=x06mL<wa<4Qezf!MZC?(D!);|{
zWj?>pU$>Z_%h*z-LBPIkUIld~m0D(@#2_HmM3JFnL1Elw30CgA3$9i7<7JHVG8JLG
zu#9C{S<|#qgM&jCjvYI3=IGHEeut;@aJcdQ{{DA=>+zIV$}}?wgP|HMM=|C&DN=+I
zg6HGNQ8A|gm%_o{I0Pw=Bdp8a2$N4Wb9>qu*ETTSRS(@LNWfh467<BxwM$LmO?yGX
zb9sqcG1Eqd&;Gozq3(;8@e#OgiXb`1Fx6qJ-~7drP?i)79dd7uK*+rSCa9~VBqRXC
zNC5y9a33#Nxm1vpbkTR0I}FgDXJg}&7vn8E_QB0Z0C2o`tf<GrJJ5gnSIv=ztpsUJ
zNtFmrr$yjyg+3)HYt2RRCq!DR2FW7<0Z#LQN(fg$DoH>O+{;D4sH&^1osEu8oKLoX
z>xY<w05|dP<{nIzX#a<&o{vVu9hT7tnRrdb-%|Ail@oNkgv(N5Naf~9IlP8zehJ8Z
zMGAVPdhp%lLbz6_3S)0?Z2RT`1^4MngBeaBu@65u*%gmBebvmpN4*tY$<^f)pQN6p
z)XS+jOBVofaU4cD8b<|v5>6|JRm%a`%XD`GSdtA7kN&M~)4e}ZZk=4s<XDA}AH4Tc
zS0WMKX6F6@Sv$J8ol)!+XK#kwi?>852?y$NKkWOJdI}C;F*$U1oDB_K`D@#zujS#U
z{*%!bA*1hKdhuvWOVf5Edk*T}psH`GC?l{^kK9Ra^pk07D<LZi?nO9FuLHOU8yviH
zcGK;hk1MxM{Ws4f*yLlE-swKFabx4%x%BG**)L-aT?=u?xuyVo(s)26MIO#qP!Eq|
ziP7CL*8kD)TkY+gKUT_3m-5hrwK4Y2#TO1I6WZ6asXswXj!~e?n5tL;m{(8k%dE`<
z1^}*^UG5)yW6PH9Pbs*WQilb+!Bk)0P@kq(so)0)9pX&i02WjhqP8ZtU|L9?hhj{L
zqEtl}Uz?a3gL}@y^UTazJr@V!iS`I)E$411iyKH4nVCwtf;)?j@#30?h1o-zG&V(>
z@t&)3;A#P@67kk8j&qTcK~svz@)0O<r~jtA3Cyc9S9T1>-Llx$)+@lZd$SM>1wAdf
zg><1c<?9<)<wPq0%D7|%Sk%&>6UmOmS!F1clw;f4DM+=BY8)w~pa{~hzRTLjRHxv|
zr5s1dMSUFGr2rLc-I`XWL6SJ7CQ0zXfjtSqprlai$_KlGg2bSr>xzIy!NPE?8qNjP
zkMR_MIk}iqt$+HdCpW8FTe5-;4?cK*2Zu{>tfI0(nGA`7HBE0|8uU|ynVg)x%(%&F
zs%zV1OSDzzRwKT+Ia79Yd}WIQyk4q;f%>{ovv6)u|CQYU$_Xgta<{=Y!fxC!2KMZI
z^0B)5w%v7g$=!SQ?0;<LMrttM+VIEm!csOpBE%EXRVlcd&pk6!y;m3JkBijIvm!Is
zEsTX8VH%f(ksB}%9s0>ZxCX{$7IEl#7^fH(!DIXOJ^F*`>4_o7SrCprFKp|&u*^}|
zj*DD&Y96lHajSPTtNH9SvKK@){ffw@UKU37BG^BjW4}Lg1kQaMY%?rOui+dA+n-5W
zn!KTxgKa;4y!!|MI*vUHa1&r-V3WA|&QbwA+wbc59x9ewI16Cs#2;Qf)CI@C1NJ$3
zyJgQRID9O~Dabr1e78|3x_0M@6Wv|5^(iT6S)Yi#dw1jH94G~_=9h<mb}*NDU0mqx
zK81_q!EU~OOVgOY7;>ql)Zi;Y|9G<d*OoaXUVZiCZ{T~p6hQB;T{|ABsR=!xZtnl=
z`@z%_$b}-<?BZkakPl!4T3cK7z^+~QJ%Vej1h|@M1|PL@YVKdl52m$8$YFB*-chgj
dzajo5zyQE`2upQ1nP30_002ovPDHLkV1lv6Ju3hJ
deleted file mode 100644
index cc3e2ee54c6bb2d9635254f406c1cc5add020ab1..0000000000000000000000000000000000000000
GIT binary patch
literal 0
Hc$@<O00001
deleted file mode 100644
index 0935f835c458c0b2e68fa87408b164f5a8504dfc..0000000000000000000000000000000000000000
GIT binary patch
literal 0
Hc$@<O00001
--- a/mobile/themes/wince/jar.mn
+++ b/mobile/themes/wince/jar.mn
@@ -16,17 +16,17 @@ chrome.jar:
images/weave.png (images/weave.png)
images/aboutBackground.jpg (images/aboutBackground.jpg)
images/arrowright-16.png (images/arrowright-16.png)
images/arrowup-16.png (images/arrowup-16.png)
images/arrowleftdark-16.png (images/arrowleftdark-16.png)
images/arrowrightdark-16.png (images/arrowrightdark-16.png)
images/ratings-18.png (images/ratings-18.png)
images/favicon-default-30.png (images/favicon-default-30.png)
- images/starred-64.png (images/starred-64.png)
+ images/star-40.png (images/star-40.png)
images/throbber.png (images/throbber.png)
images/navigation-magnifier-30.png (images/navigation-magnifier-30.png)
images/folder-30.png (images/folder-30.png)
images/stop-30.png (images/stop-30.png)
images/reload-30.png (images/reload-30.png)
images/go-30.png (images/go-30.png)
images/bookmarks-30.png (images/bookmarks-30.png)
images/addons-default-64.png (images/addons-default-64.png)
@@ -102,17 +102,16 @@ chrome.jar:
images/leftcapSSL-default-36.png (images/leftcapSSL-default-36.png)
images/preferences-active-36.png (images/preferences-active-36.png)
images/preferences-default-36.png (images/preferences-default-36.png)
images/rightcap-active-36.png (images/rightcap-active-36.png)
images/rightcap-default-36.png (images/rightcap-default-36.png)
images/settings-active-36.png (images/settings-active-36.png)
images/settings-default-36.png (images/settings-default-36.png)
images/settings-open-36.png (images/settings-open-36.png)
- images/starred-36.png (images/starred-36.png)
images/newtab-default-36.png (images/newtab-default-36.png)
images/newtab-active-36.png (images/newtab-active-36.png)
images/navigation-magnifier-16.png (images/navigation-magnifier-16.png)
images/close-16.png (images/close-16.png)
images/bookmarks-24.png (images/bookmarks-24.png)
images/go-16.png (images/go-16.png)
images/reload-16.png (images/reload-16.png)
images/stop-16.png (images/stop-16.png)