-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add email and adress #3
base: master
Are you sure you want to change the base?
Conversation
footer.php
Outdated
<p class="mb-3">© <?php echo date( 'Y' ) ?> | ||
<?php bloginfo( 'name' ); ?></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
повтор
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
в открытой теме мы по-моему используем 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
здесь не корректно выставлены type="text" id="email"
как было раньше - правильно
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ну вроде норм, font awesome только пересобрать
И может футер реорганизовать, чтобы оно одно над другим не сидело, а по колонкам
ну и чтоб mobile-first responsive
Ок, вёрстку в футере я поправлю. И такой ещё вопрос по композеру можешь,
пожалуйста, посоветовать инфу почитать/посмотреть.
чт, 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.