QFileAssetDialog does not work inside a QForm

[alex94040]

Only works inside a panel - there's an assumption inside that it has an objParentControl, not an objForm.

[alex94040]

Fix for the issue, with proper indentation

[enzo]

Hello folks

I'm checking the latest release 1.0 RC2, but I have an issue with QFileAsset, because in previous version the QDialog for QFileAsset was rendered outside the parent control. i.e

<form>
<control-fileasset>
<control-XXXXX>
<control-YYYYYY>
<control-fileasset-dialog> <--------- RIGHT PLACE
</form>

But using the latest version the html I get is

<form>
<control-fileasset>

<control-fileasset-dialog> <--------- WRONG PLACE

</control-fileasset>

<control-XXXXX>
<control-YYYYYY>
</form>

The problem with this structure is the dialog is not visible or only in part depending the parent width and height.

I tried to figure out by myself, but I couldn't find the source of this behavior

If someone could guide me in what part of code I could fix, I could fix by myself.

Regards.

enzo

[basilieus]

Looks fine alex, go ahead and patch.

[marcosdsanchez]

this should be checked in in both branches, 1.0 and 1.1

[alex94040]

Setting the correct milestone.

[alex94040]

(In [155]) Fixes #182 (QFileAssetDialog does not work inside a QForm). Code by alex94040, reviewed by Basilieus. Patching both 1.0 and 1.1 branches.

[enzo]

This problem persist.

I update to QCubed 1.1 but the QFileAsset control render the QDialog inside the control span, so the Dialog isn't visible.

Exanple at http://edukalia.anexusit.com/admin/students/add

If someone, could explain me in what part of code the dialog could be assign outside the control scope I could do by myself

please help

enzo

[enzo]

Hello folks

I did a change in QFileAssetDialog class to guaranty the modal window is a child of form, with this change I solve the issue to include in complex design with complex css.

The remain error is when the upload failure the position of window is render in a wrong position, someone could provide some clue to try to fix by myself.

Check a demo at : http://edukalia.anexusit.com/admin/students/QFileAsset/index.php

regards

enzo

P.S: please reopen this ticket I don't have rights to do that.

[marcosdsanchez]

Enzo, your patch contains jquery code.

As this fix is for version 1.0 could you please refactor the javascript code to be compatible with qcodo instead of jquery?

Thanks for your patch :)

[alex94040]

Can someone re-roll this patch, please?