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

This is supposed to fix broken corner in case it is very close to an edge (for .down direction only) #133

Merged
merged 2 commits into from
Dec 31, 2019

Conversation

vitalii-tym
Copy link
Contributor

The corner used to be corrected in case it its point is exactly at an edge, but in case the corner is not at the edge there was some visual cut-out.
Not sure, but probably it is also mentioned in the issue #33

How it used to be:
Screenshot 2019-11-04 17 09 42

After fix:
Screenshot 2019-11-04 17 10 41

@vitalii-tym
Copy link
Contributor Author

Please, don't accept it yet. Found some artifacts, will try to fix them.

…d behind the edge (only applicable for .up position) + added test buttons into the demo app
@vitalii-tym
Copy link
Contributor Author

vitalii-tym commented Nov 12, 2019

Fixed some artifacts and covered the case when target view is behind the corner.
Was after my first commit (artifacts appearing when opened from some specific positions):
Screenshot 2019-11-12 13 45 51
Fixed:
Screenshot 2019-11-12 17 05 44
Screenshot 2019-11-12 17 06 00

The change is only applicable for .down direction. Haven't had enough time to cover other directions.

@vitalii-tym vitalii-tym changed the title This is supposed to fix broken corner in case it is very close to an edge This is supposed to fix broken corner in case it is very close to an edge (for .up direction only) Nov 12, 2019
@vitalii-tym vitalii-tym changed the title This is supposed to fix broken corner in case it is very close to an edge (for .up direction only) This is supposed to fix broken corner in case it is very close to an edge (for .down direction only) Nov 12, 2019
@corin8823
Copy link
Owner

I'm sorry to be late. Good Job!

@corin8823 corin8823 merged commit 98e6564 into corin8823:master Dec 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants