[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Remove CiteThisPage from blacklist for page previews

2017-08-23 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373300 ) Change subject: Remove CiteThisPage from blacklist for page previews .. Remove CiteThisPage from blacklist for page previews Bug:

[MediaWiki-commits] [Gerrit] mediawiki...Vector[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/373299 ) 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:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[wmf/1.30.0-wmf.14]: Verify the existence of `url` key when parsing lang objects

2017-08-23 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373292 ) Change subject: Verify the existence of `url` key when parsing lang objects .. Verify the existence of `url` key when parsing lang

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: WIP: Limit usage of global jQuery in preference of ES6

2017-08-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373139 ) Change subject: WIP: Limit usage of global jQuery in preference of ES6 .. WIP: Limit usage of global jQuery in preference of ES6 *

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Use String.prototype.trim

2017-08-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373138 ) Change subject: Hygiene: Use String.prototype.trim .. Hygiene: Use String.prototype.trim No need to use $.trim when we support ES5

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Adjust TOC print styles

2017-08-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373120 ) Change subject: Adjust TOC print styles .. Adjust TOC print styles * Font size shrunk and declared in pt * Margin adjustment *

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Hygiene: Correct summary spec test

2017-08-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373093 ) Change subject: Hygiene: Correct summary spec test .. Hygiene: Correct summary spec test The spec comment incorrectly says a link will

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Do not run various toast notification browser tests on BC

2017-08-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373085 ) Change subject: QA: Do not run various toast notification browser tests on BC .. QA: Do not run various toast notification browser tests

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Disable wikitext editor on non-wikitext content models

2017-08-22 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373076 ) Change subject: Disable wikitext editor on non-wikitext content models .. Disable wikitext editor on non-wikitext content models Do not

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Make sure referenced file exists

2017-08-21 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372882 ) Change subject: Make sure referenced file exists .. Make sure referenced file exists Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Add test case for summary endpoint

2017-08-21 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372875 ) Change subject: Add test case for summary endpoint .. Add test case for summary endpoint Change-Id:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Hygiene: Remove dead config variable

2017-08-21 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372838 ) Change subject: Hygiene: Remove dead config variable .. Hygiene: Remove dead config variable Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Revert "Import the right files"

2017-08-18 Thread Jdlrobson (Code Review)
Hello Pmiazga, jenkins-bot, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/372562 to review the following change. Change subject: Revert "Import the right files" .. Revert "Import the

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Enable mediawiki.special.userlogin.common.styles on mobile a...

2017-08-18 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372561 ) Change subject: Enable mediawiki.special.userlogin.common.styles on mobile and allow skins to style it .. Enable

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: WIP: Remove special page handling

2017-08-18 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372560 ) Change subject: WIP: Remove special page handling .. WIP: Remove special page handling This is handled by MobileFrontend now, just as it

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Remove and tidy some Minerva references

2017-08-18 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372558 ) Change subject: Hygiene: Remove and tidy some Minerva references .. Hygiene: Remove and tidy some Minerva references * Update inline

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: MobileFrontend should not add Minerva modules for its specia...

2017-08-18 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372559 ) Change subject: MobileFrontend should not add Minerva modules for its special pages .. MobileFrontend should not add Minerva modules for

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Add mobile and desktop thresholds

2017-08-18 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372551 ) Change subject: Add mobile and desktop thresholds .. Add mobile and desktop thresholds This will allow us to centralise these values

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Import the right files

2017-08-18 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372539 ) Change subject: Import the right files .. Import the right files These modules are throwing Less compiler errors. The variables being

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Do not use numbered grays

2017-08-18 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372538 ) Change subject: Hygiene: Do not use numbered grays .. Hygiene: Do not use numbered grays We have names for a reason. We should limit our

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Standardise on MinervaNeue skin name

2017-08-17 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372472 ) Change subject: Standardise on MinervaNeue skin name .. Standardise on MinervaNeue skin name Bug: T171644 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Check if SpecialMobileWatchlist exist

2017-08-17 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372470 ) Change subject: Check if SpecialMobileWatchlist exist .. Check if SpecialMobileWatchlist exist If MobileFrontend doesn't exist this will

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Give the Special:preferences page on desktop some love

2017-08-17 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372461 ) Change subject: Give the Special:preferences page on desktop some love .. Give the Special:preferences page on desktop some love The

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Correct documentation

2017-08-17 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372420 ) Change subject: Hygiene: Correct documentation .. Hygiene: Correct documentation Result of code review session on 17th August 2017. We

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: MediawikiUI: Introduce mw-ui-icon-large

2017-08-17 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372403 ) Change subject: MediawikiUI: Introduce mw-ui-icon-large .. MediawikiUI: Introduce mw-ui-icon-large This will allow MobileFrontend and

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: User page should use mw-ui-icon-large

2017-08-17 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372402 ) Change subject: User page should use mw-ui-icon-large .. User page should use mw-ui-icon-large Bug: T143264 Depends-On:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Do not assume MobileFrontend special pages exist

2017-08-16 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372183 ) Change subject: Do not assume MobileFrontend special pages exist .. Do not assume MobileFrontend special pages exist If they don't exist

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: WIP: Break the MobileFrontend dependency on Minerva

2017-08-16 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372186 ) Change subject: WIP: Break the MobileFrontend dependency on Minerva .. WIP: Break the MobileFrontend dependency on Minerva Remaining

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: i18n: Add Minerva specific message key for last modified bar

2017-08-16 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372184 ) Change subject: i18n: Add Minerva specific message key for last modified bar .. i18n: Add Minerva specific message key for last modified

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: WIP: Port over minimum required LESS variables

2017-08-16 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372185 ) Change subject: WIP: Port over minimum required LESS variables .. WIP: Port over minimum required LESS variables Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Return promises from QUnit test

2017-08-16 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372173 ) Change subject: Return promises from QUnit test .. Return promises from QUnit test Simplify all our tests to return to promises Use

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Use proto current not proto relative

2017-08-16 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372167 ) Change subject: Use proto current not proto relative .. Use proto current not proto relative Per the usage instructions what we actually

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Roll page previews out to all wikis except en and de wiki

2017-08-16 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372160 ) Change subject: Roll page previews out to all wikis except en and de wiki .. Roll page previews out to all wikis except en and de wiki

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Add tests for extractLeadIntroduction and fix edge case

2017-08-15 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372079 ) Change subject: Add tests for extractLeadIntroduction and fix edge case .. Add tests for extractLeadIntroduction and fix edge case Add

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Make it easier to run unit tests

2017-08-15 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372080 ) Change subject: Make it easier to run unit tests .. Make it easier to run unit tests Currently `npm test` runs integration tests which

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: User always passed in as an argument to classes

2017-08-15 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372077 ) Change subject: User always passed in as an argument to classes .. User always passed in as an argument to classes Depends-On: To be

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Add phan config to Popups

2017-08-15 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372062 ) Change subject: Add phan config to Popups .. Add phan config to Popups Bug: T133664 Change-Id: I57de60a783cd011e93aef87c81187dbbf92cb270

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Add phan config to MinervaNeue

2017-08-15 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372061 ) Change subject: Add phan config to MinervaNeue .. Add phan config to MinervaNeue phantastic! Bug: T133664 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Add MinervaNeue Doxyfile

2017-08-15 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372060 ) Change subject: Add MinervaNeue Doxyfile .. Add MinervaNeue Doxyfile We forgot to copy this across when migrating Minerva out of

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: MobileFrontend should be responsible for adding mobile.site ...

2017-08-11 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/371498 ) Change subject: MobileFrontend should be responsible for adding mobile.site module .. MobileFrontend should be responsible for adding

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Load mobile.site on all mobile skins

2017-08-11 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/371499 ) Change subject: Load mobile.site on all mobile skins .. Load mobile.site on all mobile skins Bug: T171062 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP: A/B test infrastructure

2017-08-10 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/371129 ) Change subject: WIP: A/B test infrastructure .. WIP: A/B test infrastructure Introduce getUserBucket which determines whether a user is

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Correct config - commonswiki not commons

2017-08-09 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370924 ) Change subject: Correct config - commonswiki not commons .. Correct config - commonswiki not commons Bug: T170687 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Simplify toast notification step some more

2017-08-09 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370880 ) Change subject: QA: Simplify toast notification step some more .. QA: Simplify toast notification step some more If

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: Merge footer hooks with Hooks

2017-08-09 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370876 ) Change subject: Merge footer hooks with Hooks .. Merge footer hooks with Hooks per Baha's suggestion we have no need for two Hooks files

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Disable browser test in @chrome

2017-08-09 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370844 ) Change subject: QA: Disable browser test in @chrome .. QA: Disable browser test in @chrome This is failing consistently due to T172835.

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Workaround for T172835

2017-08-09 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370806 ) Change subject: QA: Workaround for T172835 .. QA: Workaround for T172835 Ensure the user is logged in and has made several edits to

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: POC: Send all the links for disambiguation pages in the summary

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370754 ) Change subject: POC: Send all the links for disambiguation pages in the summary .. POC: Send all the links for disambiguation pages in

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Always extract an infobox and paragraph

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370736 ) Change subject: Always extract an infobox and paragraph .. Always extract an infobox and paragraph There is undocumented logic limiting

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Empty tags are omitted from summary output

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370697 ) Change subject: Empty tags are omitted from summary output .. Empty tags are omitted from summary output Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Per spec return summary type

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370694 ) Change subject: Per spec return summary type .. Per spec return summary type Change-Id: I4e1c84cc490dc86551cae86bc4b96eafad42a89e --- M

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Pages with empty lead sections should send a 204

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370692 ) Change subject: Pages with empty lead sections should send a 204 .. Pages with empty lead sections should send a 204 If preview is an

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Relax toast notification QA steps

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370667 ) Change subject: QA: Relax toast notification QA steps .. QA: Relax toast notification QA steps It seems trying to test both the steps

[MediaWiki-commits] [Gerrit] integration/config[master]: WebdriverIO tests should look for LocalSettings.php in selen...

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370662 ) Change subject: WebdriverIO tests should look for LocalSettings.php in selenium folder .. WebdriverIO tests should look for

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: DONOTMERGE

2017-08-08 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370657 ) Change subject: DONOTMERGE .. DONOTMERGE Change-Id: I31ee31b6239106d92735519b9c25028fa92a4599 --- D

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Reflect template change

2017-08-07 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370513 ) Change subject: Reflect template change .. Reflect template change Link has changed from Help:IPA/English

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Look for dablinks when extracting hatnotes

2017-08-06 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370371 ) Change subject: Look for dablinks when extracting hatnotes .. Look for dablinks when extracting hatnotes Certain wiki pages use older

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Scrub #coordinates from response

2017-08-04 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370253 ) Change subject: Scrub #coordinates from response .. Scrub #coordinates from response Now we are using the MediaWiki coordinates prop to

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Don't output empty footer items in desktop Minerva

2017-08-04 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370215 ) Change subject: Don't output empty footer items in desktop Minerva .. Don't output empty footer items in desktop Minerva These fields

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: The main page check is not necessary

2017-08-03 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370109 ) Change subject: Hygiene: The main page check is not necessary .. Hygiene: The main page check is not necessary The main page in the

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Padding tweak for all lists

2017-08-03 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370105 ) Change subject: Padding tweak for all lists .. Padding tweak for all lists This is particularly important in references where this small

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Page.js getLeadSectionElement

2017-08-03 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/370087 ) Change subject: Page.js getLeadSectionElement .. Page.js getLeadSectionElement * Add tests * When pages are not mobile formatted return

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: QA: Move LocalSettings to folder with other browser tests

2017-08-02 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369666 ) Change subject: QA: Move LocalSettings to folder with other browser tests .. QA: Move LocalSettings to folder with other browser tests

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: MCS no longer uses mobile view API under the hood

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369580 ) Change subject: MCS no longer uses mobile view API under the hood .. MCS no longer uses mobile view API under the hood After this change

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Use css rather than class for finding toast

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369561 ) Change subject: QA: Use css rather than class for finding toast .. QA: Use css rather than class for finding toast Bug: T170890

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Exclude files from Special:ShortPages on commons

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369503 ) Change subject: Exclude files from Special:ShortPages on commons .. Exclude files from Special:ShortPages on commons Bug: T170687

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Disable page previews on a variety of pages

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369494 ) Change subject: Disable page previews on a variety of pages .. Disable page previews on a variety of pages Bug: T170893 Change-Id:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Update wordmark for Wikipedia Atikamekw

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369464 ) Change subject: Update wordmark for Wikipedia Atikamekw .. Update wordmark for Wikipedia Atikamekw Bug: T168203 * Editor should jump to

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Relax navbox hack

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369453 ) Change subject: Relax navbox hack .. Relax navbox hack Per discussion in T172078 let's relax these rules so navbox shows up in the

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Make sure toast is not empty before checking text value

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369450 ) Change subject: QA: Make sure toast is not empty before checking text value .. QA: Make sure toast is not empty before checking text

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Style Parsoid generated references

2017-08-01 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369399 ) Change subject: Style Parsoid generated references .. Style Parsoid generated references Bug: T171727 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: WIP: Do not use MobileView to source metadata

2017-07-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368959 ) Change subject: WIP: Do not use MobileView to source metadata .. WIP: Do not use MobileView to source metadata Bug: T103362 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Source coordinates from GeoData

2017-07-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368954 ) Change subject: Source coordinates from GeoData .. Source coordinates from GeoData Rather than rely on Parsoid HTML, source geodata from

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: DRY: Save repetition of formatversion and format api parameters

2017-07-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368953 ) Change subject: DRY: Save repetition of formatversion and format api parameters .. DRY: Save repetition of formatversion and format api

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: QA: Minor tweaks to how we check toast notifications

2017-07-31 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368942 ) Change subject: QA: Minor tweaks to how we check toast notifications .. QA: Minor tweaks to how we check toast notifications A toast

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Fix pronounication portion of API

2017-07-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368526 ) Change subject: Fix pronounication portion of API .. Fix pronounication portion of API Bug: T170325 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...mobileapps[master]: Fix broken tests (do not skip)

2017-07-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368520 ) Change subject: Fix broken tests (do not skip) .. Fix broken tests (do not skip) Skipping should only be used as a short term measure.

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: WIP: Having 2 folders for browser tests is confusing.

2017-07-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368457 ) Change subject: WIP: Having 2 folders for browser tests is confusing. .. WIP: Having 2 folders for browser tests is confusing. Should we

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: MwContentProvider should honour the skin name

2017-07-28 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368452 ) Change subject: MwContentProvider should honour the skin name .. MwContentProvider should honour the skin name As we've discovered the

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Allow blacklisting certain namespaces in Special:ShortPages

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368325 ) Change subject: Allow blacklisting certain namespaces in Special:ShortPages .. Allow blacklisting certain namespaces in

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Code review session: DONOTMERGE

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368314 ) Change subject: Code review session: DONOTMERGE .. Code review session: DONOTMERGE Please comment on the lines of code in this patch.

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: mediawiki.legacy.shared can be run in mobile target

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368313 ) Change subject: mediawiki.legacy.shared can be run in mobile target .. mediawiki.legacy.shared can be run in mobile target Minerva

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Blank parsoid styles, update thumbnail selectors for Parsoid...

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368306 ) Change subject: Blank parsoid styles, update thumbnail selectors for Parsoid content .. Blank parsoid styles, update thumbnail selectors

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Allow content to be sourced from Parsoid

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368287 ) Change subject: Allow content to be sourced from Parsoid .. Allow content to be sourced from Parsoid This helps with testing Apps

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Browser tests: Use generic toast step

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368260 ) Change subject: Browser tests: Use generic toast step .. Browser tests: Use generic toast step Change-Id:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Compute list of English Wikipedia logos

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368237 ) Change subject: Compute list of English Wikipedia logos .. Compute list of English Wikipedia logos Change-Id:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Define custom logos using wmg

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368236 ) Change subject: Define custom logos using wmg .. Define custom logos using wmg The merge strategy of arrays is problematic. To avoid

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Run stylelint in `npm test`

2017-07-27 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368202 ) Change subject: Run stylelint in `npm test` .. Run stylelint in `npm test` This ensures Jenkin will complain if stylelint errors are

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: DONOTMERGE: testing someting

2017-07-26 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368115 ) Change subject: DONOTMERGE: testing someting .. DONOTMERGE: testing someting Change-Id: I6908872ece922772ddcf5f22daae6bb61ab8ddaa --- M

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Toast selector should be more flexible

2017-07-26 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368114 ) Change subject: Hygiene: Toast selector should be more flexible .. Hygiene: Toast selector should be more flexible A toast can carry

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: DONOTMERGE: Remove main page special casing code

2017-07-26 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/368109 ) Change subject: DONOTMERGE: Remove main page special casing code .. DONOTMERGE: Remove main page special casing code Change-Id:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Update several Wikipedia projects to existing wordmarks

2017-07-26 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367938 ) Change subject: Update several Wikipedia projects to existing wordmarks .. Update several Wikipedia projects to existing wordmarks Bug:

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Print styles: Border bottom of headings should not overlap i...

2017-07-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367817 ) Change subject: Print styles: Border bottom of headings should not overlap infoboxes .. Print styles: Border bottom of headings should

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Print styles: Apply padding and margins to ol as well as ul

2017-07-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367814 ) Change subject: Print styles: Apply padding and margins to ol as well as ul .. Print styles: Apply padding and margins to ol as well as

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Print mode: Define when to use serif and when to use sans-serif

2017-07-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367812 ) Change subject: Print mode: Define when to use serif and when to use sans-serif .. Print mode: Define when to use serif and when to use

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: Restore default MediaWiki codesniffer configuration

2017-07-25 Thread Jdlrobson (Code Review)
Jdlrobson has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/365042 ) Change subject: Restore default MediaWiki codesniffer configuration .. Restore default MediaWiki codesniffer configuration

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Hygiene: Drop MFAllowNonJavaScriptEditing and Special:Mobile...

2017-07-25 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367688 ) Change subject: Hygiene: Drop MFAllowNonJavaScriptEditing and Special:MobileEditor .. Hygiene: Drop MFAllowNonJavaScriptEditing and

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Fix stylesheet

2017-07-24 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367627 ) Change subject: Fix stylesheet .. Fix stylesheet You can't nest like this. It creates the invalid selector

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: SkinMinerva should deal with edit section link not MobileFor...

2017-07-24 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367477 ) Change subject: SkinMinerva should deal with edit section link not MobileFormatter .. SkinMinerva should deal with edit section link not

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Allow us to run mobile formatter on desktop skin

2017-07-24 Thread Jdlrobson (Code Review)
Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367471 ) Change subject: Allow us to run mobile formatter on desktop skin .. Allow us to run mobile formatter on desktop skin We won't use this

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