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

поддержка webp для обложек подкастов #238

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

akellbl4
Copy link
Collaborator

@akellbl4 akellbl4 commented May 24, 2021

  • ресайзим обложки через встроенный механизм Hugo, а так же генерим webp версию
  • картинки переехали в папку hugo/assets/images/radio-t, потому что Hugo умеет их брать только от туда
  • изменены ссылки на обложки (из ссылки убран домен)

@umputun,

  • кажется что тут надо будет где-то что-то менять чтобы для новых подкастов складывать обложки в нужную папку
  • больше не нужно генерит webp версию обложки
  • не нужно добавлять домен в ссылку на картинку
  • заметил что в тело md файла так же встраивается картинка, кажется ее тоже можно убить, потом можно пройтись и удалить из всех выпусков

@github-actions
Copy link

github-actions bot commented May 24, 2021

size-limit report 📦

Path Size
static/build/app.js 14.85 KB (+0.51% 🔺)
static/build/manifest.js 724 B (0%)
static/build/modernizr-bundle.js 1.98 KB (0%)
static/build/vendor.js 91.31 KB (0%)
static/build/app.css 6.31 KB (-3.63% 🔽)
static/build/app-dark.css 6.35 KB (-3.61% 🔽)
static/build/vendor.css 5.51 KB (-0.09% 🔽)
static/build/vendor-dark.css 5.55 KB (-0.08% 🔽)

@akellbl4
Copy link
Collaborator Author

Хуго научился сам конвертировать картинки в WEBP, поэтому можно удалить все картинки WEBP и конвертировать на лету во время сборки

@akellbl4 akellbl4 marked this pull request as ready for review July 18, 2021 12:30
@akellbl4 akellbl4 requested a review from umputun July 18, 2021 12:38
@akellbl4
Copy link
Collaborator Author

Поправил формат для создания нового поста
a3792f1

@webkonstantin
Copy link
Collaborator

здесь обсуждали, что картинки в теле лучше оставить #27 (comment)

@akellbl4
Copy link
Collaborator Author

@webkonstantin спасибо за коммент, гляну.
Можешь рассказать почему выкладка сломалась?

@webkonstantin
Copy link
Collaborator

ERROR 2021/07/22 13:32:46 render of "page" failed: execute of template failed: template: _default/single.html:28:50: executing "main" at <partial "podcast-cov...>: error calling partial: "/opt/build/repo/hugo/layouts/partials/podcast-cover.html:3:21": execute of template failed: template: partials/podcast-cover.html:3:21: executing "partials/podcast-cover.html" at <resources.Get>: error calling Get: stat /opt/build/repo/hugo/assets/images/cover-default.jpg: no such file or directory

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

Successfully merging this pull request may close these issues.

2 participants