1

Тема: Оцініть роботу будь-ласка!

Оцініть роботу будь-ласка.Проект навчальний.Пробував експериментувати.Цікавить критика та відгуки (кому не лінь).
http://p73051h8.beget.tech

2

Re: Оцініть роботу будь-ласка!

Я бачу звичайну верстку на бутстрапі і трохи JS, який в загальному можна замінити на css. Взагалі оцінювати є не багато що

  • картинки без розмірів

  • нащо це в коді "<!---->" ?

  • <a href="#"></a> - так не можна лишати тег <a>

  • class="footer-menu-right-link" - що станеться з блоком, якщо його доведеться перенести ліворуч? Або по центру, чи згори. Невже кожного разу перейменовувати?

  • аналогічно щодо класу class="container-footer-right"> та інших

  • можна обійтися без id="ourDesign" замінивши на класи

  • background-repeat: no-repeat;
    background-size: 21px 21px;
    background-position: 90% 50%;


    Краще скоротити все це одним background

Подякували: Tar, Monolith, A.N.Onim, ostap34PHP4

3

Re: Оцініть роботу будь-ласка!

Дякую за відгук та критику.Переглянув ваші зауваження буду виправляти!А як на рахунок оцінки верстки ? Важлива ваша думка! Я початківець, пробую щось писати для себе.

Подякували: ostap34PHP1

4

Re: Оцініть роботу будь-ласка!

Викладіть код своєї сторінки на github - тоді можна буде детальніше оцінити

Подякували: Tar, ostap34PHP2

5

Re: Оцініть роботу будь-ласка!

Соромно зізнатись але не знаю як і для чого це робити, але спробую розібратись!

Подякували: ostap34PHP1

6 Востаннє редагувалося A.N.Onim (02.03.2017 14:04:30)

Re: Оцініть роботу будь-ласка!

Щодо юзабіліті:

  • В меню кнопка "Join". Зліві відступ здоровий. Не файно виглядає.

  • Зроби всі переходи поступовими - коли натискаєш на кнопку вверх, або стрілки кожного розділу. Бо зараз воно просто перекидує миттєво - користувачі такого як на мене не полюбляють - та і мене бісить. :-)

  • Lorem ipsum dolor sit amet, consectetur adipisicing - внизу уберіть - а то ви вже копірайт, рік написали а потім щещось пишете - не логічно.

  • випадаючий список вверху - можна гарніше оформити за допомогою якогось плагіну - наприклад nice-select.

  • Кнопка "піднятися вгору" - зробіть її фіксовану до нижнього кута - зараз виглядить прикольно - але припустимо людина зайшла з телефону - і в футері вона просто не побачить цю кнопку. :-)

  • Крайній правий пункт меню зліплений с кнопкою "join".

  • Особисто я не люблю фіолетовий - це досить специфічний колір.

А так - взагалі достатньо гарно виглядить - мені подобається)

Подякували: Tar, ostap34PHP2

7

Re: Оцініть роботу будь-ласка!

Щодо юзабіліті:
В меню кнопка "Join". Зліві відступ здоровий. Не файно виглядає.
Зроби всі переходи поступовими - коли натискаєш на кнопку вверх, або стрілки кожного розділу. Бо зараз воно просто перекидує миттєво - користувачі такого як на мене не полюбляють - та і мене бісить. :-)
Lorem ipsum dolor sit amet, consectetur adipisicing - внизу уберіть - а то ви вже копірайт, рік написали а потім щещось пишете - не логічно.
випадаючий список вверху - можна гарніше оформити за допомогою якогось плагіну - наприклад nice-select.
Кнопка "піднятися вгору" - зробіть її фіксовану до нижнього кута - зараз виглядить прикольно - але припустимо людина зайшла з телефону - і в футері вона просто не побачить цю кнопку. :-)
Крайній правий пункт меню зліплений с кнопкою "join".
Особисто я не люблю фіолетовий - це досить специфічний колір.
А так - взагалі достатньо гарно виглядить - мені подобається)

1.Відступ поправив.
2.Переходи зробив плавними
3.Lorem ipsum dolor sit amet, consectetur adipisicing - забрав.
4.скористався вашою порадою при оформленні select використав nice-select(може прикрутив я його криво(перший раз використовую плагіни а з jQ ще не знайомий потрохи розбираю JS))
5.Кнопку "піднятись вгору" зробив.
6.Крайній правий пункт меню прилипав до кнопки join - поправив.
7.До фіолетового кольору також ставлюсь не дуже але вирішив його залишити.

Подякували: ostap34PHP1

8

Re: Оцініть роботу будь-ласка!

Tar написав:

Соромно зізнатись але не знаю як і для чого це робити, але спробую розібратись!

як - навчитеся
https://www.codeschool.com/courses/try-git
https://www.codeschool.com/courses/mastering-github

а от для чого -моя відповідь така - без системи контролю версій нічого масштабного, командного, з можливістю розвитку/вдосконалення/супроводу робити не зможете.

Подякували: Tar, ostap34PHP2

9

Re: Оцініть роботу будь-ласка!

Дякую за відповідь! Починаю розбиратись!Правда успіхи поки не дуже.Для чого  git вже зрозумів(в мене досвіду мало декілька навчальних маленьких проектів для себе тому такої нагальної потреби в git поки не відчував(хоча вже декілька раз з`являлась думка ,що не погано було б це якось організувати постійно щось переписуєш ,переправляєш а потім кінці з кінцями звести не можеш,щось не там зберіг, кусок там кусок там і в кінці кінців каша).

Подякували: ostap34PHP1

10

Re: Оцініть роботу будь-ласка!

Викладіть код своєї сторінки на github - тоді можна буде детальніше оцінити

з git мені ще потрібно почитати, пробувати,працювати.Код https://github.com/Taras19/Eclipse-7
https://taras19.github.io/Eclipse-7/
Як що є можливість оцініть верстку?

Подякували: ping, ostap34PHP2

11

Re: Оцініть роботу будь-ласка!

На Github є можливість тримати статичні сайти з нормальною адресою, все дуже легко робиться https://pages.github.com/

Попередні зауваження не виправлені. Нове, що помітив:

  • яке призначення у id="body" ? Аналогічне питання щодо id="to-next-page-1"

  • & краще замінити на &amp;

  • push-youself-p - чому вибрана така назва класу?

Подякували: Tar, ostap34PHP2

12

Re: Оцініть роботу будь-ласка!

А на якій платформі створено сайт???

Подякували: ostap34PHP1

13

Re: Оцініть роботу будь-ласка!

А на якій платформі створено сайт???

Можна детальніше про платформу.Що це і для чого.Де можна про плаформи почитати?

Подякували: ostap34PHP1

14

Re: Оцініть роботу будь-ласка!

Ну CMS чи ще щось подібне

Подякували: ostap34PHP1

15

Re: Оцініть роботу будь-ласка!

kisilam написав:

Ну CMS чи ще щось подібне

повідомлення №10 містить код на гітхаб.

з нього видно - bootstrap + jquery

Подякували: Tar, ostap34PHP2

16

Re: Оцініть роботу будь-ласка!

CMS не використовував, використав сітку bootstrap, плагін nice-select,affix, скрипт для плавної прокрутки(сам написати не можу тому використав готове рішення).Все що можу намагаюсь писати сам, довго нудно але так мені цікавіше і навчитись чогось можна.

Подякували: kisilam, ostap34PHP2

17

Re: Оцініть роботу будь-ласка!

- від всіх id відмовився
-

& краще замінити на &amp;

- замінив
- всім img задав розмір
- <!----> - використовува просто для зручності щоб візуально розділяти код забув знищити поправив
-

<a href="#"></a> - так не можна лишати тег <a>

- не знаю чи правильно вас зрозумів поправив на

<div class="logo-container">
      <a href="http://p73051h8.beget.tech" class="logo"></a>
 </div>

- з назвами класів в мене взагалі біда, критично подивився потрібно міняти напевне всі назви і тут цікаве питання до вас Якщо ця робота щось вартує то сяду і перепишу,а як що це нічого не змінить тоді в наступному навчальному проекті буду звертати особливу увагу на назви класів. Висловлюйте свою думку!
- з git поки нормально не розібрався буду вивчати та практикувати тому зміни буду поки проводити за адресою http://p73051h8.beget.tech
Чекаю ваших відгуків та пропозицій

18

Re: Оцініть роботу будь-ласка!

Ще один пункт
- було зауваження по background поправив(скорочений запис)

Подякували: ostap34PHP1

19

Re: Оцініть роботу будь-ласка!

До

<a href="http://p73051h8.beget.tech" class="logo"></a>

потрібно ще додати title

Подякували: ostap34PHP, Tar2

20

Re: Оцініть роботу будь-ласка!

Якось все геть статично. Жодної анімації чи hover-ефекту. Не цікаво.