Skip to content

fix(drag-drop): error during device emulation on firefox - #19396

Merged
jelbourn merged 1 commit into
angular:masterfrom
crisbeto:19385/emulated-firefox-error
May 21, 2020
Merged

fix(drag-drop): error during device emulation on firefox#19396
jelbourn merged 1 commit into
angular:masterfrom
crisbeto:19385/emulated-firefox-error

Conversation

@crisbeto

Copy link
Copy Markdown
Member

The events that Firefox emits when it emulates a mobile device in the dev tools are incomplete which causes an error to be thrown at the end of a drag seequence. These changes add a bit of extra logic so everything doesn't break down.

Fixes #19385.

@crisbeto crisbeto added P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: patch This PR is targeted for the next patch release labels May 19, 2020
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label May 19, 2020
@crisbeto crisbeto changed the title fix(drag-drop): error during browser emulation on firefox fix(drag-drop): error during device emulation on firefox May 19, 2020
The events that Firefox emits when it emulates a mobile device in the dev tools are incomplete which causes an error to be thrown at the end of a drag seequence. These changes add a bit of extra logic so everything doesn't break down.

Fixes angular#19385.
@crisbeto
crisbeto force-pushed the 19385/emulated-firefox-error branch from 9dfd5bb to 240cde9 Compare May 19, 2020 18:48

@jelbourn jelbourn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jelbourn jelbourn added lgtm action: merge The PR is ready for merge by the caretaker labels May 19, 2020
@jelbourn
jelbourn merged commit d309ee2 into angular:master May 21, 2020
jelbourn pushed a commit that referenced this pull request May 21, 2020
The events that Firefox emits when it emulates a mobile device in the dev tools are incomplete which causes an error to be thrown at the end of a drag seequence. These changes add a bit of extra logic so everything doesn't break down.

Fixes #19385.
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Jun 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug([dragAndDrop]): [Error when using drag and drop on mobile with firefox]

3 participants