* Change save_settings call of main and viewer windows to
'delete-event' signal handler at modest-window-mgr.
* Review Copy menu item dimming rules.
* Fixes: NB#62502, NB#62342

pmo-trunk-r2584

Renamed a symbol that has changed in tinymail

pmo-trunk-r2583

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.

pmo-trunk-r2582

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.

pmo-trunk-r2581

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.

pmo-trunk-r2580

* show the size unit correctly for the local Total size
  Fixes: NB#62594

(the bug is about MCC folders, but this issues applies to
   all local folders)

pmo-trunk-r2579

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.

pmo-trunk-r2578

* src/maemo/modest-main-window.c:
        * Now proper Contextual menus for header view
          are shown, depending on the type of folder (drafts, sent,
          outbox, inbox...) (fixes NB#60352, NB#60355, NB#60356).

pmo-trunk-r2577

* closing ModestMsgEditWindows  with attachments no longer crashes modest
  there was some confusion when TnyMimeParts were treated as gchar*s....

Fixes: NB#62598

pmo-trunk-r2576

* copy the list of attachments; this fixes the first
  crash when closing a edit window with a message with an attachment

pmo-trunk-r2575

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.

pmo-trunk-r2574

* src/widgets/modest-folder-view.c:
        * Now folders get proper expanded/collapsed emblems. This
          way we can know if a folder has children (fixes NB#60310).

pmo-trunk-r2573

Added new bugs, bulding package for w27-04-1830.

pmo-trunk-r2572

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

pmo-trunk-r2571

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

pmo-trunk-r2570

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.

pmo-trunk-r2569

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!

pmo-trunk-r2568

2007-07-04  Johanens 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)

pmo-trunk-r2567

Corrected the versioning mistake.

pmo-trunk-r2566

* Added a method to unregister a header in the window manager in case of error
* Changed the way the information dialogs are shown, now they do not use gtk_dialog_run
* Refactored the code that opens a message in the msg view window
* Refactored the code that asks the user for a download approval
* Reverted some changes in the modest_mail_operation_get_msg, the callback must be always called even in case of error
* Modified all the modest_mail_operation_get_msg callbacks to handle properly errors in the mail operation

pmo-trunk-r2565

Added new fixes made.

pmo-trunk-r2564

Added redo in editor (fixes NB#60257).
* src/widget/modest-msg-edit-window.h,
src/maemo/modest-msg-edit-window.c:
        * Added implementation for redo action, using wpeditor backend.
* src/widgets/modest-msg-edit-window-ui.h:
        * Added redo action.
* src/modest-ui-actions.[ch]:
        * Added redo ui action with implementation for editor.
* src/maemo/ui/modest-msg-edit-window-ui.xml:
        * Added redo action.

pmo-trunk-r2563

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.

pmo-trunk-r2562

* src/maemo/modest-msg-view-window.c:
        * Now we show a confirmation dialog for "next message"
          and "previous message" actions, when the message is
          not currently cached (fixes NB#59472).

pmo-trunk-r2561

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.

pmo-trunk-r2560

* Changes on xfer_folder async mail operation to add two new
parameters (user callback function).
* Add a new user callback funciton to destroy ifnromation
banner about progress of paste operation.
* Fixes: NB#62302

pmo-trunk-r2559

* src/maemo/modest-msg-edit-window.c:
        * If you try to move to plain text, but do not confirm
          the action, the menu radio action should return to plain
          text (fixes NB#60650).

pmo-trunk-r2558

* src/modest-text-utils.[ch]:
        * Now validate_email_address and validate_recipient also
          return the position of the invalid char in case validation
          failed because of this.
        * Added & as invalid character.
* src/maemo/modest-address-book.c:
        * Now we show a proper information banner when we enter invalid
          characters (fixes NB#59494).

pmo-trunk-r2557

* src/maemo/modest-msg-view-window.c:
        * Fixed a crash after scrolling up/down with keyboard the
          msg view. I forgot to provide a receptor for return value
          of a signal emition (fixes NB#62434).

pmo-trunk-r2556

* fix some memory leaks (valgrind helps)

pmo-trunk-r2555

* src/maemo/modest-maemo-global-settings-dialog.c:
* Now minimum and maximum values information banners use the
  correct translation domain (fixes NB#62471).

pmo-trunk-r2554

* src/modest-ui-dimming-rules.c:
* Now unable to paste message is obtained from the correct
  translation domain (fixes NB#62441).

pmo-trunk-r2553

* src/modest-tny-msg.c:
        * Now copy the mime parts of TnyMsg attachments. This does not fix
          the problem with forwarding attached messages, but at least now
          the contents are forwarded (in a bad format unfortunately).

pmo-trunk-r2552

Upgraded the version number for integration.

pmo-trunk-r2551

Added all the bugs sent by the testers.

pmo-trunk-r2549

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.

pmo-trunk-r2548

* Commented some code that is causing a hang on the device, we still need to investigate a little bit more why it was happening
* Changed the nofify_end function, it does not need the locking because the signal handlers are executed into the main lock

pmo-trunk-r2547

* Commented some code that seems to be unmaintained, and that causes a crash in the device

pmo-trunk-r2546

* fix for #62336

pmo-trunk-r2545

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.

pmo-trunk-r2544

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.

pmo-trunk-r2543

* small cleanup + fix runtime gtk warning

pmo-trunk-r2542

* This and another previous changes fixes NB#61982

pmo-trunk-r2541

Fixed some bugs in Escape HW key managment.

pmo-trunk-r2540

* src/maemo/modest-platform.c:
        * (modest_platform_get_icon_name): now gnome-mime-message
          and gnome-mime-message-rfc822 obtain the modest app icon
          (fixes NB#58262).

pmo-trunk-r2539

* Fixes: NB#61649

pmo-trunk-r2538

* Fixed a crash when removing messages from the viewer
* When removing messages from the viewer, if there are more messages, go to the next

pmo-trunk-r2537

Updated the changelog with the Fixes made.

pmo-trunk-r2536

Reverted

pmo-trunk-r2535

* commented out the stuff that depends on the w27 gtkhtml for now

pmo-trunk-r2534

2007-07-02  Murray Cumming  <murrayc@murrayc.com

* src/modest-tny-account.c: 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.

pmo-trunk-r2533

* src/maemo/modest-main-window-ui.h:
        * Now Email/New menu option is shown as Email/New message
          as requested in specs (NB#60222).

pmo-trunk-r2532

* make the second row in the header list grey; thus it's closer to the
  the details of the ui spec

pmo-trunk-r2531

* src/widgets/modest-msg-view.c:
        * Now that gtkhtml0 includes a way to get selected area,
          activate the scroll on finding in page in view
          (fixes NB#58611).

pmo-trunk-r2530

* src/maemo/modest-main-window.c:
        * Now moving with keyboard focus from folder view to header
          view updates the window title properly (fixes NB#61023).

pmo-trunk-r2529

* src/widgets/modest-window.c:
        * Fix keyboard handling.

pmo-trunk-r2528

2007-07-02  Murray Cumming  <murrayc@murrayc.com>

* src/modest-tny-account.h:
* src/modest-tny-account.c:
(modest_tny_account_new_from_server_account),
(modest_tny_account_new_from_server_account_name),
(modest_tny_account_new_from_account): Added a TnyCamelSession parameter,
because this must now be set before setting the proto. This is usd in
modest_tny_account_new_from_server_account(). It avoids a (maybe harmless)
warning in camel_session_get_service() caused by
tny_camel_transport_account_prepare().

* src/modest-tny-account-store.c: (get_server_accounts): Pass the new
session parameter.

pmo-trunk-r2527

* Fixes NB#62003, application do not longer hang when forwarding messages
* the reply_forward_cb only allows now to reply/forward more than one mail, did that in order to get progress information, while get_msgs_full is not migrated to use the async calls to tinymail

pmo-trunk-r2526

* add checks that session != NULL

pmo-trunk-r2525

* src/maemo/modest-msg-edit-window.c:
        * Added insensitve information banner for copy menu action. Fixes NB#60661.

pmo-trunk-r2524

* fix removing of messages:
  for imap folders: mark as deleted, for pop, local: really remove

pmo-trunk-r2523

* src/maemo/modest-main-window.c:
        * Now the window title is updated in more cases when you go
          from split view to single view (fix NB#61490).

pmo-trunk-r2522

* Fixes some bugs in modest-ui-actions, related with
mail operation IDs.
* Add a tree_model_filter_refilter call after remove_message
mail operation.

pmo-trunk-r2521

* Fixed a compilation warning on ARM

pmo-trunk-r2520

* don't expunge messages after marking them as deleted

pmo-trunk-r2519

* Add a new dimming rule to prevetn move messages
which are already opened in a viewer.
* Fixes: NB#58799

pmo-trunk-r2518

* Fixed some compilation errors.

pmo-trunk-r2517

* Add a notification dialog to avoid move messages
to the same folder.
* Fixes: NB#58330

pmo-trunk-r2516

* modest-main.c:
- return 0 iff there are no errors
* modest-ui-actions:
- clear the queue when closing down
* modest-runtime.h:
- remove unimplemented modest_runtime_init_ui
* modest-tny-account-store:
- debugging messages

pmo-trunk-r2515

* don't set the window icon explicitly; maemo seems to
  ignore it, and as we could not find the icon anyway,
  this saves a runtime warning

pmo-trunk-r2514

* Signal handlers are executed in the main lock, so no need to block

pmo-trunk-r2512

2007-06-30  Johannes Schmid <johannes.schmid@openismus.com>

* src/maemo/modest-connection-specific-smtp-edit-window.c:
(on_value_changed),
(modest_connection_specific_smtp_edit_window_init):

Fixed project.maemo.org bug NB#60479 by using \"notify::value\" instead
of \"changed\" for now

pmo-trunk-r2511

* remove unneeded stuff from modest-main (the batch mode stuff which wasn't working anyway
* cleanup the initialization, which should make it more clear what happens
* don't do separate ui/core init

pmo-trunk-r2510

* leak fix; unref the icon

pmo-trunk-r2509

* leak fixes

pmo-trunk-r2508

* Add a error_handler for send_receive operation.
* Review dimming rules of 'Cancel sending' command.
* Fixes: NB#61952

pmo-trunk-r2507

* Review dimming rules of 'delete' and 'delete folder'
* Fixes: NB#61254

pmo-trunk-r2506

2007-06-29  Murray Cumming  <murrayc@murrayc.com

* src/modest-tny-account.c:
(modest_tny_account_new_from_server_account):
Handle the TnyAccount::connection-status-changed signal, which tinymail
now uses to request a connection when it needs one, and which it uses
to tell us that something is wrong with the connection, so we can force
the accounts offline to prevent further problems.

* src/modest-defs.h:
* src/modest-init.c: (init_default_settings):
* src/modest-mail-operation.c:
(modest_mail_operation_update_account):
* src/modest-main.c: (on_show): Do not try to connect explicitly on startup.
This should not be necessary anymore. Do not set the conf key that
determined this (it was always TRUE and not set anywhere else).

* src/modest-ui-actions.h: Remove unused and unimplemented
modest_ui_actions_on_online_toggle_toggled().

pmo-trunk-r2505

* Fixed a compilation warning

pmo-trunk-r2504

* fix a memleak in the msg edit window
* fix a memleak when registering the update_interval for the alarm daemon

pmo-trunk-r2503

* src/widgets/modest-header-view-render.c:
        * Added some implementation work to retrieve the send status of
          mails in outbox. get_status_string() obtains the proper string
          for a status, get_status_of_uid() should retrieve the current
          status trying in all send queues, the cell renderer has a
          specific path for the case of outbox.
        * As currently it does not get the proper status from the
          send queue, we set always "waiting".
* src/widgets/modest-header-view-priv.h:
        * Added an enum of the types of compact cell renderer.
* src/widgets/modest-header-view.c:
        * Pass the proper mode to the compact cell renderer (incoming,
          outgoing, outbox).

pmo-trunk-r2502

* Use an user handler for 'key-press-event' signal on
ModestWindow class.
* Fixes: NB#61828, NB#61825

pmo-trunk-r2501

Updated the changelog with the fixes made on week26 29th june.

pmo-trunk-r2500

Fixed the bug 61219, changed from showing dialog to showing banner if the account already exists.

pmo-trunk-r2499

* Fixed bug in select_next function.

pmo-trunk-r2498

* fix logical error, don't call
  search_mime_part_and_child_parts
   if msg == NULL. This fixes a crash if msg happens to be NULL

pmo-trunk-r2497

* src/maemo/modest-msg-edit-window.c:
        * Now the font size toolbutton always get the default font face
          (it was getting the default font, but then it was replaced
          with Serif on first font size change).

pmo-trunk-r2496

* src/maemo/modest-msg-edit-window.c:
        * Fixed some typo errors in set_format_state function.
        * Use set_attribute instead of set_format in font size change
          handler.

pmo-trunk-r2495

2007-06-29  Murray Cumming  <murrayc@murrayc-desktop>

* src/maemo/modest-main-window.c: (connect_signals):
Don't connect to delete_event twice. Maybe this was harmless.

pmo-trunk-r2494

2007-06-29  Murray Cumming  <murrayc@murrayc.com

* libmodest-dbus-client/libmodest-dbus-client.c: Improved the documentation slightly.
* src/modest-mail-operation.c: (modest_mail_operation_xfer_folder):
Debug output to give some more clue when a folder move is not allowed.

pmo-trunk-r2493

* update the various places where the modest-window-mgr was used to register
  the header *before* the message is opened (see previous patch).
* also update the dimming rules, so opened messages will  have the delete
  button in the main window dimmed, as well, as when selecting multiple of
  which one or more are opened already.

pmo-trunk-r2492

* update the modest-window-manager; together with the next patch, this
  fixes NB#57297
- we do this by adding modest_window_mgr_register_header,
modest_window_mgr_find_registered_header. in the next patch, these will
be called *before* actually opening the message, so it will be impossible
to open the same message several times. the problem used to be that
the windows were only registered after the message was retrieved, so
plenty time to have doubles, esp. on a slow connection
- also fix a bug where really the registered *window* is returned,
instead of the GList* (casting hid this bug), this enables us to popup
a msg-view-window if it was already opened for the activated message

pmo-trunk-r2491

* Reverted a do not wanted change

pmo-trunk-r2490

* Do "pulse" when done=0 and total=0

pmo-trunk-r2489

* Do not block signal emissions into idles callbacks, because signal handlers are already executed into the main lock

pmo-trunk-r2488

* Fixes some warning about uninitialized valrables.

pmo-trunk-r2487

small change for Fixed bug 58983, changed the labels from the previous commit.

pmo-trunk-r2486

Fixed bug 58983, changed the order and labels for delete account.

pmo-trunk-r2485

* Review 'move to' callback from viewer window with pop
accounts.
* Fixes: NB#61929

pmo-trunk-r2484

* Fixes: NB#61955

pmo-trunk-r2483