Use ems instead of pixels for checkbox

Mixing pixels and ems results in a negative height, which is invalid.
Use ems instead of pixels for @focusBottomBorderSize.

Follow up: Ia6adc4bda000508f14acf4214e778118efa35be0

Bug: 71506
Change-Id: Ia619916a172a501365e0104e9927c7fae9de8a3f

Merge "User rights API: Abstract out some stuff about core's form into separate methods"

User rights API: Abstract out some stuff about core's form into separate methods

To we can make a CentralAuth API module for global user rights.

Bug: 71495
Change-Id: I139c43e5eaa1fb63b75baddbafb7caa3c964d6b5

Merge "mediawiki.jqueryMsg: Support arbitrary expressions in plural forms"

Merge "Use skinStyles for mediawiki.ui.icon"

Use skinStyles for mediawiki.ui.icon

This allows skin customization.  This was missed since it was under
code review when the others went in.

Change-Id: I2c640fda4d2c6e40be0add52f6fc948a8e07c010

mediawiki.jqueryMsg: Support arbitrary expressions in plural forms

In I15c167f245 mw.jqueryMsg.htmlEmitter#plural was modified to
convert the plural params to flat html. This caused extlink params
as plural forms expanded as text node and escaped.

This patch improves the plural support by:
* Allowing valid parser expressions as plural form parameters. They wont
  be converted to html, parsed nodes will be preserved in plural
  calculation
* For explicit plural forms like n=expression, the expression can be any
  valid parser expression. They also will not be coverted to html,
  nodes will be preserved
* Move the splitting by equal sign logic for explicit plural forms to
  jqueryMsg, avoiding mediawiki.language#convertPlural to do
  parsing logc. Explicit plural forms, if any, are passed as a parameter
  to mediawiki.language#convertPlural

Includes test cases for all combinations of the above cases.

mediawiki.jqueryMsg.peg can be improved to support digit=expression as an
additional form for templateParam to better handle this. But it looks
it can potentially disturb other parser functions. Not attempted.

Bug: 70617
Change-Id: I3c3cbb6d3676903941c40b658032b4c543556e06

Update OOjs UI to v0.1.0-pre (30b0407428)

New changes:
3386c39 Make this.$.$iframe actually point to an iframe again
bb7bbf5 ClippableElement: Use 'overflow-[xy]: scroll' rather than 'auto'
71176eb TextInputWidget: Don't mix inner and outer heights in calculations
f1dd60e TextInputWidget: Work around Blink height miscalculation
954a369 WindowManager: Emit 'resize' event when a window is resized
3232d6f Try to stop user from tabbing outside of open dialog box
6c4baa3 Tool: Use FlaggedElement mixin to let tools be flagged

Change-Id: Ia0a0b45e5f210314592733114a05583973b011d6

Merge "Skip MailAddressTest::testNewFromUser on windows"

Merge "CSSMin: Don't generate double rules for IE < 8 when embedding SVG files"

Merge "CSSMin: Do not base64-encode non-binary files when embedding"

Merge "Update OOjs UI to v0.1.0-pre (afa3241e8e)"

Update OOjs UI to v0.1.0-pre (afa3241e8e)

New changes:
8daef61 TextInputWidget: Add setValidation function to change pattern used to validate text
953c46c [BREAKING CHANGE] Create a MediaWiki theme
e2a3bae PopupWidget: Position body/anchor using 'margin-left' instead of 'left'
4f50c34 Correct the left positioning of a popup wrapper
afa3241 Move references to pending.gif to the themes that provide it

Update changes:
* Update links to OOjs UI distribution files

Change-Id: Ic59650b56de44807348db4df46c538756ed9329e

CSSMin: Don't generate double rules for IE < 8 when embedding SVG files

Bug: 71003
Change-Id: Ic232e8d62b164940003afdfe7cce9f964d7e9cbc

CSSMin: Do not base64-encode non-binary files when embedding

Do not base64-encode non-binary files (containing only whitespace and
printable ASCII characters, which matches sane SVG files).

For SVG files the percent-encoded URIs are actually slightly longer
than the base64-encoded ones (~10%), but compress a lot better
resulting on 15-20% less data to transfer after gzip compression.

(The effect is best seen on the 'oojs-ui' module, which consists
mostly of SVG icons – especially after commenting out everything other
than 'oojs-ui.svg.css'.)

I tried this for binary files too, just in case; but as expected, they
suffer from a noticeable size increase even with compression (~15%).

Bug: 67341
Change-Id: Iddaf863b6be98570a2bb8e606f13946a96345f65

Fix GitInfoTest::testValidJsonData on windows

GitInfo.php is replacing DIRECTORY_SEPARATOR by '-'.
On windows this is \, which was not set in the tests.

1) GitInfoTest::testValidJsonData
Failed asserting that false is true.

Change-Id: I1871c342b61a335b59a98755d8b824cc58c1543e

Localisation updates from https://translatewiki.net.

Change-Id: I8d868812d775114923187d2db27201080017d452

Merge "Introduce Message::plaintextParam"

Introduce Message::plaintextParam

I have run into numerous issues trying to utilize unsafe user
provided content as an argument to a Message instance.  Specific
cases are enumerated in MessageTest.php

Typically the solution to using user provided text is to use
Message::rawParam, but this pushes escaping of the parameter to
the caller.  This patch introduces Message::plaintextParams which
handles escaping of the string parameter to match the requested
output format.

The functionality is:
* plain and text: exactly like rawParams()
* escaped, parse and parseAsBlock: escape it but don't do brace expansion

Additionaly, similar to Message::rawParam, plaintext parameters are not
valid parser function arguments.

Change-Id: I320645cd23c98fea4bfc32ab22b7ef8d320957cb

Disable "linkPrefixExtension" on Central Kurdish

Disable prefix on Ckb and format it like Fa, it currently links
"«[[Link]]»" to "<a>«Link</a>»" on Ckb but on Fa will "«<a>Link</a>»"
which per a community member is much better and expected

Change-Id: I1d17fc6ea511f13bde4fef7538117c9c2eebc6db

Merge "Restore ability for getAttributes to set HTMLTextField type"

Restore ability for getAttributes to set HTMLTextField type

Change I47db5eab broke SecurePoll, which was relying on this.

Change-Id: Ib11bf9d34eb12f14b9636498c00ff4040b35f130

Fix version comments in thumbnail prerendering code

Change-Id: I74b52d2ecc382dfc0b55615ce512e8c64d02e9cb
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/301

Merge "Added missing JsonFormat::parse() RELEASE NOTES, fixed docs"

Added missing JsonFormat::parse() RELEASE NOTES, fixed docs

Constant values were changed to be above 0xFF - this way
we can easily decide to allow depth-parsing-limit to be OR-able:

  FormatJson::parse( $value, 30 | FormatJson::FORCE_ASSOC )

Follows-up Ic0eb0a7 and I1c4f37a.

Change-Id: I9bfd67a5ca4ea1d399821549c7e63ffdecd56ad1

Implement autocompletion for Performer field on Special:Log

Bug: 48241
Change-Id: I5e6af5aba56de613f10f862aaf064aaa32b5b176

Add test cases for wfAppendQuery

Change-Id: I51e0e1764ad70d922466d62f1ddf3f8e3fb4568f

Remove deprecated "jquery.json" module

Announced in https://www.mail-archive.com/wikitech-l%40lists.wikimedia.org/msg78108.html

Change-Id: Icc9dec0146cd374787124b7e1ed295c77737d11b

mediawiki.api: Remove deprecated function parameters

* 'ok' and 'err' parameters to various functions.
  Tracked with mw.track( 'mw.deprecate' ) as 'api.cbParam'.
  Deprecated since MW 1.20, printing console warnings since MW 1.23.
* 'async' parameter to mediawiki.api.category #getCategories.
  Tracked with mw.track( 'mw.deprecate' ) as 'api.async'.
  Deprecated and printing console warnings since MW 1.23.

Also removed is mw.Api#normalizeAjaxOptions, which is no longer needed
and which has never been used by anything outside mediawiki.api module.

Announced in https://www.mail-archive.com/wikitech-l%40lists.wikimedia.org/msg78108.html

Change-Id: I0a650fdb4affd394ae419e21d54baf790116f6f7

mediawiki.user: Remove deprecated name() and anonymous() methods

Removed mw.user#name and mw.user#anonymous which have been deprecated
in favor of mw.user#getName and mw.user#isAnon since 1.20.

Announced in https://www.mail-archive.com/wikitech-l%40lists.wikimedia.org/msg78108.html

Change-Id: I3be1cfecb79faee5701708136ede133870b9940d

PasswordFactory::newFromPlaintext( null ) needs to work

Various code passes null around to mean "an invalid password". It
shouldn't all have to test for null and specially handle that.

This also fixes a codepath where User::$mNewpassword could get set to an
empty string rather than a password object, which would cause problems
later when anything else tries to use it.

Bug: 71421
Change-Id: Ib5f94b52c07e7dba89328b98fb43c86db95ee09f

Merge "HistoryAction: Use correct classes for buttons"

Localisation updates from https://translatewiki.net.

Change-Id: I46d7b8b70d02685d9b5c20bd14a0da56dffbcf2b

HistoryAction: Use correct classes for buttons

Bug: 71141
Change-Id: I95ac411e7c06d3b7aec966527f196ef347f1c710

Merge "API: Rewrite queries for list=allusers"

Merge "Special:Contributions: Use mediawiki.ui"

Merge "Bidi-isolate and nowrap parser tags on Special:Version"

Merge "Do not read metadata from non-existing djvu image"

Merge "Fixed spacing"

Merge "Register missing messages for mediawiki.special.upload"

Register missing messages for mediawiki.special.upload

Follows-up I43bf828654d342893a19656ddd4151c2558d2f63.
The confirm close needs messages and jQueryMsg for parsing "int:".

Change-Id: If9a69749dcff2196dfe51a4484c6faeb98674ffa

Merge "Introduce ContentAlterParserOutput hook"

Merge "Lowered ActiveUsers query period to reduce timeouts"

Do not read metadata from non-existing djvu image

Tests are printing:
*** [1-11711] Failed to open 'some-nonexistent-file': No such file or
directory.
*** (ByteStream.cpp:693)
*** 'DJVU::GUTF8String DJVU::ByteStream::Stdio::init(const DJVU::GURL&,
const char*)'
djvutxt: [1-11711] Failed to open 'some-nonexistent-file': No such file
or directory.
djvutxt: 'ByteStream.cpp:693'

That just spamming the logs, so adding a valid check before do anything.

Change-Id: Ia77377e632587ede76af4c722a1b8ac34fa50e1c

Fixed spacing

- Added newline at end of file
- Removed double spaces/newlines
- Added space after if/function and parentheses/brackets
- Removed space before comma/cast
- Fixed indent of some lines

Change-Id: I29867ffdffdfb7d2b56997e9393497c7dc12f7d3

Introduce ContentAlterParserOutput hook

This hook is the counterpart of ContentGetParserOutput: it gives
high-level access to the parsing of a content object (as opposed
to lower-level parser hooks like ParserBefore*/ParserAfter* which
are called for all kinds of non-content input like interface
messages and don't have the necessary context to know what kind of
input is being parsed), but it is called when the parsing has
already finished.

The intention is to provide a convenient location for ParserOutput
modifications which depend on some global property of the parsed
content, but need to happen before LinksUpdate. Currently there are
no hooks between parsing (initiated by WikiPage::doEditUpdates()
calling WikiPage::prepareContentForEdit()) and updating the link
tables (initiated by the same method calling
Content::getSecondaryDataUpdates()), so any hook an extension would
try to use would face one of the following problems:
* the parsing output is not available yet
* LinksUpdate has run already, modifying list properties is unsafe
* there is not enough context to tell what's being parsed

A typical use of this hook would be to add tracking categories when
something is missing from the text (e.g. there are no references).
For the concrete use case, see I43ed79b6a54cd31820ecae8139e29c5880f5dd1b

Alternative approaches that have been suggested and do not require
a new hook but are not robust / do not rely on a clear contract:
* use ParserAfterTidy or a similar hook, assume that the input
  being parsed is the main page content iff limit reporting is
  enabled
* use ParserAfterTidy or a similar hook, assume that the input
  being parsed is the main page content iff
  ParserOptions::getInterfaceMessage() is false (this would yield
  some false positives, but in those cases adding a tracking
  category would probably have no effect)

Change-Id: I685b285fcc772382993116f7822a832eeecc0681

Lowered ActiveUsers query period to reduce timeouts

Change-Id: If34266cbac79d319fe33e1f362791fb438577fd7

Special:Contributions: Use mediawiki.ui

 * Use 'mw-ui-input-inline' for tagfilter input
 * Use 'mw-ui-input-inline' for IP/Username
 * Use 'mw-ui-progressive' for Search button
 * Vertically align the select dropdown

Bug: 71137
Change-Id: I20b0ebeb194ce0505abede6fb3bdd69b28350f04

Add "returntoquery" URL parameter support in Special:ChangeEmail

For consistency with other special pages, e.g. Special:ChangePassword.

Change-Id: I97eac516c8a366624413bfcd0f4bd9f1185b6419

Merge "Revert "Replace wfMsgReplaceArgs by RawMessage""

Merge "Article: Handle double redirects in data for mediawiki.action.view.redirect"

Revert "Replace wfMsgReplaceArgs by RawMessage"

This was merged too quickly, has several issues.

This reverts commit 6abefb274ff9cd86f75d5d13f713e1971e1b27ce.

Change-Id: Ibf3e464fa9eb534d599274dd9262a30d5a6a6948

Article: Handle double redirects in data for mediawiki.action.view.redirect

Bug: 71002
Change-Id: Id7dd37a4865635d3b68aac70179890be6ab89210

Replace wfMsgReplaceArgs by RawMessage

The replace of $n params can also be done using the Message class. In
case there are no message keys, the RawMessage class is used.

Deprecated wfMsgReplaceArgs, but added no warning, because it is used in
some extensions and from deprecated functions

Change-Id: I62091b09e4490e59ed7258566e0ddf2f8ee799d2

Merge "Move addTrackingCategory from Parser to ParserOutput"

API: Rewrite queries for list=allusers

The existing query only works with a single value for augroup, or mostly
if it's combined with auprop=groups or auprop=rights (since most users
don't have every possible group).

When used with multiple values for augroup, it will raise an error if it
happens to encounter enough users who have more than one of the
specified groups. And further, it will lead to repeated groups for these
users if combined with auprop=groups or auprop=rights.

An attempt to use EXISTS instead of a JOIN to query user_groups failed
because it made the planner scan the user table where a smaller scan of
user_groups plus a filesort was faster. So instead, let's just fix it
directly by acknowledging that supplying multiple groups for
augroup is going to give us duplicate rows.

At the same time, let's try using a subquery (with GROUP_CONCAT) to
fetch the actual groups the user belongs to, both to avoid processing so
many duplicate rows and to simplify the row-processing code. And let's
kill the forced index, it's probably not needed anymore.

Bug: 70496
Change-Id: Ic87dbc3b2d933775ca71a72932e0658e2f082bb6

Merge "Don't clobber revision size of 0"

Don't clobber revision size of 0

When a revision's size is 0, don't clobber it by changing it to null. This
causes, among other things, moves of empty pages to not show a revision
size. If we get to where the else used to be, either mSize is 0 (and doing
nothing is correct), or mSize is already null, in which case this didn't
do anything anyway. We know this because, above, we initialized it with
$this->mSize = isset( $row['len'] ) ? intval( $row['len'] ) : null;

Change-Id: I3d9b9163368c57b9a7906f6569119081b945d57e

Merge "Watchlist: changed "all" for the max number of days available"

Merge "Set rup/roa-rup autonym to "armãneashti""

Move addTrackingCategory from Parser to ParserOutput

addTrackingCategory is more in line with ParserOutput's functionality
(addLink, addCategory etc), and tracking categories are useful even for
content types which do not use the parser at all. There is no reason to
require the caller to obtain a Parser object just to be able to add
tracking categories.

Change-Id: I89d9ea1db3a4e6486e77eee940bd438f7753b776

Merge "Add window close warning to Special:Upload"

Merge "Implement mediawiki.confirmCloseWindow module"

Add window close warning to Special:Upload

Change-Id: I43bf828654d342893a19656ddd4151c2558d2f63

Localisation updates from https://translatewiki.net.

Change-Id: Icb08b905fced272db07a100be7f15ebed1ce2ccd

mediawiki.action.view.redirect: Remove deployment hack

Follow-up to c49bd9cb.

Change-Id: Ic3904d40e89533483434e074b2021d4ec3db6831

Fix case of SpecialAllPages/SpecialAllMessages in SpecialPageFactory

[autoloader] Class SpecialAllpages was loaded using incorrect case
[autoloader] Class SpecialAllmessages was loaded using incorrect case

Follow-Up: Ic137bc7adbc4a9ed96448ba0fee4807b67b3112d
Follow-Up: I21a164af0b6ec123bf654cd1e4e7085b1192f067
Change-Id: Idae65141de0de41bf4244408218a91c4b87d5b7d

Fix case of SpecialShortPages in SpecialPageFactory

[autoloader] Class ShortpagesPage was loaded using incorrect case

Just for information: This was not visible on Special:Specialpages.
In SpecialPageFactory::$list the class LongPagesPage is before
ShortPagesPage and loads that class due to "extends", when
ShortPagesPage should be loaded, the class is already there and no
warning is generated. The warning is shown, when reording the $list.

Change-Id: Ie0ac708b40ff51372545278bb694865650fa8b1f

Update CREDITS

Add myself as a developer, and sort the whole thing alphabetically.

Change-Id: Ib089c09fa0c52d8adba2058251c626d9161b048b

Merge "Remove dead code after removal of cancel button on Special:ChangeEmail"

Merge "FormatJson::parse( TRY_FIXING ) - remove trailing commas"

Merge "Bidi-isolate extension version on Special:Version"

Remove dead code after removal of cancel button on Special:ChangeEmail

Since the removal of the cancel button, the "wpCancel" parameter
cannot be present anymore, thus this check is now unneccessary.

Change-Id: Id72b29484554989ce8a639f2e0d3ecc07622edad

Merge "Remove executable bit from non-executable files"

Remove executable bit from non-executable files

Change-Id: I6def7588b970a15c4e6bd49dc4f7a7138687a99e

Localisation updates from https://translatewiki.net.

Change-Id: I2932704fa919944fc2ddf8c0578e4947e010974d

FormatJson::parse( TRY_FIXING ) - remove trailing commas

Removes trailing commas from json text when parsing
Solves very common cases like [1,2,3,]

Resulting status will be set to OK but not Good to warn caller

Change-Id: Ic0eb0a711da3ae578d6bb58d7474279d6845a4a7

Skip MailAddressTest::testNewFromUser on windows

1) MailAddressTest::testNewFromUser
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'Real name <foo@bar.baz>'
+'foo@bar.baz'

Change-Id: I0c9724130730c39a355c22d06b2c0f0629c1d0e3

Remove explict autoload from phpunit tests

The phpunit tests are loading all classes, so there is no need for an
explicit load.

Change-Id: Ifb26a5082e3366c44f6a1e65baf64c6ff9bf8449

Merge "Add release notes for skins/common/ removal"

Update OOjs UI to v0.1.0-pre (98e770ce46)

New changes:
888f1c6 Fix for endless loop in SelectWidget
28f6b94 PopupWidget: Make $element not be a 0x0 box

Change-Id: Id9c9e79e4a1fb4d3a42f7fec65c1360b3b192bfd

Merge "Remove Preferences::trySetUserEmail (deprecated 1.20)"

Merge "Remove LogEventsList::getDisplayTitle (deprecated 1.20)"

Merge "Remove XmlDumpWriter::schemaVersion (deprecated 1.20)"

Merge "Remove ApiQueryUsers::getAutoGroups (deprecated 1.20)"

Remove Preferences::trySetUserEmail (deprecated 1.20)

Change-Id: I31b008c93eae54b2aecc18154d0c2e3df3fdb712

Remove LogEventsList::getDisplayTitle (deprecated 1.20)

Change-Id: If8306adec1cd64064280f8ffa1163237ee427cf2

Remove XmlDumpWriter::schemaVersion (deprecated 1.20)

Change-Id: I725d19e46d4728d46d58bfbd3d5a9ad54c4df24a

Remove ApiQueryUsers::getAutoGroups (deprecated 1.20)

Change-Id: Idb43eca139c3e63f8a53ef076b2dfa7c5e3941b5

Merge "buttons: Update focus for quiet buttons"

Merge "buttons: Update focus state"

buttons: Update focus for quiet buttons

Bug: 62924
Change-Id: Ide6174c5da3a04b8e6c4370fbc191783e976f6ee

buttons: Update focus state

Currently the hover and focus state are exactly the same. The hover
state is subtle and making it hard to navigate using a keyboard.

Now the buttons have a border when focused.

Bug: 62924
Change-Id: Ibd75f7caaf5f34d8bcc53381335a048cede26950

Set site logo in mediawiki.skinning.interface module instead of inline styles

Requires skins use the 'mw-wiki-logo' class on the logo element for the styles
to apply and to load the 'mediawiki.skinning.interface' module (or implement
their own skin module that uses the ResourceLoaderSkinModule class).

This is backwards compatible with skins using inline styles as the class only
applies when used.

Bug: 56257
Change-Id: I4e6db89a688843ac24539f8fd1b408c0d04881b3

Localisation updates from https://translatewiki.net.

Change-Id: I0117cee6fead0262213b250e7844183362b03562

i18n: Use native digits on 'difference-missing-revision'

Change-Id: I19cd68562690c4ab8b077681d0f3d18d4286a5cd

Merge "QA: Do not mention Wikimedia Foundation wikis in browser tests"

Merge "API: Add ApiPageSet accessors for just good and missing titles"

COPYING: Update the list of CC licenses icons used

Change-Id: If2e28e63be25c0e2284c1469a466954f497db457

Merge "Added FormatJson::parse( $value, $options = 0 ) returning Status"