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

Do we want rounded corners on the video player? #1229

Closed
owi92 opened this issue Aug 22, 2024 · 6 comments
Closed

Do we want rounded corners on the video player? #1229

owi92 opened this issue Aug 22, 2024 · 6 comments
Labels
area:player Paella and its integration in Tobira priority:low Low priority

Comments

@owi92
Copy link
Member

owi92 commented Aug 22, 2024

I think it looks nicer with rounded corners rather than shap ones 🤷
Though maybe having sharp corners is deliberate and was decided before I came on board.
However if it is only because paella does it like that, it is possible to override.
Bildschirmfoto 2024-08-22 um 14 57 56

@owi92 owi92 added the area:player Paella and its integration in Tobira label Aug 22, 2024
@oas777
Copy link
Collaborator

oas777 commented Aug 23, 2024

"No, thanks" was my initial response of course. But then I saw most/all (?) other elements have rounded corners, so that's a "Yes" from me for consistency, if it is.

@owi92
Copy link
Member Author

owi92 commented Aug 23, 2024

all (?) other elements have rounded corners

I think that is the case. Just out of curiosity, why was your initial response "no, of course not"?
I went back to look at Lisa's design and there the player does actually have rounded corners, though only slightly (which I think works well, maybe better than the radius I used in my screenshot).
Did you veto that back then? I don't remember discussing it to be honest.

@oas777
Copy link
Collaborator

oas777 commented Aug 26, 2024

I don't remember either, but the reason for my initial rejection comes from "preserving" the original video which doesn't have round corners.

@owi92 owi92 added the priority:low Low priority label Aug 26, 2024
@LukasKalbertodt
Copy link
Member

Interesting, never considered that. You of course have the problem that you cut of a tiny bit of information from the video. But with the small radius that we are talking about, I really can't imagine that being a problem. I would stick to the 4px radius that Lisa suggested though, not using a bigger one. That should already remove the sharpness of the edge. Maybe prepare a PR and we can test out how we all like it?

@dagraf
Copy link
Collaborator

dagraf commented Sep 3, 2024

I would vote for round corners with a small radius. YouTube has them and I never noticed them being there - untill now.

@owi92 owi92 changed the title Do we want round borders on the video player? Do we want rounded corners on the video player? Sep 4, 2024
@owi92
Copy link
Member Author

owi92 commented Oct 12, 2024

We decided against this in #1237 (see comments).

@owi92 owi92 closed this as completed Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:player Paella and its integration in Tobira priority:low Low priority
Projects
None yet
Development

No branches or pull requests

4 participants