Face UI fixes (#18000)

* ensure portrait cams are contained in dialog

* always show rename and delete buttons on mobile and tablet

without a group-hover state, a double-tap was required to select a face from the dropdown
This commit is contained in:
Josh Hawkins 2025-05-02 07:44:45 -05:00 committed by GitHub
parent 92f7b54371
commit f98795abfd
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -497,7 +497,7 @@ function LibrarySelector({
<Button
variant="ghost"
size="icon"
className="size-7 opacity-0 transition-opacity group-hover:opacity-100"
className="size-7 lg:opacity-0 lg:transition-opacity lg:group-hover:opacity-100"
onClick={(e) => {
e.stopPropagation();
setRenameFace(face);
@ -515,7 +515,7 @@ function LibrarySelector({
<Button
variant="ghost"
size="icon"
className="size-7 opacity-0 transition-opacity group-hover:opacity-100"
className="size-7 lg:opacity-0 lg:transition-opacity lg:group-hover:opacity-100"
onClick={(e) => {
e.stopPropagation();
setConfirmDelete(face);
@ -661,7 +661,7 @@ function TrainingGrid({
<div className="text-sm">{formattedDate}</div>
</div>
<img
className="w-full"
className="mx-auto max-h-[60dvh] object-contain"
loading="lazy"
src={`${baseUrl}api/events/${selectedEvent?.id}/${selectedEvent?.has_snapshot ? "snapshot.jpg" : "thumbnail.jpg"}`}
/>