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

Attempt to make page breaks work again, not complete #72

Closed
wants to merge 1 commit into from

Conversation

tmccormi
Copy link
Contributor

@tmccormi tmccormi commented Oct 9, 2015

Rod or anyone.

This is a change that works when run from the appointments report, sort of. This seems like it would be simple, but it has never been simple.

This change works good in Chrome with margins and headers turned off or in Firefox if you select to shrink the pages to 90% or 95%

But ... for some reason there is a blank first page, and the first superbill on the list comes out blank.

Any ideas how to make this reliable short of rewriting it to produce native PDF?

@tmccormi
Copy link
Contributor Author

tmccormi commented Oct 9, 2015

just to be clear, if there are 5 appointments on the report you get 6 pages of output, 2 blank and the last 4 appts are fine.

@Wakie87
Copy link
Member

Wakie87 commented Sep 7, 2016

Hi @tmccormi

We thank you for your submitting a pull request to OpenEmr. Without contributions like this from the community these types of projects go stale.

Currently we are going through and doing a spring clean of old and outdated pull requests. We are giving contributors the chance to update the code they submitted if they think it is still relevant, close the pull requests themselves, or if we do not hear back within 3 days we will close the pull request ourselves. If you believe that closing the pull request is an error, let us know and we will look at re-opening it.

Thanks

The OpenEMR Team
http:https://www.open-emr.org/

@bradymiller bradymiller closed this Dec 8, 2016
adunsulag added a commit to DiscoverAndChange/openemr that referenced this pull request Aug 2, 2024
Changes up get patient documents to do the following.
Change verbiage from "Select All Toggle" to "Select All Documents".- Fixes openemr#70
Add info message to say zip file will be downloaded - Fixes openemr#71
Change category path structure to use folder separator instead of
hyphens and fix display if files stored in root category - Fixes openemr#72
adunsulag added a commit to DiscoverAndChange/openemr that referenced this pull request Aug 2, 2024
Changes up get patient documents to do the following.
Change verbiage from "Select All Toggle" to "Select All Documents".- Fixes openemr#70
Add info message to say zip file will be downloaded - Fixes openemr#71
Change category path structure to use folder separator instead of
hyphens and fix display if files stored in root category - Fixes openemr#72
adunsulag added a commit that referenced this pull request Aug 9, 2024
Changes up get patient documents to do the following.
Change verbiage from "Select All Toggle" to "Select All Documents".- Fixes #70
Add info message to say zip file will be downloaded - Fixes #71
Change category path structure to use folder separator instead of
hyphens and fix display if files stored in root category - Fixes #72
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants