feat: replace static premium indicator with editable input and dynamic slider range

[Catrya]

fix #442

Summary by CodeRabbit

• New Features

  • Premium section now has an editable input with synchronized real-time slider, formatting, and dynamic bounds validation
  • Added an "Edit" hint string in English, Spanish and Italian

• UI/UX Improvements

  • Icon backgrounds can render as transparent when specified

[coderabbitai]

Walkthrough

This PR makes FormSection icon background conditional (transparent vs circular), converts PremiumSection into a StatefulWidget with debounced text input, focus management, and dynamic slider/input synchronization, and adds a new localization key premiumEditHint in English, Spanish, and Italian.

Changes

Cohort / File(s)	Summary
Icon rendering lib/features/order/widgets/form_section.dart	Icon background now conditional: if iconBackgroundColor is Colors.transparent the icon is rendered without the 36×36 circular container; otherwise the circular colored container is shown.
Premium input & slider (stateful) lib/features/order/widgets/premium_section.dart	PremiumSection converted from StatelessWidget → StatefulWidget. Adds TextEditingController, FocusNode, debounce timer, editing flag, lifecycle methods, debounced input parsing/clamping/formatting, and synchronized slider ↔ text input with dynamic bounds and divisions.
Localization lib/l10n/intl_en.arb, lib/l10n/intl_es.arb, lib/l10n/intl_it.arb	Added premiumEditHint translations: "Edit" (en), "Editar" (es), "Modifica" (it).

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant TextField
    participant DebounceTimer
    participant Parser
    participant Slider
    participant Parent

    User->>TextField: type value / edit
    TextField->>DebounceTimer: schedule debounced update
    DebounceTimer->>Parser: parse, clamp, format value
    Parser->>Slider: update slider position/value
    Slider->>Parent: onChanged(value)
    Parent->>TextField: propagate external value update

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• Bug when selecting the premium or discount for an order #123 — touches premium_section.dart, related slider/divisions behavior and input syncing.
• New range order UX #253 — modifies form_section.dart header/icon rendering; overlaps icon/background changes.
• Improve info button UX and standardize dialog styling #227 — refactors FormSection/PremiumSection UI and info handling; directly related to these widgets.

Suggested reviewers

• grunch
• mostronator
• AndreaDiazCorreia

Poem

🐰 Hopping through pixels, I tweak and mend,

Icons lose circles—transparent trend.
Sliders hum softly, inputs debounce light,
Percentages settle, values sit right.
Translations whisper: Edit, Edit, Modifica.

🚥 Pre-merge checks | ✅ 6

✅ Passed checks (6 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: replacing a static premium indicator with an editable input and dynamic slider range.
Linked Issues check	✅ Passed	The changes implement dynamic slider bounds based on current value, allowing premium range beyond the fixed -10% to +10% limit as required by issue #442.
Out of Scope Changes check	✅ Passed	All changes directly support the PR objective: editable input UI, dynamic slider bounds, conditional icon rendering, and localization strings for the edit hint.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch premiun-discount-range

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

lib/features/order/widgets/premium_section.dart (1)

192-198: Add const to TextStyle.

This TextStyle has only compile-time constant arguments and should be const for consistency with the rest of the file (e.g., lines 123, 241, 246).

Suggested fix

         Text(
           S.of(context)!.premiumEditHint,
-          style: TextStyle(
+          style: const TextStyle(
             color: AppTheme.textSubtle,
             fontSize: 10,
           ),
         ),

As per coding guidelines, "Use const constructors where possible to optimize widget performance."

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@lib/features/order/widgets/premium_section.dart`:
- Around line 154-156: The input formatter currently uses
FilteringTextInputFormatter.allow(RegExp(r'^-?\d{0,3}')) which uses a leading ^
anchor and causes user input loss; update the formatter in the TextField/widget
that sets inputFormatters (the array containing
FilteringTextInputFormatter.allow) to either remove the anchor and allow
individual digit/minus characters (e.g., use a pattern without ^ so the
formatter matches characters rather than only from position 0) and rely on the
existing _commitTextValue clamping, or replace it with a
TextInputFormatter.withFunction that tests the entire resulting string
(newValue.text) against RegExp(r'^-?\d{0,3}$') and returns oldValue when the
full-string match fails. Ensure you modify the formatter where
FilteringTextInputFormatter.allow(RegExp(r'^-?\d{0,3}')) is declared in
premium_section.dart.

🧹 Nitpick comments (1)

lib/features/order/widgets/premium_section.dart (1)

73-78: 2-second debounce may feel sluggish for a premium input.

The debounce delay of 2 seconds before committing the text value is quite long. Users might expect faster feedback, especially since the slider and labels won't update until the debounce fires. Consider reducing to ~500-800ms for a snappier feel, or committing immediately when the input is a complete valid number.

[grunch]

solid! tACK