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

displayed a message when no resources are found matching the filters #305

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

nandu-99
Copy link

@nandu-99 nandu-99 commented Jul 8, 2024

Fixes

Description

Added a feature to display a message when no resources match the selected combination of filters in the topic, medium, and language categories. This enhancement ensures that users are informed when their filter criteria do not yield any results, preventing confusion and providing clarity on the absence of matching resources.

Screenshots

Screenshot 2024-07-08 at 10 50 56 PM

Checklist

  • My pull request has a descriptive title (not a vague title like Update index.md).
  • My pull request targets the default branch of the repository (main or master).
  • My commit messages follow [best practices][best_practices].
  • My code follows the established code style of the repository.
  • I added or updated tests for the changes I made (if applicable).
  • I added or updated documentation (if applicable).
  • I tried running the project locally and verified that there are no
    visible errors.

@nandu-99 nandu-99 requested a review from a team as a code owner July 8, 2024 17:22
@nandu-99 nandu-99 requested review from Shafiya-Heena and possumbilities and removed request for a team July 8, 2024 17:22
@Murdock9803 Murdock9803 self-requested a review July 8, 2024 18:38
docs/_assets/css/style.css Show resolved Hide resolved
docs/_layouts/listing.html Outdated Show resolved Hide resolved
@@ -50,6 +50,9 @@ <h1>Resources</h1>

<!-- This is the section that shows all the resource cards in the site.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This documentation should be shifted to just before the #thumbnaillist, and a different descriptive comment should be added here related to the noThumbnailsMessage

docs/_assets/js/listing.js Outdated Show resolved Hide resolved
@Murdock9803 Murdock9803 removed the request for review from Shafiya-Heena July 24, 2024 06:21
@@ -47,6 +47,20 @@ h1 {
list-style: none;
}

.hidden {
display: none !important;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we want to avoid the use of !important in our cascade.

display: none !important;
}

.visible{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't need a class of visible, and should operate off of things being visible until made invisible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

[Feature] display a message when no resources are found matching the filters
3 participants