Fix #21 (keyboard long press) while addressing #9 (launching apps wit…

…h mouse/touch)

* Modify FocusKeyboardListener according to the current needs
* Remove automatic release creation for now

.github/workflows/continuous-release.yml

@@ -45,16 +45,6 @@ jobs:
       - name: Upload APKs as artifact
         uses: actions/upload-artifact@v4
         with:
-          name: flauncher-continuous-release
+          name: flauncher-release
           path: ${{ env.OUTPUT_DIR }}/flauncher-*.apk
           retention-days: 30
-      - name: Delete last continuous release
-        continue-on-error: true
-        env:
-          GH_TOKEN: ${{ secrets.CI_ACCESS_TOKEN }}
-        run: gh release delete continuous --yes
-      - name: Create release
-        env:
-          GH_TOKEN: ${{ secrets.CI_ACCESS_TOKEN }}
-        run: gh release create continuous --title "Continuous release" --notes "" $OUTPUT_DIR/flauncher-*.apk
-

lib/widgets/app_card.dart

@@ -97,22 +97,7 @@ class _AppCardState extends State<AppCard> with SingleTickerProviderStateMixin {
 
   @override
   Widget build(BuildContext context) => FocusKeyboardListener(
-      onPressed: (key) {
-        // A duplicate press occurs on "app cards" if both the
-        // FocusKeyboardListener and another widget
-        // (InkWell or GestureDetector, see below) handle the onPress and onTap,
-        // respectively, causing an attempt to open applications twice,
-        // though this is only visible when opening the settings side panel on TVs.
-        // Still, both handlers are needed to open applications using a touch
-        // screen, TV remote, and keyboards, as well as being able to drag the
-        // card in the "dragging mode", this is why only the Enter key is ignored
-        // here.
-        if (key == LogicalKeyboardKey.enter) {
-          return KeyEventResult.ignored;
-        }
-
-        return _onPressed(context, key);
-      },
+      onPressed: (key) => _onPressed(context, key),
       onLongPress: (key) => _onLongPress(context, key),
       builder: (context) {
         return AspectRatio(

lib/widgets/focus_keyboard_listener.dart

@@ -43,27 +43,29 @@ class _FocusKeyboardListenerState extends State<FocusKeyboardListener> {
   @override
   Widget build(BuildContext context) => Focus(
         canRequestFocus: false,
+        // Using "onKeyEvent", in favor of the deprecated "onKey"
+        // seems to break the fix for issue #21 so, keep using the old property
         onKey: (_, rawKeyEvent) => _handleKey(context, rawKeyEvent),
         child: Builder(builder: widget.builder),
       );
 
   KeyEventResult _handleKey(BuildContext context, RawKeyEvent rawKeyEvent) {
     switch (rawKeyEvent.runtimeType) {
       case RawKeyDownEvent:
-        return _keyDownEvent(context, rawKeyEvent.logicalKey, (rawKeyEvent.data as RawKeyEventDataAndroid));
+        return _keyDownEvent(context, rawKeyEvent.logicalKey);
       case RawKeyUpEvent:
         return _keyUpEvent(context, rawKeyEvent.logicalKey);
     }
     return KeyEventResult.handled;
   }
 
-  KeyEventResult _keyDownEvent(BuildContext context, LogicalKeyboardKey key, RawKeyEventDataAndroid data) {
+  KeyEventResult _keyDownEvent(BuildContext context, LogicalKeyboardKey key) {
     if (!longPressableKeys.contains(key)) {
       return widget.onPressed?.call(key) ?? KeyEventResult.ignored;
     }
-    if (data.repeatCount == 0) {
+    if (_keyDownAt == null) {
       _keyDownAt = DateTime.now().millisecondsSinceEpoch;
-      return KeyEventResult.ignored;
+      return KeyEventResult.handled;
     } else if (_longPress()) {
       _keyDownAt = null;
       return widget.onLongPress?.call(key) ?? KeyEventResult.ignored;

pubspec.yaml

@@ -1,7 +1,7 @@
 name: flauncher
 description: Flutter-based Android TV launcher.
 
-version: 0.0.1
+version: 2024.09.002+13
 
 environment:
   sdk: ">=3.4.3"