Settings panel

[mateobelanger]

• Added settings tab to navdrawer
• Persistent settings with streamed_shared_preferences package (previously added by Mourad)
• Cute UI blocks with settings_ui package
• Added Age, Sex and AcquisitionBoard settings
• 'Not Set' values when app is initially launched
• Settings widget can be added to Record 'Sleep sequence' when a value has not been setted.

Misc

• Changed sliverbar backgound, I think it's nice
• Changed the app logo to new dodo

[MouradLachhab]

Really nice. The only other thing is that when I started checking how to dynamically change between Bluetooth and USB repository, I looked into streaming_shared_preferences that extends shared_preferences. So unless we go for the solution where we automatically connect to the device instead of selecting it, we might have to swap for that to do dynamic changes. It allows you to subscribe to preferences and be notified of changes. They are very similar and changes should be minimal.

[WilliamHarvey97]

such wow! good job!

[mateobelanger]

Je n'aime pas du tout le widget pour la sélection du board d'acquisition? Qu'est-ce que vous diriez d'utiliser des radio buttons dans un modal? J'ai déjà fait un widget pour ça:

import 'package:flutter/material.dart';
import 'package:flutter/src/widgets/framework.dart';
import 'package:flutter_hooks/flutter_hooks.dart';

class RadioButtonItem {
  final Widget title;
  final dynamic value;

  RadioButtonItem({@required this.title, @required this.value});
}

class RadioButtonsDialog<T> extends HookWidget {
  final Widget title;
  final List<RadioButtonItem> radioButtonItems;
  final T initialValue;
  final String cancelLabel;
  final String confirmLabel;
  final void Function(T) onConfirm;

  RadioButtonsDialog({
    Key key,
    this.title,
    @required this.radioButtonItems,
    @required this.initialValue,
    @required this.cancelLabel,
    @required this.confirmLabel,
    @required this.onConfirm,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final selection = useState(initialValue);
    return AlertDialog(
      title: title,
      contentPadding: EdgeInsets.symmetric(vertical: 8),
      content: Column(
        mainAxisSize: MainAxisSize.min,
        children: [
          ...radioButtonItems.map(
            (e) => RadioListTile(
              title: e.title,
              value: e.value,
              groupValue: selection.value,
              onChanged: (value) => selection.value = value,
            ),
          )
        ],
      ),
      actions: <Widget>[
        FlatButton(
          child: Text(cancelLabel.toUpperCase()),
          onPressed: () {
            Navigator.of(context).pop(false);
          },
        ),
        FlatButton(
          child: Text(
            confirmLabel.toUpperCase(),
            style: TextStyle(
              color: Theme.of(context).accentColor,
            ),
          ),
          onPressed: () {
            onConfirm(selection.value);
            Navigator.of(context).pop(true);
          },
        ),
      ],
    );
  }
}

(Ici je juge que c'est une bonne chose d'avoir un state local au widget, car c'est uniquement un state de présentation et non pas un state d'application. Par contre, il faudra sauvegarder la carte sélectionnné dans un bloc)

j'ai enlevé l'option du board d'acquisition parce que c'était plus nécessaire

[WilliamHarvey97]

j'ai enlevé l'option du board d'acquisition parce que c'était plus nécessaire

Pour le sexe alors? Aussi, si c'est le genre de widget que tu aimes j'aurais aussi un NumberInputDialog

[mateobelanger]

Je n'aime pas du tout le widget pour la sélection du board d'acquisition? Qu'est-ce que vous diriez d'utiliser des radio buttons dans un modal? J'ai déjà fait un widget pour ça:

import 'package:flutter/material.dart';
import 'package:flutter/src/widgets/framework.dart';
import 'package:flutter_hooks/flutter_hooks.dart';

class RadioButtonItem {
  final Widget title;
  final dynamic value;

  RadioButtonItem({@required this.title, @required this.value});
}

class RadioButtonsDialog<T> extends HookWidget {
  final Widget title;
  final List<RadioButtonItem> radioButtonItems;
  final T initialValue;
  final String cancelLabel;
  final String confirmLabel;
  final void Function(T) onConfirm;

  RadioButtonsDialog({
    Key key,
    this.title,
    @required this.radioButtonItems,
    @required this.initialValue,
    @required this.cancelLabel,
    @required this.confirmLabel,
    @required this.onConfirm,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final selection = useState(initialValue);
    return AlertDialog(
      title: title,
      contentPadding: EdgeInsets.symmetric(vertical: 8),
      content: Column(
        mainAxisSize: MainAxisSize.min,
        children: [
          ...radioButtonItems.map(
            (e) => RadioListTile(
              title: e.title,
              value: e.value,
              groupValue: selection.value,
              onChanged: (value) => selection.value = value,
            ),
          )
        ],
      ),
      actions: <Widget>[
        FlatButton(
          child: Text(cancelLabel.toUpperCase()),
          onPressed: () {
            Navigator.of(context).pop(false);
          },
        ),
        FlatButton(
          child: Text(
            confirmLabel.toUpperCase(),
            style: TextStyle(
              color: Theme.of(context).accentColor,
            ),
          ),
          onPressed: () {
            onConfirm(selection.value);
            Navigator.of(context).pop(true);
          },
        ),
      ],
    );
  }
}

(Ici je juge que c'est une bonne chose d'avoir un state local au widget, car c'est uniquement un state de présentation et non pas un state d'application. Par contre, il faudra sauvegarder la carte sélectionnné dans un bloc)

j'ai enlevé l'option du board d'acquisition parce que c'était plus nécessaire

j'ai enlevé l'option du board d'acquisition parce que c'était plus nécessaire
pour le widget de sélection dropdown. J'me suis inspiré des settings de Spotify. C'est purement une préférence subjective d'ergonomie. J'trouve ça moins intrusif.

[WilliamHarvey97]

Please use contants like MAX_AGE, MIN_AGE.

Maybe use a factory constructor so you can throw custom exceptions that you could handle in the future, in place of assert statements.

outdated