|
|
#46 | |
|
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,157
Karma: 1512904
Join Date: Sep 2017
Location: Buenos Aires, Argentina
Device: moon+ reader, kindle paperwhite
|
Quote:
I asked him to do the steps you mentioned. |
|
|
|
|
|
|
#47 |
|
Chalut o/
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 714
Karma: 721246
Join Date: Dec 2017
Device: Kobo
|
Calibre 7 arrive right aroud the corner, so I present the pre-release of Comments Cleaner that now support Category Notes cleanning (ammong many other thing)
You can download the beta version on the Github page. Note that this version is for Calibre 5 and above. |
|
|
|
|
|
#48 |
|
want to learn what I want
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 2,414
Karma: 9656783
Join Date: Sep 2020
Device: Libra Colour
|
|
|
|
|
|
|
#49 |
|
Chalut o/
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 714
Karma: 721246
Join Date: Dec 2017
Device: Kobo
|
version 1.17.1 - 2025/09/29
Hey, do you remeber the issue that was raise and that I was unable to reproduce and correct it? Welll, I migrated to Linux and guess what? Fatal freeze. The most obscure aspect of this issue is that it only appears when Comments Cleaner is run on its own, not when it is run via Action Chain. The worst is that come from my common_utils lib, so Mass Search/Replace and ePub Extended Metadata was also affected. The dump, it was very easy to fix. I apologize to everyone who has had to deal with this issue, which I have not been able to fix until today, but unfortunately it is clear that it has been an OS-specific issue all along. I could only properly debug and fix it once I was personally confronted with it and could therefore tinker with my code in every possible way to be sure of finding the source of the problem. Have a nice day, I will have a nice sleep. |
|
|
|
|
|
#50 |
|
Enthusiast
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 37
Karma: 39966
Join Date: Aug 2023
Device: Kindle
|
Subject: Request: Add support for icon overrides / theming
Hi, Could you please add support for Calibre's built‑in icon theming in a future update? Currently your plugin uses a custom get_icon() function, which prevents users from overriding the toolbar icon with their own custom images or Calibre icon themes. The fix is simple: replace the custom function with Calibre's native get_icons(). For example: python # Instead of a custom get_icon() function... # Use Calibre's built-in function: from calibre.customize.zipplugin import get_icons icon = get_icons('icon.png', 'Your Plugin Name') self.qaction.setIcon(icon) This would allow users to place a custom icon.png in: calibre/resources/images/Your Plugin Name/icon.png And it would also make the plugin respect the user's active Calibre icon theme. Your plugin plugin also has a notes feature that uses NOTES_ICON. So you would need to update both icons to fully support theming. Thank you for considering this. I really appreciate your work on this plugin. Thanks |
|
|
|
|
|
#51 | |
|
Chalut o/
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 714
Karma: 721246
Join Date: Dec 2017
Device: Kobo
|
Quote:
The thing is that, unlike your example, the main icon use "plugin.png" as file name (not "icon.png"). To see the real names used by the plugin, so that you can overide it, open the ZIP file of the plugin and look inside the "image/" folder. The get_icon() function may have confused you, but in fact I uses a custom function with the same name. |
|
|
|
|
|
|
#52 |
|
Chalut o/
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 714
Karma: 721246
Join Date: Dec 2017
Device: Kobo
|
New version 1.20:
Add a experimental option to add the comment cleaner action into the toolbar of the Comments and Note editors (disable by default) |
|
|
|
|
|
#53 |
|
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,157
Karma: 1512904
Join Date: Sep 2017
Location: Buenos Aires, Argentina
Device: moon+ reader, kindle paperwhite
|
Updated and tested Spanish and French translations.
|
|
|
|
|
|
#54 |
|
Chalut o/
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 714
Karma: 721246
Join Date: Dec 2017
Device: Kobo
|
thanks, updated.
|
|
|
|
|
|
#55 | |
|
null operator (he/him)
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 22,785
Karma: 34252730
Join Date: Mar 2012
Location: Sydney Australia
Device: none
|
Quote:
If it's enabled, suggest Comments Cleaner be added to the context menu for Normal view, or the Advanced flyout. BR |
|
|
|
|
|
|
#56 | |
|
Chalut o/
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 714
Karma: 721246
Join Date: Dec 2017
Device: Kobo
|
Quote:
A context menu will requiring a official support from calibre. However, since I do that, I'm starting to think seriously about propose such thing but no guarantee at all. |
|
|
|
|
|
|
#57 |
|
null operator (he/him)
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 22,785
Karma: 34252730
Join Date: Mar 2012
Location: Sydney Australia
Device: none
|
Compromise, add shortcut (Ctrl+T) for Toggle (Show/Hide) Toolbar.
BR |
|
|
|
![]() |
|
Similar Threads
|
||||
| Thread | Thread Starter | Forum | Replies | Last Post |
| [GUI Plugin]Clean Comments | Terisa de morgan | Plugins | 17 | 08-15-2026 08:27 AM |
| [GUI Plugin] Manga plugin | mastertea | Plugins | 7 | 10-30-2025 12:13 AM |
| [Conversion Plugin] Language Cleaner (profanity filter) | jdanders | Plugins | 12 | 07-08-2023 09:21 PM |
| [GUI Plugin] Save Virtual Libraries To Column (GUI) | chaley | Plugins | 14 | 04-04-2021 05:25 AM |
| [GUI Plugin] Plugin Updater **Deprecated** | kiwidude | Plugins | 159 | 06-19-2011 12:27 PM |