angular-ui/ui-grid

emptyBaseLayer.directive and expandable.directive collision

Open

#6.830 aberto em 8 de ago. de 2018

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)JavaScript (2.496 forks)batch import
good first issuegrid-empty-base-layer

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Because of the same priority with emptyBaseLayer.directive, expandable.directive on line: var rowRepeatDiv = angular.element($elm.children().children()[0]);

Gets the rowRepeatDiv of wrong element: div.ui-grid-empty-base-layer-container.ui-grid-canvas instead of: div.ui-grid-canvas (which is the second element of the array - $elm.children().children()[1])

Becouse of that, sub-rows wont get rendered. Raising expandable.directive priority fixes the rendering.

Is it a good case for a pullrequest? Or is it because of my configuration? I also use:

  • ui-grid-pagination
  • ui-grid-auto-resize
  • ui-grid-selection

I tried ui-grid-empty-base-layer="false", but that didn't help.

Guia do colaborador

Direção de pesquisa
Investigue as prioridades das diretivas no ui grid, especificamente compare emptyBaseLayer.directive e expandable.directive. O problema sugere aumentar a prioridade de expandable.directive para corrigir a colisão. Confirme examinando as definições das diretivas e a estrutura DOM na grade. Verifique se a correção introduz quaisquer efeitos colaterais para outros recursos, como paginação ou redimensionamento automático. Consulte o problema e os comentários vinculados para obter contexto adicional.
Pilha de tecnologia
javascriptangular
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
AngularGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75