Добавлена обработка изображения + обновление конфигурации #4
Conversation
niknov80
commented
Apr 4, 2023
niknov80
commented
- Добавлена обработка изображений
- Добавлен sourcemap для стилей и скриптов
- Сделано модульное подключение скриптов
- Исправлены пути скриптов и стилей для pixelGlass
- Добавлено удаление папки build перед сборкой
1. Добавлена обработка изображений 2. Добавлен sourcemap для стилей и скриптов 3. Сделано модульное подключение скриптов 4. Исправлены пути скриптов и стилей для pixelGlass 5. Добавлено удаление папки build перед сборкой
gulpfile.js
Outdated
| import sharpOptimizeImages from "gulp-sharp-optimize-images"; | ||
| import terser from 'gulp-terser'; | ||
| import sourcemaps from 'gulp-sourcemaps'; |
There was a problem hiding this comment.
единообразие кавычек + давай в алфавитном порядке)
gulpfile.js
Outdated
| @@ -150,8 +153,43 @@ export const sprite = () => src(`${path.img.icons}**/*.svg`) | |||
| .pipe(rename('sprite.svg')) | |||
| .pipe(dest(path.img.save)) | |||
|
|
|||
| const images = () => src(`${path.images.root}/**/*.{png,jpg}`) | |||
| .pipe(dest(path.images.save)) | |||
| export const img = () => src(`${path.img.root}/**/*.{png,jpg}`) | |||
gulpfile.js
Outdated
| .pipe(dest(path.images.save)) | ||
| export const img = () => src(`${path.img.root}/**/*.{png,jpg}`) | ||
| .pipe(plumber()) | ||
| .pipe(sharpOptimizeImages({ |
There was a problem hiding this comment.
давай отдельно вынесем
{
webp: {
quality: 80,
lossless: false,
},
png_to_png: {
quality: 80,
lossless: false,
},
jpg_to_jpg: {
quality: 80,
mozjpeg: true,
},
}There was a problem hiding this comment.
Вынес в отдельную переменную
src/static/js/main.js
Outdated
| @@ -0,0 +1,3 @@ | |||
| import {test} from "./script.js"; | |||
src/templates/base.j2
Outdated
| <link href="{{ url_for('static', filename='pp/styles.css') }}" rel="stylesheet"> | ||
| <script src="{{ url_for('static', filename='pp/script.js') }}"></script> | ||
| <link href="{{ url_for('static', filename='../pp/styles.css') }}" rel="stylesheet"> | ||
| <script src="{{ url_for('static', filename='../pp/script.js') }}"></script> |
There was a problem hiding this comment.
папка pp в статике должна быть, не надо на уровень выше делать
There was a problem hiding this comment.
Исправил пути здесь и в gulpfile.js
src/templates/base.j2
Outdated
| <script src="{{ url_for('static', filename='js/script.min.js') }}"></script> | ||
| <script src="{{ url_for('static', filename='js/main.js') }}" type="module"></script> |
There was a problem hiding this comment.
я бы оставил script.min.js и прикрутил бандлер (сборщик всех модулей в один файл)
для разработки — type=module можно оставить
There was a problem hiding this comment.
Добавил сборщик и сделал его в двух режимах dev и build
# Conflicts: # gulpfile.js # package.json
gulpfile.js
Outdated
| message: 'Error: <%= error.message %>' | ||
| }))) | ||
| .pipe(sourcemaps.init()) | ||
| .pipe(webpack({ |
There was a problem hiding this comment.
я бы не тянул тяжелый вабпак, тем более он жирный бандл делает, можем обсудить что лучше: parcel, rollup, fusebox, esbuild, vite...
There was a problem hiding this comment.
|
Kudos, SonarCloud Quality Gate passed!
|
