Added kotlin runnable samples #315
Conversation
Stepic/WebStepViewController.swift
Outdated
| } | ||
|
|
||
| let scriptsString = "\(Scripts.localTexScript)\(Scripts.clickableImagesScript)" | ||
| let scriptsString = "\(Scripts.localTexScript)\(Scripts.clickableImagesScript)\(Scripts.kotlinRunnableSamples)" |
There was a problem hiding this comment.
И этот скрипт будет подгружаться в каждом степе? К тому же он удаленный.
There was a problem hiding this comment.
Не уверен, что там не кешируется ничего, но поправлю, да
There was a problem hiding this comment.
Даже если это локальный скрипт, то он может быть тяжелым довольно. А если учесть, что у нас в обычных степах (в адаптивных ок) нет никакого лоадера (просто не появляется текст, пока webview не догрузила) – это плохо.
| } | ||
|
|
||
| let scriptsString = "\(Scripts.localTexScript)\(Scripts.clickableImagesScript)\(Scripts.kotlinRunnableSamples)" | ||
| var scriptsString = "\(Scripts.localTexScript)\(Scripts.clickableImagesScript)" |
There was a problem hiding this comment.
Ниже замечательный вызов buildHTMLStringWith, в которой похожие вещи делаются для аудио контрола. Давай туда это помещать.
There was a problem hiding this comment.
Да, согласен.
Я лучше на следующей неделе отдельным PR-ом перепишу HTMLBuilder, добавив нормальный css туда
|
А как это потестить можно? |
Задача: #APPS-1952
Коротко для Release Notes, в формате «Сделали/Добавили/Исправили N»:
Описание:
Добавил скрипт для парсинга Kotlin runnable samples