[wp-trac] [WordPress Trac] #3051: QuickTags now work in Safari browser

WordPress Trac wp-trac at lists.automattic.com
Sat Aug 19 08:59:03 GMT 2006


#3051: QuickTags now work in Safari browser
----------------------------+-----------------------------------------------
 Reporter:  mattyrob        |       Owner:  MattyRob        
     Type:  enhancement     |      Status:  new             
 Priority:  low             |   Milestone:  2.1             
Component:  Administration  |     Version:  2.0.4           
 Severity:  trivial         |    Keywords:  safari quicktags
----------------------------+-----------------------------------------------
 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
 }}}

-- 
Ticket URL: <http://trac.wordpress.org/ticket/3051>
WordPress Trac <http://wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list