https://bugzilla.wikimedia.org/show_bug.cgi?id=40329

--- Comment #17 from TMg <[email protected]> 2012-09-23 18:17:40 UTC ---
(In reply to comment #16)
> We just define most of them to work the same way as HTML defines them.

No, you do not "define" anything. All you do is whitelisting. Everything is
simply passed through or not. MediaWiki never did any translations and never
should. This is confusing and bogus.

> We output br in the same way whether you write <br> or <br />. Whether you
> write class=foo, class="foo", or class='foo' we output it as class="foo".

Fixing slashes and quotes is not the same. class=foo and class="foo" still have
the same meaning. The replaced align stuff does *not* have the same meaning.

> we always decode entities you input into an id.

Again, non of this changes the *meaning* of the input.

> we already have the related code saying that only urls matching our
> protocol whitelist are allowed.

Adding or removing something from a whitelist is not the same as *changing*
something with something else that has a *different* meaning. As said I'm fine
with removing the align attribute from the whitelist.

> Our job isn't to replicate browser quirks.

Are you kidding? This is what you currently do. This is what this bug report is
about. You are trying to replicate a browser quirk and you are doing it bad.
Drop this please.

> And manually replacing only the things that are
> actually broken is simpler than replacing everything.

What about *not* breaking anything?

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to