Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify accordion-card #65

Closed
wants to merge 1 commit into from

Conversation

szepeviktor
Copy link
Contributor

@szepeviktor szepeviktor commented May 15, 2024

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á!

Copy link

netlify bot commented May 15, 2024

Deploy Preview for cone-development-site ready!

Name Link
🔨 Latest commit f2bd710
🔍 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
Copy link
Contributor Author

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
Copy link
Member

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.

@adamlaki adamlaki closed this May 16, 2024
@szepeviktor
Copy link
Contributor Author

szepeviktor commented May 16, 2024

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
Copy link
Contributor Author

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
Copy link
Contributor Author

@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
Copy link
Member

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
Copy link
Member

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
Copy link
Contributor Author

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?

@szepeviktor szepeviktor deleted the patch-1 branch June 12, 2024 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants