PDF Viewing Feature

[VitorVieiraZ]

This feature can be used to open PDF files using the user’s system default application viewer, and can open formats such as links, PDFs, Docx, etc.

1. In a form that allows multiline entries, add text to the "Form Layout" widgets. This can be done in the "Default Value" section, or within the "HtmlWidget" field.
2. To embed a PDF file in the project, use the following HTML format:

• For the "Default Value" section, enter:
  - '<a href="project://dummy.pdf">Open File</a>' (include the HTML code inside single quotes).
• For direct entry in the "HtmlWidget", use:
  - <a href="project://dummy.pdf">Open File</a> (able to enter the HTML code without quotes).

3. After synchronizing the project, check the app to ensure that the fields display the link reference correctly.

HTML Link	Invalid File

5. When clicking into the HTML button, the PDF file will be opened using the default application of the system.
6. Additional Note:

• In the case of a file inside a folder in the project directory, use the following format:
  <a href="project://files/sample.pdf">Open PDF File inside folder "files"</a>
  sample.pdf file inside 'files' folder
• In the case of a link use the following format:
• <a href="https://www.qgis.org/">Visit QGIS</a>

[tomasMizera]

Please merge master branch here to make CI working. We can then proceed with testing

[github-actions]

Pull Request Test Coverage Report for Build 9207444860

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 275 unchanged lines in 3 files lost coverage.
• Overall coverage decreased (-0.03%) to 59.953%

---

Files with Coverage Reduction	New Missed Lines	%
input/app/ios/iosutils.cpp	2	17.24%
input/app/androidutils.cpp	12	4.76%
input/app/inpututils.cpp	261	50.74%

Totals
Change from base Build 9161804447:	-0.03%
Covered Lines:	7668
Relevant Lines:	12790

---

💛 - Coveralls

[tomasMizera]

Windows build is failing here @VitorVieiraZ, please check the error log

[tomasMizera]

Just one small comment, otherwise it looks good :)

[tomasMizera]

Let's merge to master and then test from there

[jozef-budac]

I just tried this feature on the APK from this ticket and I always get an error message, that the file does not exist. Can you please check it?

My project is tc05_forms_derived and there is a tab called Files

[VitorVieiraZ]

I just tried this feature on the APK from this ticket and I always get an error message, that the file does not exist. Can you please check it?

My project is tc05_forms_derived and there is a tab called Files

Tests successfully conducted. In the case of a file inside a folder in the project directory, use the following format:

<a href="project://files/sample.pdf">Open PDF File inside folder "files"</a>

sample.pdf file inside 'files' folder

[jozef-budac]

@VitorVieiraZ I checked the project and I don't see a problem in the configuration. Do you see it? I tried it again and I was able to open TXT and DOCX files, but the PDF file was still unavailable.
The configuration for it is '<a href="project://sample.pdf">Open PDF File</a>' and the sample.pdf file it is located in the root directory of the project. Am I doing something wrong?

Screen_Recording_20240514_094720_My.Files.mp4

[jozef-budac]

Now it looks great! I didn't find any more problems.

[jozef-budac]

Unfortunately, the files in sub-directories stopped working

Screen_Recording_20240515_230806.mp4

[tomasMizera]

@VitorVieiraZ please merge master branch here - it will upload the build to iOS then too :)

[tomasMizera]

@jozef-budac Can you please retest this PR? We are mainly interested in PDFs so if they work we can merge it and fix other file types later. This is the build to test it with: https://github.com/MerginMaps/mobile/actions/runs/9158230910?pr=3353

[jozef-budac]

Right now I can't install the package

[tomasMizera]

@jozef-budac you can now test it on iOS too :)

[tomasMizera]

@jozef-budac you can now test it on iOS too :)

24.5.622711

[tomasMizera]

I can confirm that the apk is now invalid @VitorVieiraZ

[tomasMizera]

It says Failure, intent tag may have at most one action

[tomasMizera]

After removing these two lines

mobile/cmake_templates/AndroidManifest.xml.in

Lines 110 to 111 in d3293db

	<action android:name="android.intent.action.OPEN_DOCUMENT" />
	<action android:name="android.intent.action.GET_CONTENT" />

the code works as expected for me in the tc05_forms_derived

[tomasMizera]

@VitorVieiraZ Can you please remove these two lines

mobile/cmake_templates/AndroidManifest.xml.in

Lines 110 to 111 in d3293db

	<action android:name="android.intent.action.OPEN_DOCUMENT" />
	<action android:name="android.intent.action.GET_CONTENT" />

I believe we are ready for merge then 🥳

[jozef-budac]

PDF files are working now of both Android and iOS>
The docx and xlsx are not working on Android. But if only PDF is mandatory, we can go with this to production

[tomasMizera]

For me all of them worked :)

…

On Thu, 23 May 2024 at 14:39, Jozef Budáč ***@***.***> wrote: PDF files are working now. The docx and xlsx are not. But if only PDF is mandatory, we can go with this to production — Reply to this email directly, view it on GitHub <#3353 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AFLI4IRKLZ744SVUKYZLMCDZDXPONAVCNFSM6AAAAABGUMKYDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRXGAYDKMRWG4> . You are receiving this because your review was requested.Message ID: ***@***.***>

[jozef-budac]

Here is a video from Android 14

Screen_Recording_20240523_145441.mp4

[tomasMizera]

@jozef-budac do you have some app to open xls installed? If yes, we can debug then. Anyways,let's merge it as pdfs are working now :)

[tomasMizera]

🚀 🔥

[jozef-budac]

@tomasMizera Yes I have programs to open such files. In some of the previous APKs I was able to open the files. I wrote a note "Now it looks great! I didn't find any more problems." But then it doesn't work anymore.