Re: [Freeipa-devel] [Patch] 0001-Remove-search-field-on-group-button.patch

2010-08-09 Thread Rob Crittenden

Adam Young wrote:
In reusing the search functionality for the group listing, we don't want 
the search field or button.  THis patch hides it, while also performing 
some initialization/cleanup for the Enroll button


ack

rob

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [Patch] 0001-Remove-search-field-on-group-button.patch

2010-08-09 Thread Adam Young

On 08/09/2010 01:27 PM, Rob Crittenden wrote:

Adam Young wrote:
In reusing the search functionality for the group listing, we don't 
want the search field or button.  THis patch hides it, while also 
performing some initialization/cleanup for the Enroll button


ack

rob

pushed to master

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


[Freeipa-devel] [Patch] 0001-Remove-search-field-on-group-button.patch

2010-08-06 Thread Adam Young
In reusing the search functionality for the group listing, we don't want 
the search field or button.  THis patch hides it, while also performing 
some initialization/cleanup for the Enroll button
From e9c411d6874bf5d1b4a16cad58bcdb97bde21601 Mon Sep 17 00:00:00 2001
From: Adam Young ayo...@redhat.com
Date: Fri, 6 Aug 2010 15:09:29 -0400
Subject: [PATCH 1/2] Remove serach field on group button

Hide the search bar when showing the groups listed for a user, and resotre it 
when dsoing other searches.

The enroll button is added only on the groups page, and removed along with 
anything else in the searchButtons div when a new search is started.
---
 install/static/index.xhtml |4 ++--
 install/static/search.js   |3 +++
 install/static/user.js |8 +++-
 3 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/install/static/index.xhtml b/install/static/index.xhtml
index a33b083..b22a5ba 100644
--- a/install/static/index.xhtml
+++ b/install/static/index.xhtml
@@ -49,8 +49,8 @@
 /div
 
 div id=search  style=visibility:hidden
-  div class=searchControls 
-   span class=filter  
+  div id=searchControls  class=searchControls 
+   span id=filter class=filter  
  input id=queryFilter type=text/
  input id=query type=submit value=find /
  input id=new type=submit value=new /
diff --git a/install/static/search.js b/install/static/search.js
index 6b0be6d..de11f2e 100644
--- a/install/static/search.js
+++ b/install/static/search.js
@@ -15,6 +15,9 @@ function showSearch(){
 $('#content').css(visibility,hidden);
 $('#search').css(display,block);
 $('#content').css(display,none);
+$(#filter).css(display,block);
+$(#searchButtons).html();
+
 
 }
 
diff --git a/install/static/user.js b/install/static/user.js
index f91fc01..53a8435 100644
--- a/install/static/user.js
+++ b/install/static/user.js
@@ -66,7 +66,7 @@ function renderUserLinks(current, cell){
 }).appendTo(cell);
 
 $(a/,{
-   href: #tab=userfacet=detailspkey=+current.uid,
+   href: #?tab=userfacet=grouppkey=+current.uid,
click:setupUserGroupMembership,
html: [G]
 }).appendTo(cell);
@@ -243,16 +243,14 @@ function populateUserEnrollments(userData){
 
 function setupUserGroupMembership(){
 
-$(#searchButtons).html();
-
+showSearch();
+$(#filter).css(display,none);
 $(input/,{
type:  'button',
value: 'enroll',
click: setupUserGroupEnrollmentSearch
 }).appendTo(#searchButtons);
 
-
-showSearch();
 var columnHeaders  = document.createElement(tr);
 for (var i =0 ; i != groupMembershipColumns.length ;i++){
var th = document.createElement(th);
-- 
1.7.1

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel