fix(#553): make the Build value part of the 7-tap target

The build stamp rendered as SelectableText, which ate taps for selection, so
only the plain 'Build' label triggered the unlock. When a row is tappable it
now uses a plain Text so the whole row (label + value) is one InkWell target;
copy stays on the button. Part of #509/#553.

Agent: QuietSnow (session 31eaba02)
pull/556/head
Strycher 2 months ago
parent fb649e746d
commit f9064735e9

@ -954,7 +954,19 @@ class _SettingsScreenState extends State<SettingsScreen> {
crossAxisAlignment: CrossAxisAlignment.start,
children: [
Expanded(
child: SelectableText(
// When the row is tappable, use a plain Text so the value is
// part of the InkWell tap target (a SelectableText would eat
// the tap for selection). Copy is still available via the
// button. (#553)
child: onTap != null
? Text(
value,
style: theme.textTheme.bodyLarge?.copyWith(
color: valueColor,
fontWeight: FontWeight.w500,
),
)
: SelectableText(
value,
style: theme.textTheme.bodyLarge?.copyWith(
color: valueColor,

Loading…
Cancel
Save

Powered by TurnKey Linux.