Most of this code was already typed, but not everything. Using
language-level type declarations allows us to remove extra PHPDoc
blocks that just repeat the same information.
I'm also using the more narrow UserIdentity instead of User in a
few places where this is possible.
Change-Id: I7661824fcb34180af1a4fd3030fcd6c0b7d34089
The following rules are failing and were disabled:
* modules/ve-mw/tests:
* implicit-arrow-linebreak
Change-Id: If857233c0de24c8cf619dbb1347ebb375f3ab1ba
New changes:
c82195c52 JSDoc: convert @cfg to @param
204f48a3e ve.ui.FindAndReplaceDialog: Use flexbox instead of tables
9205fbc97 Tests: Use spread syntax on string literals
84be2efbd Avoid Array.concat
4e956f400 Localisation updates from https://translatewiki.net.
Bug: T250843
Change-Id: Ic68fd49b00d87306a264f4a2d31cbe8955b03f16
I'm going to assume the order in production with Chrome, as asserted
by the unit test, is the preferred order whereby negative numbers are
intended to be prepended before the positive numbers.
This came up in I6ecbd5743f3e, where this was the only unit test across
MediaWiki core and WMF gated extensions, that fails in Firefox.
It is reproducible locally with plain MW core + VisualEditor as well.
Bug: T366299
Change-Id: I6f8546e6e604cbb41e11bd2b59f8b5f19350c676
Previously, it was possible to close a dialog with active edits by
pressing the "<" button or pressing escape. A change was made to confirm
the user's intent before abandoning their changes, see Ia8935b5b1acb
This patch fixes a bug where the user's intent is always confirmed while
editing a template, even if the user has made no changes. This was
because for technical reasons we trimmed whitespace before making a
comparison with the new template case, but that caused the comparison
with the edit case to always fail because existing templates are padded
with whitespace.
This could have been solved by moving the trim operation into the new
template flow. This patch would still have been necessary to prevent
a bug if the default value had trimmable whitespace. I have opted to
keep the whitespace behaviour for edits for consistency.
Bug: T334513
Change-Id: I7b3370c86df67c36fc63a1f1d0e7004a098a1950
Same as in If6c7e85 in the TemplateWizard codebase. [a-z] is already
part of \p{L}, and [0-9] is part of \p{N}.
Change-Id: I8008ac7e504739b8f4b5fb5dd732b0deca20cefc
New changes:
8a1eda8af build: Updating jsdoc-wmf-theme to 1.0.1
7e2e7ebaa JSDoc: Fix generated documentation of Storage objects
2a0229ea1 JSDoc: Fix more warnings
227a38f79 JSDoc: Use upstream version of externals plugin
e1b2cbc3b Localisation updates from https://translatewiki.net.
a588efb7c Convert inline functions to arrow functions so context variables can be removed
2a12cfd15 Localisation updates from https://translatewiki.net.
27296e0e3 Disable MutationObserver in ve.ce.Surface
Deleted files:
- build/jsdoc-externals.js
Bug: T230323
Bug: T365052
Change-Id: Ia9710a52cf407c1b1cc11216ed9e54cf89e3478d
