{"comment":{"url":"https://api.github.com/repos/concord-consortium/lab/comments/12535882","html_url":"https://github.com/concord-consortium/lab/commit/09a87595ad58a1b21f11d4f77f294aa4599113a2#commitcomment-12535882","id":12535882,"user":{"login":"pjanik","id":767857,"avatar_url":"https://avatars.githubusercontent.com/u/767857?v=3","gravatar_id":"","url":"https://api.github.com/users/pjanik","html_url":"https://github.com/pjanik","followers_url":"https://api.github.com/users/pjanik/followers","following_url":"https://api.github.com/users/pjanik/following{/other_user}","gists_url":"https://api.github.com/users/pjanik/gists{/gist_id}","starred_url":"https://api.github.com/users/pjanik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pjanik/subscriptions","organizations_url":"https://api.github.com/users/pjanik/orgs","repos_url":"https://api.github.com/users/pjanik/repos","events_url":"https://api.github.com/users/pjanik/events{/privacy}","received_events_url":"https://api.github.com/users/pjanik/received_events","type":"User","site_admin":false},"position":15,"line":56,"path":"src/lab/common/controllers/basic-dialog.js","commit_id":"09a87595ad58a1b21f11d4f77f294aa4599113a2","created_at":"2015-08-05T09:08:55Z","updated_at":"2015-08-05T09:08:55Z","body":"Sorry for that, but argument used to have `$` prefix (`$content`), so I wouldn't expect anyone passing string there.\r\nIs it difficult to fix it? It's super easy to restructure this function too, so it still works with strings. Just:\r\n```javascript\r\nvar $content = $('<div>').append(content);\r\n```\r\ninstead of:\r\n```javascript\r\nvar $content = $(content);\r\n```\r\nI'm fine with both approaches, up to you, as I'm not familiar with sensor use case."}}