genanki icon indicating copy to clipboard operation
genanki copied to clipboard

genanki.Note should have a field for the paths to its media files

Open kerrickstaley opened this issue 5 years ago • 1 comments

Instead of having a .media_files attribute on Package, it may be more ergonomic to have a .media_files attribute on each genanki.Note instance. The .media_files for the overall package will then be computed by combining the lists for each individual Note (removing duplicates).

Need to think a little more about whether this design will handle all use-cases and whether it will be genuinely easier to use. One consideration is "static" media files; those still need to be specified at the package level.

kerrickstaley avatar Nov 29 '20 06:11 kerrickstaley

I vote keeping it the way that it is now. The same media file can be referenced in many different ways. Might as well save yourself a headache engineering something so specific in and force the user to make sure they know what they want when specifying the media writes. The greatest utility your module provides is not having to manually generate that sqlite db. KISS where possible.

yash-fn avatar Feb 03 '22 22:02 yash-fn