mokhtarakle / connect-your-tribe-profile-card

Redesign je visitekaartje op basis van een bestaande API.

Mokhtar

Hi there 👋

My name is Mokhtar. I am an 23 year old frontend developer living in Amsterdam.

Stats

Does not include forked repositories

Skills

💻 Frontend Developer

connect-your-tribe-profile-card's Issues

Design

Je kan misschien de kaarten op een ander manier kleuren en designen

Er ontbreekt een wiki

Er een ontbreekt een wiki, het kan handig zijn die wel te gebruiken om je proces te beschrijven.

JS tip

Je kan let gebruiken in plaats van var, sinds 2015 is het obsoleet geworden

Lege div

Op het einde heb je een lege div staan, je kan even checken of die gebruikt wordt.

https://github.com/MokhtarAkle/connect-your-tribe-profile-card/blob/main/views/index.ejs#:~:text=%3Cdiv%20id%3D%22openButton,%3C/div%3E

svg

Je kan je svg ook inklappen in je HTML

Efficienter werken in JS

https://github.com/MokhtarAkle/connect-your-tribe-profile-card/blob/main/public/script.js#:~:text=var%20showButton%20%3D,frontBackside%22)%3B

Ik zie hier dat je vaak elementen selecteerd op deze manier: "document.querySelector". Dit kan efficienter worden gedaan door het niet steeds opnieuw te typen door de volgende functies in je script te plaatsen:
const $ = element => document.querySelector(element)
const $all = element => document.querySelectorAll(element)
Nieuwe code wordt dan:
var showButton = $("#openButton");
var frontAnimation =$("#animationFront");
var middleAnimation = $("#animationMiddle");
var rightAnimation = $("#animationRight");
var flipButton = $("#showCards");
var frontBackanim = $("#frontBackside");

En bij querySelectorAll : $all(".voorbeeld-class')

Json tip

Hi, Mokthar vandaag hadden wij een code en design review. Al met al tof visite kaartje met leuke animaties. Echter heb ik wel nog als kleine tip voor het overzichtelijk maken van Json bestand. Daarvoor heb ik in de onderstaande link een hele mooie plugin er voor:

https://chrome.google.com/webstore/detail/json-formatter/bcjindcccaagfpapjjmafapmmgkkhgoa