Re: [evolution-patches] calendar dialog HIG tweaks



Yes, good catch.

Updated to include this. Also changed the <span weight="bold"> tag to <b> to be consistent with the event page.

Thanks,
Jon

Gary Ekker wrote:
	  <property name="padding">0</property>
-	  <property name="expand">False</property>
+	  <property name="expand">True</property>
	  <property name="fill">True</property>

Shouldn't expand and fill both be set to False here so that the behavior
in the task editor is consistent with that of the event editor?

There seems to be a big block of padding at the bottom of the editor if
these are set to True. Just a thought, since I have no real say in the
matter :) This has been bugging me for a while now, so I'm grateful for
the patch.
? gui/goto-dialog.gladep
? gui/dialogs/event-page.gladep
? gui/dialogs/task-page.gladep
Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution/calendar/ChangeLog,v
retrieving revision 1.2444
diff -p -u -r1.2444 ChangeLog
--- ChangeLog	13 Jul 2004 11:07:19 -0000	1.2444
+++ ChangeLog	13 Jul 2004 21:41:42 -0000
@@ -1,3 +1,11 @@
+2004-07-13  William Jon McCann  <mccann jhu edu>
+
+	* gui/dialogs/task-page.glade:
+	* gui/dialogs/event-page.glade: Make Description box expand
+	with window.
+	* gui/goto-dialog.glade:
+	* gui/dialogs/new-calendar.glade: Remove separator per HIG.
+
 2004-07-08  Radek Doulik  <rodo ximian com>
 
 	* gui/e-cal-model.c (e_cal_model_date_value_to_string): as below
Index: gui/goto-dialog.glade
===================================================================
RCS file: /cvs/gnome/evolution/calendar/gui/goto-dialog.glade,v
retrieving revision 1.13
diff -p -u -r1.13 goto-dialog.glade
--- gui/goto-dialog.glade	29 Mar 2004 15:24:00 -0000	1.13
+++ gui/goto-dialog.glade	13 Jul 2004 21:41:42 -0000
@@ -10,7 +10,7 @@
   <property name="modal">False</property>
   <property name="resizable">True</property>
   <property name="destroy_with_parent">False</property>
-  <property name="has_separator">True</property>
+  <property name="has_separator">False</property>
 
   <child internal-child="vbox">
     <widget class="GtkVBox" id="dialog-vbox1">
Index: gui/dialogs/event-page.glade
===================================================================
RCS file: /cvs/gnome/evolution/calendar/gui/dialogs/event-page.glade,v
retrieving revision 1.20
diff -p -u -r1.20 event-page.glade
--- gui/dialogs/event-page.glade	23 Jun 2004 00:18:02 -0000	1.20
+++ gui/dialogs/event-page.glade	13 Jul 2004 21:41:42 -0000
@@ -173,44 +173,6 @@
 	      </child>
 
 	      <child>
-		<widget class="GtkScrolledWindow" id="scrolledwindow12">
-		  <property name="visible">True</property>
-		  <property name="hscrollbar_policy">GTK_POLICY_NEVER</property>
-		  <property name="vscrollbar_policy">GTK_POLICY_AUTOMATIC</property>
-		  <property name="shadow_type">GTK_SHADOW_IN</property>
-		  <property name="window_placement">GTK_CORNER_TOP_LEFT</property>
-
-		  <child>
-		    <widget class="GtkTextView" id="description">
-		      <property name="visible">True</property>
-		      <property name="can_focus">True</property>
-		      <property name="editable">True</property>
-		      <property name="overwrite">False</property>
-		      <property name="accepts_tab">True</property>
-		      <property name="justification">GTK_JUSTIFY_LEFT</property>
-		      <property name="wrap_mode">GTK_WRAP_WORD</property>
-		      <property name="cursor_visible">True</property>
-		      <property name="pixels_above_lines">0</property>
-		      <property name="pixels_below_lines">0</property>
-		      <property name="pixels_inside_wrap">0</property>
-		      <property name="left_margin">0</property>
-		      <property name="right_margin">0</property>
-		      <property name="indent">0</property>
-		      <property name="text" translatable="yes"></property>
-		    </widget>
-		  </child>
-		</widget>
-		<packing>
-		  <property name="left_attach">1</property>
-		  <property name="right_attach">2</property>
-		  <property name="top_attach">4</property>
-		  <property name="bottom_attach">5</property>
-		  <property name="x_options">fill</property>
-		  <property name="y_options">fill</property>
-		</packing>
-	      </child>
-
-	      <child>
 		<widget class="GtkLabel" id="label69">
 		  <property name="visible">True</property>
 		  <property name="label" translatable="yes">Classi_fication:</property>
@@ -296,7 +258,7 @@
 		      <property name="yalign">0.5</property>
 		      <property name="xpad">0</property>
 		      <property name="ypad">0</property>
-		  <property name="mnemonic_widget">source</property>
+		      <property name="mnemonic_widget">source</property>
 		    </widget>
 		    <packing>
 		      <property name="padding">0</property>
@@ -371,6 +333,43 @@
 	      </child>
 
 	      <child>
+		<widget class="GtkScrolledWindow" id="scrolledwindow12">
+		  <property name="visible">True</property>
+		  <property name="hscrollbar_policy">GTK_POLICY_NEVER</property>
+		  <property name="vscrollbar_policy">GTK_POLICY_AUTOMATIC</property>
+		  <property name="shadow_type">GTK_SHADOW_IN</property>
+		  <property name="window_placement">GTK_CORNER_TOP_LEFT</property>
+
+		  <child>
+		    <widget class="GtkTextView" id="description">
+		      <property name="visible">True</property>
+		      <property name="can_focus">True</property>
+		      <property name="editable">True</property>
+		      <property name="overwrite">False</property>
+		      <property name="accepts_tab">True</property>
+		      <property name="justification">GTK_JUSTIFY_LEFT</property>
+		      <property name="wrap_mode">GTK_WRAP_WORD</property>
+		      <property name="cursor_visible">True</property>
+		      <property name="pixels_above_lines">0</property>
+		      <property name="pixels_below_lines">0</property>
+		      <property name="pixels_inside_wrap">0</property>
+		      <property name="left_margin">0</property>
+		      <property name="right_margin">0</property>
+		      <property name="indent">0</property>
+		      <property name="text" translatable="yes"></property>
+		    </widget>
+		  </child>
+		</widget>
+		<packing>
+		  <property name="left_attach">1</property>
+		  <property name="right_attach">2</property>
+		  <property name="top_attach">4</property>
+		  <property name="bottom_attach">5</property>
+		  <property name="x_options">fill</property>
+		</packing>
+	      </child>
+
+	      <child>
 		<widget class="GtkLabel" id="label68">
 		  <property name="visible">True</property>
 		  <property name="label" translatable="yes">_Description:</property>
@@ -404,8 +403,8 @@
 	</widget>
 	<packing>
 	  <property name="padding">0</property>
-	  <property name="expand">False</property>
-	  <property name="fill">False</property>
+	  <property name="expand">True</property>
+	  <property name="fill">True</property>
 	</packing>
       </child>
 
Index: gui/dialogs/new-calendar.glade
===================================================================
RCS file: /cvs/gnome/evolution/calendar/gui/dialogs/new-calendar.glade,v
retrieving revision 1.5
diff -p -u -r1.5 new-calendar.glade
--- gui/dialogs/new-calendar.glade	12 Jan 2004 21:19:09 -0000	1.5
+++ gui/dialogs/new-calendar.glade	13 Jul 2004 21:41:42 -0000
@@ -12,7 +12,7 @@
   <property name="modal">False</property>
   <property name="resizable">False</property>
   <property name="destroy_with_parent">False</property>
-  <property name="has_separator">True</property>
+  <property name="has_separator">False</property>
 
   <child internal-child="vbox">
     <widget class="GtkVBox" id="dialog-vbox1">
Index: gui/dialogs/task-page.glade
===================================================================
RCS file: /cvs/gnome/evolution/calendar/gui/dialogs/task-page.glade,v
retrieving revision 1.28
diff -p -u -r1.28 task-page.glade
--- gui/dialogs/task-page.glade	22 Jun 2004 22:35:41 -0000	1.28
+++ gui/dialogs/task-page.glade	13 Jul 2004 21:41:42 -0000
@@ -2,6 +2,7 @@
 <!DOCTYPE glade-interface SYSTEM "http://glade.gnome.org/glade-2.0.dtd";>
 
 <glade-interface>
+<requires lib="gnome"/>
 
 <widget class="GtkWindow" id="task-toplevel">
   <property name="title" translatable="yes">window1</property>
@@ -26,7 +27,7 @@
       <child>
 	<widget class="GtkLabel" id="label21">
 	  <property name="visible">True</property>
-	  <property name="label" translatable="yes">&lt;span weight=&quot;bold&quot;&gt;Basics&lt;/span&gt;</property>
+	  <property name="label" translatable="yes">&lt;b&gt;Basics&lt;/b&gt;</property>
 	  <property name="use_underline">False</property>
 	  <property name="use_markup">True</property>
 	  <property name="justify">GTK_JUSTIFY_LEFT</property>
@@ -287,6 +288,31 @@
 	      </child>
 
 	      <child>
+		<widget class="GtkLabel" id="label18">
+		  <property name="visible">True</property>
+		  <property name="label" translatable="yes">Description:</property>
+		  <property name="use_underline">False</property>
+		  <property name="use_markup">False</property>
+		  <property name="justify">GTK_JUSTIFY_CENTER</property>
+		  <property name="wrap">False</property>
+		  <property name="selectable">False</property>
+		  <property name="xalign">0</property>
+		  <property name="yalign">0</property>
+		  <property name="xpad">0</property>
+		  <property name="ypad">0</property>
+		  <property name="mnemonic_widget">description</property>
+		</widget>
+		<packing>
+		  <property name="left_attach">0</property>
+		  <property name="right_attach">1</property>
+		  <property name="top_attach">3</property>
+		  <property name="bottom_attach">4</property>
+		  <property name="x_options">fill</property>
+		  <property name="y_options">fill</property>
+		</packing>
+	      </child>
+
+	      <child>
 		<widget class="GtkScrolledWindow" id="scrolledwindow1">
 		  <property name="visible">True</property>
 		  <property name="hscrollbar_policy">GTK_POLICY_NEVER</property>
@@ -320,32 +346,6 @@
 		  <property name="top_attach">3</property>
 		  <property name="bottom_attach">4</property>
 		  <property name="x_options">fill</property>
-		  <property name="y_options">fill</property>
-		</packing>
-	      </child>
-
-	      <child>
-		<widget class="GtkLabel" id="label18">
-		  <property name="visible">True</property>
-		  <property name="label" translatable="yes">Description:</property>
-		  <property name="use_underline">False</property>
-		  <property name="use_markup">False</property>
-		  <property name="justify">GTK_JUSTIFY_CENTER</property>
-		  <property name="wrap">False</property>
-		  <property name="selectable">False</property>
-		  <property name="xalign">0</property>
-		  <property name="yalign">0</property>
-		  <property name="xpad">0</property>
-		  <property name="ypad">0</property>
-		  <property name="mnemonic_widget">description</property>
-		</widget>
-		<packing>
-		  <property name="left_attach">0</property>
-		  <property name="right_attach">1</property>
-		  <property name="top_attach">3</property>
-		  <property name="bottom_attach">4</property>
-		  <property name="x_options">fill</property>
-		  <property name="y_options">fill</property>
 		</packing>
 	      </child>
 	    </widget>
@@ -358,7 +358,7 @@
 	</widget>
 	<packing>
 	  <property name="padding">0</property>
-	  <property name="expand">False</property>
+	  <property name="expand">True</property>
 	  <property name="fill">True</property>
 	</packing>
       </child>
@@ -366,7 +366,7 @@
       <child>
 	<widget class="GtkLabel" id="label24">
 	  <property name="visible">True</property>
-	  <property name="label" translatable="yes">&lt;span weight=&quot;bold&quot;&gt;Date and Time&lt;/span&gt;</property>
+	  <property name="label" translatable="yes">&lt;b&gt;Date and Time&lt;/b&gt;</property>
 	  <property name="use_underline">False</property>
 	  <property name="use_markup">True</property>
 	  <property name="justify">GTK_JUSTIFY_LEFT</property>
@@ -512,7 +512,7 @@
 		  <property name="top_attach">0</property>
 		  <property name="bottom_attach">1</property>
 		  <property name="x_options">fill</property>
-		  <property name="y_options">fill</property>
+		  <property name="y_options"></property>
 		</packing>
 	      </child>
 
@@ -558,7 +558,7 @@
 		  <property name="top_attach">1</property>
 		  <property name="bottom_attach">2</property>
 		  <property name="x_options">fill</property>
-		  <property name="y_options">fill</property>
+		  <property name="y_options"></property>
 		</packing>
 	      </child>
 	    </widget>
@@ -571,8 +571,8 @@
 	</widget>
 	<packing>
 	  <property name="padding">0</property>
-	  <property name="expand">True</property>
-	  <property name="fill">True</property>
+	  <property name="expand">False</property>
+	  <property name="fill">False</property>
 	</packing>
       </child>
     </widget>


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]