Jump to content

Оцените пожалуйста верстку

darknebel

By darknebel,
in Discussion of works

 Share

Recommended Posts

klierik Middle Developer

klierik

Posted
Posted

1. почему не работает меню? при клике должно проскролится к соответствующему пункту

2. в .person засунул <p> -- зачем, если это не абзац? используй <div> (ошибка повторяется не только тут)

3. для создания спрайтов и иконок посмотри в сторону: http://draeton.github.io/stitches/

4. не исползуй <br> для оформления:

<button>Subscribe to our<br><span>newsletter</span></button>

два <span> тут смотрелись бы правильнее, потому что высота строки у тебя может изменится, да и контент может изменится.

 

5. http://www.myworks.96.lt/anhiora/css/main.css -- как обычно нечитабельный и трудно поддерживаемый код. проще написать с нуля, чем такое поддерживать.

6. скрипты засунуть перед </body>

 

в остальном вроде как даже нормалёк.

Link to comment
Share on other sites
darknebel Explorer

darknebel

Posted
  • Author
Posted
1. почему не работает меню? при клике должно проскролится к соответствующему пункту

 

Получается оно должно еще и фиксироваться на верху экрана при прокрутке?

 

5. http://www.myworks.9...ra/css/main.css -- как обычно нечитабельный и трудно поддерживаемый код. проще написать с нуля, чем такое поддерживать.

 

Вы имеете ввиду неправильные названия для классов или вообще весь код?

 

6. скрипты засунуть перед

 

Все, даже библиотеки, слайдеры и т.п.?

 

в остальном вроде как даже нормалёк.

 

Значит я все-таки не безнадежен! :) 

 

Большое спасибо за критику!!!

Link to comment
Share on other sites
klierik Middle Developer

klierik

Posted
Posted

Получается оно должно еще и фиксироваться на верху экрана при прокрутке?

 

 

да. пример: http://pagescroller.com/

 

 

Вы имеете ввиду неправильные названия для классов или вообще весь код?

 

его оформление.

 

 

Все, даже библиотеки, слайдеры и т.п.?

 

 

ну а чего бы и нет? в <head> засуть стили, а в конце страницы остальное, пример:

    ...    ...    ...    <div class="footer">        ...    </div>    <!-- jQuery init -->    <script src="http://code.jquery.com/jquery-1.10.1.min.js"></script>    <script>window.jQuery || document.write('<script src="js/jquery.min.js"><\/script>')</script>    <!-- jQuery UI -->    <script src="js/jquery-ui-1.10.4.custom/js/jquery-ui-1.10.4.custom.min.js"></script>    <!-- Bootstrap init -->    <script src="bootstrap/dist/js/bootstrap.min.js"></script>    <!-- jQuery plugins -->    <!-- Project scripts -->    <script type="text/javascript">        /*<![CDATA[*/        jQuery(document).ready(function ($) {            $(function() {                $( "#datepicker" ).datepicker({                    nextText:           '<i class="glyphicon glyphicon-circle-arrow-right"></i>',                    prevText:           '<i class="glyphicon glyphicon-circle-arrow-left"></i>',                    changeMonth: true,                    changeYear: true                });                $("#dialog").dialog();            });        });        /*]]>*/    </script></body></html>

работать будет отлично, а задержек при загрузке страницы ты избежиш.

Link to comment
Share on other sites
darknebel Explorer

darknebel

Posted
  • Author
Posted

 

Получается оно должно еще и фиксироваться на верху экрана при прокрутке?

 

 

да. пример: http://pagescroller.com/

 

 

Вы имеете ввиду неправильные названия для классов или вообще весь код?

 

его оформление.

 

 

Все, даже библиотеки, слайдеры и т.п.?

 

 

ну а чего бы и нет? в <head> засуть стили, а в конце страницы остальное, пример:

    ...    ...    ...    <div class="footer">        ...    </div>    <!-- jQuery init -->    <script src="http://code.jquery.com/jquery-1.10.1.min.js"></script>    <script>window.jQuery || document.write('<script src="js/jquery.min.js"><\/script>')</script>    <!-- jQuery UI -->    <script src="js/jquery-ui-1.10.4.custom/js/jquery-ui-1.10.4.custom.min.js"></script>    <!-- Bootstrap init -->    <script src="bootstrap/dist/js/bootstrap.min.js"></script>    <!-- jQuery plugins -->    <!-- Project scripts -->    <script type="text/javascript">        /*<![CDATA[*/        jQuery(document).ready(function ($) {            $(function() {                $( "#datepicker" ).datepicker({                    nextText:           '<i class="glyphicon glyphicon-circle-arrow-right"></i>',                    prevText:           '<i class="glyphicon glyphicon-circle-arrow-left"></i>',                    changeMonth: true,                    changeYear: true                });                $("#dialog").dialog();            });        });        /*]]>*/    </script></body></html>

работать будет отлично, а задержек при загрузке страницы ты избежиш.

 

Спасибо!

Link to comment
Share on other sites
darknebel Explorer

darknebel

Posted
  • Author
Posted
Привет, а можешь выложить псд и мини ТЗ этого шаблона? Тоже бы хотелось проверить на нём свои силы.

 

http://htmlforum.ru/index.php?showtopic=47561 -- отсюда брал psd.

http://www.xiper.net/collect/weekdays-front-end-dev/the-work-in-general/tehnicheskoe-zadanie-na-verstku.html -- а здесь можно составить для себя ТЗ.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
 Share
Go to topic listing
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy

  I accept