185 fixing table formatting in PPT outputs

[kartikeyakirar]

this PR fixes #185

Inspired by autoslidR, I have integrated flextable functionality to convert "data.frame," "rtables," "TableTree," and "ElementaryTable" into flextable, which offers a more accurate representation. In this context, I have enhanced the TableBlock to facilitate the conversion of tables into flextable.

Output screenshot

[github-actions]

Code Coverage Summary

Filename              Stmts    Miss  Cover    Missing
------------------  -------  ------  -------  ----------------------------------------------------------------------------------
R/AddCardModule.R       140       1  99.29%   195
R/Archiver.R             25       0  100.00%
R/ContentBlock.R         18       2  88.89%   38-44
R/DownloadModule.R      207      49  76.33%   89-95, 138, 163-168, 177-181, 184-188, 196-200, 203-207, 214-218, 221-225, 262-266
R/FileBlock.R            13       0  100.00%
R/NewpageBlock.R          2       0  100.00%
R/PictureBlock.R         30       2  93.33%   15, 79
R/Previewer.R           295      56  81.02%   183, 197, 199-202, 205, 208-216, 325-369
R/RcodeBlock.R           15       0  100.00%
R/Renderer.R             90      19  78.89%   96, 104, 113, 115-133
R/ReportCard.R           77       4  94.81%   180, 219, 224, 245
R/Reporter.R             94       1  98.94%   254
R/ResetModule.R          55       0  100.00%
R/SimpleReporter.R       29       0  100.00%
R/TableBlock.R            9       0  100.00%
R/TextBlock.R            13       0  100.00%
R/utils.R               139      69  50.36%   7, 38-97, 99, 102-109, 158-160
R/yaml_utils.R           74       2  97.30%   41, 239
TOTAL                  1325     205  84.53%

Diff against main

Filename          Stmts    Miss  Cover
--------------  -------  ------  --------
R/Renderer.R         +6      +6  -5.63%
R/TableBlock.R       +1       0  +100.00%
R/utils.R           +61      +3  +34.98%
TOTAL               +68      +9  +0.12%

Results for commit: 214841d

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[github-actions]

Unit Tests Summary

    1 files    18 suites   13s ⏱️
202 tests 202 ✔️ 0 💤 0 ❌
341 runs  341 ✔️ 0 💤 0 ❌

Results for commit 0e9765b.

♻️ This comment has been updated with latest results.

[gogonzo]

I just checked exploratory example from teal.gallery and table is not included in the reporter previewer. Instead, table is printed to graphical device, so extra window in my IDE gets open

[kartikeyakirar]

Instead, table is printed to graphical device, so extra window in my IDE gets open

In PR I was saving a 'flextable' instead of 'rtables,' in rds which was not being converted into an HTML element in preview module. This problem has been fixed using the 'flextable::htmltools_value' function.

[averissimo]

Great job @kartikeyakirar 💯

Feature-wise it performs very well! I found no issues when testing against cards that contained a table or table + plot (see screenshot below)

It works with {teal.gallery} longitudinal app and a bunch of others.

I have some comments on the code that I think would help with the readability of the code for "future us" 😅

[averissimo]

Looks good, great work @kartikeyakirar 🎉

2 small (last) suggestions to keep up with test naming convention. Feel free to merge after 😁

I have addressed the requested modifications to resolve the issue of the table not appearing in the preview. However, the request still remains in the checklist. @gogonzo , I will submit another pull request if further changes are necessary.