Skip to content

Transfer Learning Page Creation #10

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

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

Conversation

giuliocaputi
Copy link
Collaborator

I have created the transfer learning page. This also required changing the header of all other pages, adding images to the "assets" folder, and making slight changes to the "open-roles" section

giuliocaputi and others added 2 commits August 3, 2025 10:48
I have created the transfer learning page. This also required changing the header of all other pages, adding images to the "assets" folder, and making slight changes to the "open-roles" section
@giacomo-ciro
Copy link
Collaborator

@giuliocaputi ho cambiato e messo il colore #6600cc che abbiamo usato per il resto del sito. Per le prossime volte, cerchiamo di non definire gli stili in tag, ma usiamo il css. In un futuro dovremmo anche riorganizzare il css in moduli

<div class="content d-flex flex-row justify-content-center align-items-center gap-3">
<a href="https://forms.gle/dc1yNsVX4GqrVWCZ9"
class="read-more align-self-start" target="_blank">
<span>Apply as an Associate</span><i class="bi bi-arrow-right"></i>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qui aggiungerei link a pagina transfer learning (che dalla home non è subito visibile) invece che straight to form, o anche una Q su questo nelle FAQ sotto

Copy link
Collaborator

@giacomo-ciro giacomo-ciro Aug 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In realta' ha senso cosi perche' nella pagina "Transfer Learning" non c'e' il link al form, andrebbe ripensato il funnel se vogliamo fare come dici. La modifica alle FAQ invece ha senso, magari aggiungiamo una domanda e poi mergiamo. Se riesco faccio io piu' tardi che ora non ho il pc coi permessi giusti, senno' chi e' disponibile vada pure.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nella pagina "Transfer Learning" c'è il link al form. Andrebbe forse reso più visibile, magari con un bottone

@davide-beltrame
Copy link
Member

I primi due dei miei tre commit:

Before

Screenshot 2025-08-06 at 11 33 02

After

Screenshot 2025-08-06 at 11 33 44

Copy link
Member

@davide-beltrame davide-beltrame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vedete commenti precedenti. Per me ok, aspetto vostro feedback alla review @giacomo-ciro @giuliocaputi

Copy link
Collaborator

@giacomo-ciro giacomo-ciro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aggiungiamo la domanda su Transfer Learning in FAQ e poi mergiamo

@giuliocaputi
Copy link
Collaborator Author

I primi due dei miei tre commit:

Before

Screenshot 2025-08-06 at 11 33 02 ### After Screenshot 2025-08-06 at 11 33 44

Così è più bello, controlla però che anche da telefono si veda bene (mentre creavo la pagina ho scartato un paio di idee perché da telefono non rendevano bene). Il design deve essere anche flessibile all'aggiunta di nuove foto (per esempio, mi sono dimenticato una società nella box "Quantitative Finance", la aggiungo appena posso). Comunque io approvo, al momento non ho il computer quindi non posso vedere troppo bene tutto, ma mi sembra ok

@giuliocaputi
Copy link
Collaborator Author

Riguardo al template, non mi sembra siamo più legati a quello originale (almeno per questa pagina)

@giuliocaputi
Copy link
Collaborator Author

Also, ora che mi ci fate pensare, avrebbe senso aggiungere una sezione FAQs nella pagina "Transfer Learning", usando la stessa struttura delle FAQs nella Home del sito. Ci posso lavorare io dal 14, quando riprenderò il computer

@davide-beltrame
Copy link
Member

davide-beltrame commented Aug 7, 2025

Così è più bello, controlla però che anche da telefono si veda bene (mentre creavo la pagina ho scartato un paio di idee perché da telefono non rendevano bene). Il design deve essere anche flessibile all'aggiunta di nuove foto (per esempio, mi sono dimenticato una società nella box "Quantitative Finance", la aggiungo appena posso). Comunque io approvo, al momento non ho il computer quindi non posso vedere troppo bene tutto, ma mi sembra ok

@giuliocaputi Hai ragione, ho controllato, fatto un fix veloce al titolo e sembra che funzioni da telefono e sia robusto all'aggiunta di loghi:

IMG_6910

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