Skip to content

Conversation

@aduth
Copy link
Member

@aduth aduth commented Oct 5, 2018

Regression of #9977

This pull request seeks to resolve a failing E2E test, failed to have been updated in #9977.

Testing instructions

Verify E2E tests pass:

npm run test-e2e

@aduth aduth added [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. [Type] Bug An existing feature does not function as intended labels Oct 5, 2018
@aduth aduth added this to the 4.0 milestone Oct 5, 2018
Copy link
Member

@tofumatt tofumatt left a comment

Choose a reason for hiding this comment

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

Why the comment-removal?


// When returning to Visual mode, backspace in selected block should
// reset to an unmodified default block.
// Reset.
Copy link
Member

Choose a reason for hiding this comment

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

Concise. (The old comment seemed better. 🤔)

Copy link
Member Author

Choose a reason for hiding this comment

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

the old comment was wrong (outdated). we don't switch modes

Copy link
Member

Choose a reason for hiding this comment

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

It can probably just be removed then. Or expanded. "Remove text to reset the block." or something. 🤷‍♂️

Copy link
Member

@tofumatt tofumatt left a comment

Choose a reason for hiding this comment

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

more-red-than-green

@aduth aduth merged commit baff0e6 into master Oct 5, 2018
@aduth aduth deleted the fix/writing-flow-e2e branch October 5, 2018 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants