Philantrop / calibre-marvin-manager

A calibre User Interface Action plugin supporting Marvin
6 stars 6 forks source link

Request: Do not force MXD window to the front when processing #23

Closed WyndhamMisio closed 10 years ago

WyndhamMisio commented 10 years ago

I have this as a request although it really feels like a bug to me.

When performing processing on several books, such as updating metadata, MXD forces the MXD window to the front (on top), as well as the progress bar - floating status dialog, every time another book is processed and this means it is virtually impossible to do anything else on your desktop until all the processing has completed.

Calibre flashes the icon on the task bar to indicate a process has completed. Would this not be possible for MXD?

WyndhamMisio commented 10 years ago

Note that the floating status dialog is always on top, but that on its own doesn’t prevent a user from doing anything else on the PC.

GRiker commented 10 years ago

Please update to MXD version 1.0.93.

This version puts all status updates in the upper right corner of the MXD window and disables user interaction while updates are active.

This should resolve the issue of the floating status window interfering with other apps.

GRiker commented 10 years ago

Please update to MXD version 1.0.95.

This version adds a Cancel button for certain operations, and aggregates errors when updating metadata for multiple books. It has been lightly tested. Please try updating metadata on the three books that you reported (State of Siege, Splinter Cell (2004), Fallout (2007) to test that the error reporting is handled properly at the end of the update rather than after each book. I would suggest that you update just a few books, maybe 10, with those books included.

WyndhamMisio commented 10 years ago

Installed MXD 1.0.95

Connected and selected the books suggested plus about three others, and chose to update metadata Calibre to Marvin.

Like the Cancel button.

Process completed with combined log (nice), but showed unsuccessful updates.

Some other books apparently successfully updated: The Coffee Trader – David Liss, Code to Zero – Ken Follett, The Chancellor Manuscript – Robert Ludlum, The Chamber – John Grisham.

I forgot to run this in debug mode, so I will do this again oops

From: GRiker [mailto:notifications@github.com] Sent: 08 December 2013 22:59 To: GRiker/calibre-marvin-manager Cc: WyndhamMisio Subject: Re: [calibre-marvin-manager] Request: Do not force MXD window to the front when processing (#23)

Please update to MXD version 1.0.95 https://www.dropbox.com/s/six0c50asmvkjm8/Marvin%20XD.zip .

This version adds a Cancel button for certain operations, and aggregates errors when updating metadata for multiple books. It has been lightly tested. Please try updating metadata on the three books that you reported (State of Siege, Splinter Cell (2004), Fallout (2007) to test that the error reporting is handled properly at the end of the update rather than after each book. I would suggest that you update just a few books, maybe 10, with those books included.

— Reply to this email directly or view it on GitHub https://github.com/GRiker/calibre-marvin-manager/issues/23#issuecomment-30095623 . https://github.com/notifications/beacon/abTCOjnj04utPBsVQiZPJBdrFHF0PBo5nSNmivMKiZ4KqvrXMGZZY8ao8rvnQ0pU.gif


This email is free from viruses and malware because avast! Antivirus protection is active. http://www.avast.com

WyndhamMisio commented 10 years ago

Second run at the job, this time with debug set.

Question: Is there any chance of setting the sort to be by title (within condition) – it seems to be in reverse title order. If you sort on column headings then you lose the overall grouping by condition. Perhaps there could be an option to retain condition groupings when sorting.

Debug txt to follow

From: GRiker [mailto:notifications@github.com] Sent: 08 December 2013 22:59 To: GRiker/calibre-marvin-manager Cc: WyndhamMisio Subject: Re: [calibre-marvin-manager] Request: Do not force MXD window to the front when processing (#23)

Please update to MXD version 1.0.95 https://www.dropbox.com/s/six0c50asmvkjm8/Marvin%20XD.zip .

This version adds a Cancel button for certain operations, and aggregates errors when updating metadata for multiple books. It has been lightly tested. Please try updating metadata on the three books that you reported (State of Siege, Splinter Cell (2004), Fallout (2007) to test that the error reporting is handled properly at the end of the update rather than after each book. I would suggest that you update just a few books, maybe 10, with those books included.

— Reply to this email directly or view it on GitHub https://github.com/GRiker/calibre-marvin-manager/issues/23#issuecomment-30095623 . https://github.com/notifications/beacon/abTCOjnj04utPBsVQiZPJBdrFHF0PBo5nSNmivMKiZ4KqvrXMGZZY8ao8rvnQ0pU.gif


This email is free from viruses and malware because avast! Antivirus protection is active. http://www.avast.com

GRiker commented 10 years ago

Thank you for confirming that the this issue may now be closed. It seems to be working properly for you, as the MXD window or its children do not obscure other app windows.

With respect to the 3 books that don't update, Marvin is reporting errors during the operation, and MXD is reporting them to you. That's working correctly. As to why these books aren't updating, that's a separate issue. Please open a new issue with a title something like Unable to update metadata for some books. Include a copy of the error messages you're seeing.

Question: Is there any chance of setting the sort to be by title (within condition) – it seems to be in reverse title order. If you sort on column headings then you lose the overall grouping by condition. Perhaps there could be an option to retain condition groupings when sorting.

If you want to request an enhancement concerning sorting, please do so in a separate issue.

I am now closing this issue, as the original problem has been resolved.