Skip to content

Conversation

@martonmiklos
Copy link
Contributor

Add support drawing strikethrough texts in Input/Output/String/Numbers.

Restructured the text drawing components, depends on Open-Agriculture/AgIsoStack-plus-plus#525

@martonmiklos martonmiklos marked this pull request as draft January 7, 2025 16:01
@martonmiklos martonmiklos force-pushed the dry_numbers branch 3 times, most recently from dc924b7 to 48a6ab5 Compare January 11, 2025 22:09
@martonmiklos martonmiklos marked this pull request as ready for review January 11, 2025 22:57
@martonmiklos martonmiklos force-pushed the dry_numbers branch 2 times, most recently from 2961c60 to e05b69e Compare January 11, 2025 23:01
ad3154
ad3154 previously requested changes Mar 9, 2025
@martonmiklos
Copy link
Contributor Author

Hi @ad3154

I think all of the issues were resolved this with except this:
#73 (comment)

I have not found a better way to solve this with smart pointers. I tried to use enabled_shared_from_this but I have not been able to compile the code to store generic pointers in constructor.

@martonmiklos martonmiklos force-pushed the dry_numbers branch 2 times, most recently from eda301c to 6cbabe7 Compare November 29, 2025 19:40
@martonmiklos martonmiklos requested a review from GwnDaan November 29, 2025 22:57
@martonmiklos martonmiklos force-pushed the dry_numbers branch 3 times, most recently from 8cbd1d7 to 0c92e58 Compare November 30, 2025 10:51
@martonmiklos martonmiklos dismissed ad3154’s stale review December 23, 2025 17:48

Fixed by creating a vtObject() member

@martonmiklos martonmiklos merged commit 4d01c46 into Open-Agriculture:main Dec 23, 2025
6 checks passed
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.

3 participants