Jump to content

Критика и т.д.


a1rborn
 Share

Recommended Posts

День добрый)

У кого есть желание, посмотрите на сайт http://kombinatorika.p.ht/

Собственно, тема: Известный японский писатель Харуки Мураками.

Критика приветствуется любого рода, особое внимание уделить верстке и дизайну)

Сайт еще до конца контентом не наполнен, решил пока есть время выложу сюда, посмотрю что скажут люди, а там и с инфой закончу

Вдруг какие проблемы будут критические? Лучше раньше исправлю

Замечания и вопросы

1. По клику на любой картинке из средней колонки, будет в дальнейшем появляться инфа, пока, повторяюсь, не дописал

2. Интересует ваше мнение по поводу

а) Кнопка переключения среднего меню, никак не могу найти или нарисовать подходящую, мб подскажите что нибудь?

б) Подчеркивание li оставить как есть? или у списка изменить на точечную, а у подсписков на сплошную?

в) Цвет шрифта и сам шрифт - оставлять как есть, или искать дальше и пробовать? Довольно много времени на это убил...

г) Скорости разных анимацией - делать быстрее, медленнее?

Вот собственно и все)

Спасибо за внимание

PS Не обращайте внимание на название линка

Edited by a1rborn
Link to comment
Share on other sites

Мне кажется, что ссылкам в меню было бы неплохо присвоить hover-эффект для "оживления" странички

JQuery понравился)) Немного давит дизайн, темно как-то. А в целом аккуратный такой сайт..

Только что зашел в валидатор, мне кажется вам тоже следует туда зайти. 52 ошибки...

Многовато однако.

Link to comment
Share on other sites

Большинство ошибок исправил, осталась парочка, где валидатор говорит что ul и h2 не могут быть дочерними в ul. Везде все работает, исправлять думаю нет смысла (если есть, объясните)

До сих пор жду, по возможности, ответы на мои вопросы)

Замечания и вопросы

1. По клику на любой картинке из средней колонки, будет в дальнейшем появляться инфа, пока, повторяюсь, не дописал

2. Интересует ваше мнение по поводу

а) Кнопка переключения среднего меню, никак не могу найти или нарисовать подходящую, мб подскажите что нибудь?

б) Подчеркивание li оставить как есть? или у списка изменить на точечную, а у подсписков на сплошную?

в) Цвет шрифта и сам шрифт - оставлять как есть, или искать дальше и пробовать? Довольно много времени на это убил...

г) Скорости разных анимацией - делать быстрее, медленнее?

Link to comment
Share on other sites

Мое мнение просто как человека, а не спеца. У меня сайт не входит по ширине, При прокрутке вниз, меню опускается, это классно, но, когда я выбираю другой пункт меню прокрутка стоит на месте, хотя переход происходит. С этим что-то нужно делать

Link to comment
Share on other sites

Сайт можно сделать резиновым и задать при этом параметр max-width, тогда и картинки при растяжении не пострадают и у пользователей с маленьким разрешением сайт будет отображаться нормально.

Link to comment
Share on other sites

а) Кнопка переключения среднего меню сейчас обозначает обновление. Чтобы было понятно, что она делает, нужно нарисовать стрелку влево при развёрнутом меню и стрелку вправо при свёрнутом. При такой маленькой ширине этого меню я бы вообще убрал сворачивание/разворачивание.

б) Подчеркивание у li, по-моему, лишнее. На меню набранное прописными буквами смотреть сложно. Ширину меню нужно подобрать так, чтобы названия влезали в одну строку.

в) Цвет шрифта и сам шрифт — искать дальше и пробовать... ) Сейчас как-то не очень (особенно Times). Мне всегда было сложно читать больше пары предложений на тёмном фоне. Хотя это на любителя, конечно.

г) Скорости разных анимацией нормальные. Только анимация ездящего меню раздражает (можно закрепить меню без анимации).

И ещё:

— Текстуры фона смотрятся некачественно.

— Логика работы среднего меню мне совсем не понравилась. Т. е. я нахожусь в разделе «Об авторе», нажимаю «Сборники рассказов», а меняется только узкая полоска с картинками, которую мне ещё и прокручивать приходится. Как пользователь я ожидал увидеть книги, занимающие всё свободное пространство, да ещё и с краткими описаниями.

— Весь сайт на одной странице. Поисковик его забанит )

Link to comment
Share on other sites

  • У меня кнопка "назад" не работает.
  • Заголовки не меняются (да их и вообще нет)
  • При переходе по страницам адрес URL тоже не меняется, поэтому поисковикам будет приятно узнать, что ваш сайт выдает для них контент, отличный от браузера пользователя.

Link to comment
Share on other sites

  • У меня кнопка "назад" не работает.
  • Заголовки не меняются (да их и вообще нет)
  • При переходе по страницам адрес URL тоже не меняется, поэтому поисковикам будет приятно узнать, что ваш сайт выдает для них контент, отличный от браузера пользователя.

Где вы нашли кнопку назад?

Какой у Вас браузер? Заголовки у всех были и менялись

Последнее вообще не понял

Link to comment
Share on other sites

Блин, да что ж тут понимать-то? Дайте мне, пожалуйста, ссылку, по которой я сразу смогу перейти в раздел «Юность», например.

Такого линка нет, потому что сайт, выполнен пока что на одной странице, и если Вы не заметили, то все меняется по средствам jq ;)

Link to comment
Share on other sites

Такого линка нет, потому что сайт, выполнен пока что на одной странице, и если Вы не заметили, то все меняется по средствам jq ;)

Вам об этом и говорят. Пользы от такой «навигации», когда даже нельзя дать ссылку на конкретный раздел, нет ни практической, ни для поисковиков.

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

  • Similar Content

    • By zeiger2
      Здравствуйте! У меня стоит задача, что при наведении на блок li строка должна поменять цвет, в том числе и картинка. Я меняю картинку с помощью 
      background-image: none;     background: url(../img/check_icon_red.png) left no-repeat;   Но теперь картинка позицианируется не там где должна, её можно поставить на место только вручную, через -100px. Нужно поставить ровно туда, где она была. Должна быть в одном ряду с другими
    • By Mix9
      есть див с 5 img, при уменьшении экрана див выходит за него. Я добавил overflow: auto для этого div в надежде на то, что я смогу прокручивать фотки с помощью скроллбара, однако даже с ним почему-то я не вижу часть фоток которые вышли за границу. Что с этим можно сделать? класс video повторяется 5 раз, я тут оставил только 1 
      .content{ width: 90%; background-color: #333; } .video{ margin: 0px 4px 0px 4px; width: 310; display: flex; flex-direction:column; } .video_button_text{ margin-top: 10px; display: flex; flex-direction: row; font-size: 20px; color: white; } .video_text_div{ display: inline-block; width: 250px; } .video_text{ text-align: justify-all; margin: 0px; display: -webkit-box; -webkit-line-clamp: 2; -webkit-box-orient: vertical; overflow: hidden; } .slidan_videos{ margin: 0px 10px 0px 20px; overflow: auto; width: auto; margin-bottom: 50px; display: flex; flex-direction: row; justify-content: space-around; } <div class="content"> <div class = slidan_videos> <div class = video> <div> <a href = 'ссылка'><img class="img" src=""картинка"></a> </div> <div class = video_button_text> <div class = avatarka_div> <a href="ссылка" target="_blank"><img class = avatarka src="картинка"></a> </div> <div class = video_text_div> <p class = video_text><a href="ссылка">текст</a></p> </div> </div> </div>
    • By Марко
      Добрый день. Начинающий программист, столкнулся с проблемой. Селектор .class не работает должным образом. Несмотря на правильное, я надеюсь, описание, на web-странице не отображается ни одно изображение. С чем может быть связано? Заранее спасибо за помощь. 



    • By Kaido
      Использую готовый плагин для модальных окон(от MaxGraph). Проблема в том, что когда у меня открыто два модальных окна, для примера Форма + Политика конфендициальности, и мне нужно закрыть политику вместе с ней закрывается и другое модальное окно. В JS я не сильно разбираюсь(собственно из за этого и использую готовый плагин), можете помочь кто работал с этим плагином? Я примерно понимаю как он работает, но реализовать чтобы закрывалось только одно не получается.
       
        <div class="content"> <button class="modal-btn" data-path="first" data-animation="fadeInUp" data-speed="1500">Открыть окно 1</button> </div> <div class="modal"> <div class="modal__wrapp" data-target="first"> <div class="modal__content"> <button class="modal__close">Закрыть</button> модальное окно <button data-path="policy">Политика</button> </div> </div> <div class="modal__wrapp" data-target="policy"> <div class="modal__content"> <button class="modal__close">Закрыть</button> политика </div> </div> </div> .modal { --transition-time: 0.3s; position: fixed; left: 0; top: 0; right: 0; bottom: 0; z-index: 1000; cursor: pointer; overflow-y: auto; overflow-x: hidden; text-align: center; opacity: 0; visibility: hidden; transition: opacity var(--transition-time), visibility var(--transition-time); } .modal__wrapp { display: none; cursor: default; width: fit-content; height: fit-content; } .modal__content{ position: absolute; left: 500px; width: 500px; height: 500px; display: flex; color: white; flex-direction: column; text-align: left; background-color: #000; } .modal__content button{ width: 200px; height: 50px; margin: 50px 0; } .modal.is-open { opacity: 1; visibility: visible; transition: opacity var(--transition-time), visibility var(--transition-time); } .modal__wrapp.modal-open { display: flex; } .disable-scroll { position: relative; overflow: hidden; height: 100vh; position: fixed; left: 0; top: 0; width: 100%; } .fade { opacity: 0; transition: opacity var(--transition-time); } .fade.animate-open { opacity: 1; transition: opacity var(--transition-time); } .fadeInUp { opacity: 0; transform: translateY(vw(-100)); transition: opacity var(--transition-time), transform var(--transition-time); } .fadeInUp.animate-open { opacity: 1; transform: translateY(0); transition: opacity var(--transition-time), transform var(--transition-time); } .modal__wrapp[data-target="policy"] .modal__content{ left: 1050px; background-color: #000; opacity: .5; } class Modal { constructor(options) { let defaultOptions = { isOpen: () => {}, isClose: () => {}, } this.options = Object.assign(defaultOptions, options); this.modal = document.querySelector('.modal'); this.speed = false; this.animation = false; this.isOpen = false; this.modalContainer = false; this.previousActiveElement = false; this.fixBlocks = document.querySelectorAll('.fix-block'); this.focusElements = [ 'a[href]', 'input', 'button', 'select', 'textarea', '[tabindex]' ]; this.events(); } events() { if (this.modal) { document.addEventListener('click', function(e){ const clickedElement = e.target.closest('[data-path]'); if (clickedElement) { let target = clickedElement.dataset.path; let animation = clickedElement.dataset.animation; if (clickedElement.classList.contains('modal-close')) { this.close(); } let speed = clickedElement.dataset.speed; this.animation = animation ? animation : 'fade'; this.speed = speed ? parseInt(speed) : 300; this.modalContainer = document.querySelector(`[data-target="${target}"]`); this.open(); return; } if (e.target.closest('.modal__close')) { this.close(); return; } }.bind(this)); window.addEventListener('keydown', function(e) { if (e.keyCode == 27) { if (this.isOpen) { this.close(); } } if (e.keyCode == 9 && this.isOpen) { this.focusCatch(e); return; } }.bind(this)); this.modal.addEventListener('click', function(e) { if (!e.target.classList.contains('modal__wrapp') && !e.target.closest('.modal__wrapp') && this.isOpen) { this.close(); } }.bind(this)); } } open() { this.previousActiveElement = document.activeElement; this.modal.style.setProperty('--transition-time', `${this.speed / 1000}s`); this.modal.classList.add('is-open'); this.disableScroll(); this.modalContainer.classList.add('modal-open'); this.modalContainer.classList.add(this.animation); setTimeout(() => { this.options.isOpen(this); this.modalContainer.classList.add('animate-open'); this.isOpen = true; this.focusTrap(); }, this.speed); } close() { if (this.modalContainer) { this.modalContainer.classList.remove('animate-open'); this.modalContainer.classList.remove(this.animation); this.modal.classList.remove('is-open'); this.modalContainer.classList.remove('modal-open'); this.enableScroll(); this.options.isClose(this); this.isOpen = false; this.focusTrap(); } } focusCatch(e) { const focusable = this.modalContainer.querySelectorAll(this.focusElements); const focusArray = Array.prototype.slice.call(focusable); const focusedIndex = focusArray.indexOf(document.activeElement); if (e.shiftKey && focusedIndex === 0) { focusArray[focusArray.length - 1].focus(); e.preventDefault(); } if (!e.shiftKey && focusedIndex === focusArray.length - 1) { focusArray[0].focus(); e.preventDefault(); } } focusTrap() { const focusable = this.modalContainer.querySelectorAll(this.focusElements); if (this.isOpen) { focusable[0].focus(); } else { this.previousActiveElement.focus(); } } disableScroll() { let pagePosition = window.scrollY; this.lockPadding(); document.body.classList.add('disable-scroll'); document.body.dataset.position = pagePosition; document.body.style.top = -pagePosition + 'px'; } enableScroll() { let pagePosition = parseInt(document.body.dataset.position, 10); this.unlockPadding(); document.body.style.top = 'auto'; document.body.classList.remove('disable-scroll'); window.scroll({ top: pagePosition, left: 0 }); document.body.removeAttribute('data-position'); } lockPadding() { let paddingOffset = window.innerWidth - document.body.offsetWidth + 'px'; this.fixBlocks.forEach((el) => { el.style.paddingRight = paddingOffset; }); document.body.style.paddingRight = paddingOffset; } unlockPadding() { this.fixBlocks.forEach((el) => { el.style.paddingRight = '0px'; }); document.body.style.paddingRight = '0px'; } } const modal = new Modal({ isOpen: (modal) => { console.log(modal); console.log('opened'); }, isClose: () => { console.log('closed'); }, });  
×
×
  • 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