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

Block editing doesn't grey out date of videos #1285

Open
owi92 opened this issue Nov 20, 2024 · 1 comment
Open

Block editing doesn't grey out date of videos #1285

owi92 opened this issue Nov 20, 2024 · 1 comment
Labels
area:frontend Everything frontend related priority:low Low priority

Comments

@owi92
Copy link
Member

owi92 commented Nov 20, 2024

See screenshot
Bildschirmfoto 2024-11-20 um 10 58 20

@owi92 owi92 added area:frontend Everything frontend related time:minutes Time estimate: less than an hour priority:low Low priority labels Nov 20, 2024
@owi92 owi92 removed the time:minutes Time estimate: less than an hour label Nov 27, 2024
@owi92
Copy link
Member Author

owi92 commented Nov 27, 2024

Turns out this might be a little more complicated due to us not having a global "in edit mode" state or something. The date is handled differently since there is a tooltip behind it, and that means it needs to be focusable on it's own and can't be part of the clickable link area of it's video tile in a series block. That is currently archieved by a higher zIndex. I'd need to check in appkit and its use of the floating UI library to see if there's anything we can do about it.
Since this is a low priority issue and we have more pressing matter at the moment, I'll leave it as is for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:frontend Everything frontend related priority:low Low priority
Projects
None yet
Development

No branches or pull requests

1 participant