After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 84463 - Use infobar for scripts directory instead of dialog
Use infobar for scripts directory instead of dialog
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Scripts facilities
2.11.x
Other Linux
: Normal minor
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
: 312400 680321 (view as bug list)
Depends on: 329906
Blocks:
 
 
Reported: 2002-06-07 07:38 UTC by Henri Mannerberg
Modified: 2012-09-19 18:19 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
Add a clue bar to special locations like Templates and scripts (14.60 KB, patch)
2012-08-14 02:42 UTC, William Jon McCann
needs-work Details | Review
Don't show a dialog every time the scripts dir is opened (1.87 KB, patch)
2012-08-14 02:42 UTC, William Jon McCann
committed Details | Review
Add a clue bar to special locations like Templates and scripts (14.15 KB, patch)
2012-08-14 13:44 UTC, William Jon McCann
committed Details | Review

Description Henri Mannerberg 2002-06-07 07:38:35 UTC
Hey,

  There is no way to "disable" the "About Scripts"-dialog shown when
opening the scripts folder. It's very annoying to see it every time - there
should be a "don't show this again" checkbox or something.
Comment 1 Christian Kellner 2003-03-22 15:01:11 UTC
This is still true for nautilus version 2.2.2 (Debian/unstable).
I am changing the GNOMVER keyword to "GNOMEVER2.2".
Comment 2 Sebastien Bacher 2003-11-25 11:33:57 UTC
Still true with Nautilus 2.4.1 =>  GNOMEVER2.4
Comment 3 Sebastien Bacher 2005-08-14 13:29:30 UTC
*** Bug 312400 has been marked as a duplicate of this bug. ***
Comment 4 Joachim Noreiko 2006-02-04 17:37:50 UTC
See also Bug 329906
Comment 5 William Jon McCann 2012-07-20 15:44:18 UTC
I think we should just use an infobar for this directory that allows going to the documentation.
Comment 6 William Jon McCann 2012-08-14 02:42:18 UTC
Created attachment 221088 [details] [review]
Add a clue bar to special locations like Templates and scripts
Comment 7 William Jon McCann 2012-08-14 02:42:21 UTC
Created attachment 221089 [details] [review]
Don't show a dialog every time the scripts dir is opened

This should be moved to the documentation.
Comment 8 Cosimo Cecchi 2012-08-14 09:13:46 UTC
Review of attachment 221088 [details] [review]:

Thanks, looks mostly good.

::: src/nautilus-special-location-bar.c
@@ +72,3 @@
+	if (message == NULL) {
+		gtk_widget_destroy (GTK_WIDGET (bar));
+		return;

I think you could remove this code, as you g_assert() in the no-message case of get_message_for_special_location()

@@ +124,3 @@
+nautilus_special_location_bar_finalize (GObject *object)
+{
+	G_OBJECT_CLASS (nautilus_special_location_bar_parent_class)->finalize (object);

This doesn't seem to be useful

::: src/nautilus-special-location-enum-types.h.in
@@ +1,3 @@
+/*** BEGIN file-header ***/
+#ifndef NAUTILUS_SPECIAL_LOCATION_ENUM_TYPES_H
+#define NAUTILUS_SPECIAL_LOCATION_ENUM_TYPES_H

I'd rather have a general nautilus-enum-types shared for all the objects that need one, rather than a special one just for NautilusSpecialLocationBar. I also think it's cleaner to just generate the boilerplate completely using automake...[1] has an example for how to do this.

[1] http://git.gnome.org/browse/empathy/tree/libempathy/Makefile.am#n179

::: src/nautilus-view.c
@@ +7635,3 @@
 				      view);
 
 	/* Translators: %s is a directory */

This comment isn't needed anymore.
Comment 9 Cosimo Cecchi 2012-08-14 09:16:04 UTC
Review of attachment 221089 [details] [review]:

Looks good, but we should at least file a bug against gnome-user-docs requesting this text to be added to the guide.
Comment 10 William Jon McCann 2012-08-14 13:44:51 UTC
Created attachment 221139 [details] [review]
Add a clue bar to special locations like Templates and scripts
Comment 11 Cosimo Cecchi 2012-08-14 13:54:19 UTC
Review of attachment 221139 [details] [review]:

Looks good!
Comment 12 Cosimo Cecchi 2012-09-19 18:19:42 UTC
*** Bug 680321 has been marked as a duplicate of this bug. ***