-
Notifications
You must be signed in to change notification settings - Fork 0
Code Review #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
jurecki
wants to merge
17
commits into
code-review
Choose a base branch
from
main
base: code-review
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Code Review #2
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
614aee1
Add task-runner
jurecki 0583beb
Add scss base structure
jurecki 8f58318
Add header, Add homesection
jurecki d60016a
Add about section
jurecki e5f0207
About section in progress
jurecki 358984c
Add portfolio section
jurecki f35836a
Add contact section
jurecki dc4c17a
Add footer section
jurecki 33510f7
RWD in progress
jurecki 4dd7ca5
Add RWD
jurecki 35ec1fc
Optimize image in progress
jurecki 60b4240
Add webP, jp2 formats to photos
jurecki 70c8ecf
Add google fonts
jurecki 5ad64f8
Fixed gallery on small screen
jurecki 67dc3d3
Update README.md
jurecki aa9b147
Update README.md
jurecki 45c485c
Update README.md
jurecki File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,116 @@ | ||
| # Logs | ||
| logs | ||
| *.log | ||
| npm-debug.log* | ||
| yarn-debug.log* | ||
| yarn-error.log* | ||
| lerna-debug.log* | ||
|
|
||
| # Diagnostic reports (https://nodejs.org/api/report.html) | ||
| report.[0-9]*.[0-9]*.[0-9]*.[0-9]*.json | ||
|
|
||
| # Runtime data | ||
| pids | ||
| *.pid | ||
| *.seed | ||
| *.pid.lock | ||
|
|
||
| # Directory for instrumented libs generated by jscoverage/JSCover | ||
| lib-cov | ||
|
|
||
| # Coverage directory used by tools like istanbul | ||
| coverage | ||
| *.lcov | ||
|
|
||
| # nyc test coverage | ||
| .nyc_output | ||
|
|
||
| # Grunt intermediate storage (https://gruntjs.com/creating-plugins#storing-task-files) | ||
| .grunt | ||
|
|
||
| # Bower dependency directory (https://bower.io/) | ||
| bower_components | ||
|
|
||
| # node-waf configuration | ||
| .lock-wscript | ||
|
|
||
| # Compiled binary addons (https://nodejs.org/api/addons.html) | ||
| build/Release | ||
|
|
||
| # Dependency directories | ||
| node_modules/ | ||
| jspm_packages/ | ||
|
|
||
| # Snowpack dependency directory (https://snowpack.dev/) | ||
| web_modules/ | ||
|
|
||
| # TypeScript cache | ||
| *.tsbuildinfo | ||
|
|
||
| # Optional npm cache directory | ||
| .npm | ||
|
|
||
| # Optional eslint cache | ||
| .eslintcache | ||
|
|
||
| # Microbundle cache | ||
| .rpt2_cache/ | ||
| .rts2_cache_cjs/ | ||
| .rts2_cache_es/ | ||
| .rts2_cache_umd/ | ||
|
|
||
| # Optional REPL history | ||
| .node_repl_history | ||
|
|
||
| # Output of 'npm pack' | ||
| *.tgz | ||
|
|
||
| # Yarn Integrity file | ||
| .yarn-integrity | ||
|
|
||
| # dotenv environment variables file | ||
| .env | ||
| .env.test | ||
|
|
||
| # parcel-bundler cache (https://parceljs.org/) | ||
| .cache | ||
| .parcel-cache | ||
|
|
||
| # Next.js build output | ||
| .next | ||
| out | ||
|
|
||
| # Nuxt.js build / generate output | ||
| .nuxt | ||
| dist | ||
|
|
||
| # Gatsby files | ||
| .cache/ | ||
| # Comment in the public line in if your project uses Gatsby and not Next.js | ||
| # https://nextjs.org/blog/next-9-1#public-directory-support | ||
| # public | ||
|
|
||
| # vuepress build output | ||
| .vuepress/dist | ||
|
|
||
| # Serverless directories | ||
| .serverless/ | ||
|
|
||
| # FuseBox cache | ||
| .fusebox/ | ||
|
|
||
| # DynamoDB Local files | ||
| .dynamodb/ | ||
|
|
||
| # TernJS port file | ||
| .tern-port | ||
|
|
||
| # Stores VSCode versions used for testing VSCode extensions | ||
| .vscode-test | ||
|
|
||
| # yarn v2 | ||
| .yarn/cache | ||
| .yarn/unplugged | ||
| .yarn/build-state.yml | ||
| .yarn/install-state.gz | ||
| .pnp.* |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,111 +1,29 @@ | ||
| # CodersCamp 2020 - Projekt HTML & CSS | ||
| **CodersCamp (coderscamp.edu.pl) - Największy otwarty kurs programowania webowego** | ||
|
|
||
| Gratulacje! | ||
| Skoro czytacie ten tekst, oznacza to, że dowiedliście swoich umiejętności ze znajomości HTML & CSS. | ||
| Zostaliście przydzieleni do zespołu, który będzie się uczył programowania i wykonywał prawdziwe aplikacje webowe. | ||
| Teraz stajecie przed pierwszym większym wyzwaniem. | ||
| Technologie, o których się uczyliście, trzeba zastosować w praktyce. | ||
| To pierwszy i ostatni na CodersCamp projekt, który będziecie robić w pojedynkę. | ||
| Jednakże już macie cały zespół i mentora, który z pewnością będzie Was wspierał. | ||
|
|
||
| ## Projekt HTML & CSS — Wizytówka / Portfolio | ||
|
|
||
| ### Zasady wykonywania projektu (wspólne dla wszystkich grup i mentorów): | ||
|
|
||
| ##### W projekcie każdy z uczestników powinien zaprezentować praktyczną znajomość poniższych zagadnień związanych z HTML & CSS: | ||
| - Box-model | ||
| - Kaskadowość CSS | ||
| - Selektory CSS | ||
| - Popularne tagi HTML | ||
| - Jak podpinać CSSa do HTMLa | ||
| - Zapisywanie kolorów | ||
| - Stylowanie tekstu | ||
| - Zewnętrzne ikony/fonty (fontawesome, google fonts) | ||
| - Flexbox i/lub CSS Grid | ||
| - Position (absolute, relative) | ||
| - Animacje keyframes | ||
| - Formularz (wysłanie formularza może powodować przeładowanie strony, gdyż w kursie nie było jeszcze jego obsługi) | ||
| - Responsive Web Design | ||
|
|
||
| Nie używaj JavaScriptu ani żadnego innego frameworka. | ||
| Nie używaj też gotowych bibliotek styli takich jak Bootstrap. | ||
| W trakcie przerabiania działu poznałeś wszystkie umiejętności, które pozwolą Ci wykonać Twoją wizytówkę od podstaw :) | ||
|
|
||
| ##### Sposób oceny projektu (dotyczy tylko projektu pierwszego): | ||
| Zapewne interesuje Was, w jaki sposób projekt zostanie „zaliczony” i oceniony. | ||
| Ocenianie będzie miało kilka etapów. | ||
|
|
||
| - Kiedy skończycie pracę nad projektem, odnotujcie ten fakt w specjalnie przygotowanym formularzu — link otrzymacie od swojego mentora. | ||
| Powinno to nastąpić najpóźniej o godzinie 23:59 dnia poprzedzającego prezentację projektu (data dostępna w harmonogramie kursu). | ||
| - Spotkajcie się w wyznaczonym dniu całym zespołem wraz z mentorem (np. za pomocą Google Meet). W trakcie spotkania zaprezentujcie wykonane przez Was projekty. | ||
| Forma prezentacji pozostaje dowolna (musi zmieścić się w 5 minutach + 5 minut na pytania). | ||
| Możecie przygotować jakieś slajdy, pokaz waszych talentów, czy nawet zaśpiewać — ale nie jest to konieczne. Na pewno musi zostać pokazana działająca strona internetowa, reszta wg Waszego uznania. | ||
| Jeśli robicie wizytówkę o sobie, to być może chcecie też opowiedzieć o swoich zainteresowaniach, żeby inni w zespole poznali Was lepiej :) ? | ||
| - Po prezentacji mentor Waszego zespołu oceni projekt wg kryteriów opisanych w specjalnym arkuszu — przykład takiego arkusza możecie zobaczyć [TUTAJ](https://docs.google.com/spreadsheets/d/1mjCi-oDXILKoCReqJlhGYP4NW-HVMCzvdcIy6ntnsog/edit?usp=sharing). | ||
| Pamiętajcie też odpowiednio opracować README.md Waszego projektu, tak aby prezentowało kto, co, jak i dlaczego zostało wykonane. | ||
| Zawartość, którą macie tutaj aktualnie (to co teraz czytacie) można usunąć — znajdziecie ją zawsze w oryginalnym repozytorium, które forkowaliście. | ||
| - Po zrecenzowaniu waszych projektów mentor powinien przekazać każdemu jego ocenę punktową i feedback jako uzasadnienie. | ||
|
|
||
|
|
||
| ### Opis projektu | ||
| Pozyskanie pierwszego klienta nie przychodzi łatwo. | ||
| Dla klienta, który chce otrzymać produkt wysokiej jakości, nie jest ważna tylko firma, która spróbuje zrealizować jego wizję. | ||
| Równie istotne jest też to, z jakimi konkretnymi osobami przyjdzie komuś współpracować przy wytwarzaniu oprogramowania. | ||
| Dobrze, żeby Wasi przyszli klienci i wasz zespół trochę lepiej Was poznali. | ||
| Dlatego jako pierwszy projekt do Waszego portfolio proponujemy przygotowanie strony — wizytówki. | ||
| Motywem przewodnim powinniście być właśnie Wy sami lub o wasze zrealizowane projekty lub zainteresowania, najważniejsze umiejętności — nie koniecznie programistycznych! | ||
| Jesteś fotografem? Możesz przygotować stronę reklamującą swoje usługi. | ||
| Jako fryzjer czy mechanik, możesz zrobić podobnie. | ||
| Świat web designu stoi przed Tobą otworem. | ||
|
|
||
| W tym pierwszym, rozgrzewkowym projekcie pracujecie jeszcze osobno. | ||
| Pamiętajcie, że macie zespół i mentora, który jest tutaj, żeby Was wspierać przez cały czas trwania kursu. | ||
|
|
||
| Dajcie się lepiej poznać dzięki internetowi i swoim umiejętnością! :) | ||
|
|
||
| #### Jak rozpocząć pracę nad projektem | ||
| Aby ułatwić prezentację projektu innym osobom, dobrze, gdyby był on dostępny bez problemów pod adresem strony internetowej. | ||
| W tym przypadku większość pracy z tym związanej zrobiliśmy już za Ciebie. | ||
| Oto co musisz zrobić, żeby zobaczyć efekt i zacząć development: | ||
|
|
||
| 1. Wykonaj fork tego repozytorium do swojego własnego profilu. | ||
| Po kliknięciu przycisku pokazanego poniżej GitHub przeniesie Cię automatycznie na Twoją kopię tego repozytorium. | ||
| To tutaj będziesz wykonywać swoją stronę. | ||
|  | ||
| 1. Przejdź do zakładki Settings w Twoim repozytorium. | ||
|  | ||
| 1. Znajdź w zakładce setting sekcję GitHub Pages i wybierz branch *main* jako source (na screenie dawna nazwa głównej gałęzi - master). | ||
|  | ||
| 1. Po wyborze brancha *main*, zatwierdź ustawienia, klikając przycisk `Save`. | ||
| Wszystko powinno wyglądać tak jak w czerwonym polu poniżej. | ||
| Dzięki temu aktywujesz GitHub Pages dla swojego repozytorium. | ||
|  | ||
| 1. Znajdź sekcję GitHub Pages w zakładce Setting jeszcze raz. | ||
| Teraz powinien znajdować się w tym miejscu link. Po kliknięciu zaprowadzi Cię do Twojej strony. | ||
| Ten link możesz przekazać teraz kolegom z zespołu i mentorowi. | ||
| Będą mogli dać Ci cenne wskazówki i pomóc z błędami bez pobierania całego repozytorium. | ||
| A Twoja strona jest od razu dostępna w internecie dla każdego, kto chciałby ją odwiedzić. I to kompletnie za darmo! | ||
|  | ||
| 1. Jeśli po kliknięciu w link, pokazała Ci się strona o tej treści, to znaczy, że wszystko jest w porządku. | ||
|  | ||
|
|
||
| Od teraz każda zmiana, jaką commitujesz do głównego brancha, będzie się pokazywać pod tym adresem. | ||
| Masz już działające środowisko, więc na co czekasz? Do dzieła! | ||
|
|
||
| ### Porady odnośnie do projektu | ||
|
|
||
| #### Interfejs | ||
| Możesz posłużyć się projektem interfejsu strony — wizytówki, wykonanym przez organizatorów projektu. | ||
| Jest to tylko dla ułatwienia Ci pracy. | ||
| Możesz w trakcie implementacji go dowolnie przekształcać / zmieniać czy zrobić coś zupełnie innego. | ||
| https://www.figma.com/file/WHtiMfcNHt4tc7mDamNBYa/CodersCamp2020-Wizytowka?node-id=0%3A1 | ||
| Pamiętaj też, że Twoje rozwiązanie musi być responsywne! | ||
| Czyli odpowiednio dostosowywać sposób wyświetlania treści do rozmiaru okna przeglądarki. | ||
|
|
||
| Inspiracją może być także [DevChallenge.io - Portfolio](https://devchallenges.io/challenges/5ZnOYsSXM24JWnCsNFlt). | ||
| Pamiętaj tylko, że Ty korzystasz tutaj jedynie z HTML i CSS. | ||
|
|
||
| Przygotowane projekty interfejsu użytkownika są tutaj po to, żebyś bez problemów postawił swoje pierwsze kroki w praktycznym wykorzystaniu HTML i CSS. | ||
| Jednakże zachęcamy do wykonania czegoś, co ukaże Twoje wnętrze :) | ||
| Dobrej zabawy! | ||
| ## Table of contents | ||
| * [General info](#general-info) | ||
| * [Technologies](#technologies) | ||
| * [Setup](#setup) | ||
| * [Sources](#sources) | ||
|
|
||
| ## General info | ||
| This is a training project. It was written after the first module of the CodersCamp course. I had to demonstrate a practise knowledge of HTML and CSS. I couldun't use JavaScript or ready-made style libraries like Bootstrap. | ||
|
|
||
| ## Technologies | ||
| Project is created with: | ||
| * HTML 5 | ||
| * CSS / Sass | ||
| * NPM | ||
| * BEM Methodology | ||
| * Next-Gen Image Formats jp2, webp | ||
|
|
||
| ## Setup | ||
| You can see this project on https://jurecki.github.io/myPortfolio/ | ||
| To run this project, install it locally using npm: | ||
| ``` | ||
| $ npm install | ||
| $ npm watch | ||
| ``` | ||
|
|
||
| ## Sources | ||
| Combining the BEM methodology with Sass inspired by Samuraj Programowania tutorial by https://www.udemy.com/course/zaawansowane-projekty-w-css-i-javascript \ | ||
| Using RWD with mixin and rem inspired by Jonas Schmedtmann tutorial by https://www.udemy.com/course/advanced-css-and-sass/ | ||
| Photos | ||
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Brakuje najważniejszej rzeczy czyli WIELKIEGO WIDOCZNEGO LINKU DO DEMO.
Dodatkowo przydałby się screenshot - zawsze to robi dobre wrażenie na wejściu.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dobra, jest - ale to, że nie zauważyłem na początku świadczy, że powinen być bardziej widoczny.