Автоматически добавляет шапку и футер в демки при открытии вне iframe - #1355
Open
solarrust wants to merge 1 commit into
Open
Автоматически добавляет шапку и футер в демки при открытии вне iframe#1355solarrust wants to merge 1 commit into
solarrust wants to merge 1 commit into
Conversation
|
Превью контента из опубликовано. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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хэшируются и кэш-бастятся тем же механизмом, что и остальные ассеты сайтаКак проверить:
npm run buildилиnpm run previewdist/css/hover/demos/link-hover/index.htmlчерез локальный статический сервер (неfile://, иначе/styles/...не резолвится) — снизу должен появиться футер со ссылкой на статью и автора, акцент — цвет разделаdist/css/hover/index.html) — внутри iframe шапки и футера быть не должноnpm test— новые тесты вsrc/libs/__tests__/demo-frame.jsОтдельно: 28 демок (11 статей), где уже были ручные
.demo-logo/.demo-meta/?embed=1, временно будут показывать шапку/футер дважды — это отдельная задача по чистке контент-репо, не входит в этот PR