core/mobile_nebula
1
0
Fork 1
mirror of https://github.com/DefinedNet/mobile_nebula.git synced 2025-01-27 07:37:03 +00:00
Code Issues Projects Releases Packages Wiki Activity

Use CupertinoButton.tinted on iOS instead of IconButton

Browse source
This commit is contained in:
Caleb Jasik 2025-01-24 13:45:52 -06:00
parent a1df22ae75
commit 504cc206f7
No known key found for this signature in database
1 changed files with 32 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

57
lib/screens/SiteLogsScreen.dart
View file

@ -45,31 +45,7 @@ class _SiteLogsScreenState extends State<SiteLogsScreen> {
return SimplePage(
title: title,
trailingActions: [
Padding(
padding: const EdgeInsets.only(right: 8),
child: IconButton.filledTonal(
// Fixes enormous button on iOS, this *should* be the default, but it seems to be overridden.
iconSize: 24,
style: Platform.isIOS
? IconButton.styleFrom(
shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(8)),
// Match background when not selected on iOS
backgroundColor: settings.logWrap ? null : Colors.transparent,
)
: null,
isSelected: settings.logWrap,
tooltip: "Turn ${settings.logWrap ? "off" : "on"} text wrapping",
selectedIcon: const Icon(Icons.wrap_text_outlined),
icon: const Icon(Icons.wrap_text),
onPressed: () => {
setState(() {
settings.logWrap = !settings.logWrap;
})
},
),
)
],
trailingActions: [Padding(padding: const EdgeInsets.only(right: 8), child: _buildTextWrapToggle())],
scrollable: SimpleScrollable.both,
scrollController: controller,
onRefresh: () async {
@ -89,6 +65,37 @@ class _SiteLogsScreenState extends State<SiteLogsScreen> {
);
}
Widget _buildTextWrapToggle() {
return Platform.isIOS
? Tooltip(
message: "Turn ${settings.logWrap ? "off" : "on"} text wrapping",
child: CupertinoButton.tinted(
// Use the default tint when enabled, match the background when not.
color: settings.logWrap ? null : CupertinoColors.systemBackground,
sizeStyle: CupertinoButtonSize.small,
borderRadius: const BorderRadius.all(Radius.circular(8)),
child: const Icon(Icons.wrap_text),
onPressed: () => {
setState(() {
settings.logWrap = !settings.logWrap;
})
},
),
)
: IconButton.filledTonal(
isSelected: settings.logWrap,
tooltip: "Turn ${settings.logWrap ? "off" : "on"} text wrapping",
// The variants of wrap_text seem to be the same, but this seems most correct.
selectedIcon: const Icon(Icons.wrap_text_outlined),
icon: const Icon(Icons.wrap_text),
onPressed: () => {
setState(() {
settings.logWrap = !settings.logWrap;
})
},
);
}
Widget _buildBottomBar() {
var borderSide = BorderSide(
color: CupertinoColors.separator,