PATCH: Bug 82463



Removes "new window" from the context menus. This is a Calum approved UI
change.

Rational
-----------
daveB says:

Based on the theory of OO UI design, context menus
should provide menu items that perform some sort of action on the
selected item, in this case the selected window. However new window
doesn't do any "work" on the current directory so it violates this
principle.

Calum says:

I'd go along with Dave's suggestion... and a quick check on Windoze
Explorer confirms it does the same.  "Open in New Window" is fine for
a selected icon(s)' context menu, but when there's nothing selected,
"New Window" is somewhat of out of place IMHO.

Actually I'm not convinced it makes a lot of sense on the desktop
context menu either, as there's no indication of what's going to be in
the new window when it opens.  (Again, Windoze XP doesn't give you
this option either, not that I'm advocating we just copy Windoze for
everything of course...)

Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/nautilus/ChangeLog,v
retrieving revision 1.5501
diff -p -u -r1.5501 ChangeLog
--- ChangeLog	31 Aug 2002 21:59:26 -0000	1.5501
+++ ChangeLog	1 Sep 2002 03:53:05 -0000
@@ -1,3 +1,8 @@
+2002-08-31 David Bordoley <bordoley msu edu>
+
+	* src/nautilus-shell-ui.xml:
+	Removed "New Window" from the context menu. 
+
 2002-08-31  David Bordoley <bordoley msu edu>
 
 	* src/nautilus-shell-ui.xml:
Index: src/nautilus-shell-ui.xml
===================================================================
RCS file: /cvs/gnome/nautilus/src/nautilus-shell-ui.xml,v
retrieving revision 1.99
diff -p -u -r1.99 nautilus-shell-ui.xml
--- src/nautilus-shell-ui.xml	31 Aug 2002 21:59:31 -0000	1.99
+++ src/nautilus-shell-ui.xml	1 Sep 2002 03:53:11 -0000
@@ -354,8 +354,6 @@
 	<popup name="background" tearoff="0">
 		<placeholder name="Before Zoom Items" delimit="none">
 			<placeholder name="New Items" delimit="none">
-				<menuitem name="New Window" 
-				 verb="New Window"/>
 			</placeholder>
 		</placeholder>
 		<placeholder name="Zoom Items" delimit="top">


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