Skip to content

Commit

Permalink
More consistent tooltip spacing on record button
Browse files Browse the repository at this point in the history
  • Loading branch information
killergerbah committed Feb 24, 2024
1 parent bb0795a commit 4300c9b
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions extension/src/ui/components/MobileVideoOverlay.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -180,13 +180,15 @@ const MobileVideoOverlay = () => {
<Grid item>
<Tooltip title={miningButtonTooltip(model)!}>
{model.emptySubtitleTrack && model.recordingEnabled ? (
<IconButton onClick={handleMineSubtitle}>
<FiberManualRecordIcon
className={model.recording ? classes.recordingButton : classes.button}
/>
</IconButton>
// Wrap in span so that Tooltip doesn't complain about disabled child. Spacing also looks better.
<span>
<IconButton onClick={handleMineSubtitle}>
<FiberManualRecordIcon
className={model.recording ? classes.recordingButton : classes.button}
/>
</IconButton>
</span>
) : (
// Wrap in span so that Tooltip doesn't complain about disabled child
<span>
<IconButton disabled={miningButtonDisabled} onClick={handleMineSubtitle}>
<NoteAddIcon
Expand Down

0 comments on commit 4300c9b

Please sign in to comment.