Skip to content
Snippets Groups Projects

Hotline#18602 calendar config icon disappear

Merged Hotline#18602 calendar config icon disappear
hotline#18602_calendar_config_icon_disappear into stable
Merged Ghost User requested to merge hotline#18602_calendar_config_icon_disappear into stable

Merge request reports

Approval is optional

Merged by (Apr 14, 2025 3:47am UTC)

Merge details

  • Changes merged into stable with fd24238c.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Patrick Barroca Reassigned to @pbarroca

    Reassigned to @pbarroca

  • Patrick Barroca
    Patrick Barroca @pbarroca started a thread on the diff
    Last updated by Ghost User
    public/opac/js/calendrier.js
    12 12 if (url == '#')
    13 13 url = $(this).jqmData('href');
    14 14
    15 $(this).closest(".contenu").load(url+'/render/ajax',
    15 $(this).closest(".calendar").load(url+'/render/ajax .calendar > *',
    • Patrick Barroca
      Patrick Barroca @pbarroca ·
      Owner

      oula, l'url dans le load me semble plus que douteuse ".../ajax .calendar > *" me semble pas être une url. du coup, des tests sur ça ?

    • G
      Ghost User @ghost ·
      Author Contributor

      La méthode load() autorise l'utilisation de selecteurs jquery dans l'url qui permettent de sélectionner une portion seulement du contenu qu'elle renvoie. Dans notre cas, on ne veut pas tout car cela ajouterait une div.calendar dans une div.calendar à chaque clique sur un filtre mois. '.calendar > *' n'en prend que le contenu.

      En ce qui concerne l'ajout de tests, on doit en discuter car il y a énormément de travail

  • Patrick Barroca mentioned in commit 817239

    mentioned in commit 817239

Please register or sign in to reply