angular-ui/ui-grid

enableCellEditOnFocus does not work properly when a lot of columns.

Open

#6 772 ouverte le 21 juin 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (2 496 forks)batch import
good first issuegrid-editneeds: analysis

Métriques du dépôt

Stars
 (5 395 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

http://plnkr.co/edit/vpms1zqjILXS1QoeCYlm?p=preview To reproduce the problem you should сlick on the first cell, and then press the right arrow key until you force the table to initiate the scroll action. The cell outside the visible part of the table will not be active for editing. I think that the problem is in the last line of the ScrollHandler function.

function scrollHandler(evt) {
        var newScrollTop = $elm[0].scrollTop;
    var newScrollLeft = gridUtil.normalizeScrollLeft($elm, grid);

    var vertScrollPercentage = rowContainer.scrollVertical(newScrollTop);
    var horizScrollPercentage = colContainer.scrollHorizontal(newScrollLeft);

    var scrollEvent = new ScrollEvent(grid, rowContainer, colContainer, ScrollEvent.Sources.ViewPortScroll);
    scrollEvent.newScrollLeft = newScrollLeft;
        scrollEvent.newScrollTop = newScrollTop;
    if ( horizScrollPercentage > -1 ){
        scrollEvent.x = { percentage: horizScrollPercentage };
    }

    if ( vertScrollPercentage > -1 ){
        scrollEvent.y = { percentage: vertScrollPercentage };
    }

    grid.scrollContainers($scope.$parent.containerId, scrollEvent);
}

Guide contributeur

Direction de recherche
Examinez la fonction scrollHandler et tracez pourquoi la cellule en dehors de la zone visible ne devient pas modifiable lorsque enableCellEditOnFocus est activé. Concentrez vous sur la dernière ligne de scrollHandler et sur la façon dont elle interagit avec la logique d'édition de cellules.
Stack technique
javascriptangular
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitAngular
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75