Unverified Commit edbd0937 authored by Nils Reichardt's avatar Nils Reichardt Committed by GitHub

ListTile Mouse Pointer Fix (#60563)

parent 28d2f010
...@@ -946,7 +946,7 @@ class ListTile extends StatelessWidget { ...@@ -946,7 +946,7 @@ class ListTile extends StatelessWidget {
final MouseCursor effectiveMouseCursor = MaterialStateProperty.resolveAs<MouseCursor>( final MouseCursor effectiveMouseCursor = MaterialStateProperty.resolveAs<MouseCursor>(
mouseCursor ?? MaterialStateMouseCursor.clickable, mouseCursor ?? MaterialStateMouseCursor.clickable,
<MaterialState>{ <MaterialState>{
if (!enabled) MaterialState.disabled, if (!enabled || (onTap == null && onLongPress == null)) MaterialState.disabled,
if (selected) MaterialState.selected, if (selected) MaterialState.selected,
}, },
); );
......
...@@ -1517,5 +1517,21 @@ void main() { ...@@ -1517,5 +1517,21 @@ void main() {
); );
expect(RendererBinding.instance.mouseTracker.debugDeviceActiveCursor(1), SystemMouseCursors.basic); expect(RendererBinding.instance.mouseTracker.debugDeviceActiveCursor(1), SystemMouseCursors.basic);
// Test default cursor when onTap or onLongPress is null
await tester.pumpWidget(
const MaterialApp(
home: Material(
child: Center(
child: MouseRegion(
cursor: SystemMouseCursors.forbidden,
child: ListTile(),
),
),
),
),
);
expect(RendererBinding.instance.mouseTracker.debugDeviceActiveCursor(1), SystemMouseCursors.basic);
}); });
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment