Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Оффлайн Дока и места где она обитает #1038

Merged
merged 106 commits into from
Dec 21, 2023

Conversation

igsekor
Copy link
Member

@igsekor igsekor commented Nov 10, 2022

Добавляет базовые возможности кеширования Доки для работы в оффлайн

@igsekor igsekor self-assigned this Nov 10, 2022
@igsekor igsekor added кухня Работа и устройство репозитория улучшение Доработка существующего labels Nov 10, 2022
@solarrust
Copy link
Member

Лить будем?

@solarrust solarrust removed the request for review from pa7lux November 6, 2023 14:57
@solarrust
Copy link
Member

  1. Не загрузилась картинка автора в оффлайне http://localhost:8080/people/ipogorelova/
    Как вариант, если не хотим подгружать ещё и аватарки в кэш, то можно ставить картинку-заглушку, одну на всех

CleanShot 2023-11-07 at 11 12 18@2x

  1. На странице автора не подгрузились стили (?) для роли. Скорее всего связано с тем, что в пиаре просто нет этих стилей пока.

CleanShot 2023-11-07 at 11 14 51@2x

  1. Со страницы автора, Тани, не переходит по ссылке http://localhost:8080/a11y/screenreaders/#na-praktike Хотя в списке ссылка активная. По факту просто белая страница.
  2. В статье http://localhost:8080/a11y/screenreaders/ не закэшировалось видео

CleanShot 2023-11-07 at 11 17 19@2x

  1. Значки активных ссылочек не везде выглядят хорошо. Это к @skorobaeus скорее

CleanShot 2023-11-07 at 11 21 06@2x

  1. НЁХ под ссылками на разделы на главной

CleanShot 2023-11-07 at 11 23 19@2x

  1. Предложение: в разделах и/или на главной (где-то короче) выводить список всех доступных для чтения оффлайн материалов. А то приходится много скроллить в разделах, чтобы найти то, что доступно для чтения.
  2. Не знаю ОК ли это, но не все контрибьюторы статьи закэшировались. http://localhost:8080/css/flexbox-guide/

CleanShot 2023-11-07 at 11 25 36@2x

  1. В демке что-то пошло не так http://localhost:8080/css/flexbox-guide/#flex-direction

CleanShot 2023-11-07 at 11 26 50@2x

@TatianaFokina
Copy link
Member

Ничего сверх того, что нашла Алёна, пока мной не нашлось.

@igsekor
Copy link
Member Author

igsekor commented Dec 19, 2023

  1. Не загрузилась картинка автора в оффлайне http://localhost:8080/people/ipogorelova/
    Как вариант, если не хотим подгружать ещё и аватарки в кэш, то можно ставить картинку-заглушку, одну на всех

Сделал загрузку всех картинок сразу. Надо будет потом обрабатывать автоматически, чтобы оптимизировать загрузку Доки Оффлайн.

  1. На странице автора не подгрузились стили (?) для роли. Скорее всего связано с тем, что в пиаре просто нет этих стилей пока.

Это точно с этим связано.

  1. Со страницы автора, Тани, не переходит по ссылке http://localhost:8080/a11y/screenreaders/#na-praktike Хотя в списке ссылка активная. По факту просто белая страница.

Починил переходы по «якорным» ссылкам.

  1. В статье http://localhost:8080/a11y/screenreaders/ не закэшировалось видео

Теперь кешируется. Кстати, это единственное видео?

  1. Значки активных ссылочек не везде выглядят хорошо. Это к @skorobaeus скорее

Починили вместе со @skorobaeus.

  1. НЁХ под ссылками на разделы на главной

Починил.

  1. Предложение: в разделах и/или на главной (где-то короче) выводить список всех доступных для чтения оффлайн материалов. А то приходится много скроллить в разделах, чтобы найти то, что доступно для чтения.

Думаю, что всё таки не нужно показывать это специально. Может быть на страницах разделов добавить дополнительную сортировку?

  1. Не знаю ОК ли это, но не все контрибьюторы статьи закэшировались. http://localhost:8080/css/flexbox-guide/

Теперь стримы всех участников кешируются сразу.

  1. В демке что-то пошло не так http://localhost:8080/css/flexbox-guide/#flex-direction

Теперь все демки кешируются корректно (насколько я смог протестировать).

Copy link

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

@igsekor igsekor merged commit eda3ce4 into main Dec 21, 2023
4 checks passed
@igsekor igsekor deleted the feature/offline branch December 21, 2023 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
кухня Работа и устройство репозитория улучшение Доработка существующего
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants