Skip to content

Feature/keyword index#42

Merged
jya-dev merged 6 commits intojya-dev:masterfrom
stanislawkuich:feature/keyword_index
Oct 6, 2024
Merged

Feature/keyword index#42
jya-dev merged 6 commits intojya-dev:masterfrom
stanislawkuich:feature/keyword_index

Conversation

@stanislawkuich
Copy link
Copy Markdown

Added keywords created in supernote tablet to newly generated pdf as annotations. Tested on Chauvet 2.15.29 version.
Please, let me know if you have any questions or suggestions to improve the code. Its was written under specific use case so source code might be not efficient

Copy link
Copy Markdown
Owner

@jya-dev jya-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for using my tool and helping me improve.
I wrote a review comment. I would appreciate it if you could consider correcting it.

Comment thread supernotelib/converter.py Outdated
c = canvas.Canvas(buf, pagesize=self.pagesize)
keywords = self.note.get_keywords()
for n, img in enumerate(imglist):
print('Page: '+str(n))
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is good to suppress debug output.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jya-dev , Thanks for feedback - Debug output has been suppressed

Comment thread supernotelib/converter.py Outdated
found.append(keyword)
for i in found:
try:
print('Added Keyword: '+i.get_keyword()+' Keyword Page: '+str(i.get_page_number()))
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is good to suppress debug output.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jya-dev , Thanks for feedback - Debug output has been suppressed

Comment thread supernotelib/converter.py Outdated
Comment on lines +387 to +389
c.setTitle('Notes')
c.setSubject('Notes')
c.setAuthor('Author')
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these lines are unnecessary.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jya-dev , Thanks for feedback - It has been removed

Comment thread supernotelib/cmds/supernote_tool.py Outdated
parser_convert.add_argument('--exclude-background', action='store_true', default=False, help='exclude background and make it transparent (PNG and SVG are supported)')
parser_convert.add_argument('--pdf-type', choices=['original', 'vector'], default='original', help='select PDF conversion type')
parser_convert.add_argument('--no-link', action='store_true', default=False, help='disable links in PDF')
parser_convert.add_argument('--no-keyword', action='store_true', default=False, help='disable keywords in PDF')
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to set annotation feature disable by default
because annotation icons hide handwriting on Supernote tablets and make it hard to read.
This feature is useful on PC, but I would like to prioritize usability on Supernote tablets.

20240804_205616

…been disabled by default\n3. It has been removed unnecessary parts of code"
@stanislawkuich
Copy link
Copy Markdown
Author

Hi @jya-dev , Thanks for feedback - Keyword Annotation feature has been disabled by default

@jya-dev jya-dev merged commit 3ded7d9 into jya-dev:master Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants