Update SpaceSubSpacesDialog: Replace Text with SelectableText for title and error message, and add spacing in the content column to enhance user experience and maintainability.

This commit is contained in:
Faris Armoush
2025-07-06 11:00:01 +03:00
parent 6dcc851d97
commit dd735032ea
3 changed files with 12 additions and 11 deletions

View File

@ -16,7 +16,7 @@ abstract final class SpaceDetailsDialogHelper {
),
child: SpaceDetailsDialog(
context: context,
title: const Text('Create Space'),
title: const SelectableText('Create Space'),
spaceModel: SpaceModel.empty(),
onSave: print,
),
@ -36,7 +36,7 @@ abstract final class SpaceDetailsDialogHelper {
),
child: SpaceDetailsDialog(
context: context,
title: const Text('Edit Space'),
title: const SelectableText('Edit Space'),
spaceModel: spaceModel,
onSave: (space) {},
),

View File

@ -28,7 +28,7 @@ class SpaceDetailsForm extends StatelessWidget {
create: (context) => SpaceDetailsModelBloc(initialState: space),
child: BlocBuilder<SpaceDetailsModelBloc, SpaceDetailsModel>(
buildWhen: (previous, current) => previous != current,
builder: (context, state) {
builder: (context, space) {
return AlertDialog(
title: title,
backgroundColor: ColorsManager.whiteColors,
@ -39,7 +39,7 @@ class SpaceDetailsForm extends StatelessWidget {
spacing: 20,
crossAxisAlignment: CrossAxisAlignment.start,
children: [
Expanded(child: SpaceIconPicker(iconPath: state.icon)),
Expanded(child: SpaceIconPicker(iconPath: space.icon)),
Expanded(
flex: 2,
child: Column(
@ -47,17 +47,17 @@ class SpaceDetailsForm extends StatelessWidget {
crossAxisAlignment: CrossAxisAlignment.start,
children: [
SpaceNameTextField(
initialValue: state.spaceName,
isNameFieldExist: (value) => state.subspaces.any(
initialValue: space.spaceName,
isNameFieldExist: (value) => space.subspaces.any(
(subspace) => subspace.name == value,
),
),
const Spacer(),
SpaceSubSpacesBox(
subspaces: state.subspaces,
subspaces: space.subspaces,
),
const SizedBox(height: 16),
SpaceDetailsDevicesBox(space: state),
SpaceDetailsDevicesBox(space: space),
],
),
),
@ -66,7 +66,7 @@ class SpaceDetailsForm extends StatelessWidget {
),
actions: [
SpaceDetailsActionButtons(
onSave: () => onSave(state),
onSave: () => onSave(space),
onCancel: Navigator.of(context).pop,
),
],

View File

@ -56,8 +56,9 @@ class _SpaceSubSpacesDialogState extends State<SpaceSubSpacesDialog> {
@override
Widget build(BuildContext context) {
return AlertDialog(
title: const Text('Create Sub Spaces'),
title: const SelectableText('Create Sub Spaces'),
content: Column(
spacing: 12,
mainAxisSize: MainAxisSize.min,
children: [
SubSpacesInput(
@ -70,7 +71,7 @@ class _SpaceSubSpacesDialogState extends State<SpaceSubSpacesDialog> {
child: Visibility(
key: ValueKey(_hasDuplicateNames),
visible: _hasDuplicateNames,
child: const Text(
child: const SelectableText(
'Error: Duplicate subspace names are not allowed.',
style: TextStyle(color: Colors.red),
),