Skip to content
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

Баг с подгрузкой шрифтов LocalStorage в Google Chrome #16

Open
kolyaraketa opened this issue Jul 20, 2016 · 16 comments

Comments

@kolyaraketa
Copy link

Заметил один очень неприятный баг: в хроме не хотят грузиться шрифты в локал-сторедж. В эдже и фаэрфоксе все ок, шрифты грузятся отлично. Но если шрифты подключаться как и стили, тоесть через <script>loadCSS( "css/fonts.min.css?ver=1.0.0", false, "all" );</script>, то шрифты подключаются и отрабатывают нормально. Очень странное поведение, не сразу понял, почему в хроме не хочет работать локал сторедж.

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

@agragregra
Copy link
Owner

agragregra commented Jul 20, 2016

Сайт должен быть в корне домена. А не в папке.
После пертурбаций со шрифтами, надо чистить локалсторидж localStorage.clear()
Код рабочий.

@kolyaraketa
Copy link
Author

Да, действиетельно, очистка локал-стореджа помогает. Очень часто приходится верстать какой-то лендинг, который на сайте не один, и он располагается в подпапке. Тогда не нужно ложить шрифты в локал-сторедж?

ПыСы. Стыдно что не додумался очистить локалсторедж.

@agragregra
Copy link
Owner

Я обычно для работы в папке комментирую строку и дублирую просто loadCSS, как вы описали в первом сообщении. На продакшене возвращаю обратно.

@kolyaraketa
Copy link
Author

Спасибо большое за разъясения. Может быть у кого-то еще возникнут вопросы - будет где посмотреть.

@WhoAm7i7
Copy link

в одном из видео Алексей еще на этом акцентировал внимание, буквально на днях, пересматривал....
Ракета спасибо за поднятый вопрос, буду еще внимательней к этому аспекту.
Касательно чистки localStorage, не совсем понимаю, как технически это сделать, буду признателен за разъяснение.

@kolyaraketa
Copy link
Author

Почистить можно в браузере в консоли, просто вводишь команду и все. Для меня это не вариант, поскольку я работаю в компании маркетинговой, и там на одном домене куча лендингов, все они разные и не похожие друг на друга, каждый верстается отдельно, шрифты все разные, и каждый лендос в отдельной папке как отдельный проект. Пожалуй откажусь от localStorage.
Насчет видео с информацией по шрифтам - может я был еще совсем зеленый и пропустил мимо ушей, а может и видос пропустил, но это наверное видео было давненько уже, в последних я такого не помню.
А так шаблон оличный, я его под себя подстроил, добавил несколько плагинов которые всегда использую, и тягаю от проекта в проект. Спасибо огромное за разьяснение и удачи)

@WhoAm7i7
Copy link

WhoAm7i7 commented Jul 20, 2016

согласен, шаблон отличный, автору отдельное за это спасибо!
у себя можно почистить, а у клиент как, на сколько я правильно понял логику localStorage основная ее задача быстрое отображение самой страницы на стороне клиента + отличная оценка PageSpeed
Если не прав поправь.

@agragregra
Copy link
Owner

WhoAm7i7, на продакшене багов быть не должно. У клиента будет результат вашей работы, который не должен содержать подобных ошибок.

@WhoAm7i7
Copy link

agragregra, только вхожу на уровень полноценной верстки, до этого баловался сборкой и правкой сайтов на WP, с первого клиента отработанного по твоей системе и урокам, перечислю 10% от полученных денег, мне будет приятно отблагодарить тебя... не останавливайся в развитии своего YouTube канала и наполнении его поистине качественным контентом в создании новых проектов по верстке и посадки на CMS.... для многих это важно....

@inamax335
Copy link

WhoAm7i7, отчистить localStorage можно командой localStorage.clear(); в консоли браузера.

screenshot_7

@WhoAm7i7
Copy link

inamax335 за наглядное отображение, спасибо!

@pashog
Copy link

pashog commented Jul 21, 2016

А если уж накосячили, например на стейдже, или на своём тестовом сервере, который отдаёте на проверку клиенту, то можно его попросить вставить в адресную строку javascript: localStorage.clear(); и нажать Enter. Для клиента это куда проще, чем лезть в консоль.

Я так вообще добавил себе закладку c такой урлой на панель закладок, чтобы нигде ничего не прописывать ;)

@agragregra
Copy link
Owner

Можно localStorage.clear(); разместить в common.js

@pashog
Copy link

pashog commented Jul 21, 2016

Можно localStorage.clear(); разместить в common.js

Ну это уж слишком) Главное, не забыть потереть перед релизом, если уж так)

@agragregra
Copy link
Owner

О том и речь.

@WhoAm7i7
Copy link

WhoAm7i7 commented Jul 22, 2016

народ offtopic
подскажите как подключить форму ОСТАВИТЬ ЗАЯВКУ в разделе "Контакты" на сайте http://vkgroup.kiev.ua/ ?
перепробовал разные комбинации, рациональней спросить, как правильно, чем описывать все процессы которые этому сопутствовали...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants