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 607550 - Add the possibility to hide the GtkHTML Edit toolbar
Add the possibility to hide the GtkHTML Edit toolbar
Status: RESOLVED OBSOLETE
Product: evolution
Classification: Applications
Component: Composer
2.30.x (obsolete)
Other Linux
: Normal enhancement
: ---
Assigned To: Tomas Popela
Evolution QA team
evolution[composer]
: 618502 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2010-01-20 13:19 UTC by Matthew Barnes
Modified: 2021-05-19 11:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (4.33 KB, patch)
2010-01-20 13:19 UTC, Matthew Barnes
reviewed Details | Review

Description Matthew Barnes 2010-01-20 13:19:59 UTC
Created attachment 151832 [details] [review]
Proposed patch

When composing a mail, the GtkHTML Edit toolbar is highly annoying for
me ;-) - I write only plain-text mails, and the toolbar only wastes
place on my screen.

The attached patch adds the possibility to hide it, using the same menu
as where From/To/CC/BCC can be hidden (View).

Comments [or commit & push ;-)] appreciated.
Comment 1 Matthew Barnes 2010-01-20 13:20:54 UTC
This was originally posted to evolution-patches@gnome.org by Jan Holesovsky.  I'm just reposting here.
Comment 2 Matthew Barnes 2010-01-20 14:28:56 UTC
I guess I'm not opposed to this, but there's a few things to consider:

1) I'd suggest implementing most of this directly in GtkhtmlEditor (except for the GConf binding), for reasons stated below.

2) The toolbar you're trying to hide is actually intended for plain-text mails.  There's an additional edit toolbar with HTML-specific controls (Format->HTML to see it).  The option should hide both toolbars, with the menu option saying "Edit Toolbars" (plural).

3) Easiest way to do that is to pack both the plain-text and HTML toolbars into a new GtkVBox and have the toggle action control the vbox visibility.  The toolbar widgets get packed in gtkhtml-editor-private.c.

4) When adding the toggle action to gtkhtml-editor-manager.ui, we'll need a new placeholder in the View menu so Evolution can insert its header options in the correct spot.  Would also be good to add a separator between the header options and the toolbar option.  (This will also require changing evolution-composer.ui to use the new placeholder.)

5) Also on the Evolution side, you can bind the GConf key to the toggle action instead of the widget.  I tend to prefer this anyway as it helps keep the relationships clear.
Comment 3 Matthew Barnes 2010-10-13 04:04:35 UTC
*** Bug 618502 has been marked as a duplicate of this bug. ***
Comment 4 André Klapper 2012-09-22 13:38:26 UTC
Wondering if the WebKit rewrite makes this easier...
Comment 5 André Klapper 2021-05-19 11:44:58 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org. 
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org (resources are unfortunately quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/Community/GettingInTouch/BugReportingGuidelines
and create a new enhancement request ticket at
  https://gitlab.gnome.org/GNOME/evolution/-/issues/

Thank you for your understanding and your help.