search

conedevelopment / cone-site

The new 11ty-based portfolio website of Cone.
https://conedevelopment.com
3 stars 1 forks source link

Simplify accordion-card #65

Closed szepeviktor closed 4 months ago

szepeviktor commented 4 months ago

A validator azt mondta, nincs olyan, hogy rotate: 90deg;, én meg előjöttem ezzel a huszárvágással.

⚠️ Generálni kell hozzá CSS-t. De jó lenne, ha a CI ezt megcsinálná!

netlify[bot] commented 4 months ago

Deploy Preview for cone-development-site ready!

Name Link
Latest commit f2bd710c4464c36e1c9284d14f9f8346d0a4e779
Latest deploy log https://app.netlify.com/sites/cone-development-site/deploys/66450d944cebaf00084d0b7b
Deploy Preview https://deploy-preview-65--cone-development-site.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

szepeviktor commented 4 months ago

Na várjunk csak.

9:32:33 PM: $ npm run prod

A Netlify lefuttatja a "prod"-ot. Akkor minek hurcoljuk magunkkal itt a repóban a CSS fájlokat? (jó kifogás sosem rossz!)

adamlaki commented 4 months ago

Van már jó ideje rotate property jó ideje: https://caniuse.com/?search=rotate. Ezek a validátorok néhol egészen le vannak maradva.

A CSS-t azért tartjuk meg, mert ha kiszedem lokálon nem frissíti a live reload fejlesztésnél. Hogy őszinte legyek még én sem fejtettem meg miért.

szepeviktor commented 4 months ago

Van már jó ideje rotate property jó ideje

Félreértés van. Ez a PR nem a rotate-ot gyógyította meg, hanem a másik vonalat tűntette el, hogy ne kelljen a bal kezeddel megvakarni a jobb füledet.

szepeviktor commented 4 months ago

A CSS-t azért tartjuk meg, mert ha kiszedem lokálon nem frissíti a live reload fejlesztésnél. Hogy őszinte legyek még én sem fejtettem meg miért.

Most felvettem a fejlesztői overálomat. 🧑‍💼 Ha beleírok a main.scss-be, akkor mindent újra generál.

[11ty] File changed: src/css/main.css

Akkor most jön, hogy

  1. a css-eket kitörlöm a repóból, gitignore-ba rakom
  2. npm run start előtt legenerálom a CSS-eket
  3. aztán npm run start
szepeviktor commented 4 months ago

@adamlaki Nekem így működik gitignore-os CSS-sel.

  1. Egyszer az elején kell npm run dev:sass
  2. Ctrl+C-vel megszakítani
  3. npm run start és kezdődhet a munka
adamlaki commented 4 months ago

Félreértés van. Ez a PR nem a rotate-ot gyógyította meg, hanem a másik vonalat tűntette el, hogy ne kelljen a bal kezeddel megvakarni a jobb füledet.

A rotate ebben az esetben azért is van ott, mert van animáció, nem csak rejtük az egyik elemet.

adamlaki commented 4 months ago

Na, csak hogy gondolkodtam egy kicsit, azt hiszem van másik gondja is. Azt hiszem ezzel is összeakadt: https://github.com/conedevelopment/cone-site/blob/main/.eleventy.js#L20

szepeviktor commented 4 months ago

Na, csak hogy gondolkodtam egy kicsit, azt hiszem van másik gondja is. Azt hiszem ezzel is összeakadt: https://github.com/conedevelopment/cone-site/blob/main/.eleventy.js#L20

Ezt a #67-hez írod?