Fix misspellings

desktop/geary-attach

@@ -15,7 +15,7 @@ Relative paths are also supported."
     exit 1 # so that calling without parameters is counted as a failure
 fi
 
-#we dont do file checking as geary is clever enough
+#we don't do file checking as geary is clever enough
 ARG="mailto:?attachment=$1" #add first file
 shift
 while [ -n "$1" ]; do

src/client/accounts/accounts-editor.vala

@@ -159,7 +159,7 @@ public class Accounts.Editor : Gtk.Dialog {
      * Removes the current pane from the editor, showing the last one.
      */
     internal void pop() {
-        // One can't simply remove old panes fro the GTK stack since
+        // One can't simply remove old panes for the GTK stack since
         // there won't be any transition between them - the old one
         // will simply disappear. So we need to keep old, popped panes
         // around until a new one is pushed on.

src/client/application/application-controller.vala

@@ -1261,7 +1261,7 @@ public class Application.Controller : Geary.BaseObject {
         this.main_window.folder_list.add_folder(folder);
         // Since removing the folder will also remove its children
         // from the folder list, we need to check for any and re-add
-        // them. See isssue #11.
+        // them. See issue #11.
         try {
             foreach (Geary.Folder child in
                      folder.account.list_matching_folders(folder.path)) {
@@ -1909,7 +1909,7 @@ public class Application.Controller : Geary.BaseObject {
     private bool open_uri(string _link) {
         string link = _link;
 
-        // Support web URLs that ommit the protocol.
+        // Support web URLs that omit the protocol.
         if (!link.contains(":"))
             link = "http://" + link;
 
@@ -2949,7 +2949,7 @@ public class Application.Controller : Geary.BaseObject {
         }
 
         // This is going to be either an inline image, or a remote
-        // image, so either treat it as an attachment ot assume we'll
+        // image, so either treat it as an attachment to assume we'll
         // have a valid filename in the URL
         bool handled = false;
         if (url.has_prefix(ClientWebView.CID_URL_PREFIX)) {

src/client/application/geary-application.vala

@@ -352,7 +352,7 @@ public class GearyApplication : Gtk.Application {
     /**
      * Signal that is activated when 'exit' is called, but before the application actually exits.
      *
-     * To cancel an exit, a callback should return GearyApplication.cancel_exit(). To procede with
+     * To cancel an exit, a callback should return GearyApplication.cancel_exit(). To proceed with
      * an exit, a callback should return true.
      */
     public virtual signal bool exiting(bool panicked) {

src/client/components/client-web-view.vala

@@ -502,7 +502,7 @@ public abstract class ClientWebView : WebKit.WebView, Geary.BaseInterface {
      */
     protected inline void register_message_handler(string name,
                                                    JavaScriptMessageHandler handler) {
-        // XXX cant use the delegate directly, see b.g.o Bug
+        // XXX can't use the delegate directly, see b.g.o Bug
         // 604781. However the workaround below creates a circular
         // reference, causing ClientWebView instances to leak. So to
         // work around that we need to record handler ids and

src/client/conversation-list/formatted-conversation-data.vala

@@ -160,7 +160,7 @@ public class FormattedConversationData : Geary.BaseObject {
         return true;
     }
 
-    // Creates an example message (used interally for styling calculations.)
+    // Creates an example message (used internally for styling calculations.)
     public FormattedConversationData.create_example() {
         this.is_unread = false;
         this.is_flagged = false;

src/client/conversation-viewer/conversation-message.vala

@@ -729,7 +729,7 @@ public class ConversationMessage : Gtk.Grid, Geary.BaseInterface {
      * Highlights user search terms in the message view.
      *
      * Highlighting includes both in the message headers, and the
-     * mesage body. returns the number of matching search terms.
+     * message body. returns the number of matching search terms.
      */
     public async uint highlight_search_terms(Gee.Set<string> search_matches,
                                              GLib.Cancellable cancellable)

src/client/notification/in-app-notification.vala

@@ -23,7 +23,7 @@ public class InAppNotification : Gtk.Revealer {
     /**
      * Creates an in-app notification.
      *
-     * @param message The messag that should be displayed.
+     * @param message The message that should be displayed.
      * @param keepalive The amount of seconds that the notification should stay visible.
      */
     public InAppNotification(string message, uint keepalive = DEFAULT_KEEPALIVE) {

src/client/notification/new-messages-indicator.vala

@@ -24,7 +24,7 @@ public abstract class NewMessagesIndicator : Geary.BaseObject {
                                               Configuration config) {
         NewMessagesIndicator? indicator = null;
 
-        // Indicators are ordered from most to least prefered.  If more than one is available,
+        // Indicators are ordered from most to least preferred.  If more than one is available,
         // use the first.
 
 #if HAVE_LIBMESSAGINGMENU

src/engine/api/geary-revokable.vala

@@ -43,7 +43,7 @@ public abstract class Geary.Revokable : BaseObject {
     /**
      * Fired when the {@link Revokable} has been revoked.
      *
-     * {@link valid} will stil be true when this is fired.
+     * {@link valid} will still be true when this is fired.
      */
     public signal void revoked();
 
@@ -52,7 +52,7 @@ public abstract class Geary.Revokable : BaseObject {
      *
      * Some Revokables will offer a new Revokable to allow revoking the committed state.
      *
-     * {@link valid} will stil be true when this is fired.
+     * {@link valid} will still be true when this is fired.
      */
     public signal void committed(Geary.Revokable? commit_revokable);
 

src/engine/app/conversation-monitor/app-reseed-operation.vala

@@ -27,7 +27,7 @@ private class Geary.App.ReseedOperation : ConversationOperation {
                   earliest_id.to_string(), this.monitor.base_folder.to_string());
             // Some conversations have already been loaded, so check
             // from the earliest known right through to the end of the
-            // vector for updated mesages
+            // vector for updated messages
             yield this.monitor.load_by_id_async(
                 earliest_id,
                 int.MAX,

src/engine/imap-db/imap-db-account.vala

@@ -995,7 +995,7 @@ private class Geary.ImapDB.Account : BaseObject {
             //
             // Note that this uses SQLite's "standard" query syntax for MATCH, where AND is implied
             // (and would be treated as search term if included), parentheses are not allowed, and
-            // OR has a higher precendence than AND.  So the above example in standard syntax is:
+            // OR has a higher precedence than AND.  So the above example in standard syntax is:
             //
             // party* OR parti* eventful* OR event*
             StringBuilder builder = new StringBuilder();

src/engine/imap-db/imap-db-database.vala

@@ -447,7 +447,7 @@ private class Geary.ImapDB.Database : Geary.Db.VersionedDatabase {
 
                     stmt.exec();
 
-                    // reuse statment, overwrite invalid_id, fields only
+                    // reuse statement, overwrite invalid_id, fields only
                     stmt.reset(Db.ResetScope.SAVE_BINDINGS);
                 }
 
@@ -583,7 +583,7 @@ private class Geary.ImapDB.Database : Geary.Db.VersionedDatabase {
     /**
      * Rebuilds the database's FTS table index.
      *
-     * This can be used to recover from corrupt indexs, as indicated
+     * This can be used to recover from corrupt indexes, as indicated
      * by fts_integrity_check() returning false.
      */
     public void fts_rebuild() throws Error {
@@ -597,7 +597,7 @@ private class Geary.ImapDB.Database : Geary.Db.VersionedDatabase {
     /**
      * Optimises the database's FTS table index.
      *
-     * This is an expensive call, as much as performing a VACCUM.
+     * This is an expensive call, as much as performing a VACUUM.
      */
     public void fts_optimize() throws Error {
         Db.Statement stmt = prepare("""

src/engine/imap-db/imap-db-folder.vala

@@ -5,7 +5,7 @@
  */
 
 /**
- * ImapDB.Folder provides an interface for retreiving messages from the local store in methods
+ * ImapDB.Folder provides an interface for retrieving messages from the local store in methods
  * that are synonymous with Geary.Folder's interface, but with some differences that deal with
  * how IMAP addresses and organizes email.
  *
@@ -1333,7 +1333,7 @@ private class Geary.ImapDB.Folder : BaseObject, Geary.ReferenceSemantics {
     /**
      * Adds a message to the folder.
      *
-     * Note: does NOT check if message is already associated with thie
+     * Note: does NOT check if message is already associated with this
      * folder.
      */
     private void do_associate_with_folder(Db.Connection cx,

src/engine/imap-db/search/imap-db-search-folder.vala

@@ -281,7 +281,7 @@ private class Geary.ImapDB.SearchFolder : Geary.SearchFolder, Geary.FolderSuppor
                     results.add(yield fetch_email_async(ids[i], required_fields, flags, cancellable));
                 } catch (Error err) {
                     // Don't let missing or incomplete messages stop the list operation, which has
-                    // different symantics from fetch
+                    // different semantics from fetch
                     if (!(err is EngineError.NOT_FOUND) && !(err is EngineError.INCOMPLETE_MESSAGE)) {
                         fetch_err = err;
 

src/engine/imap-engine/imap-engine-generic-account.vala

@@ -515,7 +515,7 @@ private abstract class Geary.ImapEngine.GenericAccount : Geary.Account {
         throws GLib.Error {
         check_open();
 
-        // XXX work out what our public IP adddress is somehow and use
+        // XXX work out what our public IP address is somehow and use
         // that in preference to the sender's domain
         string domain = composed.sender != null
             ? composed.sender.domain

src/engine/imap/api/imap-folder-session.vala

@@ -300,7 +300,7 @@ private class Geary.Imap.FolderSession : Geary.Imap.SessionObject {
 
     // All commands must executed inside the cmd_mutex; returns FETCH or STORE results
     //
-    // FETCH commands can generate a FolderError.RETRY.  State will be updated to accomodate retry,
+    // FETCH commands can generate a FolderError.RETRY.  State will be updated to accommodate retry,
     // but all Commands must be regenerated to ensure new state is reflected in requests.
     private async Gee.Map<Command, StatusResponse>? exec_commands_async(Gee.Collection<Command> cmds,
                                                                         Gee.HashMap<SequenceNumber, FetchedData>? fetch_results,

src/engine/imap/command/imap-message-set.vala

@@ -5,7 +5,7 @@
  */
 
 /**
- * A represenation of an IMAP message range specifier.
+ * A representation of an IMAP message range specifier.
  *
  * A MessageSet can be for {@link SequenceNumber}s (which use positional addressing) or
  * {@link UID}s.

src/engine/imap/parameter/imap-atom-parameter.vala

@@ -11,7 +11,7 @@
  * This class does not check if quoting is required.  Use {@link DataFormat.is_quoting_required}
  * or {@link StringParameter.get_best_for}.
  *
- * See {@link StringParameter} for a note about class heirarchy.  In particular, note that
+ * See {@link StringParameter} for a note about class hierarchy.  In particular, note that
  * [@link Deserializer} will not create this type of {@link Parameter} because it's unable to
  * deduce if a string is an atom or a string from syntax alone.
  *

src/engine/imap/parameter/imap-list-parameter.vala

@@ -155,7 +155,7 @@ public class Geary.Imap.ListParameter : Geary.Imap.Parameter {
             return null;
 
         // Because Deserializer doesn't produce NilParameters, check manually if this Parameter
-        // can legally be NIL according to IMAP grammer.
+        // can legally be NIL according to IMAP grammar.
         StringParameter? stringp = param as StringParameter;
         if (stringp != null && NilParameter.is_nil(stringp))
             return null;
@@ -417,7 +417,7 @@ public class Geary.Imap.ListParameter : Geary.Imap.Parameter {
     }
 
     /**
-     * Returns the replaced Paramater.  Throws ImapError.TYPE_ERROR if no Parameter exists at the
+     * Returns the replaced Parameter.  Throws ImapError.TYPE_ERROR if no Parameter exists at the
      * index.
      */
     public Parameter replace(int index, Parameter parameter) throws ImapError {

src/engine/imap/parameter/imap-string-parameter.vala

@@ -152,7 +152,7 @@ public abstract class Geary.Imap.StringParameter : Geary.Imap.Parameter {
     }
 
     /**
-     * Returns the string lowercased.
+     * Returns the string in lowercase.
      */
     public string as_lower() {
         return Ascii.strdown(ascii);

src/engine/imap/response/imap-response-code-type.vala

@@ -70,7 +70,7 @@ public class Geary.Imap.ResponseCodeType : BaseObject, Gee.Hashable<ResponseCode
         if (DataFormat.is_quoting_required(ascii) != DataFormat.Quoting.OPTIONAL)
             throw new ImapError.INVALID("\"%s\" cannot be represented as a ResponseCodeType", ascii);
 
-        // store lowercased so it's easily compared with const strings above
+        // store in lowercase so it's easily compared with const strings above
         original = ascii;
         value = Ascii.strdown(ascii);
     }

src/engine/imap/response/imap-server-data-type.vala

@@ -123,7 +123,7 @@ public enum Geary.Imap.ServerDataType {
     /**
      * Examines the {@link RootParameters} looking for a ServerDataType.
      *
-     * IMAP server responses don't offer a regular format for server data declations.  This method
+     * IMAP server responses don't offer a regular format for server data declaretions.  This method
      * parses for the common patterns and returns the ServerDataType it detects.
      *
      * See [[http://tools.ietf.org/html/rfc3501#section-7.2]] for more information.

src/engine/imap/transport/imap-client-session.vala

@@ -923,7 +923,7 @@ public class Geary.Imap.ClientSession : BaseObject {
     /**
      * Prepares the connection and performs a login using the supplied credentials.
      *
-     * Preparing the connnection includes attempting compression and using STARTTLS if necessary.
+     * Preparing the connection includes attempting compression and using STARTTLS if necessary.
      * {@link Capabilities} are also retrieved automatically at the right time to ensure the best
      * results are available with {@link capabilities}.
      */

src/engine/mime/mime-multipart-subtype.vala

@@ -5,7 +5,7 @@
  */
 
 /**
- * A represenation of a MIME multipart Content-Type subtype.
+ * A representation of a MIME multipart Content-Type subtype.
  *
  * See [[https://tools.ietf.org/html/rfc2046#section-5.1]]
  */

src/engine/nonblocking/nonblocking-concurrent.vala

@@ -65,7 +65,7 @@ public class Geary.Nonblocking.Concurrent : BaseObject {
             // can't notify event here, Nonblocking.Event is not thread safe
             //
             // artificially increment the ref count of this object, schedule a completion callback
-            // on the forground thread, and signal there
+            // on the foreground thread, and signal there
             ref();
 
             Idle.add(on_notify_completed);

src/engine/smtp/smtp-authenticator.vala

@@ -5,7 +5,7 @@
  */
 
 /**
- * An abstract class describing a process for goind through an SASL authentication transaction.
+ * An abstract class describing a process for going through an SASL authentication transaction.
  *
  * Authenticators expect to use complete {@link Credentials}, i.e. user and pass must not be null.
  */

src/engine/smtp/smtp-login-authenticator.vala

@@ -5,7 +5,7 @@
  */
 
 /**
- * SASL's LOGIN authentication schema impemented as an {@link Authenticator}.
+ * SASL's LOGIN authentication schema implemented as an {@link Authenticator}.
  *
  * LOGIN is obsolete but still widely in use and provided for backward compatibility.
  *

src/engine/smtp/smtp-plain-authenticator.vala

@@ -5,7 +5,7 @@
  */
 
 /**
- * SASL's PLAIN authentication schema impemented as an {@link Authenticator}.
+ * SASL's PLAIN authentication schema implemented as an {@link Authenticator}.
  *
  * See [[http://tools.ietf.org/html/rfc4616]]
  */

test/client/application/geary-application-test.vala

@@ -28,7 +28,7 @@ class GearyApplicationTest : TestCase {
     public void paths_when_installed() throws GLib.Error {
         string[] args = new string[] {
             _INSTALL_PREFIX + "/bin/geary",
-            // Specifiy this so the app doesn't actually attempt
+            // Specify this so the app doesn't actually attempt
             // to start up
             "-v"
         };

test/mock-object.vala

@@ -98,7 +98,7 @@ public class ExpectedCall : Object {
  * the code being tested. For example, if an object being tested
  * requires certain objects to be passed in via its constructor or as
  * arguments of method calls and uses these to implement its
- * behaviour, mock objects that fulfil these requirements can be used.
+ * behaviour, mock objects that fulfill these requirements can be used.
  *
  * Mock objects provide a means of both ensuring code being tested
  * makes expected method calls with expected arguments on its

ui/client-web-view.js

@@ -57,7 +57,7 @@ PageState.prototype = {
         //
         // Note also that the delay introduced here by this last call
         // to queuePreferredHeightUpdate when the complete document is
-        // loaded seems to be important to get an acurate idea of the
+        // loaded seems to be important to get an accurate idea of the
         // final document size.
         window.addEventListener("load", function(e) {
             queuePreferredHeightUpdate();
@@ -90,7 +90,7 @@ PageState.prototype = {
     },
     loaded: function() {
         this.isLoaded = true;
-        // Always fire a prefered height update first so that it will
+        // Always fire a preferred height update first so that it will
         // be vaguegly correct when notifying of the HTML load
         // completing.
         this.updatePreferredHeight();

ui/components-inspector.ui

@@ -28,7 +28,7 @@
             <property name="visible">True</property>
             <property name="can_focus">True</property>
             <property name="receives_default">True</property>
-            <property name="tooltip_text" translatable="yes" comments="Tooltip for inspector button">Togggle appending new log entries</property>
+            <property name="tooltip_text" translatable="yes" comments="Tooltip for inspector button">Toggle appending new log entries</property>
             <property name="action_name">win.toggle-play</property>
             <property name="active">True</property>
             <child>

ui/composer-web-view.js

@@ -77,7 +77,7 @@ ComposerPageState.prototype = {
             }
         });
         // We can't use keydown for this, captured or bubbled, since
-        // that will also cause the line that the cursor is currenty
+        // that will also cause the line that the cursor is currently
         // positioned on when Enter is pressed to also be outdented.
         document.body.addEventListener("keyup", function(e) {
             if (e.keyIdentifier == "Enter" && !e.shiftKey) {

ui/conversation-web-view.css

@@ -89,7 +89,7 @@ pre {
     margin: 1em 0;
   }
 
-  /* Inline collapsable quote blocks */
+  /* Inline collapsible quote blocks */
 
   .geary-quote-container {
     position: relative;
@@ -157,9 +157,9 @@ pre {
     white-space: pre;
     -webkit-user-select: none;
     -webkit-user-drag: none;
-    /* All futher properties below are a workaround for WK Bug 166648
+    /* All further properties below are a workaround for WK Bug 166648
      * <https://bugs.webkit.org/show_bug.cgi?id=166648>. The result is
-     * we need to manually style these buttons outselves. */
+     * we need to manually style these buttons ourselves. */
     -webkit-appearance: none;
     box-sizing: border-box;
     /* The following was taken from GTK+4 trunk Adwaita theme: