JeromeDevome / GRR

GRR Officiel - Copyright Team DEVOME
https://grr.devome.com
GNU General Public License v2.0
84 stars 51 forks source link

Issue with timepicker #220

Closed remyd1 closed 11 months ago

remyd1 commented 1 year ago

Hi,

I don't know why, but it seems that timepicker.js is not working anymore since version 3.4.X.

With Firefox inspector, I see a display:none added to the css style, when I click on the div element, with class .ui-timepicker-wrapper. Am I the only one with this issue ?

Thanks,

Best regards

ynaessens commented 1 year ago

Bonjour, cela se produit dans quelle page ? Que donne le débogage de Firefox à propos du chargement des fichiers ? En particulier jquery.timepicker.min.css et jquery-ui-timepicker-addon.js sont-ils bien chargés ? Cordialement, YN

remyd1 commented 1 year ago

Bonjour,

Ca se produit sur la page de réservation/modification de la ressource edit_entry.php. Les fichiers css et js sont bien chargés au moment où je charge la page d'accueil. J'ai l'impression que c'est en lien avec la timezone; dans le script js addon:

showTimezone: null

Puis

            // Prevent displaying twice
            if ($dp.find("div.ui-timepicker-div").length === 0 && o.showTimepicker) {
                var noDisplay = ' style="display:none;"',
                html = '<div class="ui-timepicker-div' + (o.isRTL ? ' ui-timepicker-rtl' : '') + '"><dl>' + '<dt class="ui_tpicker_time_label"' + ((o.showTime) ? '' : noDisplay) + '>' + o.timeText + '</dt>' +
                '<dd class="ui_tpicker_time"' + ((o.showTime) ? '' : noDisplay) + '></dd>';

              // &<---------------------------------------- snip -----------------------------------------&<

                // Timezone
                var showTz = o.showTimezone !== null ? o.showTimezone : this.support.timezone;
                html += '<dt class="ui_tpicker_timezone_label"' + (showTz ? '' : noDisplay) + '>' + o.timezoneText + '</dt>';
                html += '<dd class="ui_tpicker_timezone" ' + (showTz ? '' : noDisplay) + '></dd>';

Est-ce qu'il faut que je définisse la Timezone quelque part ? Si oui, où ?

Sinon, est-ce qu'il y a une version récente de timepicker qui pourrait remplacer celle-ci ? Sur le site timepicker.co, bien que la dernière version des sources sur github date de 2016, ça fonctionne correctement (mais je vois que le site utilise cette version mais en date de 2018 alors qu'il y a des versions bien plus récentes (le site demo de cette version fonctionne également bien chez moi).

Cordialement,

ynaessens commented 1 year ago

Bonjour, je ne vois pas ce qui pose problème, de mon côté j'ai le timepicker avec les navigateurs classiques (Firefox, Chrome, Edge, Opera). Pour mettre les scripts à jour il faudrait que je prenne le temps de passer jquery et jquery-ui à des versions plus récentes. C'est noté ;-) Cordialement, YN