Skip to content

Search hides calendar dates permanently and breaks calendar grid layout #48

Open
Parvathy-pp wants to merge 1 commit into
NimaMaria:mainfrom
Parvathy-pp:fix-search-calendar-grid
Open

Search hides calendar dates permanently and breaks calendar grid layout #48
Parvathy-pp wants to merge 1 commit into
NimaMaria:mainfrom
Parvathy-pp:fix-search-calendar-grid

Conversation

@Parvathy-pp
Copy link
Copy Markdown

📌 Description

When using the search feature, calendar date cells that do not match the search query were hidden using display: none. However, when the search input was cleared, some cells did not reappear properly, causing the calendar grid layout to break and missing dates to remain invisible.


issue #45

🐛 Bug Fix

Updated the render logic to ensure that hidden cells are properly restored when the search input is cleared.

Now, when the search box is empty, all calendar date cells are displayed normally, and the calendar grid layout remains intact.


Screenshots

During the search
image

After the search
image

Month Switching
image

✅ Result

  • Calendar cells hide correctly during search
  • Calendar restores correctly after clearing search
  • Grid layout remains consistent and usable

🧪 How to Test

  1. Add events on multiple dates
  2. Search for a specific event
  3. Observe that only matching dates appear
  4. Clear the search input
  5. Confirm that all dates reappear correctly

@NimaMaria
Copy link
Copy Markdown
Owner

This PR is not correctly linked to issue #45, so the label is not showing. Please update the PR description and include Fixes #45 so it gets properly connected.
Also, this branch is not up to date and has some merge conflicts. Please resolve the conflicts and update the branch.

@NimaMaria
Copy link
Copy Markdown
Owner

Additionally, I couldn’t fully understand the implementation. Can you share a short video showing the changes and how it works?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants