CesiumGS/cesium

Add "disableSecurity" option to InfoBox

Open

#5 446 ouverte le 9 juin 2017

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)JavaScript (3 324 forks)batch import
category - widgetsgood first issuetype - enhancement

Métriques du dépôt

Stars
 (11 758 stars)
Métriques de merge PR
 (Merge moyen 15j 12h) (23 PRs mergées en 30 j)

Description

See this thread: https://groups.google.com/d/msg/cesium-dev/Ktn8aPQmOsQ/_Dbd7igkCQAJ

While we expose the ability to disable iframe sandboxing, some use cases need the title text to be HTML as well, which is currently using the knockout text binding instead of HTML. This will also help with the fact that most people aren't familiar with iframe sandboxing and we can provide them a single top-level property for disabling it.

This should be a simple change in InfoBox.js, see the above thread for some more details.

Guide contributeur

Direction de recherche
Examinez InfoBox.js et localisez la liaison knockout text pour le titre. Ajoutez une nouvelle option 'disableSecurity' qui, lorsqu'elle est définie, utilise la liaison HTML au lieu de la liaison text pour le titre. Voir la discussion liée pour plus de détails.
Stack technique
javascript
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptCesium
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80