For Joomla 3.x don’t use document.id(‘xxx’)

Dimitris

New Member

Posts: 2

10 år 1 måned dager siden #444 av Dimitris

For Joomla 3.x don’t use document.id(‘xxx’) opprettet av Dimitris

Joomla 3.x is supporting IE8 and newer. Using this legacy mootools code is useless. For example the following code:

<script type="text/javascript">
    js = jQuery.noConflict();
    js(document).ready(function() {
        
    });
    Joomla.submitbutton = function(task)
    {
        if (task == 'basic.cancel') {
            Joomla.submitform(task, document.getElementById('basic-form'));
        }
        else {
            
            if (task != 'basic.cancel' && document.formvalidator.isValid(document.id('basic-form'))) {
                
                Joomla.submitform(task, document.getElementById('basic-form'));
            }
            else {
                alert('<?php echo $this->escape(JText::_('JGLOBAL_VALIDATION_FORM_FAILED')); ?>');
            }
        }
    }
</script>

Can be:

if (task != 'basic.cancel' && document.formvalidator.isValid(document.getElementById('basic-form'))) {

Without any compromise.

Recently I propose removing these legacy mootools methods in Joomla PR

Thanks Dimitris

Vennligst Logg inn eller Registrer konto for å bli med i samtalen.

Andres Maeso

Platinum Member

Posts: 338

10 år 1 måned dager siden #446 av Andres Maeso

Besvart av Andres Maeso i emne For Joomla 3.x don’t use document.id(‘xxx’)

Hey there!

Thanks for your feedback, actually it's a good idea, but as you probably know we stick to Joomla Standards, so we can't remove them until they're removed from Joomla Core. But please, do not hesitate to come back to us when they accept the PR

Regards,

Andrés.

Andrés Maeso
Customer relations manager at Joomla Component Creator.

Denne e-postadressen er beskyttet mot programmer som samler e-postadresser. Du må aktivere javaskript for å kunne se den.

Vennligst Logg inn eller Registrer konto for å bli med i samtalen.

Dimitris

New Member

Posts: 2

9 år 10 måneder dager siden #582 av Dimitris

Besvart av Dimitris i emne For Joomla 3.x don’t use document.id(‘xxx’)

FYI Joomla 3.4 is scheduled for mid February and has those changes. Also There is a JTHML::(behavior.formvalidator); instead of the JTHML::(behavior.formvalidation); which uses only jQuery.

Vennligst Logg inn eller Registrer konto for å bli med i samtalen.

Andres Maeso

Platinum Member

Posts: 338

9 år 10 måneder dager siden #583 av Andres Maeso

Besvart av Andres Maeso i emne For Joomla 3.x don’t use document.id(‘xxx’)

Thank you Dimitris, duly noted. I'll pass it to our developers.

Andrés Maeso
Customer relations manager at Joomla Component Creator.

Denne e-postadressen er beskyttet mot programmer som samler e-postadresser. Du må aktivere javaskript for å kunne se den.

Vennligst Logg inn eller Registrer konto for å bli med i samtalen.

Tid til å lage siden: 0.057 sekunder

Forumløsning av: Kunena