manim icon indicating copy to clipboard operation
manim copied to clipboard

Adding tests for Paragraph functionality

Open MrDiver opened this issue 3 years ago • 2 comments

Description of proposed feature

There has been a lot of work done in

  • #2721 to address the issues with formatting of Paragraph.

To further improve the tested capabilities of the Library i suggest adding tests for the functionality presented in #2721 and also for the following issues to check if they have been fixed in relation to the PR.

Tests should also check for the following issues and close them with the new PR which adds tests if they're fixed.

  • [ ] #2673
  • [x] #2366
  • [ ] #2014

MrDiver avatar Jun 19 '22 10:06 MrDiver

I proposed a new text api which would completely replace Paragraph and let pango (the library which we use internally) handle it. https://github.com/ManimCommunity/manim/issues/2355

Do we really need tests for something which will be removed soon?

naveen521kk avatar Jun 19 '22 10:06 naveen521kk

🤔 then why do we fix functionality which will be removed ?

MrDiver avatar Jun 19 '22 10:06 MrDiver