From 344e15452a64ef9dc43a66a7e0085aef0eb7c175 Mon Sep 17 00:00:00 2001
From: Petr Vobornik <pvoborni@redhat.com>
Date: Thu, 28 Mar 2013 18:25:13 +0100
Subject: Replace id usage in App widget by class

https://fedorahosted.org/freeipa/ticket/3236
---
 install/ui/ipa.css                    | 28 ++++++++++++++--------------
 install/ui/src/freeipa/widgets/App.js | 35 ++++++++++++++++++-----------------
 2 files changed, 32 insertions(+), 31 deletions(-)

diff --git a/install/ui/ipa.css b/install/ui/ipa.css
index 0fca9ce0..5c141bc1 100644
--- a/install/ui/ipa.css
+++ b/install/ui/ipa.css
@@ -64,7 +64,7 @@ body {
 
 /* ---- Container ---- */
 
-#container {
+.app-container {
     position: absolute;
     top: 0;
     left: 0;
@@ -76,7 +76,7 @@ body {
 
 /* ---- Background ---- */
 
-#background-header {
+.background-header {
     position: absolute;
     top: 0;
     left: 0;
@@ -85,7 +85,7 @@ body {
     background: url(images/top-background.png) #0C3B00;
 }
 
-#background-navigation {
+.background-navigation {
     position: absolute;
     top: 34px;
     left: 0;
@@ -94,7 +94,7 @@ body {
     background: url(images/mainnav-background.png);
 }
 
-#background-left {
+.background-left {
     position: absolute;
     top: 106px;
     left: 0;
@@ -104,7 +104,7 @@ body {
     background-repeat: no-repeat;
 }
 
-#background-center {
+.background-center {
     position: absolute;
     top: 106px;
     left: 10px;
@@ -114,7 +114,7 @@ body {
     background-repeat: repeat-x;
 }
 
-#background-right {
+.background-right {
     position: absolute;
     top: 106px;
     right: 0;
@@ -207,7 +207,7 @@ body {
 }
 
 /*  ----  Header  ----  */
-#header {
+.header {
     position: absolute;
     top: 0;
     left: 6px;
@@ -216,25 +216,25 @@ body {
     background: transparent;
 }
 
-#header a {
+.header a {
     text-decoration: none;
 }
 
-#header a:link {
+.header a:link {
     text-decoration: none;
     color: white;
 }
 
-#header a:visited {
+.header a:visited {
     text-decoration: none;
     color: white;
 }
 
-#header span.header-logo {
+.header span.header-logo {
     padding-left: 2em;
 }
 
-#header span.header-logo a img {
+.header span.header-logo a img {
     border: 0;
 }
 
@@ -433,7 +433,7 @@ body {
 }
 
 /* ---- Content ---- */
-#content {
+.content {
     position: absolute;
     top: 151px;
     left: 6px;
@@ -441,7 +441,7 @@ body {
     bottom: 10px;
 }
 
-#content.nav-space-3 {
+.content.nav-space-3 {
     top: 175px;
 }
 
diff --git a/install/ui/src/freeipa/widgets/App.js b/install/ui/src/freeipa/widgets/App.js
index 662d0ee0..abec9754 100644
--- a/install/ui/src/freeipa/widgets/App.js
+++ b/install/ui/src/freeipa/widgets/App.js
@@ -99,7 +99,8 @@ define(['dojo/_base/declare',
 
 
             this.domNode = construct.create('div', {
-                id: this.app_id
+                id: this.app_id,
+                'class': 'app-container'
             });
 
             if (this.container_node) {
@@ -113,27 +114,27 @@ define(['dojo/_base/declare',
             construct.place(this.menu_node, this.domNode);
 
             this.content_node = construct.create('div', {
-                id: 'content'
+                'class': 'content'
             }, this.domNode);
         },
 
         _render_background: function() {
             var inner_html = ''+
-                '<div id="background-header"></div>'+
-                '<div id="background-navigation"></div>'+
-                '<div id="background-left"></div>'+
-                '<div id="background-center"></div>'+
-                '<div id="background-right"></div>';
+                '<div class="background-header"></div>'+
+                '<div class="background-navigation"></div>'+
+                '<div class="background-left"></div>'+
+                '<div class="background-center"></div>'+
+                '<div class="background-right"></div>';
 
             this.background_node = construct.create('div', {
-                id: 'background',
+                'class': 'background',
                 innerHTML: inner_html
             }, this.domNode);
         },
 
         _render_header: function() {
             this.header_node = construct.create('div', {
-                id: 'header'
+                'class': 'header'
             }, this.domNode);
 
             // logo
@@ -147,13 +148,13 @@ define(['dojo/_base/declare',
             construct.place(''+
             '<span class="header-right">'+
                 '<span class="header-passwordexpires"></span>'+
-                '<span id="loggedinas" class="header-loggedinas" style="visibility:hidden;">'+
-                    '<a href="#"><span id="login_header">Logged in as</span>: <span class="login"></span></a>'+
+                '<span class="loggedinas header-loggedinas" style="visibility:hidden;">'+
+                    '<a href="#"><span class="login_header">Logged in as</span>: <span class="login"></span></a>'+
                 '</span>'+
                 '<span class="header-loggedinas" style="visibility:hidden;">'+
-                    ' | <a href="#logout" id="logout">Logout</a>'+
+                    ' | <a href="#logout" class="logout">Logout</a>'+
                 '</span>'+
-                '<span id="header-network-activity-indicator" class="network-activity-indicator">'+
+                '<span class="header-network-activity-indicator network-activity-indicator">'+
                     '<img src="images/spinner-header.gif" />'+
                 '</span>'+
             '</span>', this.header_node);
@@ -161,10 +162,10 @@ define(['dojo/_base/declare',
 
             this.password_expires_node = query('.header-passwordexpires', this.header_node)[0];
             this.logged_nodes = query('.header-loggedinas', this.header_node);
-            this.logged_header_node = dom.byId('login_header');// maybe ditch the id?
-            this.logged_user_node = query('#loggedinas .login', this.header_node)[0];
-            this.logged_user_link_node = query('#loggedinas a', this.header_node)[0];
-            this.logout_link_node = dom.byId('logout');
+            this.logged_header_node = query('.login_header')[0];
+            this.logged_user_node = query('.loggedinas .login', this.header_node)[0];
+            this.logged_user_link_node = query('.loggedinas a', this.header_node)[0];
+            this.logout_link_node = query('.logout')[0];
 
             on(this.logout_link_node, 'click', lang.hitch(this,this.on_logout));
             on(this.logged_user_link_node, 'click', lang.hitch(this,this.on_profile));
-- 
cgit