Клуб API Карт

Ошибка в ymaps-pie-chart-clusterer (ветка с layout)

ivan-zykov
15 февраля 2016, 00:30

Нашёл небольшую ошибку в ветке https://github.com/yandex/ymaps-pie-chart-clusterer/tree/layout. Кластеры не будут видны на Opera 12. А может и ещё где-то. Для проверки достаточно открыть демо в Opera 12. Хоть и старичёк, а многие его до сих пор используют.

Файл PieChartClustererLayout.js, строки 58-60.

element.style['line-height'] = size[1] + 'px';
element.style['background-image'] = 'url(' + iconUrl + ')';
element.style['background-size'] = size[0] + 'px ' + size[1] + 'px';

Не следует так присваивать стили элементам. Правильно так:

element.style.lineHeight = size[1] + 'px';
element.style.backgroundImage = 'url(' + iconUrl + ')';
element.style.backgroundSize = size[0] + 'px ' + size[1] + 'px';

Исправьте, если посчитаете нужным.

6 комментариев
Сделай, пожалуйста, pull-request на гитхабе
наблюдаю такую-же ерунду, когда вывожу из в 1Ске диаграммы в поле HTML...
malv@automacon.ru,
в 1С допустимо использовать только коммерческую версию API. Если с ней возникли сложности, пожалуйста, напишите в тех поддержку, указав Ваш ключ доступа к API.
ymapsapi,
это в курсе. просто перед покупкой хотел проверить, как оно вообще работает.