From 8e515d577ef5a7c37780d138bbd7488fe7ee4b85 Mon Sep 17 00:00:00 2001
From: Coleman Watts <coleman@civicrm.org>
Date: Sat, 6 Sep 2014 13:32:05 -0400
Subject: [PATCH] CRM-15249 - ProfileSelector view cleanup and css fix

---
 css/crm.designer.css                   |  3 ---
 js/jquery/jquery.crmProfileSelector.js | 14 +++++++-------
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/css/crm.designer.css b/css/crm.designer.css
index e5cd2c4ce8..a1d9ff7efe 100644
--- a/css/crm.designer.css
+++ b/css/crm.designer.css
@@ -1,6 +1,3 @@
-.crm-profile-selector {
-    display: none;
-}
 .crm-profile-selector-preview-pane {
     width: 75%;
     min-width: 500px;
diff --git a/js/jquery/jquery.crmProfileSelector.js b/js/jquery/jquery.crmProfileSelector.js
index 7008da85a2..f973335520 100644
--- a/js/jquery/jquery.crmProfileSelector.js
+++ b/js/jquery/jquery.crmProfileSelector.js
@@ -25,9 +25,9 @@
     return this.each(function() {
       // Hide the existing <SELECT> and instead construct a ProfileSelector view.
       // Keep them synchronized.
-      var select = this;
+      var matchingUfGroups,
+        $select = $(this).hide().addClass('rendered');
 
-      var matchingUfGroups;
       if (options.groupTypeFilter) {
         matchingUfGroups = ufGroupCollection.subcollection({
           filter: function(ufGroupModel) {
@@ -39,15 +39,15 @@
       }
 
       var view = new CRM.ProfileSelector.View({
-        ufGroupId: $(select).val(),
+        ufGroupId: $select.val(),
         ufGroupCollection: matchingUfGroups,
         ufEntities: options.entities
       });
       view.on('change:ufGroupId', function() {
-        $(select).val(view.getUfGroupId()).change();
+        $select.val(view.getUfGroupId()).change();
       });
       view.render();
-      $(select).after(view.el);
+      $select.after(view.el);
       setTimeout(function() {
         view.doPreview();
       }, 100);
@@ -55,9 +55,9 @@
   };
 
   $('#crm-container').on('crmLoad', function() {
-    $('.crm-profile-selector:not(.rendered)', this).addClass('rendered').each(function() {
+    $('.crm-profile-selector:not(.rendered)', this).each(function() {
       $(this).crmProfileSelector({
-        groupTypeFilter: $(this).attr('data-group-type'),
+        groupTypeFilter: $(this).data('groupType'),
         entities: $(this).data('entities')
       });
     });
-- 
2.25.1