@ppelberg: did you mean to merge T239861 to T239873 instead. Because this one seems like a different issue to me. T239873 is about not being able to post a comment and this one is about it being posted multiple times.

In T157956#5643561, @Schnark wrote:

This only fixes pastes inside VE, not into external applications, does it? And these are probably not fixable at all?

In T235692#5700616, @matmarex wrote:

In T235692#5592687, @Ryasmeen wrote:

Got it. So ^ icon is sliding out of view just fine. About the slight animation of having slight left padding then to have none, it seems to me this is still happening on Beta, not sure though. Sharing screen capture of how it behaves both on Beta and Production.

Yeah, it definitely is wrong in that Beta video, but I couldn't reproduce this when I tested on Beta just now – the heading was aligned perfectly when it transitioned. I'm not sure if this is a device-specific issue, or if perhaps some change to the mobile site in the meantime fixed it. Can you try again?

@matmarex: What do I need to test here?

Getting the following error still when closing out the Basic Reference dialog with link inspector being open inside it:
Uncaught TypeError: Cannot read property 'getDocument' of null

In T229074#5668083, @ssastry wrote:

@Ryasmeen: the 'new page creation' bug is now resolved and new code is on beta cluster as of y'day. Let us know if you find any other bugs. Or, a confirmation that you could run all your QA tests successfully! Thanks!

Checked the change using iPhone 5s and iPhone 6s. Looks good.

In T233470#5624849, @ppelberg wrote:

In T233470#5624784, @ppelberg wrote:

As for QA, @Ryasmeen, do you have access to a device running iOS 12 you're able to use to QA this? Ed did an initial pass; ideally, we have a device we can test this on first-hand.

Just chatted with @Ryasmeen, she thinks she'll be able to test this patch on both iOS 12 and iOS 13. Tho, I'll leave Rummana to confirm.

@Esanders: Is the expected result here, it will strip references pasted from LO to VE as well. For now, what I see is nothing gets pasted into VE when I copy-paste the reference from LO after step 4.

Okay, checked it for the listed articles on en.wiki, scroll is now moving past the infobox for those articles.

This seems to be working for me.

In T234108#5585009, @matmarex wrote:

@Ryasmeen In your video, the site is in a weird mix of Italian and English. Do you have automatic translation enabled in your browser or something? I would imagine that would mess with things.

Clicking on the "+" actually works regardless of that, as far as I can see. The placeholder is replaced with a normal, empty table cell, as expected.

The wiggle seems to be harmless, but I think I found a way to prevent it.

In T235692#5588325, @Esanders wrote:

When you load the editor the ^ icon should slide out of view. Once the editor has finished loading there should be no change in appearance of the heading. Currently the headings jump from having a slight left padding to having none.

@Esanders/@matmarex: Great that we fixed this, but could you please tell me what are the things that I need to check here.

@matmarex: This is not yet on en.wiki right? Why did @ReleaseTaggerBot not tag this? 😕

@matmarex: I tried to verify it by changing the font size to 20px and it seems fixed to me, since on Beta the text "New section" is aligned correctly with the rest of the page and on production it's not.
Is there anything I need to check to verify it?

@matmarex: This still looks a bit broken to me, I see that the "+" icon is still there, however clicking on it doesn't do anything, it just wiggles the table a bit. Also on the context menu, the "Property" button is not clickable instead there is a small button next to it and when I click on it, it brings up the Properties dialog.

In T213120#5549335, @ppelberg wrote:

In T213120#5095449, @Ryasmeen wrote:

ok, this time it's real. Still seeing this on bn.wiki. Here is the video capture:

ScreenRecording_04-08-2019 13-24-59.mp41 MBDownload

I seem to be encountering the same issue Rummana encountered above:
Actual behavior

1. Edit the body of a section
2. Publish changes

⚠️ 3. View is scrolled to the top of the article in read mode
Expected behavior

1. Edit the body of a section
2. Publish changes

✅ 3. View is scrolled to the top of the section I was just editing
Configuration
Browser: Safari
Device: iPhone Xs
OS: iOS 13.1.2
Video
https://youtu.be/fikbhfZjmJE

@matmarex: You're right. It was happening couple hours ago, but not anymore. Guess it was not updated at that point.

I am still getting the following error with moving (drag-drop) and cutting references.
Check the screenshot for the error:

In T232790#5545449, @ppelberg wrote:

In T232790#5544653, @DLynch wrote:

Everything Rummana saw looks correct, it's just a confusion about what the event name is. For the VisualEditorFeatureUse events it's two-part, there's a feature and action component, with the name being written here as [feature].[action].
e.g. this console log shows the mwSave.checkbox-wpWatchthis event:

Got it. Thank you for demonstrating this and for updating the task description accordingly, David.

In T232790#5544672, @DLynch wrote:

That said, the "Yes, but it's getting fired twice" shouldn't be firing twice. But I can't reproduce that myself, so I might need more information.

Never mind, worked it out, patch incoming.

Excellent.

Next steps...

• @Ryasmeen, all that's left now is to test the two actions in the table below that you correctly identified as being fired twice in your initial testing:

Action	Event that should fire	Did event fire? Y/N	Notes
tap "This is a minor edit" checkbox in the "Save your changes" screen	VisualEditorFeatureUse: mwSave.checkbox-wpMinoredit	Yes, but it's getting fired twice
tap the "Watch this page" checkbox in the "Save your changes" screen	VisualEditorFeatureUse: mwSave.checkbox-wpWatchthis	Yes, but it's getting fired twice