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

feat: add about us page #18

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
58 changes: 58 additions & 0 deletions app/about_us/page.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,58 @@
import React from "react";
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
import React from "react";

This is not needed in Next.js


export default function about_us() {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
export default function about_us() {
export default function AboutUs() {

The naming convention is to use Pascal Case.

return (
<main>
<div className="mb-20 flex select-none flex-col">
<h1 className="text-center font-orbitron text-4xl font-semibold">
Centro de Estudantes de Engenharia Informática da Universidade do
Minho
</h1>
</div>
<div className="flex select-none flex-col gap-4">
<p className="items-center text-justify font-inter text-base font-normal text-gray-500">
&quot;Decorria no ano de 1995, a Licenciatura em Engenharia de
Sistemas e Informática (LESI) da UM. (...) Foi neste contexto que um
grupo de alunos do 3º ano pediu à Direção de Curso (DC), que eu então
dirigia, apoio para criar oficialmente um Centro de Estudantes. (...)
A partir desse momento iniciou-se o caminho deste núcleo estudantil
que, 20 anos depois, mantém o nome e sigla e mantém o propósito
determinado na projeção da imagem do curso e na defesa dos interesses
dos seus associados&quot;
</p>
<p className="text-right text-justify font-inter text-base font-normal">
Pedro Rangel Henriques <br /> Diretor do Departamento de Informática -
2017
</p>
</div>
<div className="mt-20 flex select-none flex-col">
<h1 className="text-center font-orbitron text-3xl font-semibold underline decoration-cesium-900">
About Us
Copy link
Member

Choose a reason for hiding this comment

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

I don't think it makes sense to be mixing Portuguese and English.

</h1>
</div>
<div className="mt-12 flex select-none flex-row text-justify font-inter text-base">
<div className="mr-12 font-normal">
<h2 className="mb-4 text-xl">O que é o CeSIUM?</h2>
<p className="text-gray-500">
O CeSIUM é um Centro de estudantes de Engenharia Informática da
Universidade do Minho composto por alunos voluntários que têm como
objectivo principal representar e promover o curso. É um núcleo
unido e dinâmico capaz de proporcionar experiências únicas e
enriquecedoras para a tua futura vida profissional.
</p>
</div>
<div className="ml-12 font-normal">
<h2 className="mb-4 text-xl">O que fazemos?</h2>
<p className="text-gray-500">
Procuramos organizar actividades de contéudo semelhante e
complementar ao curso, assim como promover e divulgar acções de
lazer e socialização entre estudantes e ex-alunos. Representamos
ainda os interesses colectivos dos alunos de Engenharia Informática
junto dos orgãos ou instituições para que a sua carreira académica
seja mais rica.
</p>
</div>
</div>
</main>
);
}
2 changes: 1 addition & 1 deletion components/Navbar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export default function Navbar() {
<Link
key={title}
className={`transition-colors hover:border-cesium-900 hover:text-cesium-900`}
href={`/${title.toLowerCase()}`}
href={`/${title.toLowerCase().replace(/\s+/g, "_")}`}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
href={`/${title.toLowerCase().replace(/\s+/g, "_")}`}
href={`/${title.replace(/\s+/g, "")}`}

This needs to be updated to match the naming convention above.

>
{title}
</Link>
Expand Down
2 changes: 1 addition & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
"autoprefixer": "10.4.14",
"eslint": "8.42.0",
"eslint-config-next": "13.4.4",
"next": "13.4.4",
"next": "^13.4.4",
Copy link
Member

Choose a reason for hiding this comment

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

Is there any reason for this change? I don't think it is necessary

"npm-run-all": "^4.1.5",
"postcss": "8.4.24",
"prettier": "^3.0.0",
Expand Down