315 scrollbar for firefox #574
Conversation
👷 Deploy Preview for confort-plus processing.
|
| } | ||
|
|
||
| setScrollAspect = (value: any): void => { | ||
| console.log('ScrollAspectService - setScrollAspect ') |
There was a problem hiding this comment.
Oh oh... tu as laissé quelques console.log()
| .${PREFIX}big-scroll::-webkit-scrollbar-thumb, | ||
| .${PREFIX}big-scroll *::-webkit-scrollbar-thumb { | ||
| background-color: ${this.scrollColor}; | ||
| background-color: red; |
There was a problem hiding this comment.
Pourquoi fixer en rouge ? Un oubli lors du débug ?
| let styleScroll = ` | ||
| html { | ||
| scrollbar-width: none; | ||
| width: calc(100% - 40px); |
There was a problem hiding this comment.
Normalement, on habitude de fixer les tailles des éléments en rem plutôt qu'en pixel.
|
|
||
| function onMouseUp() { | ||
| isDragging = false; | ||
| document.removeEventListener('mousemove', onMouseMove); |
There was a problem hiding this comment.
Remplacer tous les évènements de type Mouse Event par ceux de type Pointer Event (mousemove-> pointermove), pour gérer le tactile.
| .filter((el: any) => !el.disabled && el.tabIndex >= 0); | ||
| } | ||
|
|
||
| addCustomScroolBar = (): void => { |
There was a problem hiding this comment.
| addCustomScroolBar = (): void => { | |
| addCustomScrollBar = (): void => { |
(penser à le remplacer les appels)
| scrollDown.addEventListener('click', function() { | ||
| window.scrollBy(0, 100); | ||
| }); | ||
| let navetteContainer = document.createElement('div'); |
There was a problem hiding this comment.
pour éviter le mélange anglais / français dans le code, tu peux renommer navette en thumb.
| scrollDownTriangle.className = 'cf-scroll-down-triangle'; | ||
| let scrollDown = document.createElement('div'); | ||
| scrollDown.className = 'cf-scroll-down'; | ||
| scrollUp.appendChild(scrollUpTriangle); |
There was a problem hiding this comment.
pour éviter le mélange anglais / français dans le code, tu peux renommer triangle par arrow.
src/app/services/dom.service.ts
Outdated
| /* | ||
| .cf-scroll-down-triangle { |
There was a problem hiding this comment.
Je suis partagé sur le dessin en css du bouton. J'aurais eu tendance à ajouter l'icone en SVG (si ,elle n'existe pas déjà) et l'appeler via le composant app-icon.
Le problème dans ce cas, c'est que pour chaque taille de triangle, tu as un css différent, ça ne me paraît pas très maintenable.
Dans solaris, tu as "Chevron down" par exemple.
2 participants
No description provided.