id,summary,reporter,owner,description,type,status,priority,milestone,component,version,severity,resolution,keywords,cc
3051,QuickTags now work in Safari browser,mattyrob,MattyRob,"The Safari browser has not behaved as expected when the QuickTags are used. A post at [http://www.alexking.org/blog/2006/08/15/safari-cursor-aware/] made me look into the fact that they may now work. I've tested and it now works as expected meaning the disabling code can be removed from admin-functions.


{{{
Index: admin-functions.php
===================================================================
--- admin-functions.php	(revision 4106)
+++ admin-functions.php	(working copy)
@@ -1228,8 +1228,6 @@
 }
 
 function the_quicktags() {
-	// Browser detection sucks, but until Safari supports the JS needed for this to work people just assume it's a bug in WP
-	if (!strstr($_SERVER['HTTP_USER_AGENT'], 'Safari')) {
 		echo '
 		<div id=""quicktags"">
 			';
@@ -1237,7 +1235,7 @@
 		echo '			<script type=""text/javascript"">if ( typeof tinyMCE == ""undefined"" || tinyMCE.configs.length < 1 ) edToolbar();</script>
 		</div>
 ';
-	} else echo '
+	echo '
 <script type=""text/javascript"">
 function edInsertContent(myField, myValue) {
 	//IE support
}}}
",enhancement,closed,low,2.0.6,Administration,2.0.5,trivial,fixed,safari quicktags maint-candidate,