Talvez eu seja um júnior (não sei por que nunca fui contratado como um, mas enfim), e gostaria de algumas dicas para melhorar (code review) o meu portfólio. Se alguém puder me ajudar, ficarei grato.
Ah, o projeto é open source, então se quiser usá-lo, está tudo bem.
Repositório: https://github.com/Jailsonr12/Portfolio2.0
Link do portfólio: https://meet-jailson-jailsonr12s-projects.vercel.app/
Achei interessante seu portfólio, mas ele contém alguns erros de design:
1 - Os ícones na versão desktop ao passar o mouse tem a barra de scroll horizontal aparecendo, provavelmente você aumentou o tamanho do svg para um valor superior a div/button que ele está.
2 - As imagens com prints do projeto são grandes demais, deveria ao máximo manter em kb o tamanho delas, nunca em mb no caso que está utilizando.
3 - Uso de sombras excessivo, apenas nos botões, não tem necessidade de adicionar texto, apenas confunde o usuário.
4 - Suas sombras me parecem estar projetadas na direção errada, a maior parte deve ficar na parte inferior ou em quantias iguais. Mas essa parte pode ser só impressão minha.
5 - Remova seu portfólio dos projetos que fez, nós já estamos nele, não precisa mencionar ele novamente.
6 - Mude a ordem dos projetos, coloque aqueles que trazem mais valores para os usuários dos mesmos em primeiro lugar, também buscaria ter como o usuário visualizar e interagir com as páginas reais, a maior parte das pessoas não têm paciência para ver um vídeo, principalmente quando ultrapassa o tempo de um shorts/ tik tok.
Obrigado pela ajuda com o design, não é o meu ponto forte. Em relação às sombras, vou tentar reduzi-las e melhorar os tamanhos das imagens. Quanto ao scroll horizontal, foi um problema que tentei resolver antes, mas pelo visto deixei passar. O portfólio que coloquei é a primeira versão, bem básica. Sobre a direção das sombras, é uma novidade para mim; posso ver como fica com as mudanças sugeridas. Ainda tenho mais projetos para adicionar, não tinha pensado na importância de cada projeto; vou melhorar a organização. Em relação ao vídeo, vou pensar em outra solução. Obrigado pelas dicas :)
Não sei se é seu objetivo, mas acredito que seria interessante realizar algumas melhorias no seu código. Observei algumas questões:
Como a página Home é uma parte importante, seria interessante torná-la um módulo separado. Assim, você poderá declarar e importar apenas o necessário para ela.
Não sei se você planeja criar mais páginas no projeto, mas seria vantajoso separar os componentes genéricos dos específicos. Notei que todos os componentes estão em uma única pasta chamada "components". Uma abordagem mais organizada seria deixar nessa pasta apenas os componentes globais e criar uma pasta "components" em cada módulo para os componentes específicos daquele módulo.
Notei que você definiu estilos para o body dentro do estilo do componente Home. Nesse caso, seria mais interessante definir esses estilos de forma global, a menos que seja um ajuste muito específico e necessário, o que não parece ser o caso aqui.
Já que você está utilizando SCSS, é possível aproveitar as funcionalidades disponíveis. Você pode conferir alguns recursos básicos na documentação em https://sass-lang.com/documentation/style-rules/parent-selector.
Não foi seguido um padrão de idioma no projeto. Em alguns lugares, as variáveis são declaradas em português, em outros, em inglês. O ideal seria seguir um padrão, preferencialmente em inglês. Isso inclui nomes de arquivos, componentes, variáveis, funções, etc., e não o idioma que o seu projeto apresenta (no caso, seu portfólio).
Embora não seja estritamente necessário, seria interessante tipar suas variáveis, métodos e atributos. Mesmo que o TypeScript possa inferir os tipos, é uma boa prática realizar essa tipagem explicitamente.
É importante definir modificadores de acesso para seus atributos e métodos. Se não forem definidos, eles ficam públicos por padrão.
Essas são sugestões de melhorias que podem acrescentar qualidade ao seu projeto. Se eu estivesse avaliando-o, esses pontos certamente seriam considerados.
Em resumo, o código está bem escrito e organizado. Parabéns pelo trabalho!
Obrigado pelo elogio :). Nossa, tenho bastante coisa para fazer, ótimos pontos. Irei executar todos quando tiver tempo. Sinceramente, eu não sabia disso sobre o SCSS, pensava que era a mesma coisa que CSS. Vou tentar seguir um padrão melhor e melhorar a organização. Muito obrigado pelas dicas!
É muito bom receber feedback, assim posso saber o que melhorar e tudo mais.
seu portifólio aparenta estar quebrado no mobile. Achei 2 letras por linha bem agressivo para um celular
pois é, eu tbm não gostei muito, e foi minha primeira vez fazendo algo para mobile, entao nao sei como posso testar todas as responsividade
Para testar a responsividade durante o desenvolvimento pode usar a extensão: https://www.webmobilefirst.com/en/ , a versão free já suprime todas suas necessidades.
Darei uma olhada para ver se consigo me orientar melhor. Obrigado.
Não entendo muito de Angular mas me parece código que um jr faria, e acho que a culpa disso é por vc seguir a linha de fullstack maninho, vc vai ter que ralar o dobro pra subir de nível e alcançar uma excelencia de código maior.
Fico feliz por ser considerado um júnior, obrigado. No entanto, em relação a ser fullstack, essa foi a única experiência que tive no mercado, enquanto estagiário de desenvolvimento web Java fullstack e sinceramente eu gostei, mas sim vou ter q ralar mais.