Intercom Acquire(Chat application) Integration

Hey @arunkumark, this is great, and I would be happy to include this feature in the module. I've just got a couple of questions about the code first. Also, pardon my ignorance since I've not actually used the Intercom Acquire feature before so sorry if these questions are obvious. Anyway ...

1. +++ b/intercomio.admin.inc
   @@ -100,6 +100,33 @@ function intercomio_admin_form() {
   +    '#title'=>t('Intercom Acquire(Chat application)'),
   

   Can you add an extra space here between Acquire, and (Chat, so that it reads, "Intercom Acquire (Chat application)". I think it'll be easier for people to read that way. Same with the other labels in this form.

2. +++ b/intercomio.admin.inc
   @@ -100,6 +100,33 @@ function intercomio_admin_form() {
   +  $form['intercomio_chat']['intercomio_chat_type']=array(
   +    '#type' => 'radios',
   +    '#title' => t('Acquire Integration type'),
   +    '#description' => t('Specify type of Intercom Acquire.'),
   +    '#options' => array('anonymous' =>t('Leads (logged-out)'), 'authenticated' =>t('Users (logged-in)')),
   +    '#default_value' => variable_get('intercomio_chat_type','anonymous'),
   

   I've not used this Intercom.io feature yet. Is it possible to have both Leads, and Users chat enabled at the same time? Or can you only ever do one or the other?

3. +++ b/intercomio.module
   @@ -117,6 +117,28 @@ function intercomio_page_alter(&$page) {
   +        name:" . $user->name . ", ¶
   +        email:" . $user->mail .", ¶
   

   I wonder if we should escape these values using the check_plain() function? Seems like this might be a good idea in oder to prevent possible XSS or similar issues.