fix: Text on the scanner should not be selectable

[bhattabhi013]

What

Text on the scanner should not be selectable

Screenshot

Record_2022-03-31-23-45-21_cc729dbb1ea2c751a543590b5da9708c.mp4

Fixes bug(s)

• Text on the scanner should not be selectable #1426

Part of

• 🎯 What can I work on ? #525
  (please be as granular as possible)

[M123-dev]

Heyyy, some minor comments:

What about instead of using a conditional Text() or Text().selectable() everywhere just updating the Selectable Text extension for allowing to pass a bool if selectable or not and then deciding there wether to return Text or SelectableText

extension Selectable on Text {
  Widget selectable({bool isSelectable = false}) {
    return isSelectable
        ? SelectableText( ... )
        : Text( ... )

Also noticed the quantity text in the product title card is still always selectable

[codecov-commenter]

Codecov Report

❗ No coverage uploaded for pull request base (develop@35457d3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             develop   #1434   +/-   ##
=========================================
  Coverage           ?   9.15%           
=========================================
  Files              ?     158           
  Lines              ?    6380           
  Branches           ?       0           
=========================================
  Hits               ?     584           
  Misses             ?    5796           
  Partials           ?       0           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35457d3...51d33bf. Read the comment docs.

[bhattabhi013]

Thanks for the review and for providing a better approach to the solution. I've made the changes in the selectable extension.

[M123-dev]

The style is missing here

[M123-dev]

Looks good, thanks @bhattabhi013