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

Remake #3

Merged
merged 38 commits into from
Jul 10, 2024
Merged

Remake #3

merged 38 commits into from
Jul 10, 2024

Conversation

samuelemusiani
Copy link
Member

Mentre mi esercitavo per Tecweb ho deciso di rifare questo sito che mi sembrava un po' antiquato. L'ho fatto in Vue perché è il framework che ho imparato. È vero che Dynamik è in svelte però secondo me ci sta differenziare un po' i framework così possiamo attirare più gente per contribuire.

Ritengo di essere già arrivato ad un buon punto per una PR. Ci sarebbero altre cose da fare per migliorare il codice ma le farei piano piano nel tempo senza appesantire ancora di più questa PR

Se viene approvata poi lo metterei anche hostato in adm magari sotto diario.students.cs.unibo.it

@samuelemusiani samuelemusiani requested review from foxyseta, ali-benny and a team July 8, 2024 12:15
@foxyseta foxyseta requested a review from kocierik July 8, 2024 13:08
@foxyseta
Copy link
Member

foxyseta commented Jul 8, 2024

Ho messo come recensore il man @kocierik perché ci ha lavorato lui e io non ho mai fatto un tirocinio.

@foxyseta foxyseta removed their request for review July 8, 2024 13:08
@kocierik
Copy link
Member

kocierik commented Jul 8, 2024

Ci guardo in giornata 👍

@samuelemusiani
Copy link
Member Author

samuelemusiani commented Jul 8, 2024

Mi sono scordato di mettere il deploy per pages nuovo, dopo domani lo faccio

src/components/InfoMenu.vue Outdated Show resolved Hide resolved
@kocierik
Copy link
Member

kocierik commented Jul 9, 2024

A livello di funzionalita' direi che va piu' che bene, ho notato che da mobile c'e' uno scroll orizzontale dovuto a un qualche padding penso

@samuelemusiani
Copy link
Member Author

Ho aggiustato l'overflow sul menu e ho risolto un piccolo problema che si aveva con i pulsanti del menu in schermi piccoli. Non ho trovato lo scroll orizzontale che dici, puoi dirmi nello specifico come lo hai ottenuto?

Dopo aggiungo il deploy per pages e direi che ci siamo

@samuelemusiani
Copy link
Member Author

@kocierik dire che ci siamo anche con il deploy

@kocierik
Copy link
Member

Ho aggiustato l'overflow sul menu e ho risolto un piccolo problema che si aveva con i pulsanti del menu in schermi piccoli. Non ho trovato lo scroll orizzontale che dici, puoi dirmi nello specifico come lo hai ottenuto?

Dopo aggiungo il deploy per pages e direi che ci siamo

avevo provato dal mio telefono muovendomi da sinistra verso destra, se non c'e' da te magari avevo zoomato io per errore

@samuelemusiani
Copy link
Member Author

avevo provato dal mio telefono muovendomi da sinistra verso destra, se non c'e' da te magari avevo zoomato io per errore

Mergio e provo anche su altri telefoni, se lo noto risolvo

@samuelemusiani samuelemusiani merged commit 3b0a47b into main Jul 10, 2024
2 checks passed
@samuelemusiani samuelemusiani deleted the remake branch July 10, 2024 09:11
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.

3 participants