css ошибки в apostrophe-ui, лучший подход к их решению?

Я вел свой блог через валидатор CSS и обнаружил, что некоторые шрифты, которые выходят из коробки, не определены правильно, а точнее, это в апострофе / lib / modules / apostrophe-ui / public / css / global / fonts .меньше

@apos-ui-font-path: '/modules/apostrophe-ui/fonts/';

.apos-add-font('roboto', @apos-ui-font-path + 'roboto-regular-webfont');
.apos-add-font('roboto', @apos-ui-font-path + 'roboto-light-webfont', normal, 300);
.apos-add-font('roboto', @apos-ui-font-path + 'roboto-bold-webfont', normal, 500);

.apos-add-font('karla', @apos-ui-font-path + 'karla-regular-webfont');
.apos-add-font('karla', @apos-ui-font-path + 'karla-bold-webfont', normal, 700);

использование переменных добавляло пробел в путь, кроме того, в скомпилированном / минифицированном файле css было несколько лишних символов. Как и в следующей строке:

src: url(''/modules/apostrophe-ui/fonts/' 'roboto-regular-webfont'.eot');

Чтобы исправить это, мне пришлось прекратить использовать переменную пути, примерно так:

.apos-add-font('roboto', '/modules/apostrophe-ui/fonts/roboto-regular-webfont');
.apos-add-font('roboto', '/modules/apostrophe-ui/fonts/roboto-light-webfont', normal, 300);
.apos-add-font('roboto', '/modules/apostrophe-ui/fonts/roboto-bold-webfont', normal, 500);

.apos-add-font('karla', '/modules/apostrophe-ui/fonts/karla-regular-webfont');
.apos-add-font('karla', '/modules/apostrophe-ui/fonts/karla-bold-webfont', normal, 700);

Кроме того, перемещение папки apostrophe-ui в мою собственную папку lib привело к дублированию css, поэтому я внес изменения в исходную папку в node_modules, но это означает, что все изменения будут перезаписаны в следующем обновлении npm. Можно ли удалить модуль из node_modules, чтобы избежать дублирования кода css?


person DavePlusPlus    schedule 08.08.2018    source источник


Ответы (1)


Привет, спасибо, что подняли этот вопрос. Похоже, что правильный способ объединения строковой переменной с другой строкой выглядит так: https://github.com/apostrophecms/apostrophe/pull/1604/files

Недавно сгенерированный CSS передает (эту часть) валидатору :)

Это должно быть частью следующего выпуска

person Stuart Romanek    schedule 19.08.2018
comment
Большое спасибо за это, я забыл упомянуть одну вещь: жирная версия Roboto, которая поставляется с Apostrophe, называется не roboto-bold-webfont, а roboto-medium-webfont, поэтому это приводит к ошибке 404. - person DavePlusPlus; 19.08.2018