New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Select all checkbox #150
Select all checkbox #150
Conversation
Personally I would like it to not be a checkbox like all the other normal ones. Maybe a button, next to the "Select books" label would be better. I wonder if it is a common use case. I actually haven't really used it like that at all, except in testing, since I am never actively reading more than more than one book at the same time. |
8702bc5
to
8eaeec9
Compare
I wanted to do a "toggle all" button to begin with but the logic could be complicated when only some checkboxes are ticked, so it'd be cleaner to just have 2 separate buttons VS.mov |
Would this not work? checkbox.setChecked(not checkbox.isChecked()) Again my concern is that the buttons take up space for the selections Honestly I'm not sure this is a good idea |
8eaeec9
to
d15b1cb
Compare
Ok, now there's no new UI elements, the checkboxes simply keep their checked status after an import |
don't actually store the python list in the settings, since it would be written as some QVariant binary thing. Store them with json strings (json.dumps, loads) and read them out |
also let's just get rid of all the truncate_middle calls, because I think they cause issues with the filtering and don't really have benefits anyways (the viewport can scroll). |
d15b1cb
to
0402721
Compare
simple checkbox to make it easier to extract new words when reading multiple books
2024-04-18.23-17-54.mp4