2007-08-04 Johannes Schmid <johannes.schmid@openismus.com>
[modest] / ChangeLog2
index 579cdb2..934bd57 100644 (file)
@@ -1,3 +1,837 @@
+2007-08-04  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/modest-text-utils.c:
+       Fixed invalid jump due to incorrect string/pointer
+       operation. 
+       (See https://projects.maemo.org/trac/email/wiki/ValgrindErrors
+       for details)
+
+2007-08-01  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       Added modest_account_mgr_get_first_account_name(), 
+       using code from modest_account_mgr_set_first_account_as_default().
+
+       * src/maemo/modest-main-window.c: Added set_at_least_one_account_visible().
+       (account_number_changed): Call set_at_least_one_account_visible() so that 
+       the first account is visible when first added, without the user having to 
+       use the View menu.
+       I do wonder how this worked before.
+
+2007-08-01  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (show_password_warning_only),
+       (show_wrong_password), (request_password_and_wait), (get_password),
+       (modest_tny_account_store_alert): Remove the (incomplete anyway) 
+       attempts to show UI only in the main context, which used g_idle_add() 
+       and mainloops, because tinymail now guarantees that it only calls these 
+       callbacks in the mainloop with the gdk lock. This makes everything much 
+       simpler.
+
+2007-08-01  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
+       Handle TNY_ACCOUNT_ERROR_TRY_CONNECT_SERVICE_UNAVAILABLE in the same 
+       way as TNY_ACCOUNT_ERROR_TRY_CONNECT_HOST_LOOKUP_FAILED, showing an 
+       error dialog, instead of ignoring it.
+
+2007-08-01  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-msg-view-window.c:
+       (modest_msg_view_window_get_header),
+       (modest_msg_view_window_last_message_selected):
+       Added comments to explain the use of priv->header_model.
+       * src/modest-ui-dimming-rules.c:
+       (modest_ui_dimming_rules_on_open_msg),
+       (modest_ui_dimming_rules_on_mark_as_read_msg),
+       (modest_ui_dimming_rules_on_mark_as_unread_msg),
+       (modest_ui_dimming_rules_on_remove_attachments),
+       (_invalid_attach_selected), (_selected_msg_marked_as):
+       Renamed _marked_as_deleted() to _selected_message_is_marked_as_deleted(), 
+       and changed the parameter to a ModestMainWindow to make it clearer that 
+       it never uses any other kind of window.
+       Added _message_is_marked_as_deleted() for examining a ModestMsgViewWindow 
+       instead of the selected message.
+       I notice that much of the dimming code and the ModestMsgViewWindow code 
+       hard-codes the assumption that the viewed message is always selected in 
+       the header view. This does not seem wise or simple.
+
+       (modest_ui_dimming_rules_on_delete_msg): For a ModestMsgViewWindow, 
+       examine the actual message instead of whatever might be selected 
+       in the header view. This stops the delete toolbar button and menu item
+       from being dimmed when opening a message view the search UI, 
+       partially fixing projects.maemo.org bug NB#63811.
+
+       However, when actually deleting, I now get this warning:
+       "tny_header_set_flags: This is a header instance for a new message. 
+       Only priority and attachment flags are supported"
+
+2007-07-30  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/modest-ui-actions.c: (modest_ui_actions_on_send):
+       
+       Show account wizard if no accounts are defined while sending
+       (automatic sending after the account wizard finished does not
+       work because it takes some time until the account manager
+       synchronizes everything. (projects.maemo.org NB#62792)
+       
+2007-07-30  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (on_account_removed),
+       (on_account_changed): Emit the ACCOUNT_UPDATE signal so that the 
+       account is removed from the View menu. We have TnyAccountStore 
+       and ModestTnyAccountStore signals that seem very similar, but this 
+       seems to work for now.
+
+       * src/maemo/modest-main-window.c:
+       (on_show_account_action_activated): Prevent dereference 
+       of a NULL if the accoutn does not really exist, though that 
+       should not happen anymore.
+
+       This fixes projects.maemo.org bug NB#64606.
+
+2007-07-30  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c: (run_account_setup_wizard),
+       (modest_ui_actions_on_accounts): Revert a change that set these 
+       to modal dialogs, freezing the UI when they show secondary dialogs.
+       This fixed projects.maemo.org bug NB#64519.
+
+2007-07-27  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-header-view-render.c:
+       (_modest_header_view_compact_header_cell_data):
+       Some checks and initialization to try to deal with a crash, 
+       probably ineffective.
+
+2007-07-27  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-maemo-utils.c:
+       (modest_maemo_utils_get_supported_secure_authentication_methods):
+       Make sure that we have a connection, asking the user for one 
+       if necessary.
+       * src/maemo/modest-main-window.c: (modest_main_window_on_show): 
+       Check for a connection before showing the wizard, because 
+       the wizard needs a connection to check for server capabilities.
+
+2007-07-27  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-view-window.c:
+       (on_new_button_clicked): Remember the wizard dialog instance, 
+       so we can just present it again if necessary. This prevents 
+       multiple windows from appearing if you click again quickly on the 
+       New button before the dialog appears, fixing projects.maemo.org 
+       bug NB#64169.
+
+2007-07-27  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-main-window.c: (modest_main_window_on_show):
+       Offer a connection dialog if there is no connection when 
+       starting modest. This should fix projects.maemo.org bug NB#61134.
+
+2007-07-27  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-msg-edit-window.c: (replace_with_attachments):
+       Reverted my last commit. I was able to fix the build by doing 
+       apt-get install wpeditor0 (replacing other wpeditor packages) and 
+       an explicit apt-get install wpeditor-dev to upgrade the -dev 
+       package.
+
+2007-07-27  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-msg-edit-window.c: (replace_with_attachments):       
+       Commented out the use of wp_text_buffer_replace_image() 
+       because it does not exist, to break the build. Added a TODO for it.
+
+2007-07-26  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (recreate_all_accounts):
+       Recreate them even if they were NULL before. I guess something 
+       else was already creating them before, or this would never 
+       have worked.
+       (modest_tny_account_store_get_server_account): Make sure that 
+       the tny accounts have been created before trying to find one.
+       This allows libmodest_dbus_client_compose_mail() to work 
+       when the UI is not yet visible.
+       This is probably useful in other cases too.
+
+       (get_password): When the password is wrong after it is entered 
+       in the protected password dialog, show an info banner (in an 
+       idle handler) before showing the protected password dialog 
+       again.
+
+2007-07-26  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.h:
+       * src/modest-ui-actions.c: Added 
+       modest_do_refresh_current_folder() containing some code from 
+       modest_ui_actions_on_send_receive().
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_send_receive):
+       Call modest_do_refresh_current_folder() before refreshing the 
+       requested account, as in modest_ui_actions_on_send_receive(). 
+       This is strange, but seems to be necessary to show new mails.
+       Now new emails are finally visible in the UI when downloaded 
+       via the automatic update interval, fixing bug NB#57458.
+       This also works because bug NB#57458 has been fixed somehow.
+
+2007-07-26  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_filter):
+       printf more details when we get unhandled D-Bus methods, because 
+       it is interesting.
+
+       * src/modest-mail-operation.c: (update_account_thread):
+       Comment out the tny_camel_pop_store_account_reconnect(), because this 
+       is apparently no longer needed, and it did not work before anyway. 
+       See the comment in the code.
+
+       * src/modest-search.c: (modest_search_folder): Ignore deleted emails, 
+       by checking the header flags. This fixes projects.maemo.org bug 
+       NB#64225.
+
+2007-07-25  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-maemo-utils.h: 
+       * src/maemo/modest-maemo-utils.c: Added 
+       modest_maemo_show_information_note_in_main_context_and_forget().
+       * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
+       Use the new function to show simple error dialogs in the main 
+       context, though question dialogs are still in the current context, 
+       which is unfortunately usually not the main context. This should 
+       fix some hangs.
+
+2007-07-25  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (get_password):
+       Never save the entered username and password in gconf, 
+       because the UI spec says we should not. This helps 
+       slightly with projects.maemo.org bug NB#63671.
+
+2007-07-25  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-msg-edit-window.h:
+       * src/maemo/modest-msg-edit-window.c: (create_stream_for_uri):
+       Moved some code here to make it simpler.
+       (modest_msg_edit_window_attach_file_one): Renamed from 
+       modest_msg_edit_window_attach_file_interactive().
+       (modest_msg_edit_window_offer_attach_file): Renamed from 
+       modest_msg_edit_window_attach_file() and used 
+       modest_msg_edit_window_attach_file_one() instead of 
+       copy/pasting the code. The old code for one was not using 
+       GnomeVFS. This allows use of smb:// URLs when using send-as 
+       from external applications, fixing bug NB#63741.
+
+       (modest_msg_edit_window_insert_image),
+       (modest_msg_edit_window_add_attachment_clicked):
+       * src/modest-ui-actions.c: (modest_ui_actions_on_attach_file):
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_compose_mail):
+       Adapted to the new function names.
+
+2007-07-25  Murray Cumming  <murrayc@murrayc.com>
+
+       (modest_platform_set_update_interval): Set the alarm recurrence 
+       fields so that auto-update does not just happen the first time.
+       This should have fixed projects.maemo.org bug NB#63938.
+
+       When this happens you can see a printf when the D-Bus method is 
+       called:
+       GLIB DEBUG default - debug: modest_dbus_req_handler: method received: SendReceive
+
+       However, I do not actually seem to get new POP messages even when 
+       clicking the refresh button manually. A restart gets them. 
+       This might be a problem in today's tinymail only.
+
+2007-07-24  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-maemo-global-settings-dialog.c:
+       (current_connection):
+       Avoid a crash (probably only in scratchbox) when strcmp()ing a 
+       NULL when closing the dialog.
+
+2007-07-24  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_delete_message):
+       Remove the hacky send/receive that didn't work, and instead 
+       refilter the treemodel to make the deleted email really vanish from 
+       the UI (though the delete-from-menu code does has commented out 
+       the code that does that). This fixes projects.maemo.org bug 
+       NB#62859.
+
+2007-07-24  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_delete_message):
+       Try a couple of tricks to force the treeview to update (
+       poke the folder, and do a send/receive refresh), but the deleted 
+       email still appears with a strike-through instead of really being 
+       removed.
+
+2007-07-23  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (add_single_folder_to_list): Ignore TnyMergeFolder folders, 
+       because their ID is not human-readable (see the comment about 
+       improvements needed to the osso-email-interface API) and 
+       TnyMergeFolder::get_id_func() currently crashes with DBC enabled. 
+
+2007-07-23  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c: (modest_dbus_req_handler),
+       (add_folders_to_list), (on_dbus_method_get_folders),
+       (modest_dbus_req_filter):
+       * src/maemo/modest-maemo-utils.c:
+       (on_camel_account_get_supported_secure_authentication):
+       * src/maemo/modest-main-window.c: (modest_main_window_set_style),
+       (on_header_view_focus_in):
+       * src/modest-email-clipboard.c: (modest_email_clipboard_set_data):
+       * src/modest-formatter.c: (modest_formatter_create_body_part):
+       * src/modest-mail-operation.c: (recurse_folders),
+       (update_account_thread), (get_msgs_full_thread),
+       (modest_mail_operation_get_msgs_full),
+       (modest_mail_operation_xfer_msgs):
+       * src/modest-search.c: (search_mime_part_and_child_parts),
+       (modest_search_account), (modest_search_all_accounts):
+       * src/modest-tny-account.c:
+       (modest_tny_account_get_special_folder), (recurse_folders):
+       * src/modest-tny-local-folders-account.c:
+       (modest_tny_local_folders_account_query_passes):
+       * src/modest-tny-msg.c: (copy_mime_part),
+       (modest_tny_msg_find_body_part_from_mime_part):
+       * src/modest-tny-send-queue.c: (modest_tny_send_queue_cancel),
+       (modest_tny_send_queue_try_to_send):
+       * src/modest-ui-actions.c: (modest_ui_actions_on_delete_message),
+       (_modest_ui_actions_open), (download_uncached_messages),
+       (reply_forward), (modest_ui_actions_on_copy),
+       (modest_ui_actions_on_details), (has_retrieved_msgs),
+       (msgs_move_to_confirmation), (open_msg_for_purge_cb),
+       (do_headers_action):
+       * src/modest-ui-dimming-rules.c: (_already_opened_msg),
+       (_selected_msg_marked_as), (_msg_download_completed),
+       (_selected_msg_sent_in_progress):
+       * src/widgets/modest-attachments-view.c:
+       (modest_attachments_view_set_message):
+       * src/widgets/modest-msg-view.c: (find_cid_image):
+       * tests/check_folder-transfer.c: (find_folders):
+       * tests/check_update-account.c: (func):
+       * tests/dbus_api/test_get_folders.c: (main):
+       Checked all uses of tny_iterator_get_current(), making sure 
+       that the object is unreferenced (3 or 4 places did not), 
+       and adding checks for NULL.
+
+2007-07-23  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (on_idle_open_default_inbox): Make sure that the application UI is 
+       visible.
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (on_entry_invalid_character): Show the word whitespace instead 
+       of (null) in the banner when whitespace was entered.
+       (show_error): Add a comment that  this does not work in 
+       Maemo Bora, though we do not know why.
+
+2007-07-23  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account.c:
+       (modest_tny_account_get_special_folder): Use an if instead of 
+       an assert for when the parent account name is NULL, because this 
+       might be causing a crash in projects.maemo.org bug 63192.
+       * src/modest-mail-operation.c:
+       (modest_mail_operation_send_new_mail_cb),
+       (modest_mail_operation_remove_folder): unref the folder that is 
+       returned from modest_tny_account_get_special_folder().
+
+2007-07-23  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c:
+       (modest_ui_actions_on_password_requested):
+       Show mail_ib_login_cancelled info banner when the user cancels 
+       the protected password dialog, to fix projects.maemo.org 
+       bug NB#63617.
+
+2007-07-23  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-settings-dialog.c:
+       (create_page_outgoing): Put this page in a GtkScrolledWindow, 
+       matching the UI mock-up in the UI spec, and allowing access 
+       to all widgets even when the on-screen keyboard is active.
+
+2007-07-23  Murray Cumming  <murrayc@murrayc.com>
+
+       Reverted this previous commit from Johannes, 
+       because the scrollbars are always visible and because 
+       I think we only need it on onepage of the account 
+       settings, not the account wizard.
+       (I wrote the wrong instructions in the bug.)
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       Added scrollbars to account wizard dialog.
+       Fixed projects.maemo.org NB #62744
+
+2007-07-22  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (create_page_user_details):
+       Use auto-cap for name (projects.maemo.org NB#63716)
+       * src/widgets/modest-account-view.c: (on_account_busy_changed):
+
+2007-07-22  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/modest-ui-actions.c: Use current account instead of
+       default account for creating viewer windows, so that the viewer
+       window knows to which account the viewed message belongs.
+
+       * src/maemo/modest-msg-view-window.c: Destroy the window when the
+       corresponding account is deleted. This fixes projects.maemo.org bug
+       NB#62936.
+
+2007-07-18  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/modest-connection-specific-smtp-edit-window.c: Don't let
+       the dialog close after entering an invalid port number and clicking
+       OK. This fixes projects.maemo.org bug NB#63602.
+
+2007-07-18  Armin Burgmeier  <armin@openismus.com>
+
+       * src/modest-tny-account-store.c:
+       (get_server_accounts): Set parent modest account name for each
+       connection-specific transport account so that special folders of such
+       an account can be looked up. This fixes a crash when changing account
+       settings. This might also be related to projects.maemo.org bug
+       NB#63594.
+
+2007-07-16  Armin Burgmeier  <armin@openismus.com>
+
+       * src/modest-ui-actions.c: Added a button to create a folder to the
+       move-to dialog. This fixes projects.maemo.org bug NB#.
+
+2007-07-16  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       Added scrollbars to account wizard dialog.
+       Fixed projects.maemo.org NB #62744
+
+2007-07-16  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/widgets/modest-msg-view.h:
+       * src/widgets/modest-msg-view.c:
+       Added modest_msg_view_get_message_is_empty() but this needs 
+       to be implemented.
+       * src/maemo/modest-msg-edit-window.c: (message_is_empty):
+       Check if there is any text in the message yet.
+       But this needs to be implemented properly.
+       (modest_msg_edit_window_toggle_find_toolbar):
+       * src/maemo/modest-msg-view-window.c:
+       (modest_msg_view_window_toggle_find_toolbar):
+       Show a banner warning if there is no text to search, when 
+       trying to start a search. This partially fixes 
+       projects.maemo.org bug NB#62350.
+
+2007-07-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-mail-operation.c: (update_account_thread):
+       Initialized num_new_headers to fix the build.
+
+2007-07-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr.c: (modest_account_mgr_account_names):
+       Ignore modest accounts whose server accounts don't exist in gconf.
+       This prevents us trying to create TnyAccount instances for those 
+       server accounts when the server account has been removed from gconf, 
+       but the parent modest account has not yet been removed from modest.
+       I don't know why this is only noticeable now. It was probably just 
+       a harmeless g_warning() anyway.
+
+2007-07-13  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-dimming-rules.c: (_selected_folder_is_root):
+       Fix my previous commit, to avoid unrefing a NULL TnyFolderStore.
+
+2007-07-12  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/modest-main-window.c: Make default account appear bold
+       in view, tools/send & receive and toolbar/send & receive SCM menus.
+       This fixes projects.maemo.org bug NB#59010.
+
+2007-07-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-dimming-rules.c: (_selected_folder_is_root):
+       Add a short-cut, by checking if the folder store is an account.
+       This ensures that the Sort menu item and toolbar is dimmed for 
+       the local-folders account root item, fixing 
+       projects.maemo.org bug NB#62545.
+
+2007-07-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (get_password):
+       Show the password dialog (via the signal) only in the mainloop, via 
+       an idle handler that we wait for. This could theoretically prevent 
+       hangs, though I worry that it could introduce some too.
+
+2007-07-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/gnome/modest-platform.c: (modest_platform_connect_and_wait):
+       * src/maemo/modest-platform.c: 
+       (modest_platform_connect_and_wait):
+       Take a TnyAccount* parameter and try to set it to online 
+       (a TnyCamel-only API), because tinymail sets accounts to offline 
+       when the user cancels a password dialog, and doesn't ask again until 
+       it has been set to online.
+       However, this is commented out because it causes a hang now.
+
+       (modest_platform_connect_and_wait_if_network_account):
+       * src/modest-mail-operation.c:
+       (modest_mail_operation_update_account):
+       * src/modest-platform.h:
+       * src/modest-ui-actions.c: (_modest_ui_actions_open),
+       (download_uncached_messages), (modest_ui_actions_on_send): Adapted.
+
+2007-07-12  Armin Burgmeier  <armin@openismus.com>
+
+       * src/modest-error.h: Added MODEST_MAIL_OPERATION_FOLDER_EXISTS error.
+
+       * src/modest-mail-operation.c:
+       * src/modest-ui-actions.c: Check for already existing folder before
+       querying the server, show error message from camel if the request
+       still fails. This fixes projects.maemo.org bug NB#58701.
+
+2007-07-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.h:
+       * src/modest-tny-account-store.c: 
+       Added modest_tny_account_store_forget_password_in_memory(),
+       (get_server_accounts): Called modest_tny_account_store_forget_password_in_memory() 
+       when creating a new account, so that new passwords will be used, instead of 
+       us thinking that it's a second request as a result of a failed password again.
+
+2007-07-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (on_idle_open_message): If the message is in drafts then open it 
+       in an edit window instead of a view window. This fixes 
+       projects.maemo.org bug NB#63005.
+
+2007-07-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_open_message):
+       Avoided a harmless g_warning().
+
+2007-07-12  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_open_message),
+       Pass the modest account name, not the TnyAccount display name to 
+       modest_msg_view_window_new() so that the reply feature works,
+       fixing projects.maemo.org bug NB#62892.
+
+       * src/gnome/modest-msg-view-window.c: (modest_msg_view_window_new),
+       (modest_msg_view_window_new_with_header_model):
+       * src/maemo/modest-msg-view-window.c:
+       (modest_msg_view_window_new_with_header_model),
+       (modest_msg_view_window_new):
+       * src/modest-ui-actions.c: (reply_forward):
+       * src/widgets/modest-msg-view-window.h:
+       * src/widgets/modest-window.c: (modest_window_set_active_account):
+       Rename an account parameter to modest_account_name to make this clearer.
+
+2007-07-11  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.h:
+       * src/modest-ui-actions.c: (headers_action_delete): Moved some code into 
+       modest_do_message_delete().
+       * src/dbus_api/modest-dbus-callbacks.c: (on_idle_delete_message):
+       Call modest_do_message_delete(), hoping that this would cause messages 
+       to be really deleted from search instead of just marked with strike through.
+       Does not seem to help.
+
+2007-07-11  Armin Burgmeier  <armin@openismus.com>
+
+       * src/widgets/maemo-folder-view.c (modest_folder_view_on_map): Set
+       priv->reselect to FALSE prior to calling
+       modest_folder_view_select_first_inbox_or_local() to prevent endless
+       recursion and thus a crash on startup.
+
+2007-07-11  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/modest-account-settings-dialog.h: Include
+       <gtk/gtknotebook.h> to fix the build (at least for me).
+
+       * src/maemo/modest-platform.c (modest_platform_run_new_folder_dialog):
+       Use "New FolderN" as default suggested name if "New folder" already
+       exists, with N being the first free number, though never exceed "New
+       folder99" in which case we just fall back to "New Folder" even though
+       it already exists. This fixes projects.maemo.org bug NB#58663.
+
+2007-07-11  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-view-window.c: (on_new_button_clicked):
+       Do not use gtk_dialog_run() to avoid modality hang problems. This fixes 
+       projects.maemo.org bug NB#62778.
+
+2007-07-11  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: (create_account):
+       Ignore the wrong (for now) secure-smtp setting in the presets - 
+       just default to requiring secure authentication for SMTP, because 
+       that is most common.
+
+2007-07-11  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/widgets/modest-folder-view.c: (filter_row)
+       Do not show drafts folder (see UI specs)
+
+2007-07-11  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/modest-ui-actions.c:
+       Remove call to show_non_move_folders(TRUE) which was moved to a wrong
+       place and also not necessary.
+       
+       * src/widgets/modest-folder-view.c: (filter_row)
+       Guess folder type and hide Sent & Outbox if necessary.
+       Fixed project.maemo.org NB#57328
+
+2007-07-11  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.h:
+       * src/modest-ui-actions.c:
+       (modest_ui_actions_on_delete_message):
+       Move some code into a new 
+       modest_ui_actions_refresh_message_window_after_delete() 
+       function so we can use it from elsewhere.
+
+       * src/dbus_api/modest-dbus-callbacks.c:
+       (on_idle_delete_message), (on_delete_message): Used an idle 
+       handler, as we do for the other D-Bus method handlers.
+       Call modest_ui_actions_refresh_message_window_after_delete() 
+       if the deleted message was open in a window. This fixed 
+       projects.maemo.org bug NB#62862.
+
+2007-07-11  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-main.c: (main): Only show the UI at the start if 
+       the showui command line option was given. For instance,
+       run-standalone src/modest showui
+       Otherwise, we wait for the top_application D-Bus method to 
+       be sent by hildon-desktop when the application is selected 
+       from the menu.
+       This allows us to run without a UI when osso-global-search 
+       uses our D-Bus methods.
+
+2007-07-11  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/dbus_api/modest-dbus-api.h: Added a define for a 
+       top-application D-Bus method.
+       * src/dbus_api/modest-dbus-callbacks.c: 
+       (on_top_application),
+       (modest_dbus_req_handler): Handle the top-application D-Bus 
+       method, which might be an undocumented D-Bus signal sent by 
+       hildon-desktop when it starts the application from the menu.
+       But I am not sure and I need to test it on the device, because 
+       my scratchbox does not have the application in the menu now.
+       This method shows the main window, so that it can be invisible 
+       when started, for instance, just for email searching from 
+       osso-global-search.
+
+       * src/maemo/modest-main-window.c: (restore_settings),
+       (modest_main_window_new): Do not show the window immediately 
+       upon creation. We show it later.
+       * src/modest-widget-memory.c: (save_settings_paned):
+       Try (unsuccessfully) to avoid saving the paned position when 
+       it is not even visible. 
+       (restore_settings_paned): Do not allow a silly paned position.
+       This seems necessary when not showing the window at first.
+
+       * tests/dbus_api/Makefile.am:
+       * tests/dbus_api/test_top_application.c: (main): Test for the 
+       new D-Bus method.
+
+2007-07-10  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-dimming-rules.c:
+       (modest_ui_dimming_rules_on_new_msg),
+       (modest_ui_dimming_rules_on_reply_msg),
+       (modest_ui_dimming_rules_on_delete_msg),
+       (modest_ui_dimming_rules_on_details),
+       (modest_ui_dimming_rules_on_view_window_move_to),
+       (modest_ui_dimming_rules_on_view_previous),
+       (modest_ui_dimming_rules_on_view_next),
+       (_invalid_folder_for_purge): Added checks before 
+       casting the window. This looks like an actual 
+       logic error in _invalid_folder_for_purge(), because it 
+       tries to use a ModestMainWindow as a message window.
+
+2007-07-10  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-account-settings-dialog.c:
+       (show_error): Use the same implementation as in 
+       modest-easysetup-wizard.c:
+       (on_entry_invalid_character): The logical ID contains a %s. 
+       Guessed what it should be.
+       (on_entry_max), (check_data), (on_response): Adapt to 
+       changed show_error().
+       (create_page_account_details): Really call 
+       modest_validating_entry_set_func() on the correct widgets.
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (on_entry_invalid_character): Fix the logical ID here too.
+ (create_page_account_details),
+       (create_page_user_details):
+       Really call 
+       modest_validating_entry_set_func() on the correct widgets.
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
+       Do not try to use the account if it is NULL.
+
+2007-07-10  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/modest-ui-actions.c: (create_move_to_dialog),
+       (modest_ui_actions_on_main_window_move_to),
+       (modest_ui_actions_on_msg_view_window_move_to):
+       * src/widgets/modest-folder-view.c: (modest_folder_view_init),
+       (filter_row), (modest_folder_view_show_non_move_folders):
+       * src/widgets/modest-folder-view.h:
+
+       Hide Sent & Outbox folder in "move to"-dialog. Does not work for
+       the Sent folder yet as it does not report the correct folder type.
+
+2007-07-10  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/modest-msg-edit-window.c: (init_window):
+       Fixed bora build
+
+2007-07-10   Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: (show_error):
+       Take a parent widget instead of a parent window, and 
+       implement this with hildon_banner_show_information() instead.
+       Non-Maemo builds can just ifdef that there.
+       This fixed projects.maemo.org bug NB#62766.
+
+       (on_entry_invalid_character): Use show_error() again instead 
+       of using hildon directly, and use the _CS() macro to use the correct 
+       translation domain.
+       (create_account), (check_for_supported_auth_methods), (on_entry_max):
+       Pass a widget instead od a window to show_error().
+
+2007-07-10  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
+       Adapt to the changed tinymail API, using the new TnyAccount* parameter.
+       This allows us to show the server name in the error message, fixing 
+       projects.maemo.org bug NB#62904.
+       Also, use the appropriate logical ID message for the account protocol.
+
+2007-07-10  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (on_idle_wrong_password):
+       A compilation fix, though the build wasn't broken for me.
+
+2007-07-09  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c: (modest_ui_actions_on_send_receive):
+       Do not try to unref a NULL folder store, to avoid a runtime warning.
+       This seems to happen sometimes when changing accounts settings.
+
+2007-07-09  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c:
+       (modest_tny_account_store_instance_init),
+       (modest_tny_account_store_finalize),
+       (on_account_settings_hide), (on_idle_wrong_password):
+       Use a GHashTable to ensure that we only show one account settings 
+       dialog per modest account. This avoids us having a dialog open for 
+       both the store and transport accounts, or due to repeated get_password() 
+       calls that shouldn't happen anyway.
+
+2007-07-09  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-maemo-utils.c: (on_hide),
+       (modest_maemo_show_dialog_and_forget): Handle the response, because 
+       the window is not even hidden by default.
+
+       * src/maemo/modest-account-view-window.c: (on_edit_button_clicked):
+       * src/modest-ui-actions.c: (modest_ui_actions_on_accounts):
+       Use modest_maemo_show_dialog_and_forget() instead of gtk_dialog_run() 
+       which seems to prevent some modality problems.
+
+       * src/maemo/modest-main-window.c: (on_sendqueue_error_happened):
+       Ignore user cancellation errors.
+
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       Added modest_server_account_get_password() and 
+       modest_server_account_get_has_password() to avoid direct use of 
+       conf enums.
+
+       * src/maemo/modest-account-settings-dialog.h:
+       * src/maemo/modest-account-settings-dialog.c:
+       (modest_account_settings_dialog_init): Store the notebook so we can 
+       use it later.
+       Added modest_account_settings_dialog_switch_to_user_info().
+
+       * src/modest-tny-account-store.c: 
+       (get_password): When the password in the account settings is wrong (we think), 
+       show the relevant page of the account settings dialog, in the mainloop.
+       However, we currently get many of these at once because cancel does not 
+       seem to cancel.
+       (forget_password): Do not forget the password 
+       from the account settings, because this causes side-effects when 
+       tinymail seems to call it at strange time, maybe because of how we 
+       are cancelling get_password().
+       (modest_tny_account_store_alert): Ignore user cancellations.
+
+2007-07-09  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-maemo-utils.h:
+       * src/maemo/modest-maemo-utils.c: 
+       Added modest_maemo_show_information_note_and_forget() for use instead of 
+       gtk_dialog_run() when the response is not needed, to avoid an extra 
+       mainloop which could cause a hang.
+
+       * src/maemo/modest-main-window.c: (on_sendqueue_error_happened):
+       Use the logical IDs for the error message, though they are not 
+       specific enough. Note that I used modest_maemo_show_information_note_and_forget() 
+       because gtk_dialog_run() caused a hang here (the window did not close).
+       This fixes projects.maemo.org bug NB#62646.
+
+2007-07-09  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/modest-account-mgr-helpers.h:
+       * src/modest-account-mgr-helpers.c:
+       Added modest_account_mgr_get_store_protocol().
+
+       * src/maemo/modest-account-view-window.c:
+       (on_delete_button_clicked): Show different warning text depending on 
+       the protocol (IMAP or POP) as per the UI spec. Use a HildonNote instead 
+       of a GtkDialog, which fixes the truncation, fixing projects.maemo.org bug 
+       NB#59037.
+
+2007-07-09  Murray Cumming  <murrayc@murrayc.com>
+
+       * src/maemo/modest-msg-edit-window.c:
+       (modest_msg_edit_window_attach_file): Set the file chooser dialog title 
+       as per the UI spec. This fixes proejcts.maemo.org bug NB#59056.
+
+2007-07-07  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: Translate
+       ckdg_ib_maximum_characters_reached from correct package.
+
+       * src/modest-ui-actions.c: Show an error message when a folder could
+       not be created. Tinymail does not give us specific error codes about
+       what has gone wrong, so we assume that a folder with that name
+       already exists, though there might be other reasons.
+
+2007-07-06  Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-tny-account-store.c: (modest_tny_account_store_alert):
+       Use the new TNY_ACCOUNT_ERROR_TRY_CONNECT_CERTIFICATE error code to 
+       give a more exact error message that can be partly translated.
+
+2007-07-05  Murray Cumming <murrayc@murrayc.com>
+
+       * src/maemo/modest-address-book.c:
+       (select_email_addrs_for_contact): Specify a default window size for the 
+       select-email-address dialog, so that it is probably big enough for the 
+       title. This should fix projects.maemo.org bug NB#62277.
+
 2007-07-05  Murray Cumming <murrayc@murrayc.com>
 
        * src/maemo/easysetup/modest-easysetup-wizard.c: