Ticket #7261 (closed defect: fixed)

Opened 5 months ago

Last modified 4 months ago

Press This

Reported by: paulhontz Assigned to: noel
Priority: normal Milestone: 2.6
Component: General Version:
Severity: major Keywords: Press This
Cc:

Description

When creating a new post from "Press This", the article does not link back to the original source if you click on the picture tab. If however, you stay on the default tab of text and press the link button, it will work.

These features are very important to tumblogs, or if you link to lots of blogs.

Change History

07/08/08 15:45:31 changed by ryan

I see a link back to the source. Could you provide browser information and explicit steps to reproduce?

07/09/08 18:42:19 changed by paulhontz

If you don't have any text selected when you click on "Press This", it will automatically put a link in the post with the title (of the site) as the link (example: You are on gmail.com and press the bookmarklet. Your new post would open up with a link that said "Gmail - Inbox (X) - xxxxxx@gmail.com".

However, if you have text selected when you press your bookmarklet the whole selection becomes a link. I don't believe this is the intended behavior. To make matters worse, if you then click on the "Photo" tab, the large link converts to plain text and stays like that even if you click back on the "Text" tab

Does this help clarify? I'm using Firefox 3.

however, when I use Safari, or IE7 (the css also appears to be messed up for use with ie7) the editor fails to appear at all. Instead, I see the code (example below)

<a href='http://www.autoblog.com/2008/07/09/g-power-aims-to-rival-m-with-supercharged-z4/'>a natural choice for anyone who wants a high performance BMW, but that hasn't stopped G-Power from offering go-fast parts for Bimmer ow</a>

Also, when I've tried clicking my bookmarklet in IE6, nothing appears. If I click on the link (that you're suppose to add to favorites), it works fine. If I add it to my list, and try and click on it, it fails to open anything.

07/09/08 19:03:52 changed by ryan

  • owner changed from anonymous to noel.

07/09/08 19:51:53 changed by ryan

(In [8302]) Don't format_to_edit TinyMCE inserts. Don't turn selection into a link. Restore editor contents when switching back to text tab. see #7261 #7220

07/09/08 19:52:18 changed by ryan

That fixes up some of the mess.

07/09/08 20:28:48 changed by paulhontz

Hey ryan, thanks for your help. I haven't been able to get tiny_mce editor to come up, but I think that might just be something on my end.

Also, would it be possible to have an option to automatically cite your source? Meaning that a link could be placed at the bottom of the post that leads directly back to the webpage you "Press This"ed from?

Thanks again!

07/09/08 20:40:29 changed by paulhontz

I think this might have something to do with why the IE 6 bookmarklet doesn't do anything.

http://blog.seagul.co.uk/articles/2007/12/14/bookmarklet-favelet-limits-in-internet-explorer-6-and-7

07/09/08 20:55:22 changed by paulhontz

One way to potentially get around this is have separate bookmarklets for *good* browsers, and then one for IE.

By killing off the user agent string to see if it's firefox, you can save quite a few characters, and you should be within the 508 char limit.

07/10/08 13:14:09 changed by paulhontz

I can now confirm that if you stay under the 508 character limit that IE6 is limited to, the bookmarklet will appear.

Is there anything else I could try to get the editor and a link?

07/10/08 16:41:23 changed by ryan

Is TinyMCE missing from the regular write post page too, or just from Press This?

I'll work on shortening the bookmarklet.

07/10/08 16:54:37 changed by ryan

(In [8308]) Reduce size of Press This bookmarklet. see #7261

07/10/08 18:33:47 changed by paulhontz

It is only missing in the bookmarklet.

07/11/08 20:35:41 changed by paulhontz

supposedly this patch fixes the editor not coming up.

http://trac.wordpress.org/attachment/ticket/6813/press-this-combined_7-11.patch

07/15/08 17:25:09 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.
  • milestone changed from 2.9 to 2.6.