Browse Source

! Quick reply now uses the sprite. Every little helps, right?

Signed-off-by: Peter Spicer <[email protected]>
Peter Spicer 10 years ago
parent
commit
3b892c1ff5

+ 3 - 3
Themes/default/Display.template.php

@@ -691,7 +691,7 @@ function template_main()
 			<div class="tborder" id="quickreplybox">
 				<div class="cat_bar">
 					<h3 class="catbg">
-						<a href="javascript:oQuickReply.swap();"><img src="', $settings['images_url'], '/', $options['display_quick_reply'] > 1 ? 'collapse' : 'expand', '.png" alt="+" id="quickReplyExpand" class="icon" /></a>
+						<a href="javascript:oQuickReply.swap();" class="', $options['display_quick_reply'] > 1 ? 'toggle_up' : 'toggle_down', '" id="quickReplyExpand"></a>
 						<a href="javascript:oQuickReply.swap();">', $txt['quick_reply'], '</a>
 					</h3>
 				</div>
@@ -834,8 +834,8 @@ function template_main()
 						sImagesUrl: smf_images_url,
 						sContainerId: "quickReplyOptions",
 						sImageId: "quickReplyExpand",
-						sImageCollapsed: "collapse.png",
-						sImageExpanded: "expand.png",
+						sClassCollapsed: "toggle_up",
+						sClassExpanded: "toggle_down",
 						sJumpAnchor: "quickreply",
 						bIsFull: ', !empty($options['use_editor_quick_reply']) ? 'true' : 'false', '
 					});';

+ 5 - 1
Themes/default/css/index.css

@@ -897,11 +897,15 @@ div.title_barIC h4.titlebg {
 	margin: 0;
 	padding: 6px;
 }
-img#upshrink_ic, img#newsupshrink, img.panel_toggle, img#quickReplyExpand {
+img#upshrink_ic, img#newsupshrink, img.panel_toggle {
 	float: right;
 	margin: 0;
 	padding: 4px 4px 0 4px;
 }
+#quickReplyExpand {
+	float: right;
+	margin: 2px 2px 0 2px;
+}
 table.table_list a.unreadlink, table.table_list a.collapse {
 	float: right;
 }

+ 1 - 1
Themes/default/scripts/topic.js

@@ -290,7 +290,7 @@ QuickReply.prototype.onQuoteReceived = function (oXMLDoc)
 // The function handling the swapping of the quick reply.
 QuickReply.prototype.swap = function ()
 {
-	document.getElementById(this.opt.sImageId).src = this.opt.sImagesUrl + "/" + (this.bCollapsed ? this.opt.sImageCollapsed : this.opt.sImageExpanded);
+	$('#' + this.opt.sImageId).toggleClass(this.opt.sClassCollapsed + ' ' + this.opt.sClassExpanded);
 	$('#' + this.opt.sContainerId).slideToggle();
 
 	this.bCollapsed = !this.bCollapsed;