creativecommons / creativecommons/cc-resource-archive

[Feature] Replace the dynamic styling in listing.js with class-based CSS approach

Open
#303 2 comments 0 reactions 0 assignees View on GitHub
✨ goal: improvement 🏁 status: ready for work 💻 aspect: code 🟩 priority: low
Dominant language
Dockerfile
Stars
66
Forks
137
PR merge metrics
No merged PRs in 30d

Description

## Problem
In the `_assets/js/listing.js` file, to make the resources visible, the use of dynamic styling has been done.

## Description
Instead of creating a new style element for the displaying of resource cards, It will be better to use a css property of `display: none` via a class based approach. we can create a new utility class for the task.
The link to `listing.js` file: https://github.com/creativecommons/cc-resource-archive/blob/main/docs/_assets/js/listing.js

## Implementation

- [x] I would be interested in implementing this feature.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.