User "Happy-melon" posted a comment on MediaWiki.r99369. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/99369#c25048 Commit summary:
Fix up makeLinkItem and makeLink: - Don't explicitly name attributes to use (fixes bug on language_links in 1.18) - Allow makeLink to accept options specifying how to wrap links and text Comment: What's the usecase for the <tt>text-wrapper</tt> stuff? It looks like it'd most likely be ''more'' code than just using an <tt>Html::rawElement()</tt> directly, as well as being much less clear. It's far from obvious whether the text is escaped, for instance. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
