internal/flutter_availability
5
0
Fork 0
mirror of https://github.com/Iconica-Development/flutter_availability.git synced 2025-05-19 05:03:44 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: reduce complicated ternary operators to simpler conditional statements

Browse source
This commit is contained in:
Bart Ribbers 2024-07-18 14:42:08 +02:00 committed by Freek van de Ven
parent 7b54809f6e
commit 9efbcbf48b
2 changed files with 87 additions and 97 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
packages/flutter_availability/lib/src/ui/widgets/availability_template_selection.dart
View file

@ -35,12 +35,14 @@ class AvailabilityTemplateSelection extends StatelessWidget {
var options = availabilityScope.options;
var translations = options.translations;
// TODO(Joey): Do not nest ternairy operators
var titleText = selectedTemplates.isEmpty
? translations.availabilityAddTemplateTitle
: selectedTemplates.length > 1
? translations.availabilityUsedTemplates
: translations.availabilityUsedTemplate;
var titleText = translations.availabilityAddTemplateTitle;
if (selectedTemplates.isEmpty) {
if (selectedTemplates.length > 1) {
titleText = translations.availabilityUsedTemplates;
} else {
titleText = translations.availabilityUsedTemplate;
}
}
var addButton = options.bigTextButtonWrapperBuilder(
context,

170
packages/flutter_availability/lib/src/ui/widgets/template_legend.dart
View file

@ -45,6 +45,8 @@ class _TemplateLegendState extends State<TemplateLegend> {
?.any((element) => element.template == null) ??
false;
var templatesVisible = templatesAvailable && _templateDrawerOpen;
void onDrawerHeaderClick() {
if (!templatesAvailable && !_templateDrawerOpen) {
return;
@ -72,6 +74,77 @@ class _TemplateLegendState extends State<TemplateLegend> {
),
);
Widget body = const Divider(
height: 1,
thickness: 1,
);
if (_templateDrawerOpen && !templatesLoading) {
body = SingleChildScrollView(
controller: _scrollController,
child: Container(
constraints: const BoxConstraints(maxHeight: 152),
child: Scrollbar(
controller: _scrollController,
thumbVisibility: true,
trackVisibility: true,
thickness: 2,
child: Column(
children: [
for (final (index, template) in templates.indexed) ...[
if (index == 0) ...[
Padding(
padding: const EdgeInsets.only(
top: 10,
left: 12,
),
child: _TemplateLegendItem(
name: translations.templateSelectionLabel,
backgroundColor: colors.selectedDayColor ??
colorScheme.primaryFixedDim,
borderColor: colorScheme.primary,
),
),
if (existAvailabilitiesWithoutTemplate) ...[
Padding(
padding: const EdgeInsets.only(
top: 10,
left: 12,
),
child: _TemplateLegendItem(
name: translations.availabilityWithoutTemplateLabel,
backgroundColor: colors.customAvailabilityColor ??
colorScheme.secondary,
),
),
],
] else ...[
Padding(
padding: const EdgeInsets.only(
top: 10,
left: 12,
),
child: _TemplateLegendItem(
name: template.name,
backgroundColor: Color(template.color),
),
),
],
],
Padding(
padding: const EdgeInsets.only(
top: 10,
bottom: 8,
),
child: createNewTemplateButton,
),
],
),
),
),
);
}
return Column(
children: [
// a button to open/close a drawer with all the templates
@ -106,101 +179,16 @@ class _TemplateLegendState extends State<TemplateLegend> {
constraints: BoxConstraints(maxHeight: _templateDrawerOpen ? 150 : 0),
decoration: BoxDecoration(
border: _templateDrawerOpen
? Border.all(color: theme.colorScheme.onSurface, width: 1)
? Border.all(
color: theme.colorScheme.onSurface,
width: 1,
)
: null,
),
padding: const EdgeInsets.only(right: 2),
child: _templateDrawerOpen && !templatesLoading
// TODO(Joey): A listview inside a scrollview inside the
// scrollable that each page has seems like really strange UX
// TODO(Joey): No ternary operators in the layout
? SingleChildScrollView(
child: Column(
children: [
Container(
constraints: const BoxConstraints(
// TODO(Joey): Not divisible by 4
maxHeight: 150,
),
child: Scrollbar(
controller: _scrollController,
thumbVisibility: true,
trackVisibility: true,
thickness: 2,
child: ListView.builder(
controller: _scrollController,
shrinkWrap: true,
// TODO(Joey): This seems like an odd way to
// implement appending items
itemCount: templates.length + 2,
itemBuilder: (context, index) {
if (index == 0) {
// TODO(Joey): Extract this as a widget
return Column(
children: [
Padding(
padding: const EdgeInsets.only(
top: 10,
left: 12,
),
child: _TemplateLegendItem(
name:
translations.templateSelectionLabel,
backgroundColor:
colors.selectedDayColor ??
colorScheme.primaryFixedDim,
borderColor: colorScheme.primary,
),
),
if (existAvailabilitiesWithoutTemplate) ...[
Padding(
padding: const EdgeInsets.only(
top: 10,
left: 12,
),
child: _TemplateLegendItem(
name: translations
.availabilityWithoutTemplateLabel,
backgroundColor:
colors.customAvailabilityColor ??
colorScheme.secondary,
),
),
],
],
);
}
if (index == templates.length + 1) {
return Padding(
padding: const EdgeInsets.only(
top: 10,
bottom: 8,
),
child: createNewTemplateButton,
);
}
var template = templates[index - 1];
return _TemplateLegendItem(
name: template.name,
backgroundColor: Color(template.color),
);
},
),
),
),
],
),
)
: const Divider(
height: 1,
thickness: 1,
),
child: body,
),
// TODO(Joey): This is too complex of a check to read the layout
// There are 8 different combinations parameters with 2 different
// outcomes
if (!templatesAvailable &&
(!_templateDrawerOpen || templatesLoading)) ...[
if (!templatesVisible) ...[
const SizedBox(height: 12),
if (templatesLoading) ...[
options.loadingIndicatorBuilder(context),