Commit graph

4020 commits

Author SHA1 Message Date
Eddie Greiner-Petter b0384c4314 Show title on the page named '0'
The string '0' evals to false in the if( $pageTitle ) condition, hence
the title was not shown for the page named '0'.

Bug: T163084
Change-Id: I27ab379bf67634c484c4cf5e12e391d69702f464
2017-07-30 19:50:34 +02:00
Translation updater bot 481bafcdc9 Localisation updates from https://translatewiki.net.
Change-Id: I0205c5600c1545348630bb4b5f7d5c217864d436
2017-07-29 22:14:37 +02:00
Translation updater bot 1be831129a Localisation updates from https://translatewiki.net.
Change-Id: I8db0b5d406cc057e807c14274396458248d4a4fc
2017-07-28 22:02:24 +02:00
jenkins-bot d746b6d970 Merge "Browser tests: Use generic toast step" 2017-07-27 21:33:19 +00:00
jdlrobson a6a093faab Blank parsoid styles, update thumbnail selectors for Parsoid content
This brings Parsoid styles when viewed in Minerva consistent with
MediaWiki parser output styles.

You can verify this with:
https://gerrit.wikimedia.org/r/#/c/368287/

Bug: T171727
Change-Id: If4bd7671a99aefdf074061dd1d5b0bac93620e9e
2017-07-27 14:27:47 -07:00
jdlrobson d5a60fdd43 Browser tests: Use generic toast step
Introduces a new generic
"I should see a toast with message ".*""
step reducing toast steps to two generic ones.

Change-Id: Ic8b91c78f6df088244f15223ee4ed658847a05b5
2017-07-27 13:41:54 -07:00
Translation updater bot 5046bb075b Localisation updates from https://translatewiki.net.
Change-Id: Ic9711e9404dd028bc471ff5895f95a698372c380
2017-07-27 22:32:25 +02:00
jenkins-bot ed427dd427 Merge "Don't show ImageOverlay in desktop mode - delegate to multimedia viewer" 2017-07-27 15:30:57 +00:00
jdlrobson 9500297090 Don't show ImageOverlay in desktop mode - delegate to multimedia viewer
Bug: T169622
Change-Id: Ie34297a56f8571b161802ea8a708a8bed37c9173
2017-07-27 15:26:31 +00:00
Fomafix 13bdb53845 chmod a-x resources/*/*.{less,svg}
Change-Id: Ic4a2c91c8282a30ae069cc33deffada1df66ff22
2017-07-26 22:14:13 +00:00
Translation updater bot 97f87921b6 Localisation updates from https://translatewiki.net.
Change-Id: Ia7783e37cecf767e0d89975784e39d77a1913141
2017-07-26 22:01:30 +02:00
jenkins-bot 78daa06cc1 Merge "Hygiene: Drop MFAllowNonJavaScriptEditing and Special:MobileEditor" 2017-07-26 18:40:09 +00:00
Ed Sanders d2b10cc357 Disable magnify icon in Parsoid output
Bug: T160960
Depends-On: Id66f09c54103854ccbaa54a03e7c62890b67cba9
Change-Id: I40243fa6d85263967314b2b30dd1a7cbda39234d
2017-07-26 17:40:18 +00:00
jenkins-bot 112587b227 Merge "Remove a full stop from the end of the tooltip" 2017-07-25 21:17:18 +00:00
Translation updater bot f31b3bf9df Localisation updates from https://translatewiki.net.
Change-Id: I46cb65a7b894ab607484bab82dd83efd496add5b
2017-07-25 22:01:48 +02:00
jdlrobson 3ad9c4e3c6 Hygiene: Drop MFAllowNonJavaScriptEditing and Special:MobileEditor
Bug: T125174
Change-Id: Ifc0efcb87605e5cd678343d1ac91a2ce53b33664
2017-07-25 11:07:03 -07:00
Translation updater bot e81e5cd11d Localisation updates from https://translatewiki.net.
Change-Id: Iaeefb43cbf364545db4b426d44f5bf99cb993802
2017-07-24 22:29:00 +02:00
Umherirrender 2aacf47c8d build: Updating mediawiki/mediawiki-codesniffer to 0.10.1
Change-Id: Icba559465885af1169a1912f501bed56eea23ad9
2017-07-24 18:53:04 +02:00
Piotr Miazga 6819b6e163 Hygiene: Use ExtensionRegistry to manage skin dependencies
Changes:
 - removed MobileFrontend existence check from MinervaHooks
 - added MobileFrontend as a requirement inside skin.json

Additional benefit: Skin will work only with MobileFrontend v2.0.0
and higher.

Change-Id: I1f2f5de3dbc0cf5159a06bc8f47e1cf401a2fcdb
2017-07-24 15:58:21 +00:00
Translation updater bot af2210fb87 Localisation updates from https://translatewiki.net.
Change-Id: Ic41bf09bdcf77a14ecdacc03fb050ff3cc3f8da5
2017-07-23 22:05:29 +02:00
Translation updater bot b716f70ced Localisation updates from https://translatewiki.net.
Change-Id: I4ce49b8b5fb44df5d367ee906186353fdbe03905
2017-07-22 22:06:11 +02:00
Amire80 8dcbf26815 Remove a full stop from the end of the tooltip
There are no full stops on other tooltips.

Change-Id: I807b20eeeccc343baffb42e57f6e97ad03ea3d55
2017-07-22 10:28:15 +03:00
Translation updater bot 07828dcdae Localisation updates from https://translatewiki.net.
Change-Id: I8f0d5240a7acab01724730cd24c45d8a312f2e20
2017-07-21 22:05:53 +02:00
jenkins-bot bd898e2be4 Merge "Minerva should use mediawiki.hlist" 2017-07-21 18:47:35 +00:00
jenkins-bot 7d0501127f Merge "Hygiene: Remove and update references to MobileFrontend" 2017-07-21 18:47:12 +00:00
jenkins-bot 27504d67c0 Merge "Add tooling for documentation" 2017-07-20 22:37:59 +00:00
jdlrobson efc84d2f25 Add tooling for documentation
* Add global View and Page
* Remove inheritdoc statements for globals - docs
  cannot be inherited from a class defined in another repo

Note:
you'll need to `gem install jsduck` for this to work

Bug: T170639
Change-Id: Idb3a6e9edbd45889047e7904561b84033569a689
2017-07-20 22:34:41 +00:00
jdlrobson fe8654adff Hygiene: Copy across bits of README.md from MobileFrontend
See:
Ia042ce3db5f4fed20913b54a6c0c80fe52762711

Change-Id: Idc05fa0854b113b0daaca54054cf4acaef50f157
2017-07-20 09:13:36 -07:00
jdlrobson 47546207ad Hygiene: Remove and update references to MobileFrontend
Changes:
* Update docs
* Update browser test artifacts
* Update comments
* Update phpunit test groups
* Update phpunit test namespace
* Update `die` when MobileFrontend not installed
* Remove the migrate script which is no longer needed

Change-Id: I83432b3f7f0bcd07ed08259972b8ff89147104b6
2017-07-19 14:21:50 -07:00
jdlrobson 747b712509 Minerva should use mediawiki.hlist
Override provided styles

Bug: T42062
Depends-On: I9305af5d30c8c24b44f4a0c6a466581c45fa3028
Change-Id: I7adda8ef99e5eee3df5bd7dc4da5081c1225f815
2017-07-19 13:52:09 -07:00
Umherirrender 211a54b568 Update and run existing stylelint
stylelint is part of config, but not run by default
Also update to 0.4.1, it is the current default in many wmf extensions

Change-Id: I55f81489182628c088e362e081417514e252e6d6
2017-07-19 20:31:24 +00:00
Piotr Miazga f86cfc6fc4 Do not show warning box when LoginForm was posted
Because there is no easy way to detect failed user login/account
creation, do not show warning box during POST request. This should
do the trick as there shouldn't be possible to display valid
login/usercreate form during POST request.

Bug: T149413
Change-Id: Ieb7a34068aaad99616ad6c43de48f70a6775882e
Depends-On: I6ba28ddaeb0f9af17fa875bd9d1265509db4b927
2017-07-19 17:21:09 +00:00
Kunal Mehta f7509adfbb Simplify onResourceLoaderGetLessVars hook
* Get ConfigFactory via its accessor, not getService()
* Avoid array_merge, use simpler syntax
* Don't use string interpolation when one variable is the entire string

Change-Id: Ic4c138fb3b83e6e9c6ff9a001dd39e03ca5029b1
2017-07-17 23:40:31 +00:00
Piotr Miazga 7536ca089a Enable PHPCS and lint checks via composer test
Sniffs that are currently failing are disabled in phpcs.xml.

Additional changes:
* Fix problem in test file

Change-Id: I53642e9d7bc1ef96e359cfe04a8f93dabbc977eb
2017-07-17 22:20:29 +00:00
Piotr Miazga d5461f55a1 Fix failing SkinMinerva unit test
Bug: T170624
Change-Id: I13fff8eba06abad43291e481f32d9eb30c0f031b
2017-07-17 23:39:53 +02:00
Piotr Miazga 9d90df7b0b SkinMinerva testGetContextSpecificModule test might use undefined title
Test scenario for getContextSpecificModules() mocks only Skin->getTitle()
behavior, but while executing isAllowedPageAction() Skin will create a
UserPageHelper with default RequestContext. As RequestContext is not mocked,
$context->getTitle() will return undefined what could lead to tests crash

Changes:
 - instead of mocking SkinMinerva::getTitle() pass test context with injected
title. Other tests will work properly as MediaWikiTestCase::tearDown() always
restes RequestContext to default

Bug: T170624
Change-Id: I872fddf8d9c52a6875bb6c69a12407a8125fba4c
2017-07-17 21:40:34 +02:00
jenkins-bot 8653b8b4b1 Merge "Add tooling for svg size checking" 2017-07-14 18:29:38 +00:00
jdlrobson 59a17fd0af Add tooling for svg size checking
also add pre-commit hook to ensure svgs are
compressed on commit

since extensions are accessible via http an .htaccess
is added to the dev-scripts folder for safety.

Bug: T170639
Change-Id: Ibcd5c29340d16c9cffc6e2eb90d33ee89b69874f
2017-07-14 11:07:08 -07:00
jdlrobson a81c87b897 Add missing Minerva skin description message key
Change-Id: I61457e709caa0bfa94a627236f1d04a16f94a166
2017-07-13 14:03:00 -07:00
jdlrobson 69306b0b68 QA: Fix a flakey test
This copies the approach taken in
I3161c64d68c09a9a8f7d00a27eddfbe37830fabd

Change-Id: Ifd62db9bf5862e0a524186de12f50d05f41b4799
2017-07-13 16:47:28 +00:00
jenkins-bot 171d536138 Merge "Migrate 3 messages from MobileFrontend" 2017-07-13 16:07:34 +00:00
jdlrobson 2bb0fc5113 Migrate 3 messages from MobileFrontend
These were missed in the migration.
Putting them where they belong.

Change-Id: Ifb6a2f20b43d05610845c06e4c92035a5d3327a9
2017-07-13 09:02:47 -07:00
jdlrobson f60ca174f1 minerva-neue valid skin name should not be necessary
Change-Id: I3c42424f332ff29aa0c6f14487bc150d15c520e2
2017-07-13 01:11:44 +00:00
jdlrobson b0df6367c2 Give MinervaNeue control of the minerva skin name
Change-Id: I7f004b43e11d88492b205a3584c29f72d26bad57
Depends-On: I985c4e3a88b59461d471945ccf74cd291db45a61
Bug: T166748
2017-07-12 14:49:52 -07:00
jdlrobson 2cc9516cde Minerva is its own skin
This is programmatic output from python3 scripts/migrate.py

This will result in a Minerva skin dependent on MobileFrontend.

Post merge we will rename message keys to have minerva- prefix

Bug: T166748
Change-Id: Iff1f7e63e796cc5d4a6d2ab0370e0c33248d2fce
2017-07-12 08:12:40 -07:00
jdlrobson 4d6555621f Fixes to migrate script with regards to i18n
* Make sure metadata field is present
* Make sure not to copy entire messages file

Change-Id: I9525f1cb35e6194de6594634c52d9f6c0587286d
2017-07-12 04:23:38 +00:00
jenkins-bot 7607a0a214 Merge "Add .gitignore file" 2017-07-12 04:19:07 +00:00
jdlrobson 687a10540a Add .gitignore file
Change-Id: Ie5112a5eef02fba9b70ef7483008b27090cc3f81
2017-07-11 21:14:48 -07:00
jdlrobson 7e1e5c85df QA: Add note about git history of browser tests in this repo
Bug: T168758
Change-Id: I8363eb3270f8660c0057fa42803577c9cdff400a
2017-07-11 14:57:27 -07:00
jdlrobson c0c90234d0 Port browser tests from MobileFrontend to Minerva
This moves all browser tests from MobileFrontend to the Minerva repo
in preparation for separating the two.

Note, this means browser tests will exist in both repositories for a
period of time. This is important and necessary to ensure we do not
break anything.

See:
https://lists.wikimedia.org/pipermail/mobile-l/2017-July/010536.html

Bug: T168758
Change-Id: I84ae3ea14191f672cabcd52020e80b0a40a72ce1
2017-07-11 14:11:03 -07:00