Skip to content
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.

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

RPA.PDF.HTML To PDF keyword problem outputting table with multiple cells #183

Closed
mikahanninen opened this issue May 19, 2021 · 2 comments
Closed
Assignees
Labels
bug Bad or unexpected behaviour

Comments

@mikahanninen
Copy link
Member

Related to beginner's course. Create File step has been added for confirmation that HTML contains required data.

*** Keywords ***
Export The Table As A PDF
    Wait Until Element Is Visible    id:sales-results
    ${sales_results_html}=    Get Element Attribute    id:sales-results    outerHTML
    Create File    ${CURDIR}${/}output${/}sales_results.html    ${sales_results_html}    overwrite=True
    Html To Pdf    ${sales_results_html}    ${CURDIR}${/}output${/}sales_results.pdf

Last rpaframework release this has worked is 7.6.0.

@mikahanninen mikahanninen added the bug Bad or unexpected behaviour label May 19, 2021
@mikahanninen mikahanninen self-assigned this May 19, 2021
@janipalsamaki
Copy link
Contributor

Did some testing with this one. The issue seems to be the current width attribute in the HTML table columns. The value there used to work back in the day.

This means that the columns are super-wide now. Thus it seems only one column is included in the generated PDF (although all the columns are probably there, they just don't fit). Apparently, some PDF dependency has changed.

There's a PR going in soon that changes the HTML output of the table in question (robocorp/rsb-intranet#20). After that is deployed, all the columns should fit again.

This means that we could ignore this issue here and close it. I can do the closing after the other PR is merged and we have verified that the change works in the prod site, too.

@janipalsamaki
Copy link
Contributor

The above-mentioned update to the HTML table structure fixed the Beginners' course issue. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bad or unexpected behaviour
Projects
None yet
Development

No branches or pull requests

2 participants