Skip to content

Автоматически добавляет шапку и футер в демки при открытии вне iframe - #1355

Open
solarrust wants to merge 1 commit into
mainfrom
feat/demo-frame
Open

Автоматически добавляет шапку и футер в демки при открытии вне iframe#1355
solarrust wants to merge 1 commit into
mainfrom
feat/demo-frame

Conversation

@solarrust

Copy link
Copy Markdown
Member

Demo-файлы уже открываются отдельно от статьи («Открыть демо в новой вкладке»). Теперь при таком открытии автоматически показываются лого и футер со ссылкой на статью и автора — без разметки в самой демке и без ?embed=1.

Изменения:

  • Шапка (лого) и футер (ссылка на статью и на автора) теперь подставляются автоматически на этапе сборки (gulpfile.js, src/libs/demo-frame/), без правок в самой демке
  • Видимость определяется через window.self === window.top, ?embed=1 для этого больше не нужен
  • Цвет акцента в футере берётся из config/category-colors.js по разделу статьи
  • Общие demo-frame.css/demo-frame.js хэшируются и кэш-бастятся тем же механизмом, что и остальные ассеты сайта

Как проверить:

  1. npm run build или npm run preview
  2. Открыть любую демку напрямую, например dist/css/hover/demos/link-hover/index.html через локальный статический сервер (не file://, иначе /styles/... не резолвится) — снизу должен появиться футер со ссылкой на статью и автора, акцент — цвет раздела
  3. Открыть статью с этой демкой в iframe (dist/css/hover/index.html) — внутри iframe шапки и футера быть не должно
  4. npm test — новые тесты в src/libs/__tests__/demo-frame.js

Отдельно: 28 демок (11 статей), где уже были ручные .demo-logo/.demo-meta/?embed=1, временно будут показывать шапку/футер дважды — это отдельная задача по чистке контент-репо, не входит в этот PR

@solarrust
solarrust requested a review from furtivite as a code owner July 29, 2026 23:02
@solarrust solarrust self-assigned this Jul 29, 2026
@github-actions

Copy link
Copy Markdown

Превью контента из опубликовано.

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.

1 participant