Commit Graph

2404 Commits

Author SHA1 Message Date
49ad387908 Merge pull request #228 from YACReader/feature/make_comic_vine_dialog_more_keyboard_friendly
Feature: make comic vine dialog more keyboard friendly
2021-03-06 22:33:02 +01:00
f4112e1190 Update what's new + CHANGELOG 2021-03-06 21:24:53 +01:00
3fcf31d936 Enable/disable comics actions after updating search results
Before this commit starting a search when an empty folder or an empty
reading list was selected left all comics actions disabled. Fixes #213.

When search mode is exited, we always call either
YACReaderNavigationController::loadFolderInfo() or
YACReaderNavigationController::loadListInfo(). Both of them call
LibraryWindow::disableComicsActions(), so the enabled/disabled state of
the comics actions stays up-to-date at all times.
2021-03-06 21:01:20 +01:00
c4333915c9 Library: don't leak DB model objects at exit
Currently these objects are created once at program startup and are
never destroyed. Printing debug messages in the models' destructors
confirms the leaks and proves that with this fix the objects are
destroyed at Library exit.
2021-03-06 21:00:31 +01:00
5fd5ff5b64 DB model classes: use explicit, nullptr, override 2021-03-06 21:00:31 +01:00
c5de729635 Set the default button based on the current context 2021-03-06 18:07:29 +01:00
5303e247b7 Always use toggleSkipButton 2021-03-06 18:06:05 +01:00
f98ce43465 Add a shortcut to going back 2021-03-06 18:03:04 +01:00
5459ff78cf Disable close button as the default when pressing enter
It can be triggered using ESC
2021-03-06 18:02:41 +01:00
b5251f9416 Merge pull request #224 from YACReader/feature/comic_vine_html_support
feature: Embrace HTML coming from Comic Vine
2021-03-06 09:53:02 +01:00
4c8259e835 Update what's new + CHANGELOG 2021-03-06 09:05:35 +01:00
0f6b203d34 Embrace HTML coming from Comic Vine 2021-03-06 09:05:35 +01:00
e5526de0af Render: Protect against race condition in setNumPages (#220)
* Render: Use sendPostedEvents to force processing of queued events when removing old comic objects
2021-03-04 16:45:52 +01:00
64f7e63a2a Azure: Use qt@5 brew package to install Qt5 on macOS 2021-03-04 15:55:29 +01:00
81f9b74fa2 Merge pull request #222 from YACReader/feature/drop_qtscript
fix: Replace QtScript with QJson*
2021-02-27 13:58:56 +01:00
1a527c51f9 Do not try to pack QtScript in windows installer 2021-02-27 13:09:04 +01:00
6a447c2e5d Stop installing qtscript in azure pipelines 2021-02-27 12:10:25 +01:00
51a9c72d05 Replace QtScript with QJson*
QtScript is deprecated and this was needed to start supporting Qt6
2021-02-27 12:02:57 +01:00
32e1db7134 Disable standard assert() macro in Release builds 2021-02-18 19:49:43 +01:00
6f182b2bb3 Extract CONFIG += c++11 into common config.pri 2021-02-18 19:49:43 +01:00
cb822c4f47 Add CONFIG += silent to build to reduce log pollution
Qmake by default creates lots of unnecessary information during builds
which makes it hard to extract warnings and errors from the log.
Add the undocumented silent parameter to make it less talky.
2021-02-18 19:49:16 +01:00
93596a4972 Restore needed lock 2021-02-08 09:02:19 +01:00
b55c442590 Add missing import
It fixes compilation with some compilers
2021-02-08 08:38:15 +01:00
4c93c70de6 Use the right mutex when the queue is modified 2021-02-08 08:32:42 +01:00
3a9bec55f8 Library: don't print errors when search edit's text changes
This was a temporary debug output. Not useful anymore.
2021-02-05 09:34:13 +01:00
2e90e38259 Merge pull request #193 from vedgy/dont-leak-folder-processor
Library: don't leak folderQueryResultProcessor at exit
2021-02-04 21:26:11 +01:00
6d5f3d7de3 Use the query search queue in FolderQueryResultProcessor 2021-02-04 19:15:58 +01:00
64d0e22fb5 Library: don't leak folderQueryResultProcessor at exit
FolderQueryResultProcessor has a ConcurrentQueue data member. The leak
meant that the thread was not joined before exit.
2021-02-04 17:58:33 +02:00
117b02fc7f Merge pull request #192 from vedgy/fix-info-comics-view-crashes
Library: fix InfoComicsView crashes
2021-02-01 18:38:50 +01:00
f88af80379 Library: fix InfoComicsView crashes
InfoComicsView constructor is the only function that connects to
FlowView's currentCoverChanged signal. Neither of the slots connected to
this signal handles the argument value index==-1. So when FlowView emits
this signal with index==-1, YACReaderLibrary crashes. Returning early
from either ComicsView::updateInfoForIndex() or
InfoComicsView::setCurrentIndex() when index==-1 is not sufficient - the
crash happens in the other slot then. Let us skip emitting the signal in
FlowView if index==-1 rather than return early from both slots.

Steps to reproduce 1:
1. Launch YACReaderLibrary version that matches the version of the
default library database. Alternatively, select a compatible library
after starting the application.
2. If InfoComicsView is not active, switch to it.
3. (optional) Switch to another comics view out of InfoComicsView.
4. Quit YACReaderLibrary. The application crashes during exit - after
the "YACReaderLibrary closed with exit code : 0" message is printed.

Steps to reproduce 2:
1. Launch a YACReaderLibrary version newer than the version of the
default library database.
2. Click the "No" button in the "Update needed" dialog that pops up.
3. Change between comics views until InfoComicsView becomes active. If
this view was active at the beginning, switch through all the views to
get back to it. At this point YACReaderLibrary crashes.
2021-02-01 11:27:18 +02:00
350234b099 Merge pull request #187 from YACReader/feature/fix-server-start-up-sequence
Fix launch sequence to avoid http port collisions
2021-01-28 21:04:49 +01:00
f4b2b3d8e3 Update comments 2021-01-28 19:56:35 +01:00
64478ca63f Typo 2021-01-28 19:54:10 +01:00
d96d01d963 Fix launch sequence to avoid http port collisions 2021-01-25 15:48:15 +01:00
f183fc1dd2 Fallback to a text close button if we can't load the svg image
If the svg plugin is missing the user can't see where the close button is
2021-01-22 16:35:58 +01:00
3a6fb91327 Install libqt5svg5-dev package in linux 2021-01-22 11:22:05 +01:00
74840f9914 Enable SVG explicitly 2021-01-22 10:48:13 +01:00
b80d85a49e Merge pull request #186 from YACReader/feature/whatsnew
Add what's new dialog
2021-01-21 23:28:08 +01:00
5c48fe5783 format 2021-01-21 22:31:56 +01:00
58fdf0af23 Add what's new dialog 2021-01-21 21:39:22 +01:00
3f6c44ce7a Fix SQL update query for setting manga in all the comics in a folder 2021-01-17 14:05:14 +01:00
710e9d9026 Merge pull request #176 from smalewski/develop
Fixes move action shortcuts not working
2021-01-17 11:45:32 +01:00
679bd1812f Update CHANGELOG 2021-01-17 11:44:38 +01:00
dee557c518 Use a constant reference for passing QKeySequence to moveAction 2021-01-17 11:11:25 +01:00
08ce8c3c65 Fixed move action shortcuts 2021-01-17 11:06:45 +01:00
c73f28cca1 KeySequence passed by reference 2021-01-17 11:06:45 +01:00
fccf9ab0af Fixed shortcuts for move actions 2021-01-17 11:06:45 +01:00
77bfd58c35 Merge pull request #185 from YACReader/feature/improve_manga_support
feature: improve manga support
2021-01-17 10:13:44 +01:00
20c772797b Fix typo 2021-01-16 18:59:53 +01:00
a911a4fc53 Update action names in macos menus 2021-01-16 18:45:47 +01:00