Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle file not being present in 'Open to the Side' #809

Merged
merged 3 commits into from
Sep 14, 2021

Conversation

mattseddon
Copy link
Member

@mattseddon mattseddon commented Sep 14, 2021

Mitigates this issue. Which is caused by trying to 'Open to the Side' a file that doesn't exist on disk. We now follow the same code path that we use for the left click action.

Demo:

Screen.Recording.2021-09-14.at.7.18.05.pm.mov

@mattseddon mattseddon added the product PR that affects product label Sep 14, 2021
@mattseddon mattseddon self-assigned this Sep 14, 2021
@mattseddon mattseddon force-pushed the handle-open-not-present branch from 62d63ce to 09cd80d Compare September 14, 2021 07:57
@codeclimate
Copy link

codeclimate bot commented Sep 14, 2021

Code Climate has analyzed commit 1de9863 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.0% (0.0% change).

View more on Code Climate.

@mattseddon
Copy link
Member Author

Relates to #569

@mattseddon mattseddon marked this pull request as ready for review September 14, 2021 09:17
@mattseddon mattseddon merged commit 42fe22b into master Sep 14, 2021
@mattseddon mattseddon deleted the handle-open-not-present branch September 14, 2021 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product PR that affects product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants