Skip to content

REML lists not rendered in plain text or RTF #81

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
delphidabbler opened this issue Dec 18, 2022 · 3 comments
Closed

REML lists not rendered in plain text or RTF #81

delphidabbler opened this issue Dec 18, 2022 · 3 comments
Assignees
@delphidabbler
Labels
bug Bug report completed Work has been completed on this issue and changes have been committed to `develop` branch..

Comments

@delphidabbler
Copy link
Owner

The newly implement REML lists render correctly as HTML, but not as Plain Text or RTF.

To reproduce:

  1. Create a snippet, routine kind, with REML lists in the description.
  2. Select the new snippet
  3. Menu option File | Save Annotated Source
  4. In save dialogue choose Pascal Include File type and any comment style except "No descriptive comments"
  5. Click Preview button. Note how text in header comment has now bullets, numbers or indenting.
  6. Go back to 4, but this time choose Rich Text File type a click Preview again. Same problem.
@delphidabbler delphidabbler self-assigned this Dec 18, 2022
@delphidabbler delphidabbler added accepted Accepted for implementation / fixing bug Bug report labels Dec 18, 2022
@delphidabbler delphidabbler added the started Work has started on this issue. label Dec 21, 2022
@delphidabbler delphidabbler mentioned this issue Dec 21, 2022
Closed
@delphidabbler
Copy link
Owner Author

See also issue #82 being fixed in the same feature branch

@delphidabbler
Copy link
Owner Author

This, and issue #82, are proving very difficult. The implementation of REML / active text lists is buggy and incomplete.

Having to rewrite the REML parser and modify the active text implementation.

Also REML docs are wrong for content of <li> tags. Also need to check REML help topic.

@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 started Work has started on this issue. labels Apr 8, 2023
@delphidabbler
Copy link
Owner Author

Fix at merge commit d9da938

@delphidabbler delphidabbler mentioned this issue Apr 8, 2023
Closed
@delphidabbler delphidabbler removed this from the Confirmed for next patch milestone Apr 9, 2023
@delphidabbler delphidabbler moved this to Done - Awaiting Release in CodeSnip May 23, 2023
@delphidabbler delphidabbler moved this from Done - Awaiting Release to Done - released in CodeSnip May 23, 2023
@delphidabbler delphidabbler removed this from CodeSnip Jun 18, 2023
delphidabbler added a commit that referenced this issue Jul 13, 2023
@delphidabbler
Merge branch 'feature/81+82-reml-lists-not-rendered-in-plain-text-or-…
c7a6444 
…rtf' into develop

Resolved conflicts:
*	Src/ActiveText.UHTMLRenderer.pas
*	Src/ActiveText.UMain.pas
*	Src/ActiveText.UTextRenderer.pas
*	Src/USourceGen.pas
*	Src/UTextSnippetDoc.pas

All resolved in favour of the incoming changes from the feature branch.

Fixes #81. Fixes #82.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@delphidabbler delphidabbler

Labels
bug Bug report completed Work has been completed on this issue and changes have been committed to `develop` branch..
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@delphidabbler