Skip to content

Refactor names in URTFUtils unit #100

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.

Sign up for GitHub

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

Jump to bottom
Closed
delphidabbler opened this issue Apr 7, 2023 · 0 comments
Closed

Refactor names in URTFUtils unit #100

delphidabbler opened this issue Apr 7, 2023 · 0 comments
Assignees
@delphidabbler
Labels
completed Work has been completed on this issue and changes have been committed to `develop` branch.. refactoring This issue requires refactoring only, not a change in functionality

Comments

@delphidabbler
Copy link
Owner

This unit has several RTF*** routines that could do with grouping together in a TRTF record w/ methods.

But, TRTF is currently used to encapsulate RTF markup. So rename TRTF as TRTFMarkup or TRTFCode.

The unit also contains TRichEditHelper which contains a single method to help loading RTF code into a RichEdit control. This doesn't sit well here. Perhaps it would be better if moved to its own unit, and maybe converted into a class helper.
@delphidabbler delphidabbler self-assigned this Apr 7, 2023
@delphidabbler delphidabbler added considering Issue under consideration. May be accepted or rejected refactoring This issue requires refactoring only, not a change in functionality labels Apr 7, 2023
@delphidabbler delphidabbler moved this to Considering in CodeSnip May 23, 2023
@delphidabbler delphidabbler moved this from Considering to Accepted in CodeSnip Apr 18, 2025
@delphidabbler delphidabbler added accepted Accepted for implementation / fixing and removed considering Issue under consideration. May be accepted or rejected labels Apr 18, 2025
@delphidabbler delphidabbler added completed Work has been completed on this issue and changes have been committed to `develop` branch.. and removed accepted Accepted for implementation / fixing labels Apr 18, 2025
@delphidabbler delphidabbler moved this from Accepted to Done - awaiting release in CodeSnip Apr 18, 2025
@delphidabbler delphidabbler removed this from CodeSnip May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@delphidabbler delphidabbler

Labels
completed Work has been completed on this issue and changes have been committed to `develop` branch.. refactoring This issue requires refactoring only, not a change in functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@delphidabbler