[Dooble-Development] SF.net SVN: dooble:[59] trunk/browser

[<tex...@us...>]

Revision: 59
          http://dooble.svn.sourceforge.net/dooble/?rev=59&view=rev
Author:   textfield
Date:     2008-11-11 04:55:08 +0000 (Tue, 11 Nov 2008)

Log Message:
-----------
The main window should no longer expand if the number of tabs exceeds its boundary.
The URL identifies the uniqueness of a history item, not its title.
Allow the user to open a URL without having to provide the "http://" text.

Modified Paths:
--------------
    trunk/browser/Doc/TO-DO
    trunk/browser/Source/dooble.cc
    trunk/browser/UI/mainWindow.ui

Modified: trunk/browser/Doc/TO-DO
===================================================================
--- trunk/browser/Doc/TO-DO	2008-11-08 15:27:52 UTC (rev 58)
+++ trunk/browser/Doc/TO-DO	2008-11-11 04:55:08 UTC (rev 59)
@@ -1,4 +1,4 @@
-1.   Remember main window's geometry.
+1.   Remember the main window's geometry.
 2.   Display the busy cursor when necessary. (DONE)
 3.   Display an informational progress bar. (DONE)
 4.   Allow for closing of tabs. (DONE)
@@ -12,3 +12,5 @@
 12.  Menus.
 13.  The page's "icon" in the location box should be updated whenever
      the user selects a different tab.
+14.  The main window appears to grow without bounds if the user adds too
+     many tabs. This appears to be restricted to OS X. (DONE)

Modified: trunk/browser/Source/dooble.cc
===================================================================
--- trunk/browser/Source/dooble.cc	2008-11-08 15:27:52 UTC (rev 58)
+++ trunk/browser/Source/dooble.cc	2008-11-11 04:55:08 UTC (rev 59)
@@ -199,7 +199,14 @@
 
   if(p)
     {
-      p->load(QUrl(urlText));
+      QString str(urlText.trimmed());
+
+      if(!(str.startsWith("file:///") ||
+	   str.startsWith("ftp://") ||
+	   str.startsWith("http://")))
+	str.prepend("http://");
+
+      p->load(QUrl(str));
       progressBar->setVisible(true);
       ui.stopToolButton->setEnabled(true);
     }
@@ -394,7 +401,8 @@
       QList<QAction *> actions = ui.historyMenu->actions();
 
       for(int i = 0; i < actions.size(); i++)
-	if(actions.at(i)->text() == p->title())
+	if(actions.at(i)->statusTip() ==
+	   p->url().toString(QUrl::StripTrailingSlash))
 	  {
 	    exists = true;
 	    break;

Modified: trunk/browser/UI/mainWindow.ui
===================================================================
--- trunk/browser/UI/mainWindow.ui	2008-11-08 15:27:52 UTC (rev 58)
+++ trunk/browser/UI/mainWindow.ui	2008-11-11 04:55:08 UTC (rev 59)
@@ -146,6 +146,9 @@
        <property name="elideMode" >
         <enum>Qt::ElideRight</enum>
        </property>
+       <property name="usesScrollButtons" >
+        <bool>true</bool>
+       </property>
        <widget class="QWidget" name="tab" >
         <attribute name="title" >
          <string>Tab</string>
@@ -175,7 +178,7 @@
      <x>0</x>
      <y>0</y>
      <width>950</width>
-     <height>26</height>
+     <height>22</height>
     </rect>
    </property>
    <widget class="QMenu" name="menu_File" >


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.