Extension:MobileFrontend/Deployments/2013-02-05
High Level Changes
[edit]Stories
[edit]* https://mingle.corp.wikimedia.org/projects/mobile/cards/332 * Upload photo at top of article * Note for 332: * Cancel link on the upload progress notification does not cancel the requests right now. This will be added in the next commit as it requires some bigger changes to the api wrapper. * Photo preview styling is not complete (still doesn't look exactly like design assets), this will be fixed in future commits
* https://mingle.corp.wikimedia.org/projects/mobile/cards/359 * CTA to create account from Dashboard/upload link
* https://bugzilla.wikimedia.org/show_bug.cgi?id=44330 * Login on https doesn't log you in on http
Bugs
[edit]* https://bugzilla.wikimedia.org/show_bug.cgi?id=44042 * Special pages not written for mobile have no margin
* https://bugzilla.wikimedia.org/show_bug.cgi?id=44355 * Login form error message has no margin
* https://bugzilla.wikimedia.org/show_bug.cgi?id=44139 * Anchor links broken
Uncategorized
[edit]* Localization * Message cleanup * Refactor links and url * Refactor - Add view, an abstraction over jQuery elements * Refactor - use wgMFResourceBoilerPlate * Styling - scrub inline styles on dynamically loaded pages * Always show watchlist icon on the left menu * Refactor - Reduce dependency on wgExtMobileFrontend * Fix descriptions on donate image page * Package messages for toggle module (beta) * Include mobile tag uploads over CORS * Surface image summaries on DonateImage page (beta) * Rename calls to wfArrayToCGI to wfArrayTocgi * Updated Ruby Gems * Refactor login hook into MobileFrontend.hooks.php * Standardize Ajax loader
Git Log
[edit]commit a94837d0418669b0b197138f4f1800d0243e6405 Author: Translation updater bot <l10n-bot@translatewiki.net> Date: Mon Feb 4 22:20:12 2013 +0000
Localisation updates from http://translatewiki.net. Change-Id: I507fb49468266b82e5050784c2f18f1220c23a6a
commit 7127d8d2768243ba3802b5db87c9856bc5abced6 Author: Translation updater bot <l10n-bot@translatewiki.net> Date: Sat Feb 2 21:32:48 2013 +0000
Localisation updates from http://translatewiki.net. Change-Id: I677792bc3082b36f43a1635ad88486747485f21d
commit f35146cd145e48fb49f0066df09bf4ca2a8c3f1d Author: Translation updater bot <l10n-bot@translatewiki.net> Date: Fri Feb 1 22:15:39 2013 +0000
Localisation updates from http://translatewiki.net. Change-Id: I8c613dcace78e269673bdd8535ab13b88c03e0ac
commit 9cb1c2facfd7ecabe431b63278c2a848163ab61d Author: jrobson <jrobson@wikimedia.org> Date: Fri Feb 1 11:58:18 2013 -0800
More message cleanup (minor fix) Change-Id: I79b8dbbf213314b2fdfb8ea0390fa1ad9b8eca37
commit 33d4ba6bb0a8a788ee0dbac7bfedf4c6c17ab0ec Author: Juliusz Gonera <jgonera@wikimedia.org> Date: Tue Jan 22 16:41:13 2013 -0800
Overhaul photo upload code (#332) Make photo upload code more modular. This introduces a new object, PhotoUploader, which can be reused in various places. See docs comments in mf-photo.js for details. Note: * Cancel link on the upload progress notification does not cancel the requests right now. This will be added in the next commit as it requires some bigger changes to the api wrapper. * Photo preview styling is not complete (still doesn't look exactly like design assets), this will be fixed in future commits. Change-Id: I71b6a38d7089427001972d7ffb743791a949983b
commit a8e647b6f45313f26bf44a52ca25331aa979c6dd Author: jrobson <jrobson@wikimedia.org> Date: Tue Jan 29 11:04:22 2013 -0800
Refactor links and url code in skin (code cleanup) Shift anything to do with links or urls to prepareTemplateLinks In process discover unused properties that we can throw away \o/ Change-Id: I6535f83ccbb3fcf1a484d676e0e59c3e3b18cad8
commit 66be21740a6feefc16268be2edb718eed1dc77cb Author: jrobson <jrobson@wikimedia.org> Date: Thu Jan 31 16:18:04 2013 -0800
don't send origin when cors not being used (arthur was right) Change-Id: Ia0e84349864cda38d9d65c8c39082b31fbbd99a0
commit 388b58c5f0ca77a95b2aebd8deb291fd3685c4a8 Author: Juliusz Gonera <jgonera@wikimedia.org> Date: Fri Jan 25 17:18:07 2013 -0800
Add View, an abstraction over jQuery elements. This might help us untangle the JavaScript code a bit. Inspired by Backbone's views and Knockout.js's ViewModels. See docs in the code and tests for more details. Real use case example in subsequent commits. It's the first version, we can change it or rewrite it later on when we discover our use patterns for it. Also adds Hogan as a templating engine. Change-Id: Ic70e708328e73d07c82f767d24be5299d94b9699
commit 079f0a127a535cebbed8449d85eb526a826118ec Author: jrobson <jrobson@wikimedia.org> Date: Tue Jan 29 16:20:10 2013 -0800
make use of $wgMFResourceBoilerplate (refactor) cleanup our code \o/ Change-Id: If560cb79251660c023a5c107dbefe7c241e9a19e
commit 6708d21717fa4b14bfcbcb462c164aeed92a7bf2 Author: Translation updater bot <l10n-bot@translatewiki.net> Date: Thu Jan 31 20:42:51 2013 +0000
Localisation updates from http://translatewiki.net. Change-Id: I0d6a4bb7a76e59db7a28a8a9f8f7cdf9a8e5c960
commit 7a81f79979a82ed5686d719362a047a0101bafec Author: jrobson <jrobson@wikimedia.org> Date: Thu Jan 31 11:40:56 2013 -0800
scrub inline styles on dynamically loaded pages (alpha) currently it only scrubs the initial page be more ruthless Change-Id: I0f0d59404683e1db9567c3726fc99c4998450eeb
commit 48702b248fcecdfa83b677d7e8aa4149aa851c04 Author: jrobson <jrobson@wikimedia.org> Date: Tue Jan 29 10:20:53 2013 -0800
clicking uploads when anon redirects to login (story 359) * add prepareTemplateUrls function - prepareTemplate is too meaty * add getLoginUrl method. With $wgMFForceSecureLogin the abstraction seems to be useful for code reduction * add getLoginPageHeading to allow customisation of the heading based on the returnto value * take away can upload check. Firstly this applies to the current wiki so is wrong when uploads go elsewhere e.g. commons, secondly the page should be visible to these users (in future we may want to tell them how they can upload) Change-Id: I28b4b71984928b265fe513b48de1cc36029d76ed
commit 8ce8271f6c66cab1c8d3c49eadca32e754211b16 Author: jrobson <jrobson@wikimedia.org> Date: Wed Jan 30 16:52:06 2013 -0800
overhaul of special pages (bug 44042) Give all special pages the div with class content so default pages have a margin and are readable Where no margin is required these special pages must access the skin and force it make noMargins only turn off margin on child .content elements remove unnecessary div on watchlist page Change-Id: I16562952741e93ce7a3720b34923db92afd5fdef
commit 3c75a3701a55c5d526b6fa20e54bfe198b88d13a Author: MaxSem <maxsem.wiki@gmail.com> Date: Thu Jan 31 00:13:09 2013 +0000
Revert "A stopgap measure to make Nearby to work for everyone with a link" This reverts commit 0f86877eb4bb73f8a0f0b03df34830c6ecabc068
commit 0f86877eb4bb73f8a0f0b03df34830c6ecabc068 Author: Max Semenik <maxsem.wiki@gmail.com> Date: Thu Jan 31 02:23:49 2013 +0400
A stopgap measure to make Nearby to work for everyone with a link Change-Id: I0ecf53cf912768d14bd998cb279e88a1858cb9e6
commit b79c12621d7da8c500c6b1c675383ff6b084831a Author: jrobson <jrobson@wikimedia.org> Date: Wed Jan 30 14:57:09 2013 -0800
always show watchlist icon in left menu Change-Id: I4abedd1c113261ba799c60f139313c544727743d
commit 6dc13c3f84ef4a4ca3f72869c72841c99a52b0d1 Author: Translation updater bot <l10n-bot@translatewiki.net> Date: Wed Jan 30 20:49:29 2013 +0000
Localisation updates from http://translatewiki.net. Change-Id: I08badaef658b4db4a49d3e5be5898fe80e96e604
commit 263ae0619e2d6c89421269e3f6ea160813bae979 Author: Max Semenik <maxsem.wiki@gmail.com> Date: Wed Jan 30 03:55:24 2013 +0400
Reduce dependency on $wgExtMobileFrontend Change-Id: Id21bfc41bf5b173f00607e01c424b24ab7272d5f
commit bb74936cfd5f1c868679a16f7c20ca9bf2cd7a88 Author: jrobson <jrobson@wikimedia.org> Date: Tue Jan 29 14:36:13 2013 -0800
fix descriptions on donate image page for CORS to work a origin parameter must be set Change-Id: I28fe83966241a3a372b604a21c0413a959171503
commit 49f09833a15a509a8e4f6307e1478e26087cac30 Author: jrobson <jrobson@wikimedia.org> Date: Tue Jan 29 13:54:31 2013 -0800
package messages for toggle module (beta) currently messages are marked as production only but should also be available in beta Change-Id: I8bb0ddcb3308f1a89f00a36fe6f115316f85158b
commit c43f7cbac4e6433a4d62561a7883d781cc15d3a2 Author: Translation updater bot <l10n-bot@translatewiki.net> Date: Tue Jan 29 21:02:45 2013 +0000
Localisation updates from http://translatewiki.net. Change-Id: Iac032d9aa971e27c8c4e5addfc747b71554bb539
commit 7db71f86657ba59184d2d0e0e1d61da09ec6cfc1 Author: jrobson <jrobson@wikimedia.org> Date: Mon Jan 28 16:52:28 2013 -0800
ensure we mobile tag uploads over CORS (minor) Change-Id: I757db7c6a51cc7a4b9fc94019027df55ceafaee1
commit 597b264bcd5059ad72b96ff8a391071972cb773a Author: Translation updater bot <l10n-bot@translatewiki.net> Date: Mon Jan 28 21:57:10 2013 +0000
Localisation updates from http://translatewiki.net. Change-Id: I10209648d62c3456706bb808606ba03725cd37b1
commit d93bb66f9e9d41981ce3f881f02091b0a7fc9de6 Author: jrobson <jrobson@wikimedia.org> Date: Fri Jan 25 16:26:57 2013 -0800
Login form error message has no margin (bug 44355) This was originally added for uploads page error message when not logged in. This keeps the margin there okay but fixes the one on the login form Change-Id: If57226e88f642023989003e12aeda05795067dbd
commit 0b01e18ec90eb28df3883d983ed231f9b0c697a6 Author: jrobson <jrobson@wikimedia.org> Date: Wed Jan 23 17:24:00 2013 -0800
Surface image summaries on DonateImage page (beta) Add tests Change-Id: I6f80bcaebdf5bf38511d89cca48dbb125ab0fd27
commit 1e12fe56c53e70e267191255d190479ea2c2394e Author: Siebrand Mazeland <s.mazeland@xs4all.nl> Date: Mon Jan 28 19:30:40 2013 +0100
Rename calls to wfArrayToCGI to wfArrayToCgi Follow-up to If7f4d4bf. Change-Id: Ie1ccc597573e93ba65f44ae2d5bb4f45cae52041
commit 28dbc2bf154f59873fff63aa21a487a1da006d08 Author: Zeljko Filipin <zeljko.filipin@gmail.com> Date: Mon Jan 28 13:08:39 2013 +0100
Updated Ruby gems Change-Id: Ic2a7fc91293642d9f1a59ec18b507f74cd04d1cb
commit c8c0bd4679625ed8efbe19c2fca5fb3ce923964b Author: awjrichards <arichards@wikimedia.org> Date: Fri Jan 25 18:46:45 2013 -0700
(bug 44330) When a user logs in on HTTPS, set a cookie to force them to stay on HTTPS This uses existing functionality in core to emulate how things work on the desktop site, to ensure that when a user logs in to the site using HTTPS, but attempts to view a page on HTTP, they will be redirected to HTTPS. Change-Id: I0bcd52581279f9d35087bd9be91b5dc01fa9f702
commit 60637843f27a1f40de1473bf7c8a1298dd644785 Author: awjrichards <arichards@wikimedia.org> Date: Fri Jan 25 16:13:23 2013 -0700
Refactor login hook into MobileFrontend.hooks.php * Also remove now unnecessary ExtMobileFrontend::attachHooks() * Changes 'OverloadTemplate' class to account creat/login-specific abstract class, which felt more appropriate since we had nothing else using OverloadTemplate, and we need a place to put shared logic/ handling for login/account creation-specific stuff, and it seemed silly to create yet another class. The naming of these classes could probably use some improvement, however... * Exposes RequestContext in abstract template for login/account creation templates * Moves data prep for anon watchlist addition after login to template logic Change-Id: I70bd611b6386e0e1b7f0414fec3decef0e716ac7
commit 0ea8c6cc83c74c05e4001f6cced746b94a50f6d5 Author: jrobson <jrobson@wikimedia.org> Date: Fri Jan 25 16:36:29 2013 -0800
save the hash (bug 44139) In stable clicking on a section doesn't update the hash moving logic from toggle.js into history.js Change-Id: Ie31b6736ab26e82000bed19c0b5da9ad46a4a2d3
commit 6e63a69c8e99e97724dafbd108d1a6a7b58c06c4 Author: jrobson <jrobson@wikimedia.org> Date: Fri Jan 25 15:39:35 2013 -0800
standardise ajax loader (code health) I was getting a 404 due to a bad url in mf-common.less with the generic ajax loader. This moves the image there and changes all references to it. In process standardised the name of the class that provides the ajax loader to 'loading' applied to watchlist loading star and edit screen as well as for dynamically loaded pages Change-Id: I2c679aa93f1e1313966bb4b9ab63da23a82e7320