Add email and adress #3
Conversation
footer.php
Outdated
| <p class="mb-3">© <?php echo date( 'Y' ) ?> | ||
| <?php bloginfo( 'name' ); ?></p> |
inc/classes/B4st.php
Outdated
| $time_string = '<time class="entry-date published" datetime="%1$s">%2$s</time> <time class="updated" datetime="%3$s">(updated %4$s)</time>'; | ||
| } | ||
|
|
||
| $time_string = $with_ico ? '<i class="far fa-clock"></i> ' . $time_string : $time_string; |
There was a problem hiding this comment.
в открытой теме мы по-моему используем solid font awesome (fas) а не regular (far), потому что он по большей части платный. Добавленные иконки можно смотреть здесь https://github.com/ArroWsGM/b4st/blob/master/assets/src/js/fa5-bundle.js#L54
и clock там вроде нет, т.е. его нужно добавить (из солид конечно), пересобрать фонт оссом с вебпаком (примерно что делать описано тут https://github.com/ArroWsGM/fa5-subsetter) и потом поменять классы на нужные. Ну и в конце убедиться что все работает
inc/classes/B4stThemeSettings.php
Outdated
| @@ -384,7 +395,7 @@ public function print_banners_section_info() { | |||
| public function contacts_email_callback() { | |||
| $contacts_email = isset( $this->options['contacts_email'] ) ? $this->options['contacts_email'] : ''; | |||
|
|
|||
| echo '<p><input type="email" id="contacts_email" name="' . $this->on . '[contacts_email]" value="' . $contacts_email . '" placeholder="' . get_option( 'admin_email', '[email protected]') . '" class="regular-text ltr"></p>'; | |||
| echo '<p><input type="text" id="email" name="' . $this->on . '[contacts_email]" value="' . $contacts_email . '" placeholder="' . get_option( 'admin_email', '[email protected]') . '" class="regular-text ltr"></p>'; | |||
There was a problem hiding this comment.
здесь не корректно выставлены type="text" id="email"
как было раньше - правильно
|
Ок, вёрстку в футере я поправлю. И такой ещё вопрос по композеру можешь,
пожалуйста, посоветовать инфу почитать/посмотреть.
чт, 19 марта 2020 г., 18:50 ArroWsGM <[email protected]>:
… ***@***.**** commented on this pull request.
Ну вроде норм, font awesome только пересобрать
И может футер реорганизовать, чтобы оно одно над другим не сидело, а по
колонкам
ну и чтоб mobile-first responsive
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3 (review)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJB6XB4NYIATG2EIKUV22T3RIJEO3ANCNFSM4LNDOWIA>
.
|
Не только верстку, еще яваскрипт пересобирать, от этого никуда не деться |
|
Привет. Все же по вебпаку нужна помощь. Думал, что я могу и сам во всем разобраться, че там сложного думал, в итоге убил нереальное количество времени, чтобы в нем разобраться, но реально не могу понять до конца, как собрать на нем тему. Может сможешь подсказать какие-то туториалы, вот прям, чтобы и идиоту дошло)) так как все же без помощи не могу я походу сообразить |
|
Туториал вроде этот https://www.youtube.com/watch?v=MRlBKfGktwI первый - для продакшн-сборки, последний для разработки с перекомпиляцией "на лету", т.е. он запущенный должен висеть пока нужен, средний для вп работать не будет. и пилишь-проверяешь, когда допилил прервал этот и запустил |
|
Спасибо большое. Я понимаю, что мне не с нуля это делать нужно, но все же
хочется в этом всем разобраться нормально, а не просто поверхностно для
этого проекта
вс, 5 апр. 2020 г. в 16:01, ArroWsGM <[email protected]>:
… Туториал вроде этот https://www.youtube.com/watch?v=MRlBKfGktwI
По крайней мере он у меня в закладках валяется
Но я ж не заставляю тебя с 0 все собирать, там же все просто, и все что
нужно, буквально расписано по шагам в
https://github.com/ArroWsGM/fa5-subsetter, тут практически тоже самое,
кроме того что конфиг чуть сложнее, но он уже написан, тебе просто
проинсталить зависимости npm i и запустить скрипт сборки (естественно,
после того, как добавишь нужные иконки). Скрипты сборки все можно смотреть
в package.json
...
"scripts": {
"build": "webpack --mode production",
"live": "webpack-dev-server --hot --no-info --mode development",
"dev": "webpack --watch --mode development"
},
первый - для продакшн-сборки, последний для разработки с перекомпиляцией
"на лету", т.е. он запущенный должен висеть пока нужен, средний для вп
работать не будет.
т.е. запускаешь
npm run dev
и пилишь-проверяешь, когда допилил прервал этот и запустил npm run build
можно еще обновиться npm up и запустить npm audit fix для
серьюрити-фикса, но так гарантии нет что оно все будет работать, хотя в
90+% случаев - работает нормально и это полезно делать, хотя не всегда все
проблемы исправляет
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJB6XB2JJBA6VB7EYOJMH6TRLB6MLANCNFSM4LNDOWIA>
.
|
No description provided.