nextcloud/text

Ctrl+A inside table cell should only select all of the cell's content instead of the whole document.

Ouverte

#5 785 ouverte le 10 mai 2024

Voir sur GitHubGuide contributeur
 (5 commentaires) (2 réactions) (0 personne assignée)JavaScript (125 forks)auto 404
1. to developbugfeature: tablesgood first issue

Métriques du dépôt

Stars
 (644 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Describe the bug Ctrl+A usually only selects the current cell content if the user is inside a table instead of selecting the whole document

To Reproduce Steps to reproduce the behavior:

  1. Create a new table
  2. Create some cells with text content
  3. Enter a cell and press Ctrl+A

Expected behavior Only the text inside the cell should be selected.

Server details:

  • Nextcloud version: 28.0.4
  • PHP Version: 8.3
  • Database: PostgreSQL 15

Client details:

  • OS: Arch Linux
  • Browser: Firefox
  • Browser version: 125.0.3
  • Device: Desktop

Guide contributeur

Direction de recherche
Examinez le gestionnaire d'événements keydown pour Ctrl+A dans le composant éditeur, en particulier la façon dont la sélection des cellules du tableau est actuellement gérée. Recherchez la logique de sélection existante et modifiez la pour limiter la sélection à la cellule actuelle lorsque vous êtes dans un tableau.
Stack technique
javascript
Domaine
frontend
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptDOM events
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70