gaspermlakar / hair_salon

0 stars 0 forks source link

Feedback #1

Open ramuta opened 4 years ago

ramuta commented 4 years ago

Hey Gašper, super narejeno! :)

Edini komentar, ki ga imam, je glede strukture mape za CSS fajle. Trenutna struktura tvojega projekta je takšna:

Kar je malce problematično pri tej strukturi je to, da imaš svojo custom style.css datoteko v mapi bootstrap/css. Če bi recimo kasneje projekt od tebe prevzel nek drug programer in želel posodobiti Bootstrap verzijo, bi se lahko zgodilo da bi izbrisal celo bootstrap mapo in jo z zamenjal z novo verzijo. S tem bi nevede izbrisal tudi custom style.css datoteko. :)

Boljša struktura projekta bi bila takšna:

Ta struktura ima sicer več nivojev podmap, ampak bolj efektivno razdeli knjižnice in tvojo custom kodo.

Zakaj se mapa imenuje static? Ker se tem datotekam reče tudi "statične datoteke" (static files), kar je dokaj običajno poimenovanje za tovrstne datoteke (CSS, JS, IMG, fonti ipd.)

Znotraj static mape pa naj ima vsaka zunanja knjižnica (kot je npr. Bootstrap) svojo mapo. Svoje custom datoteke pa daš v mapo custom.

Če imaš kakšno vprašanje glede tega, kar napiši ;)

gaspermlakar commented 4 years ago

Hvala. Sem popravil in dal v mape. Kako pa to sedaj to urediti na github-u?

ramuta commented 4 years ago

Ponavadi, ko izbrišeš stare mape, se tudi na GitHubu izbriše.

gaspermlakar commented 4 years ago

Hm ja ponavadi... :) Kako naj pa to sedaj zbrišem ko ni?

ramuta commented 4 years ago

Kater software uporabljaš za Git? Mogoče SourceTree? SourceTree ti ponavadi sam zazna, da si izbrisal neko datoteko, in to spremembo naredi tudi v Gitu.

gaspermlakar commented 4 years ago

PyCharm, tako kot je opisano v lesson 6. Ubistvu nisem izbrisal sem samo dodal pa prestavil.

ramuta commented 4 years ago

Ti predlagam da kar SourceTree uporabljaš: https://www.sourcetreeapp.com/ ;)