Skip to content

Commit

Permalink
Add webdev feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
diogotcorreia committed Apr 30, 2023
1 parent d54627c commit 00c5d4b
Show file tree
Hide file tree
Showing 8 changed files with 135 additions and 0 deletions.
36 changes: 36 additions & 0 deletions webdev/BeatrizVarela/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
Falta

```html
<meta charset="UTF-8" />
```

na página para dar render de chars com acentos.

---

(é assim no site original, portanto wtv)
Baixo contraste do texto em branco no header com o background,
quase não dá para ler.

---

Site está com horizontal scroll quando não deve.
É só adicionar [`box-sizing: border-box`](https://developer.mozilla.org/en-US/docs/Web/CSS/box-sizing)
ao `footer`.

---

```html
<h><br>
<br></h>
```

Isto devia ser margin/padding.

---

Tentar centrar iframe em vez de dar só `left: 300px`.

---

De resto está fixe! Good job
10 changes: 10 additions & 0 deletions webdev/InesMartins/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
Conteúdo fica escondido debaixo no footer quando se dá scroll até ao fim.

Uma solução é adicionar margin ao conteúdo, e.g., adicionar `margin-bottom: 100px` ao `main`.

---

Uso de semantic HTML, cool
Bastante clean no geral

De resto está nice
7 changes: 7 additions & 0 deletions webdev/ManuelMartins/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
For some reason o nome do ficheiro css não estava bem (style.css vs index.css)

---

Uso de flexbox e semantic HTML, nice

De resto tudo okay, talvez demasiado simples mas tudo o que está está excelente.
11 changes: 11 additions & 0 deletions webdev/MargaridaLoureiro/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
For some reason os buttons não ficam com o cursor bem quando se dá hover,
mas é só adicionar `cursor: pointer` ao CSS.

---

Em vez de uma table para os vídeos talvez usar CSS grid tivesse sido melhor ideia,
tables era o que se usava antigamente.

---

De resto fixe!
14 changes: 14 additions & 0 deletions webdev/MariaRibeiro/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
Falta a tag

```html
<meta charset="utf-8">
```

para os acentos ficarem bem.

---

Tooltips ficaram muito bem (e CSS only!)

Está excelente, congrats!

17 changes: 17 additions & 0 deletions webdev/RitaMota/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
Falta a tag

```html
<meta charset="utf-8">
```

para os chars não ASCII ficarem bem.

---

Várias imagens que estavam com o path total do sistema (i.e. `C:\Users\Rita\...`),
que não funcionam noutro computador.

---

De resto está bom

25 changes: 25 additions & 0 deletions webdev/afonsonóbrega/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
Gostei da tentativa de usar NodeJS, mas não incluiste nenhum
package.json para as dependências.
Além disso, é tudo estático, podia ser perfeitamente só HTML e CSS :)

---

Na homepage penso que faltou tirar a margin do body.

---

Está um random quadrado na navbar da login page (acho que devia dizer "log in"?).

---

Não que seja muito relevante para aqui, mas cuidado a fazer forms de login que
usam `method="GET"` (default). Forms com informação sensível (e.g. passwords)
devem ser sempre por POST (`method="POST"`).

---

Estás a usar `float` na navbar em vez de `flexbox` (alternativa moderna).

---

De resto está fixe
15 changes: 15 additions & 0 deletions webdev/sofia bouça/FEEDBACK.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
Fui rick rolled, you win :(

---

CSS está lacking, falta dar mais estilos aos buttons, à text box, etc

---

Searchbar tem fixed width, podia ser boa ideia fazer com que se adapte a ecrãs mais pequenos

---

Tem flexbox :+1:

De resto está okay, mas um bocadinho fraquinho :(

0 comments on commit 00c5d4b

Please sign in to comment.