[Tip] Remove Images from Quotes!

[heavyccasey]

Here's a quick and simple way to display a link instead of an image in a quote, without hiding smileys or complicated regular expressions:

This is tested for 1.1.5 - 1.1.8, as well as 2.0 RC1, although older 1.1 versions will probably work too.


Subs.php
Find:

Code Auswählen Erweitern

array(
'tag' => 'img',
'type' => 'unparsed_content',
'parameters' => array(


Add Before:

Code Auswählen Erweitern

// Remove images from quotes.
array(
'tag' => 'img',
'type' => 'unparsed_content',
'require_parents' => array('quote'),
'parameters' => array(
'alt' => array('optional' => true),
'width' => array('optional' => true, 'value' => ' width="$1"', 'match' => '(\d+)'),
'height' => array('optional' => true, 'value' => ' height="$1"', 'match' => '(\d+)'),
),
'content' => '<i>(<a href="$1" target="_blank" onclick="this.parentNode.innerHTML = \'<i\' + \'mg src=\\\'\' + this.href + \'\\\' /\' + \'>\'; return false;">Image removed from quote</a>.)</i>',
'validate' => create_function('&$tag, &$data, $disabled', '$data = strtr($data, array(\'<br />\' => \'\'));'),
'disabled_content' => '($1)',
),
array(
'tag' => 'img',
'type' => 'unparsed_content',
'require_parents' => array('quote'),
'content' => '<i>(<a href="$1" target="_blank" onclick="this.parentNode.innerHTML = \'<i\' + \'mg src=\\\'\' + this.href + \'\\\' /\' + \'>\'; return false;">Image removed from quote</a>.)</i>',
'validate' => create_function('&$tag, &$data, $disabled', '$data = strtr($data, array(\'<br />\' => \'\'));'),
'disabled_content' => '($1)',
),


Instead of an image, it'll look like...

(Image removed from quote.)

...and when you click it, the image will appear. No JavaScript, no problem! In that case, it'll open a new tab/window with the image.

You can mess with the HTML ('content' => '<i>...</i>',) if you want. Just don't touch the Javascript, it's hell trying to figure out how to properly escape the quotes. 

That's it!

[Antechinus]

Nice. Can you do the same for RC1?

[SA™]

aint that pretty much the same as my mod

http://custom.simplemachines.org/mods/index.php?mod=1302

[heavyccasey]

Nice. Can you do the same for RC1?

I don't know, I don't have a copy of RC1 installed.

I downloaded a copy of the source... the parse_bbc() array looks almost exactly the same as 1.1. I think the same code would work for 2.0, but I don't have anything to test on. Would you mind testing it? I'm almost certain it'll work.

aint that pretty much the same as my mod

http://custom.simplemachines.org/mods/index.php?mod=1302

Yes. In fact, I linked to it in my post. However, yours includes complicated regular expressions while this tip uses the SMF-included BBC manager.

[Antechinus]

Also this gives a hide/show option. I'm not sure if the mod does the same (because I haven't tested it) but it's a handy feature.

[Marcus Forsberg]

Moved to Tips and Tricks and added to Tips and Tricks (index). Thanks for your tip.

[Antechinus]

Doesn't appear to work in RC1.

[heavyccasey]

Doesn't appear to work in RC1.

Fixed in original post. Thanks for testing! Apparently SMF 2.0 doesn't automatically put in image dimensions anymore.

[Antechinus]

Yep, that works. Good job.

ETA: Prefer it in non-italic myself but that's easy to sort if anyone wants to.

In fact I think I'll install this on my forum right now. I get sick of people quoting the same damned image just to post a one line response. 

[heavyccasey]

If you don't want the italics, you'd better replace <i></i> with <span></span> instead of just removing it, if you haven't already figured that out.

Drives me crazy too

[Antechinus]

It works without the <span></span>. Dunno if it's supposed to but it does.

ETA: Validates that way too. No html or css or js errors. Go figure.

[heavyccasey]

It works without the <span></span>. Dunno if it's supposed to but it does.

ETA: Validates that way too. No html or css or js errors. Go figure.

It just looks like it's working. If you add text before or after the image, without the <span>, the text disappears too.

[Antechinus]

Ah. I see what you mean. The text displays fine until you click the link to show the image. I added the span tags and as you say that sorts it.

[Chas Large]

Thanks for this tip, the mod works just fine in 1.1.10.
Chas

[Simplemachines Cowboy]

Cool. Works nicely on 1.1.10.

[bbmtalk]

Can this be done to resize the image to 300 pixels in quotes if the image is larger than 300 pixel?

[Arantor]

Not without a lot of work. At that point the script does not know how big the image is, and won't without downloading it. At best you could make it scrollable, or else you'd have to enforce it go determine how big it is first.

[chrlsnicholas7]

Can we increase the pixel size if it is smaller than 100...
Will the clarity be maintained....

[Arantor]

This is currently available as a mod: http://custom.simplemachines.org/mods/index.php?mod=1302

[heavyccasey]

That mod uses its own regular expression code, while this edit uses the built-in SMF system.

[Arantor]

Ah, my bad. I just figured the two used the same method - I didn't check, but it's usually the case that if a mod is written where there is an existing tip/trick, that the mod may well have used the code from the tip.

[nwobhm]

Has anyone tried if this works with RC2

[Arantor]

I see no reason why this wouldn't work on 2.0 RC2. The BBcode listing wasn't touched IIRC.

[Antechinus]

I'll definitely be wanting this on RC2 so if it doesn't work now I'll find a way to make sure it does (if nobody beats me to it). AFAIK though it should work.

[Costa]

This work on RC2!

[Scratching my Head]

Does not appear to work for 1.1.11 

[Arantor]

No, there was a rewrite of the img tag in 1.1.11.

[Scratching my Head]

Thanks Arantor. Is there a fix/mod that achieves the desired result?

[Arantor]

Nope, it's a complete rewrite of the tip :S

[Scratching my Head]

I'm not sure I follow. As I understand it there is nothing now available that hides images in quotes?

[Arantor]

Correct. The tip is based on the old img tag code that was in versions prior to 1.1.11. It will need rewriting to work on 1.1.11.

[Scratching my Head]

Very good. Thanks muchly, Arantor.

[gbsothere]

Correct. The tip is based on the old img tag code that was in versions prior to 1.1.11. It will need rewriting to work on 1.1.11.

I've used the old Remove Images From Quote mod v1[1].0 for a long time; we have it now on my 1.1.11 board.  I can't remember, but I don't think I had to tweak anything for it to work. 



But then, I don't remember what I had for lunch, so....

[Chas Large]

I've written a mod for SMF 2.0 RC2 based on this tip but it's not been approved yet.

It also works for RC3.

It's title will be "Remove images from Quotes" I'll post here again when it's released.

[gbsothere]

Doesn't appear to work for 1.1.11

Pic of it working for 1.1.11

[Antechinus]

Works in 1.1.11 for me too. Never changed a thing.

In fact I want to tweak it to remove the quotes from quotes. Having quotes collapsed by default and expandable onclick would be rather handy I think.

[waruna]

I'm using SMF2.0 RC2. Just a quick question, is there any way to remove images from quotes only at the same page of the images? I mean the function will not work if the images are quoting from another page.

Thanks in advance.