filamentphp/filament

MDE behaves differently when submitted via mouse click vs keyBinding

Open

#12 858 ouverte le 19 mai 2024

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)PHP (4 162 forks)batch import
bughelp wantedlow priorityunconfirmed

Métriques du dépôt

Stars
 (30 736 stars)
Métriques de merge PR
 (Merge moyen 2j 15h) (72 PRs mergées en 30 j)

Description

Package

filament/forms

Package Version

3.2.80

Laravel Version

11.7.0

Livewire Version

3.4.12

PHP Version

8.2.18

Problem description

The behavior of clicking on submit vs pressing the registered keyBinding() is different. Clicking will clear the form. (correct) Pressing the shortcut key does not. (incorrect)

Expected behavior

Pressing the shortcut key should also clear the form.

Steps to reproduce

  1. Type something into the markdown editor at /admin/issue
  2. Press ctrl + enter

Reproduction repository

https://github.com/bilogic/filament-issue/tree/click-different-from-keyBindings

Relevant log output

https://github.com/filamentphp/filament/assets/946010/fb144bf9-802f-4003-8dcd-ec4c2148e742

No response

Guide contributeur

Direction de recherche
Concentrez vous sur le composant MarkdownEditor dans le package filament/forms. Le problème décrit une divergence entre les chemins de soumission par clic de souris et par raccourci clavier. Examinez les gestionnaires d'événements JavaScript (probablement dans resources/js/components/markdown editor.js ou similaire) pour identifier pourquoi le raccourci clavier ne déclenche pas la logique de nettoyage du formulaire. Consultez le dépôt de reproduction fourni (https://github.com/bilogic/filament issue/tree/click different from keyBindings) pour configurer l'environnement et tester le comportement. Passez en revue les 8 commentaires du problème pour toute suggestion des mainteneurs ou contexte supplémentaire.
Stack technique
phpjavascript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHPLaravelLivewire
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70