Commit graph

8681 commits

Author SHA1 Message Date
James D. Forrester cfb20ef8a1 ve.ce.MWTransclusionNode: Replace last use of ', ' in VE-MW with <comma-seperator>
Bug: 66894
Change-Id: Idd6d02e35b9aa0db5f71c75adb0a0400cc31c83a
2014-11-19 11:19:50 +00:00
jenkins-bot 30d8bf60f1 Merge "ViewPageTarget#onSaveErrorSpamBlacklist: Use mw.language.listToText now it exists" 2014-11-19 11:06:01 +00:00
James D. Forrester d70a3d76ed Update VE core submodule to master (cdd2691)
New changes:
6ae94bc Add tests for ve.instanceOfAny()
ea1ac17 Localisation updates from https://translatewiki.net.
a9a25cb Update OOjs UI to v0.1.0-pre (f1abca8e82)
ab55974 build: Update various devDependencies to latest
921d16a Localisation updates from https://translatewiki.net.
cdd2691 Use history as as global

Change-Id: I4ff70005591cac11c39e08a602457cf4e0eda9be
2014-11-18 19:37:47 -08:00
Timo Tijhof 420b336d31 mw.ViewPageTarget.init: Simplify hideLoading and pushState
Follows-up Ica33de675, If505a46f54.

* The setting of 'uri' looked like a redundant local alias before
  passing on to pushState, but is actually important on itself.

* Remove binding for hideLoading. It's a detachable method on
  a singleton, not an instance method.

Change-Id: Ic3536caf3f42ee893124312fd5981b67336bd480
2014-11-19 02:58:17 +00:00
Roan Kattouw bb6e01c488 Fix back button not exiting VE
When you clicked the Back button in the browser, the URL
in the address bar would change (removing veaction=edit),
but we would not go back to read mode. This was broken
by 5c0c11753 almost a month ago but apparently no one noticed.

This is because 5c0c11753 moved the pushState() calls to
be earlier (in init init), making the replaceState() call
in the ViewPageTarget constructor (which is there
specifically for this bug) run too late in those cases.

The simplest way to fix this is to duplicate these replaceState()
calls before the pushState() calls in init init.
I feel a bit bad about copying code, but not very bad
because the code I'm copying already has a FIXME comment
about how there should be a better way :P

Change-Id: I6627a5d1d9377ae815bc58bceeb059ce9f4f19ab
2014-11-19 02:07:41 +00:00
jenkins-bot aa44c8916b Merge "Consistently use location as global, and location.href for full url" 2014-11-19 02:00:35 +00:00
jenkins-bot c8152914eb Merge "Use history as global instead of via window" 2014-11-19 01:59:26 +00:00
Timo Tijhof dc648e6b06 test: Omit 'window' in mw-preload.js
Follows-up 92c38ea.

Change-Id: Ib8a37036a1a9dac980527bfccd506fff5ea8dce1
2014-11-19 01:46:38 +00:00
Timo Tijhof c0692af5f7 Use history as global instead of via window
Just like DOM interfaces such as document and location. And JS
constructors like Object.prototype, and libs like jQuery.
No need for the window host object.

Change-Id: Ie39aaea2ca17bdec5fb3afe32a4e6c9f751bd0c6
2014-11-19 01:12:11 +00:00
Timo Tijhof 82657727f9 Consistently use location as global, and location.href for full url
The location object is a global, just like document.

Using it via 'window' needlessly adds complexity and, for example,
makes it harder to catch typos in static analysis.

Standardise on location.href in place of the many different
variants for accessing and assinging the full url:

 location =
 location.href =
 location.assign() =

And each with 'window', 'document' and without host object.

Change-Id: If88515bd88fc5b93ebacd4ec41fce02fec540f18
2014-11-19 00:20:08 +00:00
Cmcmahon d27edd8d5a [BrowserTest] stop rubocop complaints for now
I'll be addressing these one by one now that all the steps are
refactored

Change-Id: I9649c261e4c63be46a14dbe73f0226e50c7022c4
2014-11-18 16:18:35 -07:00
James D. Forrester 6b61addd06 ViewPageTarget#onSaveErrorSpamBlacklist: Use mw.language.listToText now it exists
Change-Id: I1289c43ea4be1cd9dc1addb7c765d7050b311d8d
2014-11-18 14:22:30 -08:00
Translation updater bot 384232ee64 Localisation updates from https://translatewiki.net.
Change-Id: I7e799602cb9d11c2e77be38b3707ebd0b82c7d9d
2014-11-18 21:35:49 +01:00
Cmcmahon 210169c55b [BrowserTest] forgot a bit of dead code
Change-Id: I249e229d449eb74ff341f511e84290a4ca731f92
2014-11-18 13:09:17 -07:00
Cmcmahon 6489767bc7 [BrowserTest] remove sleeps from shared tests
* remove dead code

Change-Id: Id95ff81746eb440124958bcc6dd9334b7ed299a4
2014-11-18 12:59:34 -07:00
James D. Forrester 12f183a20c build: Update grunt-jscs devDependency to latest
grunt-jscs: 0.7.1 -> 0.8.1

No-op; note that this switches us from jscs 1.6.0 to 1.7.2, which provides
some new options (unused in the Wikimedia preset).

Change-Id: I1d0db2568ff673ff8b8bba9549b06c005eb7186b
2014-11-17 19:50:46 -08:00
jenkins-bot 4807d4c847 Merge "Replaces spaces with tabs in en.json" 2014-11-17 23:47:39 +00:00
Cmcmahon 06aa53747c [BrowserTest] interim step, no functional change
Change-Id: I3a7787c4ab9206c70f0f64b6c0986ae93ddb3d84
2014-11-17 15:14:32 -07:00
Cmcmahon 94bab8f918 [BrowserTest] update transclusion test for Page Object vio and RS
Change-Id: Ie80dce2d117d4d2dc836681dcd9362ae565f2aa8
2014-11-17 14:59:14 -07:00
Cmcmahon 743d035fa1 [BrowserTest] update switch-edit-mode test for RSpec3
Change-Id: Ia040f8b6e76a42c2c80bda382bf6ad6f30988ba1
2014-11-17 14:42:59 -07:00
Timo Tijhof 173181f857 Replaces spaces with tabs in en.json
Bug: 72132
Change-Id: I1cc0379a824caec957c4e63963634e67973f1cf2
2014-11-17 22:37:43 +01:00
Translation updater bot 3386871003 Merge "Localisation updates from https://translatewiki.net." 2014-11-17 21:29:42 +00:00
Translation updater bot f8d851c50a Localisation updates from https://translatewiki.net.
Change-Id: Iee3bdfaa9ef9b032cba0a958eb06ebbbb2e505a5
2014-11-17 22:29:16 +01:00
Cmcmahon fa02c60586 [BrowserTest] fix minor Page Object design violation
Change-Id: I75d282a7e176347b0995f2c2ccda8ab051aca270
2014-11-17 14:18:13 -07:00
Cmcmahon 4383dedcfc [BrowserTest] minor violation of Page Object design pattern
and RSPec syntax also fixed

Change-Id: I2bbaf460f6ddae708ea243064dd9a596bdd006f1
2014-11-17 14:07:16 -07:00
jenkins-bot 83797c4925 Merge "[BrowserTest] more update to Cite test for RSpec3" 2014-11-17 20:59:12 +00:00
jenkins-bot c5ec9cb8d5 Merge "Methods for new "maps" property in TD Root" 2014-11-17 20:56:49 +00:00
Cmcmahon 3c5e695c5d [BrowserTest] more update to Cite test for RSpec3
Change-Id: Ia2fa25bffe9bbdffb0eafc19396c06be253216c3
2014-11-17 13:56:08 -07:00
James D. Forrester a9292ce649 Update VE core submodule to master (fa572a5)
New changes:
4d00221 Move hasSignificantWhitespace and handlesOwnChildren abstracts to ve.Node.
3889f14 Don't always use updateDimensionsDebounced
ea59d25 Safari copy-paste fix
537478d Make changes to comment node before apply staging stack
6dea745 Apply comment changes using fragment
20e928d Update OOjs to v1.1.3
98bbe75 Don't allow dropping inside a handlesOwnChildren node
9e87ccb Move mobile ContextItemWidget hacks into separate class
5d98528 Replace ve.getProp() and ve.setProp() with OO aliases

Change-Id: I7eb0d0eac311d9a220e7a0ce2f920788a3a2ec3c
2014-11-17 12:51:54 -08:00
Cmcmahon 94c30f01da [BrowserTest] update Cite test for RSpec3
Change-Id: I589a65e0fe1b21ca6476a5c31b97d7632da39b13
2014-11-17 13:31:30 -07:00
Cmcmahon 87ad037442 [BrowserTest] remove sleep() from template test
Change-Id: Ia564232e02db2e6be3070d98691097350e920a3b
2014-11-17 13:13:25 -07:00
jenkins-bot 51976d6237 Merge "[BrowserTest] interim step to sort G/W/T statements properly" 2014-11-17 19:29:12 +00:00
Cmcmahon ea0fac5e4f [BrowserTest] interim step to sort G/W/T statements properly
no change to function

Change-Id: I2c2e9ccb6535c875d37ef59ab873a2f3280cca60
2014-11-17 12:27:06 -07:00
James D. Forrester 14a0f93621 Update VE core submodule to master (d418c99)
New changes:
81e2135 Remove upstreamed ve.contains function
77166d7 Update RangeFix library 0.1.0 -> 0.1.1
6fe9405 Give table overlay width so row context positions correctly in RTL
a2001e2 Fix getDirectionFromSelection to work with table selections
ca75f05 Make table inserts directional
7ce4a63 Follow-up 2075a46: Fix RTL image build by embedding after flipping
5068bcd Restore original rowspan/colspan rather than normalizing or overwriting with NaN
e1bff77 Localisation updates from https://translatewiki.net.
a92012e Use strict comparison for insertionAnnotationsChange
39f7b91 Localisation updates from https://translatewiki.net.
b5e1130 Localisation updates from https://translatewiki.net.
c577aeb Initialize mobile context as visible

Change-Id: I113250f8caec07f9a7e59116417eb582ea0e7ae5
2014-11-17 09:54:39 -08:00
jenkins-bot 7efa601837 Merge "On save, if there was no 'last modified' text at the bottom of the page, add it" 2014-11-17 17:48:50 +00:00
jenkins-bot e85550c350 Merge "Don't try to get newtimestamp from edit if no change was made" 2014-11-17 17:48:44 +00:00
jenkins-bot 20e54e1c8f Merge "Stop warnings about calling FlaggablePageView::setPageContent with no parameters" 2014-11-17 17:47:37 +00:00
Cmcmahon 9a4d2eeef9 [BrowserTest] interim step to alphabetize
no change to function

Change-Id: Id655a9777e3b8bd3e3fbb974c73912f9f975bf7c
2014-11-17 09:09:06 -07:00
Alex Monk a7294b34c0 Stop warnings about calling FlaggablePageView::setPageContent with no parameters
We don't care.

Bug: 73469
Change-Id: I2ff0ec2fd1b0d34c1168f83d808c1cb498b17511
2014-11-17 01:28:30 +00:00
Alex Monk ac6f6caeac On save, if there was no 'last modified' text at the bottom of the page, add it
Just hope that wgMaxCredits is 0.

Change-Id: Idf4a7cceb0650eaec4442244066d875a3ff38e06
2014-11-17 01:18:46 +00:00
Alex Monk ff3d76f488 Don't try to get newtimestamp from edit if no change was made
ApiEditPage can give us 'nochange' instead of the
'oldrevid'/'newrevid'/'newtimestamp' keys if we gave it a null edit.

Bug: 73463
Change-Id: Ic22597dfed11de3823471673404090a9bce12928
2014-11-17 01:03:00 +00:00
Translation updater bot fce0ed5d6c Localisation updates from https://translatewiki.net.
Change-Id: I0a857cef14050c92c1236612006ad34b34bf49bc
2014-11-16 22:53:55 +01:00
Translation updater bot 9b3b81c234 Localisation updates from https://translatewiki.net.
Change-Id: I3e9b5d8b65d6b1a1b57762ae904554a7b4d47c73
2014-11-15 23:18:28 +01:00
Cmcmahon 17f6badf0a [BrowserTest] update references test for modern RSpec syntax
simple update to a nice test

Change-Id: Iad00e779295ccc610d20241693a87b76b817fbd6
2014-11-14 15:52:51 -07:00
jenkins-bot 02407138ad Merge "[BrowserTest] remove sleep from multiedit test" 2014-11-14 22:35:31 +00:00
Cmcmahon 6d22cebb62 [BrowserTest] remove sleep from multiedit test
* put the shared step in shared_steps.rb
* replace "onfocus" with simple "click"
* update RSpec while we're at it

Change-Id: I77754abdbeba1314758f5fe10601f6601e8783bd
2014-11-14 15:30:10 -07:00
Translation updater bot e670131ea5 Localisation updates from https://translatewiki.net.
Change-Id: I934ae1c25cf6d1396dc4517d438a76556d4c209d
2014-11-14 22:27:52 +01:00
Cmcmahon 36e07b247e [BrowserTest] simple upate to a nice test
Change-Id: Ib52582a677d8dc2f22552bf7c60f48de8f20d3c7
2014-11-14 13:43:31 -07:00
Cmcmahon ea9df60aa3 [BrowserTest] RSpec update
Change-Id: I03f89eda4292482dedaa335e3546af724ee55024
2014-11-14 13:32:59 -07:00
Cmcmahon 1aa5d8905b [BrowserTest] remove sleeps from links test
Change-Id: Ib0e09735331dadd5bb494fe87683f6f08b6aee31
2014-11-14 12:30:13 -07:00