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

darknebel · March 11, 2014

Здравствуйте! Покритикуйте пожалуйста мою верстку под ie8+.

Картинка

Верстка

Буду рад любым замечаниям.

klierik · March 12, 2014

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>

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

darknebel · March 12, 2014

On 3/12/2014 at 8:12 AM, klierik said:
1. почему не работает меню? при клике должно проскролится к соответствующему пункту

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

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

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

On 3/12/2014 at 8:12 AM, klierik said:
6. скрипты засунуть перед

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

On 3/12/2014 at 8:12 AM, klierik said:
в остальном вроде как даже нормалёк.

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

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

klierik · March 12, 2014

On 3/12/2014 at 9:40 AM, 'darknebel said:
Получается оно должно еще и фиксироваться на верху экрана при прокрутке?

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

On 3/12/2014 at 9:40 AM, 'darknebel said:
Вы имеете ввиду неправильные названия для классов или вообще весь код?

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

On 3/12/2014 at 9:40 AM, 'darknebel said:
Все, даже библиотеки, слайдеры и т.п.?

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

Reveal hidden contents

    ...    ...    ...    <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>

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

darknebel · March 12, 2014

On 3/12/2014 at 11:12 AM, klierik said:
On 3/12/2014 at 9:40 AM, 'darknebel said:
Получается оно должно еще и фиксироваться на верху экрана при прокрутке?

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

On 3/12/2014 at 9:40 AM, 'darknebel said:
Вы имеете ввиду неправильные названия для классов или вообще весь код?

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

On 3/12/2014 at 9:40 AM, 'darknebel said:
Все, даже библиотеки, слайдеры и т.п.?

ну а чего бы и нет? в <head> засуть стили, а в конце страницы остальное, пример:
Reveal hidden contents
    ...    ...    ...    <div class="footer">        ...    </div>        <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>        <script src="js/jquery-ui-1.10.4.custom/js/jquery-ui-1.10.4.custom.min.js"></script>        <script src="bootstrap/dist/js/bootstrap.min.js"></script>            <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>
работать будет отлично, а задержек при загрузке страницы ты избежиш.

Спасибо!

darknebel · March 19, 2014

Верстку поправил, меню доделал. Кому интересно, можете посмотреть. Правда проблемы с оформлением кода остались...

ModMaker · March 19, 2014

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

darknebel · March 19, 2014

On 3/19/2014 at 11:32 AM, ModMaker said:
Привет, а можешь выложить псд и мини ТЗ этого шаблона? Тоже бы хотелось проверить на нём свои силы.

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 -- а здесь можно составить для себя ТЗ.

Sign In

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

Recommended Posts

darknebel

Link to comment

Share on other sites

klierik

Link to comment

Share on other sites

darknebel

Link to comment

Share on other sites

klierik

Link to comment

Share on other sites

darknebel

Link to comment

Share on other sites

darknebel

Link to comment

Share on other sites

ModMaker

Link to comment

Share on other sites

darknebel

Link to comment

Share on other sites

Join the conversation

Обсуждения

Forums

Important Information