QuickTags now work in Safari browser

[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

[mattyrob]

Ticket 3051 diff patch

[ryan]

(In [4164]) Enable quicktags for safari. Props mattyrob. fixes #3051

[MattyRob]

Can we get this into 2.0.5?

[foolswisdom]

Sorry, too late for 2.0.5

Closing FIXED for trunk.

[mattyrob]

I know I keep asking (God loves a tryer!) - as this missed 2.0.5 but is fixed in [4164] for the trunk.

So, can we get this into 2.0.6 (or 2.0.7)

[markjaquith]

(In [4640]) Enable quicktags for Safari. Props mattyrob. fixes #3051