Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Litepicker: added filter hook for amount of months displayed (mobile) #1754

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hansmorb
Copy link
Contributor

@hansmorb hansmorb commented Feb 6, 2025

Usage:

add_filter('commonsbooking_mobile_calendar_month_count', fn(): int => 2);

@danielappelt Bitte evaluieren, ob das für euch eine mögliche Lösung wäre. Ihr müsstet nur das o.g. Codesnippet in euer Child Theme oder über ein Codesnippet Plugin einfügen. Ihr könnt euch aus dem Kommentar unten gerne eine zip runterladen und das ganze mal austesten.

Dazu gehört auch wielebenwir/Litepicker#9

closes #1753

Copy link

github-actions bot commented Feb 6, 2025

Copy link

codecov bot commented Feb 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.02%. Comparing base (ed0ac9e) to head (9faa94d).
Report is 19 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1754      +/-   ##
============================================
+ Coverage     53.61%   54.02%   +0.41%     
- Complexity     2628     2631       +3     
============================================
  Files            98       98              
  Lines         12122    12131       +9     
============================================
+ Hits           6499     6554      +55     
+ Misses         5623     5577      -46     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hansmorb hansmorb added the enhancement New feature or request label Feb 6, 2025
@hansmorb hansmorb requested a review from danielappelt February 6, 2025 16:38
src/View/Calendar.php Outdated Show resolved Hide resolved
@datengraben
Copy link
Contributor

@hansmorb wollen wir für alle Hooks mal mindestens ein Beispiel anlegen? Um möglichst nah am Code zu sein, könnten wir das in einer Markdown hier im Code dokumentieren. Das machen andere WP-Plugins ähnlich.

* Also add readme link to examples
@datengraben
Copy link
Contributor

@hansmorb was sagst du?

@hansmorb
Copy link
Contributor Author

hansmorb commented Feb 7, 2025

@hansmorb was sagst du?

Wollen wir den docs Ordner wirklich schon einrichten, bevor wir ne entsprechende Lösung haben? Scheint mir aktuell noch nicht nötig. Und ansonsten warte ich auf @danielappelt für Rückmeldungen zu diesem PR, das ist ja vor allem für die Leihbar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Filter Hook für Anzahl Seiten auf Mobilgeräten
3 participants