From 525b182259442ef2f29349b8bc6ee46c0d880b23 Mon Sep 17 00:00:00 2001
From: pekdon <pekdon>
Date: Sun, 6 Jan 2002 11:34:29 +0000
Subject: Starting to use member functions of FluxboxWindow instead of
 accessing things directly

---
 src/Tab.cc | 76 +++++++++++++++++++++++++++++++-------------------------------
 1 file changed, 38 insertions(+), 38 deletions(-)

diff --git a/src/Tab.cc b/src/Tab.cc
index 176946d..17d9922 100644
--- a/src/Tab.cc
+++ b/src/Tab.cc
@@ -262,7 +262,7 @@ void Tab::stick() {
 			win->blackbox_attrib.flags ^= BaseDisplay::ATTRIB_OMNIPRESENT;
 			win->blackbox_attrib.attrib ^= BaseDisplay::ATTRIB_OMNIPRESENT;
 			win->stuck = false;
-			if (!win->iconic)
+			if (!win->isIconic())
       	win->getScreen()->reassociateWindow(win, -1, true);
 				    
 		} else {
@@ -285,8 +285,8 @@ void Tab::resize() {
 	//now move and resize the windows in the list
 	for (; first != 0; first = first->m_next) {
 		if (first!=this) {
-			first->m_win->configure(m_win->frame.x, m_win->frame.y,
-								m_win->frame.width, m_win->frame.height);
+			first->m_win->configure(m_win->getXFrame(), m_win->getYFrame(),
+								m_win->getWidth(), m_win->getHeight());
 		}
 	}
 
@@ -326,7 +326,7 @@ void Tab::shade() {
 void Tab::draw(bool pressed) {	
 	unsigned int tabtext_w;
 
-	GC gc = ((m_win->focused) ? m_win->getScreen()->getWindowStyle()->tab.l_text_focus_gc :
+	GC gc = ((m_win->isFocused()) ? m_win->getScreen()->getWindowStyle()->tab.l_text_focus_gc :
 		   m_win->getScreen()->getWindowStyle()->tab.l_text_unfocus_gc);
 
 	// Different routines for drawing rotated text
@@ -386,23 +386,23 @@ void Tab::setPosition() {
 
 	//Tab placement
 	if (m_win->getScreen()->getTabPlacement() == PTOP) {
-		pos_y = m_win->frame.y - m_size_h; 
+		pos_y = m_win->getYFrame() - m_size_h; 
 
 	} else if (m_win->getScreen()->getTabPlacement() == PBOTTOM ||
 				m_win->isShaded()) { 
 		if (m_win->isShaded())
-			pos_y = m_win->frame.y + m_win->getTitleHeight() +
+			pos_y = m_win->getYFrame() + m_win->getTitleHeight() +
 					m_win->getScreen()->getBorderWidth2x();
 
 		else
-			pos_y = m_win->frame.y + m_win->getHeight() +
+			pos_y = m_win->getYFrame() + m_win->getHeight() +
 					m_win->getScreen()->getBorderWidth2x();
 
 	} else if (m_win->getScreen()->getTabPlacement() == PLEFT) {
-		pos_x = m_win->frame.x - m_size_w;
+		pos_x = m_win->getXFrame() - m_size_w;
 			
 	} else if (m_win->getScreen()->getTabPlacement() == PRIGHT) {
-		pos_x = m_win->frame.x + m_win->frame.width +
+		pos_x = m_win->getXFrame() + m_win->getWidth() +
 				m_win->getScreen()->getBorderWidth2x();	
 	}
 
@@ -413,20 +413,20 @@ void Tab::setPosition() {
 		switch(m_win->getScreen()->getTabAlignment()) {
 			case ARELATIVE:
 			case ALEFT:
-				pos_x = m_win->frame.x; 
+				pos_x = m_win->getXFrame(); 
 				break;
 			case ACENTER:
 				pos_x = calcCenterXPos(); 
 				break;
 			case ARIGHT:
-				pos_x = m_win->frame.x + m_win->frame.width +
+				pos_x = m_win->getXFrame() + m_win->getWidth() +
 						m_win->getScreen()->getBorderWidth2x() - m_size_w;
 				break;
 		}
 	} else { //PLeft |�PRight
 		switch(m_win->getScreen()->getTabAlignment()) {
 			case ALEFT:
-				pos_y = m_win->frame.y - m_size_h + m_win->frame.height +
+				pos_y = m_win->getYFrame() - m_size_h + m_win->getHeight() +
 						m_win->getScreen()->getBorderWidth2x();
 				break;
 			case ACENTER:
@@ -434,7 +434,7 @@ void Tab::setPosition() {
 				break;
 			case ARELATIVE:
 			case ARIGHT:
-				pos_y = m_win->frame.y;
+				pos_y = m_win->getYFrame();
 				break;
 		}
 	}
@@ -448,8 +448,8 @@ void Tab::setPosition() {
 				
 		//dont move fluxboxwindow if the itterator = this
 		if (first!=this) {
-			first->m_win->configure(m_win->frame.x, m_win->frame.y, 
-					m_win->frame.width, m_win->frame.height);
+			first->m_win->configure(m_win->getXFrame(), m_win->getYFrame(), 
+					m_win->getWidth(), m_win->getHeight());
 		}	
 	}	
 
@@ -517,9 +517,9 @@ void Tab::calcIncrease(void) {
 				m_win->getScreen()->getTabPlacement() == PBOTTOM ||
 				m_win->isShaded()) &&
 				m_win->getScreen()->getTabAlignment() == ARELATIVE) {
-			if (!((m_win->frame.width +
+			if (!((m_win->getWidth() +
 					m_win->getScreen()->getBorderWidth2x()) % tabs) ||
-					i >= ((m_win->frame.width +
+					i >= ((m_win->getWidth() +
 					m_win->getScreen()->getBorderWidth2x()) % tabs)) {
 				first->setTabWidth(inc_x);
 				first->m_inc_x = inc_x;
@@ -529,9 +529,9 @@ void Tab::calcIncrease(void) {
 			}
 			first->m_inc_y = inc_y;
 		} else if (m_win->getScreen()->getTabAlignment() == ARELATIVE) {
-			if (!((m_win->frame.height +
+			if (!((m_win->getHeight() +
 				m_win->getScreen()->getBorderWidth2x()) % tabs) ||
-				i >= ((m_win->frame.height +
+				i >= ((m_win->getHeight() +
 				m_win->getScreen()->getBorderWidth2x()) % tabs)) {
 
 				first->setTabHeight(inc_y);
@@ -623,7 +623,7 @@ void Tab::buttonReleaseEvent(XButtonEvent *be) {
 
 				if (placement == PTOP || placement == PBOTTOM || m_win->isShaded()) {
 					if (placement == PBOTTOM && !m_win->isShaded())
-						dest_y -= m_win->frame.height;
+						dest_y -= m_win->getHeight();
 					else if (placement != PTOP && m_win->isShaded())
 						dest_y -= m_win->getTitleHeight();
 					else // PTOP
@@ -631,29 +631,29 @@ void Tab::buttonReleaseEvent(XButtonEvent *be) {
 
 					switch(m_win->getScreen()->getTabAlignment()) {
 						case ACENTER:
-							dest_x -= (m_win->frame.width / 2) - (m_size_w / 2);
+							dest_x -= (m_win->getWidth() / 2) - (m_size_w / 2);
 							break;
 						case ARIGHT:
-							dest_x -= m_win->frame.width - m_size_w;
+							dest_x -= m_win->getWidth() - m_size_w;
 							break;
 					}
 
 				} else { // PLEFT & PRIGHT
 					if (placement == PRIGHT)
-						dest_x = be->x_root - m_win->frame.width;
+						dest_x = be->x_root - m_win->getWidth();
 
 					switch(m_win->getScreen()->getTabAlignment()) {
 						case ACENTER:
-							dest_y -= (m_win->frame.height / 2) - (m_size_h / 2);
+							dest_y -= (m_win->getHeight() / 2) - (m_size_h / 2);
 							break;
 						case ALEFT:
-							dest_y -= m_win->frame.height - m_size_h;
+							dest_y -= m_win->getHeight() - m_size_h;
 							break;
 					}
 				}
 				//TODO: this causes an calculate increase event, even if we
 				// only are moving a window
-				m_win->configure(dest_x, dest_y, m_win->frame.width, m_win->frame.height);
+				m_win->configure(dest_x, dest_y, m_win->getWidth(), m_win->getHeight());
 			}
 		}
 	} else {
@@ -848,13 +848,13 @@ void Tab::insert(Tab *tab) {
 			// if the window we are grouping to, we need to shade the tab window
 			// _after_ reconfigure
 			if(m_win->isShaded()) {
-					tab->m_win->configure(m_win->frame.x, m_win->frame.y,
-							m_win->frame.width, m_win->frame.height);
+					tab->m_win->configure(m_win->getXFrame(), m_win->getYFrame(),
+							m_win->getWidth(), m_win->getHeight());
 					tab->m_win->shade();
 			} else {
 					tab->m_win->shade(); // switch to correct shade state
-					tab->m_win->configure(m_win->frame.x, m_win->frame.y,
-						m_win->frame.width, m_win->frame.height);
+					tab->m_win->configure(m_win->getXFrame(), m_win->getYFrame(),
+						m_win->getWidth(), m_win->getHeight());
 			}
 
 			tab->m_stoptabs = false;
@@ -862,11 +862,11 @@ void Tab::insert(Tab *tab) {
 		// both window have the same shaded state and have different sizes,
 		// checking this so that I'll only do shade on windows if configure did
 		// anything.
-		} else if ((m_win->frame.width != tab->m_win->frame.width) ||
-				(m_win->frame.height != tab->m_win->frame.height)) {
+		} else if ((m_win->getWidth() != tab->m_win->getWidth()) ||
+				(m_win->getHeight() != tab->m_win->getHeight())) {
 
-			tab->m_win->configure(m_win->frame.x, m_win->frame.y,
-				m_win->frame.width, m_win->frame.height);
+			tab->m_win->configure(m_win->getXFrame(), m_win->getYFrame(),
+				m_win->getWidth(), m_win->getHeight());
 
 			// need to shade the tab window as configure will mess it up
 			if (m_win->isShaded())
@@ -982,7 +982,7 @@ unsigned int Tab::calcRelativeWidth() {
 	//calculate num objs in list (extract this to a function?)
 	for (Tab *first=getFirst(this); first!=0; first=first->m_next, num++);	
 
-	return ((m_win->frame.width + m_win->getScreen()->getBorderWidth2x())/num);
+	return ((m_win->getWidth() + m_win->getScreen()->getBorderWidth2x())/num);
 }
 
 //------------- calcRelativeHeight -------
@@ -994,7 +994,7 @@ unsigned int Tab::calcRelativeHeight() {
 	//calculate num objs in list (extract this to a function?)
 	for (Tab *first=getFirst(this); first!=0; first=first->m_next, num++);	
 
-	return ((m_win->frame.height + m_win->getScreen()->getBorderWidth2x())/num);
+	return ((m_win->getHeight() + m_win->getScreen()->getBorderWidth2x())/num);
 }
 
 //------------- calcCenterXPos -----------
@@ -1006,7 +1006,7 @@ unsigned int Tab::calcCenterXPos() {
 	//calculate num objs in list (extract this to a function?)
 	for (Tab *first=getFirst(this); first!=0; first=first->m_next, num++);	
 
-	return (m_win->frame.x + ((m_win->frame.width - (m_size_w * num)) / 2));
+	return (m_win->getXFrame() + ((m_win->getWidth() - (m_size_w * num)) / 2));
 }
 
 //------------- calcCenterYPos -----------
@@ -1018,7 +1018,7 @@ unsigned int Tab::calcCenterYPos() {
 	//calculate num objs in list (extract this to a function?)
 	for (Tab *first=getFirst(this); first!=0; first=first->m_next, num++);	
 
-	return (m_win->frame.y + ((m_win->frame.height - (m_size_h * num)) / 2));
+	return (m_win->getYFrame() + ((m_win->getHeight() - (m_size_h * num)) / 2));
 }
 
 
-- 
cgit v0.11.2