* src/widgets/modest-msg-view.c:
[modest] / ChangeLog2
index 32f311d..37b01ad 100644 (file)
-2007-07-02  Murray Cumming  <murrayc@murrayc.com
+2007-07-26  Murray Cumming  <murrayc@murrayc.com>
 
 
-       * src/modest-tny-account.c: Added util_run_in_main_thread_and_wait() 
+       * 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:
+       (create_page_complete_easysetup): Use 
+       gtk_widget_set_size() request to ensure that the text wraps 
+       even in osso 1.1.
+
+2007-07-05  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       (on_entry_invalid_character), (create_page_account_details),
+       (create_page_user_details):
+       Use new API of ModestValidatingEntry. But for some reason the banner is
+       not shown. I will have to investigate that further.
+
+       * src/widgets/modest-validating-entry.c: (on_insert_text),
+       (modest_validating_entry_set_func):
+       * src/widgets/modest-validating-entry.h:
+       Added a new callback function to be able to react to prevented 
+       characters whitespaces to e.g show a banner.
+
+2007-07-05  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/modest-account-settings-dialog.c: Use the notify::value
+       notification of HildonNumberEditor instead of remembering the port
+       number. Thanks to Johannes for pointing this out.
+
+2007-07-05  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c:
+       Check if there are any user changes before showing a confirmation
+       dialog (projects.maemo.org NB#60946)
+
+2007-07-05  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/modest-account-settings-dialog.c: Remember custom port
+       number at the beginning of the dialog and set the modified flag at the
+       end if the port number has changed. This is necessary because
+       HildonNumberEditor does not support notification when the value has
+       changed.
+
+2007-07-05  Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c:
+       (modest_ui_actions_on_main_window_move_to): Check whether the source folder needs 
+       a connection before asking anything else. Correct the check for the destination 
+       folder so that it succeeds when online.
+       This fixed projects.maemo.org bug NB#61645.
+
+2007-07-05  Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c: (modest_ui_actions_on_send):
+       Call modest_platform_connect_and_wait() to ask for a connection if necessary, 
+       to fix projects.maemo.org bug NB#62439.
+
+2007-07-05  Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-tny-account.c: (on_connection_status_changed):
+       Really do not use tny_device_force_offline() here. See the previous comment.
+
+2007-07-04  Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-tny-account.c: (on_connection_status_changed):
+       Do not use tny_device_force_offline() when tinymail says that something 
+       went wrong with the connection. It seems to report this unnecessarily.
+
+2007-07-04  Murray Cumming <murrayc@murrayc.com>
+
+       * src/maemo/modest-connection-specific-smtp-window.c:
+       (modest_connection_specific_smtp_window_init):
+       Use gtk_widget_set_size_request(), because gtk_label_set_max_width_chars()
+       does not seem to have any effect in osso 1.1 (though it works in 
+       Maemo Bora). This fixes bug projects.maemo.org NB#59212.
+
+2007-07-04  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/maemo/modest-connection-specific-smtp-window.c:
+       Fix UI specification problems with projects.meamo.org NB#60479
+
+2007-07-04   Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-ui-actions.c: (open_msg_cb): Add a printf() to provide a 
+       clue when this has failed, because the error dialog is not always shown 
+       (at least in earlier version).
+       
+       (modest_ui_actions_get_msgs_full_error_handler): prinf() the actual 
+       GError message, because the generic error dialog is not helpful.
+       I am getting
+       'This message is not currently available (can't let a new connection go online)'
+       when trying to open an IMAP message.
+       
+       (_modest_ui_actions_open): Offer a connection when some messages would 
+       have to be downloaded and we are offline.
+
+2007-07-04  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/widgets/modest-folder-view.c:
+       Don't check for MERGE type anymore as we use the correct type
+       now.
+       NOTE: For the last two changes you need to update your tinymail copy
+       because of API changes!
+
+2007-07-04  Johannes Schmid <johannes.schmid@openismus.com>
+
+       * src/modest-tny-local-folders-account.c:
+       Set the folder type for the TnyMergeFolder to TNY_MERGE_FOLDER_OUTBOX
+       (see projects.maemo.org NB#61580)
+
+2007-07-04  Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-tny-account.c: (on_connection_status_changed):
+       Commented out the call to modest_platform_connect_and_wait() because this 
+       seems to always hang, and I would like to know if this is the only thing 
+       that causes a hang when showing the connection dialog.
+       It is not very useful anyway.
+
+2007-07-04  Murray Cumming <murrayc@murrayc.com>
+
+       * src/maemo/modest-msg-view-window.c:
+       (modest_msg_view_window_select_next_message):
+       
+       * src/modest-platform.h:
+       * src/maemo/modest-platform.c: 
+       Added modest_platform_connect_and_wait(), which calls 
+       tny_maemo_conic_device_connect() always in the main thread, by calling it 
+       in an idle handler and waiting for the idle handler to run. It should be 
+       in the main thread because it uses GTK+ UI code. It just returns TRUE 
+       if the device is already online.
+       
+       There is some nasty code to ensure that subsequent calls to 
+       modest_platform_connect_and_wait() just wait for the first one to finish, 
+       by checking repeatedly in another idle handler. This is necessary, 
+       because other threads (or even the main thread, via another idle handler) 
+       can call modest_platform_connect_and_wait() while 
+       modest_platform_connect_and_wait() is running.
+       
+       (modest_platform_connect_and_wait_if_network_account),
+       (modest_platform_connect_and_wait_if_network_folderstore):
+       Convenience functions that calls modest_platform_connect_and_wait() if 
+       the account or folder might want network access.
+       
+       * src/modest-mail-operation.c:
+       (modest_mail_operation_update_account):
+       * src/modest-ui-actions.c: (download_uncached_messages),
+       (modest_ui_actions_on_rename_folder), (delete_folder),
+       (modest_ui_actions_on_main_window_move_to),
+       (modest_ui_actions_on_msg_view_window_move_to):
+       * src/widgets/modest-folder-view.c:
+       (drag_and_drop_from_folder_view):
+       Offer a connection if we are offline.
+       
+       * src/modest-tny-account.c: (on_connection_status_changed): 
+       Document this as only handling errors during network operations in progress,
+       and replace the idle code with a simple call to 
+       modest_platform_connect_and_wait(), which does this instead.
+
+2007-07-03  Murray Cumming <murrayc@murrayc.com>
+
+       * src/modest-tny-account.c:
+       (modest_tny_account_get_special_folder): (util_on_idle),
+       Handle the GError so we can see when tny_folder_store_get_folders() 
+       fails, as it currently does while something else is connecting.
+       
+       Added util_run_in_main_thread_and_wait() 
        which uses an idlen handler and mainloop to ensure that the function 
        is called in the main thread.
        (on_connection_status_changed): Use util_run_in_main_thread_and_wait() 
        to make sure that we call attempt the connection in the mainloop only, 
        which might solve some hangs.
        which uses an idlen handler and mainloop to ensure that the function 
        is called in the main thread.
        (on_connection_status_changed): Use util_run_in_main_thread_and_wait() 
        to make sure that we call attempt the connection in the mainloop only, 
        which might solve some hangs.
+       
+2007-07-03  Murray Cumming <murrayc@murrayc.com>
+
+       (modest_tny_account_store_get_tny_account_by): Do not allow accounts to 
+       be found by their display name. This is never useful.
+       * src/modest-tny-account-store.c: (on_account_removed): Find the account 
+       by its ID, instead of trying to use its ID as its display name.
+       
+       * src/modest-tny-account-store.h:
+       * src/modest-tny-account.c:
+       (modest_tny_account_get_special_folder): Add a g_warning() when 
+       tny_folder_store_get_folders() returns an empty list for the maildir.
+       
+       * src/modest-tny-send-queue.c: (modest_tny_send_queue_get_outbox):
+       Add a g_warning() when this returns NULL, because that it not allowed 
+       by tinymail.
+
+2007-07-02  Armin Burgmeier  <armin@openismus.com>
+
+       * src/maemo/easysetup/modest-easysetup-wizard.c: Added a confirmation
+       note when trying to cancel the wizard. This fixes projects.maemo.org
+       bug NB#60946.
 
 2007-07-02  Murray Cumming  <murrayc@murrayc.com>
 
 
 2007-07-02  Murray Cumming  <murrayc@murrayc.com>