[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Avoid notices on large articles

2017-10-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382880 ) Change subject: Avoid notices on large articles .. Avoid notices on large articles Articles which have more than 500 contributors or 500

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: DONOTMERGE: check if tests are running

2017-10-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382854 ) Change subject: DONOTMERGE: check if tests are running .. DONOTMERGE: check if tests are running Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Add PHPUnit groups for unit tests

2017-10-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382851 ) Change subject: Add PHPUnit groups for unit tests .. Add PHPUnit groups for unit tests Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: WIP: Prepare to have one single template and simplify rendering

2017-10-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382752 ) Change subject: WIP: Prepare to have one single template and simplify rendering .. WIP: Prepare to have one single template and simplify

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Hygiene: Extract additional methods from renderCoverAndToc

2017-10-04 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382331 ) Change subject: Hygiene: Extract additional methods from renderCoverAndToc .. Hygiene: Extract additional methods from renderCoverAndToc

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Hygiene: renderCoverAndToc contains no logic

2017-10-04 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382332 ) Change subject: Hygiene: renderCoverAndToc contains no logic .. Hygiene: renderCoverAndToc contains no logic Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Hygiene: Make render functions dumber

2017-10-04 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382323 ) Change subject: Hygiene: Make render functions dumber .. Hygiene: Make render functions dumber Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: WIP: Refactor non-render logic out of render.

2017-10-04 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382324 ) Change subject: WIP: Refactor non-render logic out of render. .. WIP: Refactor non-render logic out of render. Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[wmf/1.31.0-wmf.2]: Correct feature phone threshold detection

2017-10-04 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/382238 ) Change subject: Correct feature phone threshold detection .. Correct feature phone threshold detection max-width should actually be one

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Delete unimplemented browser tests

2017-10-02 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381899 ) Change subject: Delete unimplemented browser tests .. Delete unimplemented browser tests These browser tests were added in

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Stop using wgNamespace and wgTitle globals

2017-10-02 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381892 ) Change subject: Stop using wgNamespace and wgTitle globals .. Stop using wgNamespace and wgTitle globals it's still using globals and I

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: WIP: Render book in one master template

2017-10-02 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381886 ) Change subject: WIP: Render book in one master template .. WIP: Render book in one master template This replaces all the templates with

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Provide message/warning/error box abstraction

2017-09-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381291 ) Change subject: Provide message/warning/error box abstraction .. Provide message/warning/error box abstraction This will help us

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Render add discussion button in PHP not JS

2017-09-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381315 ) Change subject: Render add discussion button in PHP not JS .. Render add discussion button in PHP not JS The talk page JavaScript

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Update SVGs for page actions

2017-09-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381281 ) Change subject: Update SVGs for page actions .. Update SVGs for page actions Additional changes: * userNormal was not minified. This has

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[wmf/1.31.0-wmf.1]: Revision::newFromTitle may return null

2017-09-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381259 ) Change subject: Revision::newFromTitle may return null .. Revision::newFromTitle may return null Bug: T176882 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[wmf/1.30.0-wmf.19]: Revision::newFromTitle may return null

2017-09-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381258 ) Change subject: Revision::newFromTitle may return null .. Revision::newFromTitle may return null Bug: T176882 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Restore feature phone threshold to 280px

2017-09-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381222 ) Change subject: Restore feature phone threshold to 280px .. Restore feature phone threshold to 280px Also rename to

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Load tablet device code on mobile devices

2017-09-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381098 ) Change subject: Load tablet device code on mobile devices .. Load tablet device code on mobile devices There is little benefit from

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Revision::newFromTitle may return null

2017-09-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381071 ) Change subject: Revision::newFromTitle may return null .. Revision::newFromTitle may return null Bug: T176882 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Hygiene: Merge selectors for removal in summarize

2017-09-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/381027 ) Change subject: Hygiene: Merge selectors for removal in summarize .. Hygiene: Merge selectors for removal in summarize Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Redirects return 204 inside MCS

2017-09-26 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380856 ) Change subject: Redirects return 204 inside MCS .. Redirects return 204 inside MCS These will be handled upstream in RestBase. To avoid

[MediaWiki-commits] [Gerrit] mediawiki...ElectronPdfService[master]: i18n: Wording nitpicks

2017-09-26 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380811 ) Change subject: i18n: Wording nitpicks .. i18n: Wording nitpicks Improved wording for translators and for readability of browser tests

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Only hide fmbox on editor interface

2017-09-26 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380743 ) Change subject: Only hide fmbox on editor interface .. Only hide fmbox on editor interface They also show up on pages that do not exist.

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Hygiene: Break out removeAttributes transformation

2017-09-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380581 ) Change subject: Hygiene: Break out removeAttributes transformation .. Hygiene: Break out removeAttributes transformation Allow this to

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Disable RelatedArticles instrumentation on all wikis

2017-09-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380554 ) Change subject: Disable RelatedArticles instrumentation on all wikis .. Disable RelatedArticles instrumentation on all wikis Bug:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Don't run diff test on beta cluster

2017-09-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380520 ) Change subject: Don't run diff test on beta cluster .. Don't run diff test on beta cluster While T176637 is open we should only run

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: User edit cta browser test should not run on beta cluster

2017-09-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380513 ) Change subject: QA: User edit cta browser test should not run on beta cluster .. QA: User edit cta browser test should not run on beta

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: API edits should use the Selenium user

2017-09-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/380505 ) Change subject: API edits should use the Selenium user .. API edits should use the Selenium user Otherwise every single test which uses

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Fix definition of lead introduction

2017-09-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/379936 ) Change subject: Fix definition of lead introduction .. Fix definition of lead introduction Bug: T176522 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: WIP: Improve parenthetical handling

2017-09-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/379930 ) Change subject: WIP: Improve parenthetical handling .. WIP: Improve parenthetical handling New test case shows current logic can be

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Test case for T176521

2017-09-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/379902 ) Change subject: Test case for T176521 .. Test case for T176521 Bug: T176521 Change-Id: Ifd0c47114973fd7c1cc1c2231a52947b5dfcbe34 --- M

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Strip any .reference elements

2017-09-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/379852 ) Change subject: Strip any .reference elements .. Strip any .reference elements Bug: T176519 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Third party main page logic is incorrect

2017-09-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/379783 ) Change subject: Third party main page logic is incorrect .. Third party main page logic is incorrect On a wiki where special casing of

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Avoid undefined notices when obtaining collection

2017-09-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376782 ) Change subject: Avoid undefined notices when obtaining collection .. Avoid undefined notices when obtaining collection Instead of

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Drop unused disableImages property

2017-09-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376754 ) Change subject: Hygiene: Drop unused disableImages property .. Hygiene: Drop unused disableImages property Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Allow extensions to override the print url

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376581 ) Change subject: Allow extensions to override the print url .. Allow extensions to override the print url Example usage: Checkout

[MediaWiki-commits] [Gerrit] mediawiki...ShortUrl[master]: Refactor ShortUrl to allow it to be used in printed version

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376580 ) Change subject: Refactor ShortUrl to allow it to be used in printed version .. Refactor ShortUrl to allow it to be used in printed

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Load mobile.init module for all mobile skins

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376557 ) Change subject: Load mobile.init module for all mobile skins .. Load mobile.init module for all mobile skins Bug: T173454 Depends-On:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: M.getCurrentPage and skin should be initialised by ...

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376553 ) Change subject: Hygiene: M.getCurrentPage and skin should be initialised by MobileFrontend .. Hygiene: M.getCurrentPage and skin should

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Do not pass mainMenu to Skin

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376554 ) Change subject: Hygiene: Do not pass mainMenu to Skin .. Hygiene: Do not pass mainMenu to Skin This wires the interaction between the

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Minerva uses skin instance defined by MobileFrontend

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376555 ) Change subject: Hygiene: Minerva uses skin instance defined by MobileFrontend .. Hygiene: Minerva uses skin instance defined by

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Remove MainMenu usage inside Skin

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376551 ) Change subject: Hygiene: Remove MainMenu usage inside Skin .. Hygiene: Remove MainMenu usage inside Skin Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Begin deprecation of Skin mainMenu option

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376550 ) Change subject: Hygiene: Begin deprecation of Skin mainMenu option .. Hygiene: Begin deprecation of Skin mainMenu option These are

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Cleanup initialisation files

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376546 ) Change subject: Hygiene: Cleanup initialisation files .. Hygiene: Cleanup initialisation files * Remove deprecated module definitions *

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Do not use deprecated fallback handling for History/Contribu...

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376543 ) Change subject: Do not use deprecated fallback handling for History/Contributions .. Do not use deprecated fallback handling for

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: OMG I am using const!

2017-09-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376523 ) Change subject: OMG I am using const! .. OMG I am using const! Change-Id: Ief8967e2efb1d8319608b078d42e905e5b4b902c --- M

[MediaWiki-commits] [Gerrit] mediawiki...ElectronPdfService[master]: Update banner message

2017-09-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376323 ) Change subject: Update banner message .. Update banner message Bug: T174147 Change-Id: Ie798a7363de0d58c2b8e880a90e4ff8e5a015b51 --- M

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hide last modified bar on old revisions

2017-09-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376282 ) Change subject: Hide last modified bar on old revisions .. Hide last modified bar on old revisions Bug: T153125 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Refactor SkinMinerva so it doesn't use MobilePage

2017-09-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376281 ) Change subject: Hygiene: Refactor SkinMinerva so it doesn't use MobilePage .. Hygiene: Refactor SkinMinerva so it doesn't use MobilePage

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Do not allow certain content to span entire page

2017-09-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376266 ) Change subject: Do not allow certain content to span entire page .. Do not allow certain content to span entire page Certain elements

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Improve hatnote styling

2017-09-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376261 ) Change subject: Improve hatnote styling .. Improve hatnote styling This improves the hatnote styling on pages such as Iran. Bug:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Show border of editor in Minerva edit mode

2017-09-05 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376141 ) Change subject: Show border of editor in Minerva edit mode .. Show border of editor in Minerva edit mode Bug: T174158 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: EventLogging modules should be soft dependencies

2017-09-05 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376135 ) Change subject: EventLogging modules should be soft dependencies .. EventLogging modules should be soft dependencies Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Run eslint on compiled assets to ensure code is es5 compatible

2017-09-05 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376100 ) Change subject: Run eslint on compiled assets to ensure code is es5 compatible .. Run eslint on compiled assets to ensure code is es5

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: mediawiki.experiments is a dependency of RelatedArticles

2017-09-05 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/376083 ) Change subject: mediawiki.experiments is a dependency of RelatedArticles .. mediawiki.experiments is a dependency of RelatedArticles

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[wmf/1.30.0-wmf.16]: Adjust language icon color

2017-09-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375398 ) Change subject: Adjust language icon color .. Adjust language icon color The hex code for the fill color was wrong resulting in it

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Adjust language icon color

2017-09-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375395 ) Change subject: Adjust language icon color .. Adjust language icon color icons. Bug: T174747 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: Make RelatedArticles tests QUnit 2 compatible

2017-09-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375390 ) Change subject: Make RelatedArticles tests QUnit 2 compatible .. Make RelatedArticles tests QUnit 2 compatible Bug: T174598 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP: Port Popups browser tests to selenium

2017-09-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375384 ) Change subject: WIP: Port Popups browser tests to selenium .. WIP: Port Popups browser tests to selenium Bug: T160238 Change-Id:

[MediaWiki-commits] [Gerrit] integration/config[master]: Add a Jenkins job for Popups browser tests

2017-09-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375377 ) Change subject: Add a Jenkins job for Popups browser tests .. Add a Jenkins job for Popups browser tests Bug: T174786 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: ls

2017-09-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375366 ) Change subject: ls .. ls Change-Id: I599c8c002c570d4911b7c86294f714279833e80b --- M tests/browser/LocalSettings.php 1 file changed, 1

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Hygiene: Refactor execute function

2017-08-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375075 ) Change subject: Hygiene: Refactor execute function .. Hygiene: Refactor execute function Trying to make sense of all the commands - the

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Hygiene: Book creator HTML is created via template

2017-08-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375070 ) Change subject: Hygiene: Book creator HTML is created via template .. Hygiene: Book creator HTML is created via template This is the

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Hygiene: Reorganise the ResourceLoader modules

2017-08-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375068 ) Change subject: Hygiene: Reorganise the ResourceLoader modules .. Hygiene: Reorganise the ResourceLoader modules * Organise files by

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Parenthetical stripping should not be greedy

2017-08-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375056 ) Change subject: Parenthetical stripping should not be greedy .. Parenthetical stripping should not be greedy Bug: T174698 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Fix watchstar icon

2017-08-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/375028 ) Change subject: Fix watchstar icon .. Fix watchstar icon xml tag was not correctly closed with "?>" Bug: T174607 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Make Minerva QUnit tests v2 compatible

2017-08-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374992 ) Change subject: Make Minerva QUnit tests v2 compatible .. Make Minerva QUnit tests v2 compatible Drop the checks for number of

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: QUnit: Drop assertions for expected number of tests

2017-08-30 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374908 ) Change subject: QUnit: Drop assertions for expected number of tests .. QUnit: Drop assertions for expected number of tests Drop

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Use ES5 methods

2017-08-30 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374842 ) Change subject: Hygiene: Use ES5 methods .. Hygiene: Use ES5 methods * Use Array.isArray instead of $.isArray Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Refactor: Use mustache templates to create download box

2017-08-29 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374637 ) Change subject: Refactor: Use mustache templates to create download box .. Refactor: Use mustache templates to create download box The

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: MobileFrontend shouldn't be medling with preferences on beha...

2017-08-29 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374617 ) Change subject: MobileFrontend shouldn't be medling with preferences on behalf of Minerva .. MobileFrontend shouldn't be medling with

[MediaWiki-commits] [Gerrit] mediawiki...Popups[wmf/1.30.0-wmf.15]: Remove ES6 usages in Popups distribution js

2017-08-29 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374582 ) Change subject: Remove ES6 usages in Popups distribution js .. Remove ES6 usages in Popups distribution js Bug: T174424 This merges: *

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Enable mediawiki.hlist on mobile

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374421 ) Change subject: Enable mediawiki.hlist on mobile .. Enable mediawiki.hlist on mobile Bug: T42062 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Merge skins.minerva.print.styles with skins.minerva...

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374418 ) Change subject: Hygiene: Merge skins.minerva.print.styles with skins.minerva.base.styles .. Hygiene: Merge skins.minerva.print.styles

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Remove FIXMEs now enough time has passed

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374417 ) Change subject: Hygiene: Remove FIXMEs now enough time has passed .. Hygiene: Remove FIXMEs now enough time has passed Routine

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Switch to @colorGray5 for wikitable borders

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374414 ) Change subject: Switch to @colorGray5 for wikitable borders .. Switch to @colorGray5 for wikitable borders Bug: T168029 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Show CTA at bottom of anonymous editor warning

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374412 ) Change subject: Show CTA at bottom of anonymous editor warning .. Show CTA at bottom of anonymous editor warning Bug: T164628 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Add spacing to anonymous editing message

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374408 ) Change subject: Add spacing to anonymous editing message .. Add spacing to anonymous editing message (Call to action will be added in a

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Drop Android2 check for search

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374405 ) Change subject: Drop Android2 check for search .. Drop Android2 check for search Android2 is a grade C browser so this code will never

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Remove checks for Android2

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374404 ) Change subject: Hygiene: Remove checks for Android2 .. Hygiene: Remove checks for Android2 Android2 is now a grade C browser, so we

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Enable Popups on en and de wiki for A/B test

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374390 ) Change subject: Enable Popups on en and de wiki for A/B test .. Enable Popups on en and de wiki for A/B test Bug: T172291 Change-Id:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Add missing wg Prefix

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374378 ) Change subject: Add missing wg Prefix .. Add missing wg Prefix Follow up to: Ia43bd4868e9e3a9fae97088a53a7f56bd1bc0078 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Logout icon should not be small

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374333 ) Change subject: Logout icon should not be small .. Logout icon should not be small Bug: T170362 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: fmboxes should be treated the same as ambox and tmboxes

2017-08-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374330 ) Change subject: fmboxes should be treated the same as ambox and tmboxes .. fmboxes should be treated the same as ambox and tmboxes

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Use Function.bind in favor of $.proxy

2017-08-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373962 ) Change subject: Use Function.bind in favor of $.proxy .. Use Function.bind in favor of $.proxy \$\.proxy\( ([^\,]*)\, \'([^\)\,\']*)\'?

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Control use of $.Deferred

2017-08-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373963 ) Change subject: Control use of $.Deferred .. Control use of $.Deferred Avoid using $.Deferred unless truly necessary. We have a tendency

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Enable an A/B test for page previews on EN and DE wikis

2017-08-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373920 ) Change subject: Enable an A/B test for page previews on EN and DE wikis .. Enable an A/B test for page previews on EN and DE wikis

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hide last modified text in footer

2017-08-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373916 ) Change subject: Hide last modified text in footer .. Hide last modified text in footer This repeats information in the last modified BAR

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Do not try to enable toggling if using desktop Minerva

2017-08-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373914 ) Change subject: Do not try to enable toggling if using desktop Minerva .. Do not try to enable toggling if using desktop Minerva There

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Collapse amboxes in edit view

2017-08-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373910 ) Change subject: Collapse amboxes in edit view .. Collapse amboxes in edit view In edit mode various amboxes are shown which push the

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Caption should inherit background from table

2017-08-24 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373623 ) Change subject: Caption should inherit background from table .. Caption should inherit background from table Since we are fiddling with

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Lighten table border color

2017-08-24 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373614 ) Change subject: Lighten table border color .. Lighten table border color Bug: T168029 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Tests

2017-08-24 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373560 ) Change subject: Tests .. Tests Change-Id: I00425b4b60c2567706cbd1b1de36cb8e0db7000d --- M

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Avoid global access of elements outside Views

2017-08-23 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373372 ) Change subject: Avoid global access of elements outside Views .. Avoid global access of elements outside Views PointerOverlay,

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Drop usages of inArray

2017-08-23 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373364 ) Change subject: Hygiene: Drop usages of inArray .. Hygiene: Drop usages of inArray We can guarantee ES5 support so we can guarantee

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Prepare for adoption of mw-ui-icon-large on user page

2017-08-23 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373361 ) Change subject: Prepare for adoption of mw-ui-icon-large on user page .. Prepare for adoption of mw-ui-icon-large on user page When

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Use mw-ui-icon-large and ResourceLoaderImage module for user...

2017-08-23 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373362 ) Change subject: Use mw-ui-icon-large and ResourceLoaderImage module for user page .. Use mw-ui-icon-large and ResourceLoaderImage module

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Revert "Revert "pagePreviews: Enable A/B test (BC-only)""

2017-08-23 Thread Jdlrobson (Code Review)
Hello Urbanecm, Niharika29, Florianschmidtwelzow, Jhernandez, jenkins-bot, Phuedx, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/373353 to review the following change. Change subject: Revert "Revert "pagePreviews: Enable A/B test (BC-only)""

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Set line height to 1 for sup and sub elements

2017-08-23 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373315 ) Change subject: Set line height to 1 for sup and sub elements .. Set line height to 1 for sup and sub elements Bug: T172501 Change-Id:

<    1   2   3   4   5   6   7   8   9   10   >