Skip to content
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

Add invisible text processing #276

Conversation

iljukhaput
Copy link
Contributor

No description provided.

@dimkanovikov
Copy link
Member

И что, мы теперь позволим пользователям загрузить любую PDFку и выдать за свою?)

@iljukhaput
Copy link
Contributor Author

Если распространение такого ПО не нарушает никакие законы, то не вижу в этом проблем)
Или ты предлагаешь как-то ограничить это?

@dimkanovikov
Copy link
Member

да это я так) мы на этом внимание особо заострять не будем - просто мы хорошо делаем свою работу)

@iljukhaput
Copy link
Contributor Author

Пираты - тоже люди, в конце концов))

@dimkanovikov
Copy link
Member

image
Вот тут, если не нашлось, то может возвращать типа объект полностью непрозрачный? или numeric_limits::quiet_nan?

@dimkanovikov
Copy link
Member

В остальном - капец там магия :)

@iljukhaput iljukhaput force-pushed the feat/992-invisible-text-processing branch from cba4e11 to dcd1bcc Compare October 8, 2024 04:12
@iljukhaput
Copy link
Contributor Author

Вот тут, если не нашлось, то может возвращать типа объект полностью непрозрачный? или numeric_limits::quiet_nan?

переделал на возврат полностью непрозрачного
невалидное значение тут, думаю, ни к чему все же, т.к. если альфа не задана, то по идее текст является просто непрозрачным

@iljukhaput
Copy link
Contributor Author

я ещё понял, что там надо бы некоторые моменты перенести в другие модули, ну и в целом чуть порядок навести хочется.
планирую после ремарок сделать отдельный коммит, в котором будет только рефакторинг

@dimkanovikov dimkanovikov merged commit 09e0831 into story-apps:master Oct 8, 2024
5 checks passed
@iljukhaput iljukhaput deleted the feat/992-invisible-text-processing branch October 15, 2024 04:48
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