Daan645 / lose-your-head-the-client-case

Ontwerp en maak een website voor een opdrachtgever op basis van een Headless CMS API
https://lose-your-head-the-client-case-5q6r.vercel.app/
0 stars 1 forks source link

Het maken van code conventies #7

Open driezie opened 1 month ago

driezie commented 1 month ago

Code conventies

html

css

Maak de css code DRY. Dus eerst de basiselementen stijlen, waarna je de classes stijlt Werk mobile-first !!!

Wanneer je meerdere objecten aanroept, er een enter tussen elk object komt

p,
ul,
li,
button {
    font-family: var(--font-sans-serif);
    font-size: 1.1em;
    line-height: 1.5em;
}

Wanneer je gebruik maakt van een @media-query gebruikt, de waarde binnen het object zet en niet onderaan het bestand. Zo moet je het dus niet doen.

 #navbar-toggle input:checked ~ .navbar-menu { 
     right: 0; 
     transform: none; 
 } 

 @media (min-width: 768px) { 

     header { 
         border-bottom: none; 
     } 

maar zo:

header {
    background-color: var(--primary-color-3);
    border-bottom: var(--border-style-black);
    width: 100%;

  @media (min-width: 768px) {
        border-bottom: none;
}

Naamgeving

.stickman-head

JavaScript

Commits & Pull requests

Commits

Branch

Naamgeving: Wanneer je aan een specifieke feature werkt, geef je de volgende naamgeving aan een branch. Feature + onderwerp + eigen naam. Dus bijvoorbeeld feature-header-driezie

Wanneer je een nieuwe branch wilt aanmaken, doe dit via GitHub. Dit is handiger, omdat je het anders eerst moet pushen. Dus doe dit direct via de website i.p.v GitHub Desktop

Pull request

Wanneer je een pull request maakt, benoem de volgende dingen:

Wanneer is een pull-request klaar?

OniWithTheHoodie commented 1 month ago

BEM (Block Element Modifier)


<article class="article">
  <h2 class="article__header">header</h2>
  <p class="article__text">Lorem ipsum dolor sit amet.</p>
</article>

Stel dat je meerdere van de zelfde block elementen hebt met andere stijling van elkaar. Doe dan dit voor de block elementen article--regular en article--featured zorgt dat je andere stijling kunt geven voor het hele blok.

articleheader--regular en articleheader--featured zorgt ervoor dat je specifieke elementen kunt stijlen binnen elke blok element .


<!-- Regular Article -->
<article class="article article--regular">
  <h2 class="article__header article__header--regular">Regular Article Header</h2>
  <p class="article__text article__text--regular">This is a regular article.</p>
</article>

<!-- Featured Article -->
<article class="article article--featured">
  <h2 class="article__header article__header--featured">Featured Article Header</h2>
  <p class="article__text article__text--featured">This is a featured article.</p>
</article>

CSS


/* Base block styles */
.article {
  /* Default article styles */
  padding: 20px;
  border: 1px solid #ddd;
}

/* Modifiers for different article types */
.article--regular {
  background-color: #f5f5f5;
}

.article--featured {
  background-color: #ffebcc;
  border-color: #ffa500;
}

/* Element styles */
.article__header {
  font-size: 24px;
  margin-bottom: 10px;
}

.article__text {
  font-size: 16px;
  color: #333;
}

/* Element modifiers for different article types */
.article__header--regular {
  color: #333;
}

.article__header--featured {
  color: #ff4500;
}

.article__text--regular {
  color: #666;
}

.article__text--featured {
  color: #000;
}
OniWithTheHoodie commented 1 month ago

HTML

Image

Indention

Image

Capitilaization

Image

Multimedia Fallback

Image

CSS

Class Naming

Image

Declaration Stops

Image

Property Name Stops

Image

Declaration Block Separation

Image

Selector and Declaration Separation

Image


:root,
html,
body {
    margin: 0;
    padding: 0;
    background-color: var(--color-main-bg);
    line-height: 1.6;
    font-size: 1.2rem;
    font-family: 'Open Sans', sans-serif;
}

Rule seperation

Image


<style>
  main {
    display: flex;
    justify-content: center; 
    align-items: center;     
    height: 100vh;     
  }

  .card {
    display: flex;
    flex-direction: column;
    align-items: center;
    border: 1px solid var(--color-main-text);
    width: 420px;
    height: 805px;
    border-radius: 1em;
  }

  article {
    line-height: 1.2;
    width: 400px;
  }
Daan645 commented 1 month ago

DOD

Wanneer is een project klaar om te mergen en hoe doe je dit? Degene die een code review geeft moet op het volgende ook beoordelen:

Code

Ontwerp

Responsive

Toegankelijkheid

Performance

Mergen

Merge requests aanmaken

Zet in de merge request de volgende dingen:

  1. Wat heb je gemaakt en is er veranderd?
  2. Hoe werkt het?
  3. Waar moet op gecontroleerd worden?
  4. Een screenshot en een uitleg hoe het werkt

Reviewen

Als je een merge request reviewt, gebruik dan de bovenstaande checklist en check deze af in je comment. Als er iets ontbreekt, meld dit en maak er issues van zodat dit opgepakt kan worden.