Formatting improvements for the Scale Answer (#122)

* Improve view of scale answer with multiline text and spacing plus config of value visibility * Fix unnecessary reverted text directionality
QuickBirdEng · Aug 30, 2023 · 8867f8c · 8867f8c
1 parent 5817f14
commit 8867f8c
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 15 deletions.
diff --git a/lib/src/answer_format/scale_answer_format.dart b/lib/src/answer_format/scale_answer_format.dart
@@ -11,6 +11,7 @@ class ScaleAnswerFormat implements AnswerFormat {
   final double step;
   final String maximumValueDescription;
   final String minimumValueDescription;
+  final bool showValue;
 
   const ScaleAnswerFormat({
     required this.maximumValue,
@@ -19,6 +20,7 @@ class ScaleAnswerFormat implements AnswerFormat {
     required this.step,
     this.maximumValueDescription = '',
     this.minimumValueDescription = '',
+    this.showValue = true,
   }) : super();
 
   factory ScaleAnswerFormat.fromJson(Map<String, dynamic> json) =>

diff --git a/lib/src/answer_format/scale_answer_format.g.dart b/lib/src/answer_format/scale_answer_format.g.dart
diff --git a/lib/src/views/scale_answer_view.dart b/lib/src/views/scale_answer_view.dart
@@ -66,30 +66,47 @@ class _ScaleAnswerViewState extends State<ScaleAnswerView> {
               mainAxisAlignment: MainAxisAlignment.spaceEvenly,
               crossAxisAlignment: CrossAxisAlignment.center,
               children: [
-                Padding(
-                  padding: const EdgeInsets.all(14.0),
-                  child: Text(
-                    _sliderValue.toInt().toString(),
-                    style: Theme.of(context).textTheme.headlineSmall,
+                if (_scaleAnswerFormat.showValue)
+                  Padding(
+                    padding: const EdgeInsets.all(14.0),
+                    child: Text(
+                      _sliderValue.toInt().toString(),
+                      style: Theme.of(context).textTheme.headlineSmall,
+                    ),
                   ),
-                ),
                 Column(
                   children: [
                     Padding(
-                      padding: const EdgeInsets.symmetric(horizontal: 32.0),
+                      padding: const EdgeInsets.symmetric(horizontal: 16.0),
                       child: Row(
                         mainAxisAlignment: MainAxisAlignment.spaceBetween,
+                        crossAxisAlignment: CrossAxisAlignment.center,
                         children: [
-                          Text(
-                            _scaleAnswerFormat.minimumValueDescription,
-                            style: TextStyle(
-                              fontSize: 16.0,
+                          Expanded(
+                            child: Text(
+                              _scaleAnswerFormat.minimumValueDescription,
+                              style: TextStyle(
+                                fontSize: 16.0,
+                              ),
+                              maxLines: 6,
+                              overflow: TextOverflow.ellipsis,
+                              textDirection: TextDirection.ltr,
+                              textAlign: TextAlign.start,
                             ),
                           ),
-                          Text(
-                            _scaleAnswerFormat.maximumValueDescription,
-                            style: TextStyle(
-                              fontSize: 16.0,
+                          SizedBox(
+                            width: 32.0,
+                          ),
+                          Expanded(
+                            child: Text(
+                              _scaleAnswerFormat.maximumValueDescription,
+                              style: TextStyle(
+                                fontSize: 16.0,
+                              ),
+                              maxLines: 6,
+                              overflow: TextOverflow.ellipsis,
+                              textDirection: TextDirection.ltr,
+                              textAlign: TextAlign.end,
                             ),
                           ),
                         ],