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 165241 - Sync empty alert with nautius
Sync empty alert with nautius
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: trash applet
git master
Other All
: Normal normal
: 2.10
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-01-25 22:27 UTC by Luca Ferretti
Modified: 2005-02-07 05:44 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
The patch to use the same alert appearance and text for empty trash action (4.05 KB, patch)
2005-01-25 22:30 UTC, Luca Ferretti
none Details | Review
The same patch, but without eel_* stuff (4.05 KB, patch)
2005-01-27 08:39 UTC, Luca Ferretti
none Details | Review

Description Luca Ferretti 2005-01-25 22:27:53 UTC
Well, read the summary and the attached patch
Comment 1 Luca Ferretti 2005-01-25 22:30:05 UTC
Created attachment 36525 [details] [review]
The patch to use the same alert appearance and text for empty trash action
Comment 2 Danielle Madeley 2005-01-26 04:07:19 UTC
This breaks UI freeze. Although does look vaguely correct.

I would personally avoid using any eel functions that we weren't already using
(unless there is a really good reason I haven't thought of).

Request permission with the release-team to break the UI freeze (please CC me)
and we can then proceed.

Tentatively marking G10, however if the release-team deny it, it'll obviously be
bounced back to G12.
Comment 3 Luca Ferretti 2005-01-26 07:40:04 UTC
The eel stuff come from a 'blind' cut-and-paste from nautilus code. Sorry.


I'll provide the right patch shortly.
Comment 4 Luca Ferretti 2005-01-27 08:39:19 UTC
Created attachment 36591 [details] [review]
The same patch, but without eel_* stuff

Please note that this patch silently change "translator_credits" to
"translator-credits"
Comment 5 Danielle Madeley 2005-01-27 12:01:45 UTC
I noticed the translator-credits change, it is correct and consist with other
changes we've been making.

This can be committed pending approval from the release team. Thanks.
Comment 6 Dennis Cranston 2005-01-27 19:16:46 UTC
Titles do not belong on alert dialogs.  You shouldn't remove this line;
otherwise, the title will be "Warning".  

-	gtk_window_set_title (GTK_WINDOW (dialog), "");
Comment 7 Danielle Madeley 2005-02-02 18:12:40 UTC
You have release team approval.
Comment 8 Danielle Madeley 2005-02-07 05:44:16 UTC
2005-02-07  Davyd Madeley  <davyd@madeley.id.au>

        * src/trashapplet.c: Sync Empty Trash dialog with Nautilus. Closes
          #165241. Patch from Luca Ferretti <elle.uca@libero.it>.