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

Bartosz Dziewoński <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---
           Assignee|[email protected]       |[email protected].
                   |                            |org

--- Comment #47 from Bartosz Dziewoński <[email protected]> ---
I've looked at this again now that things have cooled down. I still do not like
this solution, and I still like this alternative proposed a long time ago:

(In reply to Bartosz Dziewoński from comment #13)
> (In reply to Jon from comment #11)
> > That aside use of a single class .link-https rather than a
> > combination of class .link-https, div#content a.external[href
> > ^="https://";] would be preferable for a variety of reasons
> > readability, maintainability, reduced file size... are you really
> > telling me this is not a no-brainer?
> 
> Using the classes as you proposed does seem sane and mostly non-troublesome
> (apart from the classic caching issues). I am perfectly fine with doing that
> (as long as I'm not the one who will have to jerry-rig the Parser).

So I looked at the Parser, or in fact the Linker, and doing this is actually
amusingly trivial. Pity no one did this the first time around.

A patch will follow momentarily.

-- 
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