Add "disableSecurity" option to InfoBox
- Dominant language
- JavaScript
- Stars
- 15.7k
- Forks
- 3.9k
- Avg merge
- 4d 6h
- Merged PRs (30d)
- 34
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.
Contributor guide
Research direction
Start with InfoBox.js to trace the current title binding and iframe sandboxing option, then read the linked Cesium developer thread for the intended behavior. Done means the InfoBox exposes the requested top-level disableSecurity option and supports HTML title text without breaking the existing behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100