copilot/frosty
1
0
Fork 0
Code Issues Pull requests Projects Releases 11 Packages Wiki Activity Actions

refactor(chat): hoist shared MeasureSize wrapper out of bottom-bar branches

Browse source 
Both the fullscreen-overlay and BlurredContainer branches wrapped
bottomBarContent in an identical MeasureSize+onChange closure; build it
once and reuse. No behavior change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Tommy Chow 2026-06-13 13:08:06 -07:00
parent 77b0126e35
commit 2fa1650ec2
1 changed files with 9 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

19
lib/screens/channel/chat/widgets/chat_bottom_bar.dart
View file

@ -394,6 +394,13 @@ class ChatBottomBar extends StatelessWidget {
final needsBottomPadding =
!chatStore.assetsStore.showEmoteMenu && !isHorizontalLandscape;
// Measured once here; the list pads itself by this height so the bar
// never overlaps messages. Shared by both container branches below.
final measuredBar = MeasureSize(
onChange: (size) => chatStore.setBottomBarHeight(size.height),
child: bottomBarContent,
);
return isFullscreenOverlay
? Padding(
padding: EdgeInsets.only(
@ -401,11 +408,7 @@ class ChatBottomBar extends StatelessWidget {
? MediaQuery.of(context).padding.bottom
: 0,
),
child: MeasureSize(
onChange: (size) =>
chatStore.setBottomBarHeight(size.height),
child: bottomBarContent,
),
child: measuredBar,
)
: BlurredContainer(
gradientDirection: GradientDirection.down,
@ -414,11 +417,7 @@ class ChatBottomBar extends StatelessWidget {
? MediaQuery.of(context).padding.bottom
: 0,
),
child: MeasureSize(
onChange: (size) =>
chatStore.setBottomBarHeight(size.height),
child: bottomBarContent,
),
child: measuredBar,
);
},
);