? ? Failure

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
6 Sep 2015

Joomla! Documentation https://docs.joomla.org/Subform_form_field_type

This is the field I have suggested in #7749

Description

The field allow to include any existing form into the current form. If attribute multiple set to true then the included form will be repeatable. The Field have a couple "predefined" layouts for display the subform as table or as div container, and of course it allow to use your own layout.
Field support Default values from the included form, and from json string in default attribute. Last have higher priority.

Example:

<field name="field-name" type="subform" 
    formsource="path/to/form.xml" min="1" max="3" multiple="true"
    layout="joomla.form.field.subform.repeatable-table" groupByFieldset="true"
    label="Subform Field" description="Subform Field Description" />

Attributes:

  • formsource - (required) The form source to be included. Path to xml file or the form name to search by JForm::getInstance().
  • multiple - The multiple state for the form field. Whether the field is repeatable or not.
  • min - Count of minimum repeating in multiple mode. Default 0.
  • max - Count of maximum repeating in multiple mode. Default 1000.
  • groupByFieldset - Whether group the subform fields by it`s fieldset (true or false).
  • buttons - Which buttons to show in multiple mode. Default add,remove,move.
  • layout - The layout name for render the field inputs. Available:
    • joomla.form.field.subform.default - Render the subform in div container, without support of repeating.
    • joomla.form.field.subform.repeatable - Render the subform in div container, recommended for multiple mode. Support groupByFieldset.
    • joomla.form.field.subform.repeatable-table - Render the subform as table, recommended for multiple mode. Support groupByFieldset. By default render each field as the table column, but if groupByFieldset=true then render each fieldset as the table column.

Testing

Can use "Custom HTML" module. Add the field into the module xml:

<field name="field-name" type="subform" formsource="modules/mod_custom/test1.xml" 
   min="1" max="30" multiple="true"
    layout="joomla.form.field.subform.repeatable-table" groupByFieldset="true"
    label="Subform Field" description="Subform Field Description" />

Create the form xml in modules/mod_custom/test1.xml. Example:

<?xml version="1.0" encoding="UTF-8"?>
<form>
    <fieldset name="section1" label="Section1">

        <field name="test_text" type="text"
            label="test_text" description="test_text" />

        <field name="test_textarea" type="textarea"
            label="textarea"  width="100" />

        <field name="test_list" type="list" default="1" class="advancedSelect"
            label="test_list" description="test_list" >
            <option value="1">JYES</option>
            <option value="0">JNO</option>
        </field>

    </fieldset>
    <fieldset name="section2" label="Section2">
        <field name="test_radio" default="0" type="radio"
            label="test_radio" description="test_radio">
            <option value="1">JYES</option>
            <option value="0">JNO</option>
        </field>

        <field name="test_checkbox1" type="checkbox"
            label="checkbox1" description="test_checkbox1"  />

        <field name="test_checkboxes" type="checkboxes"
            label="checkboxes" description="test_checkboxes" >
            <option value="a">A</option>
            <option value="b">B</option>
            <option value="c">C</option>
            <option value="d">D</option>
        </field>
    </fieldset>
    <fieldset name="section3" label="Section3">
        <field name="test_color" type="color" default="#eee"
            label="color"  description="test_color" />

        <field name="test_tag" type="tag" class="advancedSelect"
            label="JTAG" description="test_tag" mode="nested"
            multiple="true" />

        <field name="test_media" type="media" directory=""
            label="media" description="test_media"/>

    </fieldset>
</form>

And of course you can use your own form.

Now go to edit the "Custom HTML" options and try add/remove the values for new field.
Try play with different field attributes multiple, min, max, buttons, layout, groupByFieldset and make sure that the field work. :neckbeard:

p.s. The pull request does not provide the Validation and Filter. But such thing can be implemented in future, at least for Validation, for the Filter can be need some JForm changes.

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar Fedik Fedik - open - 6 Sep 2015
avatar Fedik Fedik - change - 6 Sep 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Sep 2015
Labels Added: ?
avatar Fedik
Fedik - comment - 6 Sep 2015

Bonus 1 :candy:
Less code duplication: The field allow to create single form for the Article options and include it in Configuration, Menu, Article Form :yum:
ignore it :smile:

avatar zero-24 zero-24 - change - 6 Sep 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 6 Sep 2015
Milestone Added:
avatar zero-24 zero-24 - change - 6 Sep 2015
Easy No Yes
avatar zero-24 zero-24 - change - 6 Sep 2015
Category Layout Libraries
avatar shur
shur - comment - 7 Sep 2015

@Fedik This PR looks very promising.

Will it be helpful to load separate setting for different module layouts?
What I mean. Modules, e.g. latest news, can have several module layouts. They can differ a lot from each other. Let's say one layout to displays news in columns, another works as a slider. The task is to have individual settings for each of these templates - let's say number of columns and size of the columns for column layout, slider speed etc. for the second layout.

In your solution formsource="path/to/form.xml" is set strictly.
Is it possible to have several xml for each of module layouts, so that when a user select one of the module layouts in module parameters, it gets subform settings for this particular layout?

If the answers to both questions are positive, that would be really great.

avatar Fedik
Fedik - comment - 7 Sep 2015

@shur answer is No, if I right understand your question
but
in theory, you can make your own field that extend current subform field, where you can change the formsource depend from the Form state ... but this will work only after form will be "saved" with new values.

or you can make a couple subform fields with different forms, and switch them using shwoon feature, depend from selected layout :smile:
but I would chose the first way

avatar bertmert
bertmert - comment - 16 Sep 2015

@test
I tested with instructions above (mod_custom) and staging of yesterday.
Created second row with plus symbol. The second Tags selection has no chzn.
subform-chzn

avatar dgt41
dgt41 - comment - 21 Sep 2015

@Fedik This doesn’t work with the BS version of the field Media (the jlayout PR: #5871). Maybe it would be a good idea to not couple this mootools...

avatar Fedik
Fedik - comment - 22 Sep 2015

I want to note: current pull request do not fix that issue #6882 (for repeatable mode).
Possible work around in the repeatable mode: you define min attribute, and do not add/move/remove the "sections", then all scripts should work as usual :neckbeard:
In single mode all should work fine without any "hacks".

@dgt41 that will be hard :fearful: ... but you are right, will be bad if it will be broken, again :alien: ... and thanks for link that issue. Main problem with that Media field is "hard" linking to the input "id".

About Chosen script, the problem that we do not use "special" class where we want to have the Chosen script, and just apply that script for all <select> in core layouts.
So in the Repeatable script I have no possibility to check is that <select> from 3d extension, or it something we need apply Chosen, so I have decided do not force Chosen for all select here.
But if people really want I can make it, not a big problem to force :smirk_cat:

avatar Fedik
Fedik - comment - 26 Sep 2015

chosen.js should work now, but only for <select class="advancedSelect">

@dgt41 sorry man, no chance with new media field, it even more hard than was with Mootools.
I tried:

$row.find('div[id^="media_field_"]').each(function(){
    var $inpWrapp = $(this),
        $modal  = $inpWrapp.siblings('div[id^="imageModal_"]'),
        inputId = $inpWrapp.children('input[type="text"]').prop('id'),
        $btnSel = $inpWrapp.children('a[href^="#imageModal_"]'),
        $btnClr = $inpWrapp.children('a[onclick^="clearMediaInput"]'),
        $preview = $inpWrapp.children('span[id^="media_preview_"]');

    $inpWrapp.attr('id', 'media_field_' + inputId); // fix wrapper ID
    $modal.attr('id', 'imageModal_' + inputId); // fix modal ID
    $preview.attr('id', 'media_preview_' + inputId); // fix preview ID
    $btnSel.attr('href', '#imageModal_' + inputId); // fix Select button

    // fix clear btn
    var onclic = $btnClr.attr('onclick').replace(/clearMediaInput\('[0-9a-zA-Z\_\-]+', /, "clearMediaInput('" + inputId + "', ");
    $btnClr.attr('onclick', onclic);

});

This fix already looks like trash, and it just a 10% of fix that need to do for new Media field from #5871.
Hope you see the reason :wink:

avatar dgt41
dgt41 - comment - 26 Sep 2015

There should be an easier way to utilize those two functions in the js file...

avatar Fedik
Fedik - comment - 26 Sep 2015

@dgt41 any tips are welcome :wink:

avatar dgt41
dgt41 - comment - 27 Sep 2015

@Fedik some thoughts:
The initialization script should be generic and get the data from data attributes like: data-width="200"
also the inline script:

    jQuery(document).ready(function(){
        var path = "' . JUri::root() . '",
            empty = "' . JText::_('JLIB_FORM_MEDIA_PREVIEW_EMPTY') . '",
            previewWidth = ' . $previewWidth . ',
            previewHeight = ' .$previewHeight . ',
            source = "' . $src . '",
            fieldId = "' . $id . '";
        initializeMedia(path, empty, previewWidth, previewHeight, source, fieldId);
        });

could then be part of the mediafield.js. Also that script should take care of the buttons of the modal (for inserting the image to the input). The clear button with the usage of a data attribute for the field id could also be more generic. Doing something like this the repeatable code should only care for replacing the data- attributes, which should be fairly easy. Bottom line instead of trying to make the repeatable script compatible with the media field we should work the other way around…
If I get some free time this week I will try to patch the media field script

avatar roland-d
roland-d - comment - 3 Oct 2015

@Fedik Been testing this and one thing that isn't so smooth is the drag and drop. The background of the block is transparent, may be nicer to have a more solid background colour.

Using the drag and drop I get a horizontal scrollbar and the screen scrolls to the right when I hit the border there. This goes on to infinity.

All in all looking good.

avatar dgt41
dgt41 - comment - 3 Oct 2015

@Fedik I spent some time with this one and the #5871. I already made some changes in the other PR but still couldn’t get a nice solution here. My approach is the same as #7819, in short here is what I am trying to do:
Bootstrap modals html is not generated by javascript but is hardcoded from the layouts.
Therefore we need to:

  • drop the standard way of togging the modal and assign a meaningful id with something like:
$btnSel.removeAttr('data-toggle').attr('id', inputId + '_btn');
$('#butonId').click(function(){
    $('#modalId').on('show', function () {

        $('iframe').attr("src",'url with the right id');

    });
    $('#modalId').modal({show:true})
});
avatar dgt41
dgt41 - comment - 4 Oct 2015

@Fedik Forgot to post the code so here it is:

        // fix media field
        $row.find('div[id^="media_field_"]').each(function(){

            // Remove the default toggle option for BS modals
            var $inpWrapp = $(this),
                $modal  = $inpWrapp.siblings('div[id^="imageModal_"]'),
                inputId = $inpWrapp.children('input[type="text"]').prop('id'),
                $btnSel = $inpWrapp.children('a[data-toggle="modal"]'),
                $preview = $inpWrapp.children('span[id^="media_preview_"]'),
                $newIframeUrl = $btnSel.attr('data-url');

            // Set path and empty vars
            var empty = $btnSel.attr('data-emptystring'),
                path = $btnSel.attr('data-pasepath');

            $modal.attr('id', 'imageModal_' + inputId); // fix modal ID
            $inpWrapp.attr('id', 'media_field_' + inputId); // fix wrapper ID
            $preview.attr('id', 'media_preview_' + inputId); // fix preview ID

            // Set the new id for the button
            $btnSel.removeAttr('id').attr('data-target', inputId).attr('id', 'btn_' +inputId);

            // Set the new URL for the iframe
            $newIframeUrl = $newIframeUrl.replace(/&fieldid=(.+)&/, '&fieldid=' + inputId + '&');

            // The iframe tag
            var $tagIframe = '<iframe class="iframe" src="' + $newIframeUrl + '" name="Change Image" height="300px" width="800px">';

            // The new listener for modal open/close
            $('#btn_' + inputId ).attr('onclick',
                "jQuery('#imageModal_" + inputId + "').on('show', function () {" +
                    "jQuery(this).parent().find('.modal-body').html('" + $tagIframe + "'); " +
                "}); " +
                "jQuery('#imageModal_" + inputId + "').modal({show:true}); "
                );
        });

Replaces the code for // fix media field // section. This needs the #5871 and it works for the most parts.. ????

avatar Fedik
Fedik - comment - 4 Oct 2015

@dgt41 sorry for delay, and thanks for all your work!
this looks better but still hardly linked to id :wink:

I thinking about more 'dynamic way'. The idea is throw away the requirement of use the ID.
It can be done as jQuery plugin, something like:

// initialisation should looks like
$('.field-media-wrapper').fieldMedia();

And since the Modal markup is inside the field wrapper, we can simple show it by Javascript
without use of ID

// inside the jQuery plugin
this.$container.find('.modal-container').modal('show')

I remember that ID in URL need for set the value from the iframe, but this also can be solwed, at least I already use similar with another Modal library http://dimsemenov.com/plugins/magnific-popup/

$.magnificPopup.open({
    type:"iframe",
    items:{src: url},
    callbacks: {
        open: function() {
            $("iframe.mfp-iframe").load(function(){
                $(this).contents().on("change","input[name=\"cid[]\"]", function(){
                    setValue(button, this.value);
                    $.magnificPopup.close();
                    return false;
                });
            });
        }
    }
});

And I belive something like that we can do with Bootstrap modal also.

Unfortunately I have no time now, but I hope will try something in next 1-2 weeks.

@roland-d thanks for the review, I will check it also :wink:

avatar dgt41
dgt41 - comment - 4 Oct 2015

@Fedik That should work as well but you have to take care also the Save and Close button on the modal. It should be a similar solution to the one you posted above. If you have some spare time to review the changes in the other PR so if there are things we should change to make both of them compatible I would like to hear them. And adjust the code accordingly…

avatar Fedik
Fedik - comment - 16 Oct 2015

Using the drag and drop I get a horizontal scrollbar and the screen scrolls to the right when I hit the border there. This goes on to infinity.

something very strange with jQuery UI

upd: even Demo http://jqueryui.com/sortable/ has same problem :smile:

avatar dgt41 dgt41 - test_item - 25 Oct 2015 - Tested successfully
avatar dgt41
dgt41 - comment - 25 Oct 2015

I have tested this item :white_check_mark: successfully on 5bdf190

Tested also with the bootstrapped version of media and user fields, with success!


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar photodude
photodude - comment - 30 Oct 2015

@Fedik I like the concept, but I would suggest changing the attribute "multiple" to something like "repeat"
multiple is used in other fields to indicate multiple selection, I don't think it's wise to muddy the meaning by using it for a repeat type field.

suggested change
repeat - The repeat state Boolean for the form field. Indicates whether the field is repeatable or not

I would also suggest lowering the "max" Default from 1000 to 100. Most repeatable fields tend to be under 100 for most cases (probably even under 20 repeats)

avatar Fedik
Fedik - comment - 30 Oct 2015

I think use multiple is correct as it indicate also the multiple inputs (actually array value), and we do not need to make thing more complicated/confused with another property repeat/repeatable.

about 1000 - I agree with you, but people should have some more freedom than they can use, then they feels more happy :smile:

avatar waader
waader - comment - 5 Nov 2015

That´s a really nice feature. Two things I encountered:

  • In Chrome the column header shows a + symbol and in FF a - symbol, both are adding a new row when pressed
  • in IE8 I was not able to add a row, but I could delete one
avatar Fedik
Fedik - comment - 5 Nov 2015

thanks for testing! I will check more :wink:

in IE8 I was not able to add a row, but I could delete one

this is good enough for IE :smile:

avatar waader
waader - comment - 5 Nov 2015

I agree!

avatar wilsonge wilsonge - change - 6 Nov 2015
Milestone Added:
avatar wilsonge wilsonge - change - 6 Nov 2015
Milestone Removed:
avatar wilsonge wilsonge - change - 6 Nov 2015
Milestone Removed:
avatar joomla-cms-bot
joomla-cms-bot - comment - 7 Nov 2015

This PR has received new commits.

CC: @dgt41


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar Fedik
Fedik - comment - 7 Nov 2015

IE8 should be fine now,
the issue with Firefox I cannot reproduce

avatar bertmert
bertmert - comment - 11 Dec 2015

Tested successfully with a frontend plugin for contact form that loads a form XML with a subform field in it.

Staging 2015-12-08 (3.5.0 beta)
Protostar.
subform.repeatable
Firefox
IE 8 (some minor issues (sections not sortable, color picker not initialized in first section area) but not really related to this PR I think). Who cares? I could fill all fields.

All entered informations were submitted correctly.

avatar FeikeMulder FeikeMulder - test_item - 12 Dec 2015 - Tested successfully
avatar FeikeMulder
FeikeMulder - comment - 12 Dec 2015

I have tested this item :white_check_mark: successfully on c8638a0

Have tested with instructions (mod_custom and test1.xml)


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar RickR2H RickR2H - test_item - 12 Dec 2015 - Tested successfully
avatar RickR2H
RickR2H - comment - 12 Dec 2015

I have tested this item :white_check_mark: successfully on c8638a0

Works okay! Nice addition and thanks for the PR @Fedik.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar n9iels
n9iels - comment - 16 Dec 2015

What is the status of this PR? I concluded today that the current Repeatable fields are quite useless... Barely possible to loop the field row for row

avatar photodude photodude - test_item - 16 Dec 2015 - Tested unsuccessfully
avatar photodude
photodude - comment - 16 Dec 2015

I have tested this item :red_circle: unsuccessfully on c8638a0

New subform field would not display in a custom module as per testing instructions in any module position I tried in the protostar template and Joomla 3.5 beta.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar photodude photodude - test_item - 17 Dec 2015 - Tested successfully
avatar photodude
photodude - comment - 17 Dec 2015

I have tested this item :white_check_mark: successfully on c8638a0

Tested by adding the subform field to a form rather then attempting to go through a custom HTML module. Where the custom HTML module failed to render the form and field, this functions perfect in an existing form and cover's the usability issue for a repeating form element as indicated in #6882


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar photodude
photodude - comment - 17 Dec 2015

Interesting similarities to a pull request in the Framework that also adds a subform functionality. joomla-framework/form#22
Of course the Framework PR doesn't address the repeatable field issues.

avatar joomla-cms-bot
joomla-cms-bot - comment - 2 Jan 2016

This PR has received new commits.

CC: @dgt41, @FeikeMulder, @photodude, @RickR2H


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar N6REJ
N6REJ - comment - 26 Feb 2016

PLEASE create extended documentation including textual and graphical examples for this. It's obviously very intricate so will need highly detailed documentation to be truly useful imo.

avatar brianteeman brianteeman - change - 10 Mar 2016
Category Layout Libraries Fields Layout Libraries
avatar bertmert bertmert - test_item - 5 Apr 2016 - Tested successfully
avatar bertmert
bertmert - comment - 5 Apr 2016

I have tested this item :white_check_mark: successfully on 576c15d

Several tests with custom extensions (modules, plugins).


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar RickR2H RickR2H - test_item - 15 Apr 2016 - Tested successfully
avatar RickR2H
RickR2H - comment - 15 Apr 2016

I have tested this item :white_check_mark: successfully on 576c15d

All fields work


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar photodude
photodude - comment - 16 Apr 2016

@chrisdavenport can this make it into 3.6?

avatar chrisdavenport
chrisdavenport - comment - 16 Apr 2016

@photodude Thanks for bringing this PR to my attention. I really like it and I'd love to include it in 3.6. I'll add it to the alpha release manifest so it gets more attention and testing. As Bear has said, it will need to be thoroughly documented. Also needs testing with Hathor.

avatar photodude
photodude - comment - 16 Apr 2016

Thanks @chrisdavenport

I'll keep this in mind when I get around to testing in a couple of weeks

  • Needs testing with Hathor (and probably the included frontend templates: beez3, and protostar)
  • Needs to be thoroughly documented
avatar roland-d roland-d - change - 16 Apr 2016
Milestone Added:
avatar roland-d
roland-d - comment - 16 Apr 2016

@chrisdavenport I have added the 3.6 milestone to this PR, so it won't fall off the radar, at least for 3.6 ;) Whether it gets included we will have to see of course how things progress.

avatar N6REJ
N6REJ - comment - 17 Apr 2016

one thing to test for is with the current field if the default fields
are empty, OR if they were changed after the extension was installed (
like during developement ) you will get errors when your trying to read
the values because they will return null. So this would definetly be
something to test against.. It's extremely annoying to have to
completely reset the template params DB field when some kind of change
is made in the .xml
Bear

On 4/16/2016 10:12, Walt Sorensen wrote:

Thanks @chrisdavenport https://github.com/chrisdavenport

I'll keep this in mind when I get around to testing in a couple of weeks

  • Needs testing with Hathor (and probably included frontend templates)
  • Needs to be thoroughly documented


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#7829 (comment)

avatar wilsonge
wilsonge - comment - 7 May 2016

OK so this is totally broken in Hathor (the width in the custom module in the given example just makes the hathor layout fall apart):
screen shot 2016-05-07 at 17 59 31

I have however otherwise successfully tested this PR. Would be good if you could put in some Hathor overrides @Fedik :) - this looks really really awesome!

avatar Fedik
Fedik - comment - 7 May 2016

thanks, I will check more, but cannot promise much about the table layout.

There a couple available layouts for this field:
joomla.form.field.subform.repeatable and joomla.form.field.subform.repeatable-table

For the table layout it is expected, because it is not possible to fit wide table in smaller width.
For Hathor in the module options section the layout joomla.form.field.subform.repeatable works better.

avatar wilsonge
wilsonge - comment - 7 May 2016

Ah ok if that layout works thats ok. By the way I think that we should have a default layout for this. If people want to override it with the layout param that's ok. But I think that we should have a default renderer

avatar Fedik
Fedik - comment - 7 May 2016

I have made 3 different layout:
joomla.form.field.subform.default is default, but it do not support repeating

if some one want to make repeatable, then there a chose between:
joomla.form.field.subform.repeatable
joomla.form.field.subform.repeatable-table

hm, but you are right, maybe good idea to set joomla.form.field.subform.repeatable as default, when multiple="true"

avatar wilsonge
wilsonge - comment - 7 May 2016

That makes sense. Because otherwise you don't have anything rendered without a layout specified which is kinda annoying

avatar brianteeman
brianteeman - comment - 8 May 2016

@fedik will you be updating this PR with the layout changes above?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar Fedik
Fedik - comment - 8 May 2016

@brianteeman yes, of course
maybe even in this evening

avatar joomla-cms-bot
joomla-cms-bot - comment - 8 May 2016

This PR has received new commits.

CC: @bertmert, @dgt41, @FeikeMulder, @photodude, @RickR2H


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar Fedik
Fedik - comment - 8 May 2016

Ok, I have updated the field behavior when layout="" not specified.
joomla.form.field.subform.default is default
joomla.form.field.subform.repeatable is default when multiple="true"

avatar Kubik-Rubik Kubik-Rubik - test_item - 8 May 2016 - Tested successfully
avatar Kubik-Rubik
Kubik-Rubik - comment - 8 May 2016

I have tested this item :white_check_mark: successfully on 0e8f6f7

Great feature, should go into 3.6. Thank you @Fedik!


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar Fedik
Fedik - comment - 8 May 2016

hm, Travis says I have made mistake in FILE: ...s/administrator/components/com_installer/views/install/tmpl/default.php ... but I made nothing there, what is wrong? :smile:

avatar roland-d
roland-d - comment - 8 May 2016

@Fedik Travis is overworked at the moment due to our sprint. It may have failed on that file when you created your branch. Nothing to worry about. @wilsonge is keeping an eye on things.

avatar wilsonge
wilsonge - comment - 8 May 2016

It was a problem in staging - I just fixed it - if you rebase/merge in staging should be ok

avatar wilsonge
wilsonge - comment - 8 May 2016

screen shot 2016-05-08 at 18 04 39

I accidentally put the form in the wrong place and got this. I'm not sure what the right behaviour is - but not sure this is it :/

avatar wilsonge wilsonge - test_item - 8 May 2016 - Tested successfully
avatar wilsonge
wilsonge - comment - 8 May 2016

I have tested this item :white_check_mark: successfully on 0e8f6f7

I think this is at a happy state now pretty much. Given this a play around with Hathor and with ISIS with and without repeatable and I'm happy. Maybe a slight improvement to be made when the XML file can't be found(?) but other than that successful.

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7829.

avatar wilsonge wilsonge - change - 8 May 2016
Status Pending Ready to Commit
avatar Fedik
Fedik - comment - 8 May 2016

Any suggestion are welcome.
I did not get a better idea than just show Exception message.

avatar joomla-cms-bot joomla-cms-bot - change - 8 May 2016
Labels Added: ?
avatar rdeutz
rdeutz - comment - 9 May 2016

@Fedik if you fix the Code Style issues we can merge this :-)

avatar andrepereiradasilva
andrepereiradasilva - comment - 9 May 2016

@rdeutz restart travis

avatar wilsonge wilsonge - change - 9 May 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-09 21:04:02
Closed_By wilsonge
avatar wilsonge wilsonge - close - 9 May 2016
avatar wilsonge wilsonge - merge - 9 May 2016
avatar joomla-cms-bot joomla-cms-bot - close - 9 May 2016
avatar wilsonge wilsonge - reference | a28b188 - 9 May 16
avatar wilsonge wilsonge - merge - 9 May 2016
avatar wilsonge wilsonge - close - 9 May 2016
avatar joomla-cms-bot joomla-cms-bot - change - 9 May 2016
Labels Removed: ?
avatar wilsonge
wilsonge - comment - 9 May 2016

Merged - travis failure was unrelated and was due to failures we had in staging in manchester.

avatar Fedik Fedik - head_ref_deleted - 10 May 2016
avatar Fedik
Fedik - comment - 10 May 2016

I just now noticed, I have missed the year, fix in #10375

avatar WS-Theme
WS-Theme - comment - 3 Jun 2016

That's really awesome and working very well!! However I have noticed two things:

1) Form Field Media: It's not possible to add preview="true" to get the Image permanent next to the Field

2) Form Field Radio: It's not possible to add class="btn-group" to transform the Radio Buttons to the Styled Version of Radios. It works but only the first time! When adding a new Field it stops working

bildschirmfoto 2016-06-03 um 11 02 42

avatar zero-24
zero-24 - comment - 3 Jun 2016

@WS-Theme This is a closed / merged issue. Please open a new one at https://issues.joomla.org else i guess your findings get lost. Thanks

avatar WS-Theme
WS-Theme - comment - 3 Jun 2016

@zero-24 sorry my error! Opened a Issue @ Joomla

avatar zero-24
zero-24 - comment - 3 Jun 2016

Great. Thanks ????

avatar ufuk-avcu
ufuk-avcu - comment - 9 Jun 2016

Tested this on small devices and it's not responsive.
templates_ edit style -administrator_index php

avatar brianteeman
brianteeman - comment - 9 Jun 2016

Please create a new issue. No one will see a comment on a closed issue

avatar ufuk-avcu
ufuk-avcu - comment - 9 Jun 2016

@brianteeman sorry. Done!

avatar Fedik
Fedik - comment - 9 Jun 2016

no need new issue, just use correct layout layout="joomla.form.field.subform.repeatable, because table layout cannot be responsive

avatar Sieger66
Sieger66 - comment - 11 Aug 2016

Notice:

level subform in multiple=true mode does not supported for now (due some limitation).

However in single ( multiple=false ) mode should work, in theory, but I not tried 😄

from Fedik at #11551 :
https://issues.joomla.org/tracker/joomla-cms/11551#event-190388

Add a Comment

Login with GitHub to post a comment