Fix clock rendering artifact #62

Closed
ilidemi wants to merge 1 commits from ilidemi/orca:clock-seconds-radius into main
Collaborator

When rounded rect corner radius exceeds 2 * height, it stops looking like a rect. Proper fix would be to introduce clamping in oc_rounded_rectangle_path but that needs some investigation, what's the standard way of applying it.

image

When rounded rect corner radius exceeds 2 * height, it stops looking like a rect. Proper fix would be to introduce clamping in oc_rounded_rectangle_path but that needs some investigation, what's the standard way of applying it. ![image](/attachments/af98c764-1007-46ff-8e58-555c6a15850c)
1.1 KiB
ilidemi added 1 commit 2023-08-24 00:09:17 +00:00
Owner

Standard way I think would be min(height, width) / 2. Might be different if you can apply a different radius to each corner - which we will probably want in some form at some point. You can also enforce different horizontal and vertical radii to get elliptical corners. Never seen anyone use that in the wild though.

Standard way I think would be `min(height, width) / 2`. Might be different if you can apply a different radius to each corner - which we will probably want in some form at some point. You can also enforce different horizontal and vertical radii to get elliptical corners. Never seen anyone use that in the wild though.
Author
Collaborator

That's easier than I thought! Reviewer(s), feel free to choose between this and !64.

That's easier than I thought! Reviewer(s), feel free to choose between this and !64.
Collaborator

I'll merge !64 since it adresses the problem at the root!

I'll merge !64 since it adresses the problem at the root!
MartinFouilleul closed this pull request 2023-08-25 13:32:59 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
macOS
windows
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: hmn/orca#62
No description provided.