A discrete RangeSlider is not draggable inside a ListView. #51620
Labels
f: material design
flutter/packages/flutter/material repository.
found in release: 3.7
Found to occur in 3.7
framework
flutter/packages/flutter repository. See also f: labels.
has reproducible steps
The issue has been confirmed reproducible and is ready to work on
team-design
Owned by Design Languages team
triaged-design
Triaged by Design Languages team
Comments
|
Hi @serhii-k |
VladyslavBondarenko
added
the
waiting for customer response
|
Hi @VladyslavBondarenko And a recording:
|
no-response
bot
removed
the
waiting for customer response
|
By the way, I've also added onChangeStart and onChangeEnd callbacks. And here's the output (following this video): So even for the last two correct attempts, there are unnecessary intermediate callbacks. |
|
@serhii-k
|
VladyslavBondarenko
added
f: material design
|
A temporary workaround.
This is just a temporary solution. When in a scrollable parent, there is some issue with gesture recognizers inside the _RenderRangeSlider class. Notice: this solution triggers some unnecessary intermediate calls to onChangeEnd() and onChangeStart() (_tap.onTapCancel followed by _drag.onStart). |
|
I think I'm having the same issue. RangeSlider in a Scrollable parent is such a common use case. I can't believe it still exists in 1.17 "Stable". |
VladyslavBondarenko
added
has reproducible steps
|
I'm still having this problem in: |
|
Still having this prob! |
|
Commenting to add visibility to this issue. |
|
Hi @serhii-k, I verified the issue on the latest master 3.7 and it doesn't seem to reproduce. I believe the bug is fixed. Screen.Recording.2023-01-03.at.9.27.01.PM.movcode sampleimport 'package:flutter/material.dart';
void main() => runApp(const MyApp());
class MyApp extends StatelessWidget {
const MyApp({super.key});
static const String _title = 'Flutter Code Sample';
@override
Widget build(BuildContext context) {
return MaterialApp(
title: _title,
home: Scaffold(
appBar: AppBar(title: const Text(_title)),
body: const SimpleTest(),
),
);
}
}
class SimpleTest extends StatelessWidget {
const SimpleTest({super.key});
@override
Widget build(BuildContext context) {
return Scaffold(
body: ListView(
padding: const EdgeInsets.only(top: 100.0),
children: const <Widget>[
RangeTest(),
],
),
);
}
}
class RangeTest extends StatefulWidget {
const RangeTest({super.key});
@override
_RangeTestState createState() => _RangeTestState();
}
class _RangeTestState extends State<RangeTest> {
var _values = const RangeValues(2.0, 8.0);
@override
Widget build(BuildContext context) {
return RangeSlider(
values: _values,
onChanged: (rangeValues) => setState(() {
_values = rangeValues;
}),
min: 0.0,
max: 10.0,
divisions: 10,
);
}
}
flutter doctor -v (mac)cc: @koodimetsa Closing as this is no longer reproducible, Incase anyone still encounters the issue, then please feel free to write in the comments and we will reopen it. Thank you. |
maheshmnj
changed the title
|
@maheshmnj In this case RangeSlider doesn't work reliably. |
|
@koodimetsa were you able to reproduce with the code sample I shared? |
|
@maheshmnj Yes, I used that code sample 👍🏼 It seems, that if the touch event starts from "inside" of the range slider, the drag event abruptly ends. Here is a video with an example: Screen.Recording.2023-01-25.at.17.22.51.mov |
|
Thanks for the update @koodimetsa, I was able to reproduce the issue on stable 3.7 and the master channel. Screen.Recording.2023-01-26.at.3.13.33.PM.movflutter doctor -v |
maheshmnj
added
found in release: 3.7
maheshmnj
removed
the
r: fixed
maheshmnj
changed the title
32 tasks
flutter-triage-bot
bot
added
multiteam-retriage-candidate
team-design
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A RangeSlider is awesome. But when its discrete version is put inside a ListView, an annoying bug arises. For simplicity, let's say we're using the Left-to-Right directionality.
The text was updated successfully, but these errors were encountered: