Skip to content

Добавляет работу над картинками#45

Merged
nikolai-shabalin merged 9 commits into
mainfrom
feature/work-images
Nov 8, 2023
Merged

Добавляет работу над картинками#45
nikolai-shabalin merged 9 commits into
mainfrom
feature/work-images

Conversation

@nikolai-shabalin

@nikolai-shabalin nikolai-shabalin commented Oct 31, 2023

Copy link
Copy Markdown
Contributor
  1. Добавляем исходники из макета в .raw
  2. Как только понадобится заспускам npm run optimizeImages
  3. Оптимизированные картинки появляются в source/images
  4. При работе картинки только копираются из source/images в build/images

Такой подход уменьшает количество запусков оптимизаций изображения, что является трудной и долгой gulp-задачей

@nikolai-shabalin nikolai-shabalin marked this pull request as ready for review November 7, 2023 11:23
@firefoxic

firefoxic commented Nov 7, 2023

Copy link
Copy Markdown
Contributor

Как минимум я бы raw/ из source/ в корень перенёс. Рано или поздно начнутся всякие исключения, чтобы не цеплять то, что в этом source/raw/ валяется (уже бывало такое).

@firefoxic

Copy link
Copy Markdown
Contributor

@nikolai-shabalin а это к какому потоку готовится?

@nikolai-shabalin

Copy link
Copy Markdown
Contributor Author

@nikolai-shabalin а это к какому потоку готовится?

К ближайшему 30 потоку, который 27 ноября стартует

@firefoxic

Copy link
Copy Markdown
Contributor

К ближайшему 30 потоку, который 27 ноября стартует

Ого! А в программе какие изменения будут? Это же надо как минимум как-то студентам сообщить, куда надо файлы класть. Или уже даже автоматизация сразу выдаваться будет? Или как что планируется?

@nikolai-shabalin

Copy link
Copy Markdown
Contributor Author

Ого! А в программе какие изменения будут? Это же надо как минимум как-то студентам сообщить, куда надо файлы класть. Или уже даже автоматизация сразу выдаваться будет? Или как что планируется?

Чуть позже сообщу. Думаю через недельку или полторы недели. Сейчас немного другая работа ведётся

@firefoxic

Copy link
Copy Markdown
Contributor

Чуть позже сообщу. Думаю через недельку или полторы недели. Сейчас немного другая работа ведётся

Дело в том, что в этом PR как минимум есть нестыковка между source/.raw/README.md и таской оптимизации графики. И это надо исправлять. Либо одно, либо другое. Лучше другое, но и лучше сразу оставляя только webp+avif. Это и программу упростит, и… ну я уже много плюсов в треде в учительской описал. Только видимо тщетно…

@nikolai-shabalin

Copy link
Copy Markdown
Contributor Author

В этом pr не будет решаться судьба дефолтов использования форматов гоафики в виде webp/avif.

Далее если решим внедрять, то это будет другой pr. Пока ничего не решено. Надо взвесить все плюсы и минусы.

В этом надо определиться с расположением графики, форматом дефолтной оригинальной графики, судя по всему png будет достаточно и что делать с 2x

@firefoxic

Copy link
Copy Markdown
Contributor

что делать с 2x
С этим то я помогу.

судя по всему png будет достаточно
Некоторые фотки (того же кота на главной под зелёнкой) будут очень тяжёлыми 😱

Проблема оставления jpg/png (кроме перечисленного мной в треде) в том, что в raw надо держать в правильном формате: фотки в jpg, малоцветные картинки и/или с фотки прозрачностью в png. Этот выбор студенты должны были бы на html1 научиться делать. А тут можно было бы упростить: всё в png, и только конвертится в новые, которым без разницы какого характера картинка, они умеют и в прозрачность в том числе. Это сильно таску упростило бы. А с jpg можно, но будет громоздко. Очень.

@firefoxic

Copy link
Copy Markdown
Contributor

Пилю таску…

Если оставлять png в вёрстке, то:

  1. от jpg не отказаться, png слишком ограничен в применении,
  2. avif не добавить — это будет издевательством, заставлять писать вручную 9-этажные picture (не говоря уж про image-set()).

Пилю дальше…

@firefoxic firefoxic mentioned this pull request Nov 8, 2023
@nikolai-shabalin nikolai-shabalin merged commit 6a346da into main Nov 8, 2023
@nikolai-shabalin nikolai-shabalin deleted the feature/work-images branch November 8, 2023 12:12
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