From 5c5c56a10070bf2174a6e23325f588c436fb4a52 Mon Sep 17 00:00:00 2001 From: ggurdin <46800240+ggurdin@users.noreply.github.com> Date: Mon, 5 May 2025 12:47:27 -0400 Subject: [PATCH] chore: fix positioning in word zoom widget header buttons (#2664) --- .../toolbar/widgets/practice_activity/word_audio_button.dart | 4 ++++ lib/pangea/toolbar/widgets/word_zoom/lemma_widget.dart | 1 + lib/pangea/toolbar/widgets/word_zoom/word_zoom_widget.dart | 5 +++++ 3 files changed, 10 insertions(+) diff --git a/lib/pangea/toolbar/widgets/practice_activity/word_audio_button.dart b/lib/pangea/toolbar/widgets/practice_activity/word_audio_button.dart index 080fed3d9..fb31b184c 100644 --- a/lib/pangea/toolbar/widgets/practice_activity/word_audio_button.dart +++ b/lib/pangea/toolbar/widgets/practice_activity/word_audio_button.dart @@ -72,6 +72,10 @@ class WordAudioButtonState extends State { tooltip: _isPlaying ? L10n.of(context).stop : L10n.of(context).playAudio, iconSize: widget.size, + style: IconButton.styleFrom( + padding: const EdgeInsets.all(0), + ), + constraints: const BoxConstraints(), onPressed: widget.callbackOverride ?? () async { if (_isPlaying) { diff --git a/lib/pangea/toolbar/widgets/word_zoom/lemma_widget.dart b/lib/pangea/toolbar/widgets/word_zoom/lemma_widget.dart index 8882e80e7..5b278a213 100644 --- a/lib/pangea/toolbar/widgets/word_zoom/lemma_widget.dart +++ b/lib/pangea/toolbar/widgets/word_zoom/lemma_widget.dart @@ -168,6 +168,7 @@ class LemmaWidgetState extends State { } return Row( + spacing: 8.0, children: [ Text( widget.token.lemma.text, diff --git a/lib/pangea/toolbar/widgets/word_zoom/word_zoom_widget.dart b/lib/pangea/toolbar/widgets/word_zoom/word_zoom_widget.dart index 27cbcd1f8..ae0d41789 100644 --- a/lib/pangea/toolbar/widgets/word_zoom/word_zoom_widget.dart +++ b/lib/pangea/toolbar/widgets/word_zoom/word_zoom_widget.dart @@ -71,11 +71,15 @@ class WordZoomWidget extends StatelessWidget { //@ggurdin - might need to play with size to properly center SizedBox( width: 24.0, + height: 24.0, child: IconButton( onPressed: () => overlayController.updateSelectedSpan( token.text, ), icon: const Icon(Icons.close), + style: IconButton.styleFrom( + padding: EdgeInsets.zero, + ), ), ), LemmaWidget( @@ -164,6 +168,7 @@ class WordZoomWidget extends StatelessWidget { Wrap( alignment: WrapAlignment.center, crossAxisAlignment: WrapCrossAlignment.center, + spacing: 8.0, children: [ ...[ Text(