Changeset 738

Timestamp:

01/08/2004 03:30:23 AM (22 years ago)

Author:

alex_t_king

Message:

made the bookmarklet safari compatible and enabled passing in the title, task #49

Location:

trunk/wp-admin

Files:

• bookmarklet.php (modified) (1 diff)
• post.php (modified) (1 diff)

trunk/wp-admin/bookmarklet.php

@@ -53 +53 @@
         $post_title = $popuptitle;
     }
+// I'm not sure why we're using $edited_post_title in the edit-form.php, but we are
+// and that is what is being included below. For this reason, I am just duplicating
+// the var instead of changing the assignment on the lines above. 
+// -- Alex King 2004-01-07
+    $edited_post_title = $post_title;
     
     $content = $_REQUEST['content'];

trunk/wp-admin/post.php

@@ -687 +687 @@
 if ($is_NS4 || $is_gecko) {
 ?>
-    <a href="javascript:Q=document.selection?document.selection.createRange().text:document.getSelection();void(window.open('<?php echo $siteurl ?>/wp-admin/bookmarklet.php?text='+escape(Q)+'<?php echo $bookmarklet_tbpb ?>&popupurl='+escape(location.href)+'&popuptitle='+escape(document.title),'WordPress bookmarklet','scrollbars=yes,width=600,height=<?php echo $bookmarklet_height ?>,left=100,top=150,status=yes'));">Press It 
+    <a href="javascript:if(navigator.userAgent.indexOf('Safari') >= 0){Q=getSelection();}else{Q=document.selection?document.selection.createRange().text:document.getSelection();}void(window.open('<?php echo $siteurl ?>/wp-admin/bookmarklet.php?text='+escape(Q)+'&popupurl='+escape(location.href)+'&popuptitle='+escape(document.title),'WordPress bookmarklet','scrollbars=yes,width=600,height=460,left=100,top=150,status=yes'));">Press It 
     - <?php echo $blogname ?></a> 
     <?php