atomic-calendar-revive: Combining maxDaysToShow, maxEventCount and showNoEventDays does not work as expected

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

image

Relevant config:

maxDaysToShow: 5
maxEventCount: 1 # would normally be more, but is better for this bug report
showNoEventDays: true
showHiddenText: true
noEventText: NO EVENTS
hiddenEventText: HIDDEN EVENTS

When combining these settings: Days that have events filtered out because it goes over the maxEventCount will show as “No events”.

Expected behaviour

I would expect one of:

  1. These days not to be shown at all (because the settings is called MAXdaysToShow)
  2. These days to show HIDDEN EVENTS

I would prefer option 1.

Steps To Reproduce

v9.1.0 on HASS 2023.10.0

Use the above config to reproduce, play with maxEventCount a bit.

Mode

Event (default)

Card Version

9.1.0

Home Assistant Version

2023.10.0

Configuration

maxDaysToShow: 5
maxEventCount: 1
showNoEventDays: true
showHiddenText: true
noEventText: NO EVENTS
hiddenEventText: HIDDEN EVENTS

Relevant log output

None

Code of Conduct

  • I agree to follow this project’s Code of Conduct

About this issue

  • Original URL
  • State: closed
  • Created 8 months ago
  • Comments: 19

Commits related to this issue

Most upvoted comments

It works! All issues resolved. Thanks’ alot!

Den tis 23 jan. 2024 kl 10:36 skrev Steven Marks @.***>:

Hopefully this latest release resolves issues with sorting, there is still an issue with milestone sorting but that will be more complex to resolve

— Reply to this email directly, view it on GitHub https://github.com/totaldebug/atomic-calendar-revive/issues/1247#issuecomment-1905655546, or unsubscribe https://github.com/notifications/unsubscribe-auth/APD2WAGSQR4KIC65BEV5WRLYP6AADAVCNFSM6AAAAAA6LVLJL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBVGY2TKNJUGY . You are receiving this because you were mentioned.Message ID: @.***>

Same issue on my side. I’m using it with 3 calendars (each has several events), and when using maxDaysToShow +maxEventCount it doesn’t show right events. It shows the right number of event according the maxEventCount setting (eg: 10), but those 10 event are only considering “all day events” from 18 days ahead… weird. Like a wrong sorting action is done prior to filter out events exceeding the maxEventCount. So for now. no choice to stop using maxEventCount because my dashboard skip a lot of events.

I have the same issue, when I say maxDaysToShow: 14 the list is quite long, when i combine it with maxEventCount: 8, it does not show the calendar items of tomorrow.