angular-ui/ui-grid

1px padding-top on ui-grid-canvas

Open

#3275 aperta il 14 apr 2015

Vedi su GitHub
 (4 commenti) (0 reazioni) (1 assegnatario)JavaScript (2496 fork)batch import
good first issue

Metriche repository

Star
 (5395 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

1px padding was added to the top of the ui-grid-cavas in https://github.com/angular-ui/ng-grid/commit/040a5ee1acc1fdeda6a466fbe05d242f8775ef18. When you scroll the grids viewport to the top there is a 1px gap between the headers and the start of the grids rows.

It's not very noticeable with the grids default styles but if you change them, specifically if you make the cell borders a dark colour, the gap is very apparent.

Guida contributor

Direzione di ricerca
Ispeziona il commit 040a5ee1acc1fdeda6a466fbe05d242f8775ef18 che ha aggiunto il padding top di 1px a ui grid canvas. Determina se il padding è necessario o può essere rimosso e testa il comportamento di scorrimento.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CSSGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70