From: Alexander Ebert <ebert@woltlab.com>
Date: Fri, 18 Sep 2020 15:02:47 +0000 (+0200)
Subject: Missing link target causes buttons to be unresponsive in some touch browers
X-Git-Tag: 5.3.0_Beta_1~7^2~3^2~1
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=d31f60a8c5345f6f5461cb870e3a688797b32639;p=GitHub%2FWoltLab%2FWCF.git

Missing link target causes buttons to be unresponsive in some touch browers
---

diff --git a/wcfsetup/install/files/acp/js/WCF.ACP.Style.js b/wcfsetup/install/files/acp/js/WCF.ACP.Style.js
index 5f279e568a..f894733fc3 100644
--- a/wcfsetup/install/files/acp/js/WCF.ACP.Style.js
+++ b/wcfsetup/install/files/acp/js/WCF.ACP.Style.js
@@ -478,7 +478,10 @@ WCF.ACP.Style.List = Class.extend({
 			var $styleID = $list.data('styleID');
 			
 			var self = this;
-			$list.find('.jsSetAsDefault').click(function() { self._click('setAsDefault', $styleID); });
+			$list.find('.jsSetAsDefault').click(function(event) {
+				event.preventDefault();
+				self._click('setAsDefault', $styleID);
+			});
 			$list.find('.jsDelete').click(function(event) { self._delete(event, $styleID); });
 		}, this));
 	},
@@ -505,6 +508,8 @@ WCF.ACP.Style.List = Class.extend({
 	 * @param	integer		styleID
 	 */
 	_delete: function(event, styleID) {
+		event.preventDefault();
+		
 		var $confirmMessage = $(event.currentTarget).data('confirmMessageHtml');
 		if ($confirmMessage) {
 			var self = this;
diff --git a/wcfsetup/install/files/acp/templates/styleList.tpl b/wcfsetup/install/files/acp/templates/styleList.tpl
index 0f3c6dc8a5..22674aa394 100644
--- a/wcfsetup/install/files/acp/templates/styleList.tpl
+++ b/wcfsetup/install/files/acp/templates/styleList.tpl
@@ -60,9 +60,9 @@
 								<li><a href="{link controller='StyleExport' id=$style->styleID}{/link}" title="{lang}wcf.acp.style.exportStyle{/lang}" class="jsTooltip"><span class="icon icon16 fa-download"></span> <span class="invisible">{lang}wcf.acp.style.exportStyle{/lang}</span></a></li>
 								
 								{if !$style->isDefault}
-									<li><a title="{lang}wcf.global.button.{if $style->isDisabled}enable{else}disable{/if}{/lang}" class="jsTooltip"><span class="icon icon16 fa-{if !$style->isDisabled}check-{/if}square-o jsToggleButton" data-object-id="{@$style->styleID}"></span> <span class="invisible">{lang}wcf.global.button.{if $style->isDisabled}enable{else}disable{/if}{/lang}</span></a></li>
-									<li><a title="{lang}wcf.acp.style.button.setAsDefault{/lang}" class="jsSetAsDefault jsTooltip"><span class="icon icon16 fa-check-circle"></span> <span class="invisible">{lang}wcf.acp.style.button.setAsDefault{/lang}</span></a></li>
-									<li><a title="{lang}wcf.global.button.delete{/lang}" class="jsDelete jsTooltip" data-confirm-message-html="{lang __encode=true}wcf.acp.style.delete.confirmMessage{/lang}"><span class="icon icon16 fa-times"></span> <span class="invisible">{lang}wcf.global.button.delete{/lang}</span></a></li>
+									<li><a href="#" title="{lang}wcf.global.button.{if $style->isDisabled}enable{else}disable{/if}{/lang}" class="jsTooltip"><span class="icon icon16 fa-{if !$style->isDisabled}check-{/if}square-o jsToggleButton" data-object-id="{@$style->styleID}"></span> <span class="invisible">{lang}wcf.global.button.{if $style->isDisabled}enable{else}disable{/if}{/lang}</span></a></li>
+									<li><a href="#" title="{lang}wcf.acp.style.button.setAsDefault{/lang}" class="jsSetAsDefault jsTooltip"><span class="icon icon16 fa-check-circle"></span> <span class="invisible">{lang}wcf.acp.style.button.setAsDefault{/lang}</span></a></li>
+									<li><a href="#" title="{lang}wcf.global.button.delete{/lang}" class="jsDelete jsTooltip" data-confirm-message-html="{lang __encode=true}wcf.acp.style.delete.confirmMessage{/lang}"><span class="icon icon16 fa-times"></span> <span class="invisible">{lang}wcf.global.button.delete{/lang}</span></a></li>
 								{/if}
 								
 								{event name='itemButtons'}