mirror of
https://github.com/rzuasti/oott.git
synced 2026-07-08 19:21:54 +02:00
Replace device detail popup menu with inline CTA buttons
Register and Forget Device actions are now always-visible buttons (FilledButton and error-styled OutlinedButton) placed inline below the device info card, following M3 guidelines for in-content actions. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Sonnet 4.6
parent
39f382fd0d
commit
1da0306b61
@@ -65,30 +65,6 @@ class _DeviceDetailState extends State<DeviceDetail> {
|
|||||||
style: Theme.of(context).textTheme.headlineSmall,
|
style: Theme.of(context).textTheme.headlineSmall,
|
||||||
),
|
),
|
||||||
),
|
),
|
||||||
if (device != null)
|
|
||||||
PopupMenuButton<String>(
|
|
||||||
icon: const Icon(Icons.more_vert),
|
|
||||||
onSelected: (value) async {
|
|
||||||
if (value == 'forget') {
|
|
||||||
await confirmForgetDevice(context, device, _loadDevice);
|
|
||||||
} else if (value == 'register') {
|
|
||||||
await showRegisterDeviceDialog(
|
|
||||||
context,
|
|
||||||
device,
|
|
||||||
_loadDevice,
|
|
||||||
);
|
|
||||||
}
|
|
||||||
},
|
|
||||||
itemBuilder: (context) => [
|
|
||||||
if (device.isRegistered)
|
|
||||||
const PopupMenuItem(value: 'forget', child: Text('Forget')),
|
|
||||||
if (!device.isRegistered)
|
|
||||||
const PopupMenuItem(
|
|
||||||
value: 'register',
|
|
||||||
child: Text('Register'),
|
|
||||||
),
|
|
||||||
],
|
|
||||||
),
|
|
||||||
],
|
],
|
||||||
),
|
),
|
||||||
Expanded(
|
Expanded(
|
||||||
@@ -119,6 +95,8 @@ class _DeviceDetailState extends State<DeviceDetail> {
|
|||||||
const SizedBox(height: 24),
|
const SizedBox(height: 24),
|
||||||
_DeviceInfoCard(device: device),
|
_DeviceInfoCard(device: device),
|
||||||
const SizedBox(height: 24),
|
const SizedBox(height: 24),
|
||||||
|
_DeviceActions(device: device, onAction: _loadDevice),
|
||||||
|
const SizedBox(height: 24),
|
||||||
_SectionHeader(title: 'Event History'),
|
_SectionHeader(title: 'Event History'),
|
||||||
const SizedBox(height: 12),
|
const SizedBox(height: 12),
|
||||||
DeviceEventHistory(device: device),
|
DeviceEventHistory(device: device),
|
||||||
@@ -209,6 +187,35 @@ class _DeviceInfoCard extends StatelessWidget {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
class _DeviceActions extends StatelessWidget {
|
||||||
|
final Device device;
|
||||||
|
final VoidCallback onAction;
|
||||||
|
|
||||||
|
const _DeviceActions({required this.device, required this.onAction});
|
||||||
|
|
||||||
|
@override
|
||||||
|
Widget build(BuildContext context) {
|
||||||
|
final colorScheme = Theme.of(context).colorScheme;
|
||||||
|
|
||||||
|
if (device.isRegistered) {
|
||||||
|
return OutlinedButton.icon(
|
||||||
|
style: OutlinedButton.styleFrom(
|
||||||
|
foregroundColor: colorScheme.error,
|
||||||
|
side: BorderSide(color: colorScheme.error),
|
||||||
|
),
|
||||||
|
onPressed: () => confirmForgetDevice(context, device, onAction),
|
||||||
|
icon: const Icon(Icons.link_off),
|
||||||
|
label: const Text('Forget Device'),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
return FilledButton.icon(
|
||||||
|
onPressed: () => showRegisterDeviceDialog(context, device, onAction),
|
||||||
|
icon: const Icon(Icons.how_to_reg),
|
||||||
|
label: const Text('Register Device'),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
class _InfoRow extends StatelessWidget {
|
class _InfoRow extends StatelessWidget {
|
||||||
final String label;
|
final String label;
|
||||||
final String value;
|
final String value;
|
||||||
|
|||||||
Reference in New Issue
Block a user