Ref T1524
Dla kontekstu warto też przeczytać https://forum.sealcode.org/t/sealpage-przemyslenia-dot-podzialu-komponentow-na-pliki-dla-backendu-i-front-endu/1052/7
Details
Details
- Reviewers
michal.starski arkadiusz-wieczorek - Group Reviewers
Unknown Object (Project) - Maniphest Tasks
- Unknown Object (Maniphest Task)
T1524: Dodanie komponentom możliwości tworzenia plików ubocznych
Diff Detail
Diff Detail
- Repository
- rSEALPAGE Sealpage
- Branch
- t1524
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 1781 Build 1781: arc lint + arc unit
Event Timeline
| components/download-file-button/download-file-button.js | ||
|---|---|---|
| 7 | s.require jest zwięzłe ale wydaje mi się że lepiej byłoby to jawnie nazwać s.node_require? | |
| components/responsive-image/responsive-image.js | ||
| 16 | Co w przypadku kiedy docelowy obrazek będzie mniejszy niż na przykład 900x900, czy powinniśmy generować srcset dla 1000 i 1300? Przeglądarka niepotrzebnie będzie ładowała większe rozdzielczości rozpikselizowanego pliku. | |
| 45 | `images` → 'images' | |
| lib/s.js | ||
| 37 | Ten kod błędu oznacza ENOENT: 2 - No such file or directory więc może lepiej zmienić fileExists na access lub exists | |
| 50 | Tak jak wspomniałem wcześniej może lepiej użyć node_require? | |
Comment Actions
- Change s.require to s.node_require
- Change type of quotes to make the code style more consistent
- fileExists->access
| components/download-file-button/download-file-button.js | ||
|---|---|---|
| 7 | Myślałem o tym, i podobny był też mój oryginalny zamysł. Poprawię to na node_require | |
| components/responsive-image/responsive-image.js | ||
| 16 | Dodałem domyślną wartość do tego propsa tylko dlatego, że chciałem mieć działający ResponsiveImage w BodyBuilder. Raczej te rozdzielczości powinny być nadpisywane za każdym razem ręcznie lub generowane automatycznie na podstawie rozmiaru oryginalnego pliku - ale na to założę taska w niedalekiej przyszłości | |