Dotclear

Changeset 3445:26274861f3f6


Ignore:
Timestamp:
12/07/16 17:12:43 (9 years ago)
Author:
franck <carnet.franck.paul@…>
Branch:
default
Message:

Add button type to non submit button, add meta-helper class to meta editor buttons, cope with helper buttons in confirm-close.js, addresses #2226

The

chainHandler(window,'onload',function() {

confirmClosePage.getCurrentForms();

});

seems to be run before all DOM ready (with all js done), CKEditor or LegacyEditor? add some stuff to current form after that ?!?
a 1 second delay to run again confirmClosePage.getCurrentForms() is not enough…

Files:
9 edited

Legend:

Unmodified
Added
Removed
  • admin/_charte.php

    r3434 r3445  
    7979 
    8080     <div id="wrapper" class="clearfix"> 
    81           <div class="hidden-if-no-js collapser-box"><button id="collapser" class="void-btn"> 
     81          <div class="hidden-if-no-js collapser-box"><button type="button" id="collapser" class="void-btn"> 
    8282          <img class="collapse-mm visually-hidden" src="images/collapser-hide.png" alt="Cacher le menu" /> 
    8383          <img class="expand-mm visually-hidden" src="images/collapser-show.png" alt="Montrer le menu" /> 
  • admin/js/_media.js

    r3434 r3445  
    9898               } 
    9999 
    100                $('<p class="clear"><button class="enhanced-toggle">' + $msg + '</button></p>').click(function(e) { 
     100               $('<p class="clear"><button type="button" class="enhanced-toggle">' + $msg + '</button></p>').click(function(e) { 
    101101                    if ($container.hasClass('enhanced_uploader')) { 
    102102                         $msg = dotclear.msg.enhanced_uploader_activate; 
  • admin/js/common.js

    r3434 r3445  
    380380          $(e).append(document.createTextNode(' ')); 
    381381 
    382           $('<button class="checkbox-helper select-all">'+dotclear.msg.select_all+'</button>').click(function() { 
     382          $('<button type="button" class="checkbox-helper select-all">'+dotclear.msg.select_all+'</button>').click(function() { 
    383383               if (target !== undefined) { 
    384384                    target.check(); 
     
    393393          $(e).append(document.createTextNode(' ')); 
    394394 
    395           $('<button class="checkbox-helper select-none">'+dotclear.msg.no_selection+'</button>').click(function() { 
     395          $('<button type="button" class="checkbox-helper select-none">'+dotclear.msg.no_selection+'</button>').click(function() { 
    396396               if (target !== undefined) { 
    397397                    target.unCheck(); 
     
    406406          $(e).append(document.createTextNode(' ')); 
    407407 
    408           $('<button class="checkbox-helper select-reverse">'+dotclear.msg.invert_sel+'</button>').click(function() { 
     408          $('<button type="button" class="checkbox-helper select-reverse">'+dotclear.msg.invert_sel+'</button>').click(function() { 
    409409               if (target !== undefined) { 
    410410                    target.toggleCheck(); 
  • admin/js/confirm-close.js

    r2904 r3445  
    4242          for (var i=0; i<formsInPage.length; i++) { 
    4343               f = formsInPage[i]; 
    44                for (var j=0; j<f.elements.length; j++) { 
     44               for (var j=0; j < Math.min(f.elements.length,this.forms[i].length); j++) { 
    4545                    if (this.forms[i][j] != this.getFormElementValue(f[j])) { 
    4646                         return false; 
     
    7575 
    7676     getFormElementValue: function(e) { 
     77          if (e == undefined || e.classList.contains('meta-helper') || e.classList.contains('checkbox-helper')) { 
     78               return null; 
     79          } 
    7780          if (e.type != undefined && e.type == 'radio') { 
    7881               return this.getFormRadioValue(e); 
  • admin/js/meta-editor.js

    r3442 r3445  
    5353          var This = this; 
    5454 
    55           this.submit_button = $('<input type="button" value="ok" class="ib" />'); 
     55          this.submit_button = $('<input type="button" value="ok" class="ib meta-helper" />'); 
    5656          this.submit_button.click(function() { 
    5757               var v = This.meta_dialog.val(); 
     
    8181               for (var i=0; i < meta.length; i++) { 
    8282                    li = $('<li>'+meta[i]+'</li>'); 
    83                     a_remove = $('<button class="metaRemove"><img src="images/trash.png" alt="remove" /></button>'); 
     83                    a_remove = $('<button type="button" class="metaRemove meta-helper"><img src="images/trash.png" alt="remove" /></button>'); 
    8484                    a_remove.get(0).caller = this; 
    8585                    a_remove.get(0).meta_id = meta[i]; 
     
    109109                         var meta_id = $(this).text(); 
    110110                         li = $('<li><a href="' + This.meta_url + $(this).attr('uri') + '">'+meta_id+'</a></li>'); 
    111                          a_remove = $('<button class="metaRemove"><img src="images/trash.png" alt="remove" /></button>'); 
     111                         a_remove = $('<button type="button" class="metaRemove meta-helper"><img src="images/trash.png" alt="remove" /></button>'); 
    112112                         a_remove.get(0).caller = This; 
    113113                         a_remove.get(0).meta_id = meta_id; 
     
    163163 
    164164                    $(data).find('meta').each(function(i) { 
    165                          meta_link = $('<button>' + $(this).text() + '</button>'); 
     165                         meta_link = $('<button type="button" class="metaItem meta-helper">' + $(this).text() + '</button>'); 
    166166                         meta_link.get(0).meta_id = $(this).text(); 
    167167                         meta_link.click(function() { 
     
    178178 
    179179                    if (list_type == 'more') { 
    180                          var a_more = $('<button class="metaGetMore"></button>'); 
     180                         var a_more = $('<button type="button" class="metaGetMore meta-helper"></button>'); 
    181181                         a_more.append(This.text_all + String.fromCharCode(160)+String.fromCharCode(187)); 
    182182                         a_more.click(function() { 
     
    193193                         target.append(pa); 
    194194 
    195                          var a = $('<button class="metaGetList">' + This.text_choose + '</button>'); 
     195                         var a = $('<button type="button" class="metaGetList meta-helper">' + This.text_choose + '</button>'); 
    196196                         a.click(function() { 
    197197                              $(this).parent().next().removeClass('hide'); 
  • admin/media_item.php

    r3440 r3445  
    341341     echo 
    342342     '<p>'. 
    343      '<button id="media-select-ok" class="submit">'.__('Select').'</button> '. 
    344      '<button id="media-select-cancel">'.__('Cancel').'</button>'. 
     343     '<button type="button" id="media-select-ok" class="submit">'.__('Select').'</button> '. 
     344     '<button type="button" id="media-select-cancel">'.__('Cancel').'</button>'. 
    345345     form::hidden(array('type'),html::escapeHTML($media_type)). 
    346346     form::hidden(array('title'),html::escapeHTML($file->media_title)). 
     
    505505     echo 
    506506     '<p>'. 
    507      '<button id="media-insert-ok" class="submit">'.__('Insert').'</button> '. 
    508      '<button id="media-insert-cancel">'.__('Cancel').'</button>'. 
     507     '<button type="button" id="media-insert-ok" class="submit">'.__('Insert').'</button> '. 
     508     '<button type="button" id="media-insert-cancel">'.__('Cancel').'</button>'. 
    509509     form::hidden(array('type'),html::escapeHTML($media_type)). 
    510510     form::hidden(array('title'),html::escapeHTML($file->media_title)). 
  • admin/popup_link.php

    r3434 r3445  
    4040'</form>'. 
    4141 
    42 '<p><button class="reset" id="link-insert-cancel">'.__('Cancel').'</button> - '. 
    43 '<button id="link-insert-ok"><strong>'.__('Insert').'</strong></button></p>'."\n". 
     42'<p><button type="button" class="reset" id="link-insert-cancel">'.__('Cancel').'</button> - '. 
     43'<button type="button" id="link-insert-ok"><strong>'.__('Insert').'</strong></button></p>'."\n". 
    4444 
    4545'<script type="text/javascript">'."\n". 
  • admin/popup_posts.php

    r3434 r3445  
    7575echo '</div>'; 
    7676 
    77 echo '<p><button id="link-insert-cancel">'.__('cancel').'</button></p>'; 
     77echo '<p><button type="button" id="link-insert-cancel">'.__('cancel').'</button></p>'; 
    7878 
    7979dcPage::closePopup(); 
  • inc/admin/lib.dc.page.php

    r3434 r3445  
    235235          echo 
    236236          '<div id="wrapper" class="clearfix">'."\n". 
    237           '<div class="hidden-if-no-js collapser-box"><button id="collapser" class="void-btn">'. 
     237          '<div class="hidden-if-no-js collapser-box"><button type="button" id="collapser" class="void-btn">'. 
    238238          '<img class="collapse-mm visually-hidden" src="images/collapser-hide.png" alt="'.__('Hide main menu').'" />'. 
    239239          '<img class="expand-mm visually-hidden" src="images/collapser-show.png" alt="'.__('Show main menu').'" />'. 
Note: See TracChangeset for help on using the changeset viewer.

Sites map