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

Fabio #12

Open
wants to merge 62 commits into
base: main
Choose a base branch
from
Open

Fabio #12

wants to merge 62 commits into from

Conversation

fabiobillo
Copy link
Collaborator

file da esaminare landing_page.html e style_land.css

@fabiobillo
Copy link
Collaborator Author

da visionare anche la cartella progetto2

@dianaberna
Copy link
Owner

per il primo esercizio potresti provare a posizionare le tre foto di sinistra utilizzando il display: grid per esercitarti

@dianaberna
Copy link
Owner

secondo esercizio perfetto, bravo!

@fabiobillo
Copy link
Collaborator Author

per il primo esercizio potresti provare a posizionare le tre foto di sinistra utilizzando il display: grid per esercitarti

Si, dopo i primi esercizi sono entrato nell'ottica del flex e nel fare un css un po' più pulito

@fabiobillo
Copy link
Collaborator Author

da visionare progetto_mobile, ho avuto qualche problema con il player sotto ma penso di aver risolto, non so se ha senso inserire uno span vuoto per far inserire uno spazio dove l'elemento con l'absolute (il player) viene tolto dal flusso del documento.

@fabiobillo
Copy link
Collaborator Author

avendo appena visto i tuoi messaggi ho fatto un aggiornamento con grid di landing_page.html e style_land.css, anche se mi trovo meglio con flex perché per me è molto più logico

@dianaberna
Copy link
Owner

feedback esercizi html/css

landing_page 👍
slide2 👍

per i nomi delle classi cerca di utilizzare sempre la stessa lingua (sarebbe ottimo sempre l'inglese)

Non ho trovato html/css di qualche schermata mobile 🧐

@fabiobillo
Copy link
Collaborator Author

Si! di mobile c'è la cartella progetto_mobile! :)

@dianaberna
Copy link
Owner

top, ho controllato al volo ed è tutto ok, unico consiglio che ti lascio è provare a lasciare tutte le definizioni degli stili dei tag sopra così è più immediato leggerli

@fabiobillo
Copy link
Collaborator Author

Aggiunta v1.js e v2.js in card_js-mia_card

@fabiobillo
Copy link
Collaborator Author

pushata soluzione n.1 es eventi, fatta di corsa ma migliorabile, domani ci lavoro

@fabiobillo
Copy link
Collaborator Author

es calcolatrice eventi/script4.js - eventi/index4.html

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.

2 participants