[GH#3027] [img] with alt/height/width combines alt and height parameters

Started by Joshua Dickerson, September 04, 2015, 03:50:16 PM

Previous topic - Next topic

Joshua Dickerson

[img alt=Test height=100 width=100]http://avatars.simplemachinesweb.com/smf/avatar_23_1337883444.png[/img]


Test

<img src="http://avatars.simplemachinesweb.com/smf/avatar_23_1337883444.png" alt="Test height=100" width="100" class="bbc_img resized" style="cursor: pointer;">

What would be expected is:
<img src="http://avatars.simplemachinesweb.com/smf/avatar_23_1337883444.png" alt="Test" height="100" width="100" class="bbc_img resized" style="cursor: pointer;">
Come work with me at Promenade Group



Need help? See the wiki. Want to help SMF? See the wiki!

Did you know you can help develop SMF? See us on Github.

How have you bettered the world today?

Shambles

I was under the impression that 'alt' text had to be enclosed within quotes to start with...?

[img alt="Test"...]

Joshua Dickerson

Nope, but thanks for reminding me. That's another part of the bug.

Code (input) Select
[img alt="Test" height=100 width=100]http://avatars.simplemachinesweb.com/smf/avatar_23_1337883444.png[/img]

Code (output) Select
<img src="http://avatars.simplemachinesweb.com/smf/avatar_23_1337883444.png" alt="&quot;Test&quot; height=100" width="100" class="bbc_img resized" style="cursor: pointer;">

Code (expected) Select
<img src="http://avatars.simplemachinesweb.com/smf/avatar_23_1337883444.png" alt="Test" height="100" width="100" class="bbc_img resized" style="cursor: pointer;">
Come work with me at Promenade Group



Need help? See the wiki. Want to help SMF? See the wiki!

Did you know you can help develop SMF? See us on Github.

How have you bettered the world today?

Ninja ZX-10RR

Quote from: BeastMode topic=525177.msg3720020#msg3720020
It's so powerful that on this post and even in the two PMs you sent me,you still answered my question very quickly and you're apologizing for the delay. You're the #1 support I've probably ever encountered man, so much respect for that. Thank you, and get better soon.

I'll keep this in my siggy for a while just to remind me that someone appreciated what I did while others didn't.

♥ Jess ♥

STOP EDITING MY PROFILE

Oldiesmann

Michael Eshom
Christian Metal Fans

Ninja ZX-10RR

Quote from: BeastMode topic=525177.msg3720020#msg3720020
It's so powerful that on this post and even in the two PMs you sent me,you still answered my question very quickly and you're apologizing for the delay. You're the #1 support I've probably ever encountered man, so much respect for that. Thank you, and get better soon.

I'll keep this in my siggy for a while just to remind me that someone appreciated what I did while others didn't.

♥ Jess ♥

STOP EDITING MY PROFILE

Kindred

maybe....  however, it's such an unused feature that it has taken years for anyone to report it...
Слaва
Украинi

Please do not PM, IM or Email me with support questions.  You will get better and faster responses in the support boards.  Thank you.

"Loki is not evil, although he is certainly not a force for good. Loki is... complicated."

Joshua Dickerson

Yeah and I didn't find a fix yet so it would take time for them to fix it. If/when I find the fix, it will be easier to include it in the next release.
Come work with me at Promenade Group



Need help? See the wiki. Want to help SMF? See the wiki!

Did you know you can help develop SMF? See us on Github.

How have you bettered the world today?

Illori

seem strange to anyone?

array(
'tag' => 'img',
'type' => 'unparsed_content',
'parameters' => array(
'alt' => array('optional' => true),
'title' => array('optional' => true),
'width' => array('optional' => true, 'value' => ' width="$1"', 'match' => '(\d+)'),
'height' => array('optional' => true, 'value' => ' height="$1"', 'match' => '(\d+)'),
),
'content' => '<img src="$1" alt="{alt}" title="{title}"{width}{height} class="bbc_img resized">',


and in the resulting page source you see it in a different order?

<img class="bbc_img resized" width="100" title="" alt="Test height=100" src="http://avatars.simplemachinesweb.com/smf/avatar_23_1337883444.png" style="cursor: pointer;">

what is changing the order? alt should be before title then width and height with the class last.

Joshua Dickerson

Come work with me at Promenade Group



Need help? See the wiki. Want to help SMF? See the wiki!

Did you know you can help develop SMF? See us on Github.

How have you bettered the world today?

Ninja ZX-10RR

Quote from: BeastMode topic=525177.msg3720020#msg3720020
It's so powerful that on this post and even in the two PMs you sent me,you still answered my question very quickly and you're apologizing for the delay. You're the #1 support I've probably ever encountered man, so much respect for that. Thank you, and get better soon.

I'll keep this in my siggy for a while just to remind me that someone appreciated what I did while others didn't.

♥ Jess ♥

STOP EDITING MY PROFILE

Antes


Illori

that was in firefox, so not a chrome bug.

also antes that is for bbc not for an avatar. so different things using different code.

Ninja ZX-10RR

Quote from: BeastMode topic=525177.msg3720020#msg3720020
It's so powerful that on this post and even in the two PMs you sent me,you still answered my question very quickly and you're apologizing for the delay. You're the #1 support I've probably ever encountered man, so much respect for that. Thank you, and get better soon.

I'll keep this in my siggy for a while just to remind me that someone appreciated what I did while others didn't.

♥ Jess ♥

STOP EDITING MY PROFILE

Antes

Quote from: Illori on September 21, 2015, 05:09:01 AM
that was in firefox, so not a chrome bug.

also antes that is for bbc not for an avatar. so different things using different code.

the screenshot Ninja posted your avatar with chrome,
the screenshot I posted your avatar with firefox.

Illori

the issue i am reporting is with an image posted in a TOPIC, it has NOTHING at all to do with any other image that shows up anywhere else.

this bug is reported on the img bbc tag.

margarett

Over the discussion about the BBC parsing which might cause a huge memory consumption, I seem to remember someone (Spuds?) finding out that the current implementation doesn't return all possible permutations of the arguments. Maybe this is related?
Se forem conduzir, não bebam. Se forem beber... CHAMEM-ME!!!! :D

QuoteOver 90% of all computer problems can be traced back to the interface between the keyboard and the chair

Ninja ZX-10RR

Could be, it's just way too old and would need to be somehow made "up-to-date" entirely, lol.
Quote from: BeastMode topic=525177.msg3720020#msg3720020
It's so powerful that on this post and even in the two PMs you sent me,you still answered my question very quickly and you're apologizing for the delay. You're the #1 support I've probably ever encountered man, so much respect for that. Thank you, and get better soon.

I'll keep this in my siggy for a while just to remind me that someone appreciated what I did while others didn't.

♥ Jess ♥

STOP EDITING MY PROFILE

Antes

Issue closed/resolved. If you are having such issues please open new topic while using latest build.

Joshua Dickerson

Come work with me at Promenade Group



Need help? See the wiki. Want to help SMF? See the wiki!

Did you know you can help develop SMF? See us on Github.

How have you bettered the world today?

Advertisement: