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

[feat]: Add an infinite image slider issue #954 #957

Closed
wants to merge 4 commits into from
Closed

[feat]: Add an infinite image slider issue #954 #957

wants to merge 4 commits into from

Conversation

purnasth
Copy link

Pull Request: Adding Infinite Image Slider Mini-Project

Description

This pull request addresses the feature request issue #954, where we propose the addition of a new mini-project, the Infinite Image Slider, to the javascript-mini-projects repository. The project has been created using HTML, CSS, and JavaScript, providing an engaging and educational resource for learners and web developers.

Attachments

image

Please review this pull request and consider merging it into the main project repository. If any further adjustments are needed or if you have any feedback, please let us know. Thank you for your consideration.

Closes #954

Copy link
Collaborator

@PBJI PBJI left a comment

Choose a reason for hiding this comment

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

@purnasth please rename directory to ImageSlider, instead of image-slider. Thank you.

@purnasth
Copy link
Author

The directory has been renamed on c7ae41e commit. Now, it's ready to be merged.
@PBJI

@PBJI
Copy link
Collaborator

PBJI commented Oct 16, 2023

you just created two directories image-slider and imageSlider what I am asking is ImageSlider which is pascal cased, also delete the former two directories before I can merge your changes.

@purnasth

@purnasth
Copy link
Author

Sorry for that problem, finally the task has now been completed on 4d89800 commit. It's ready to be merged.

@PBJI PBJI added the changeRequested The requested changes should be adhered to before further review or approval. label Oct 16, 2023
@purnasth purnasth requested a review from PBJI October 16, 2023 14:35
@PBJI
Copy link
Collaborator

PBJI commented Oct 16, 2023

ou just created two directories image-slider and imageSlider what I am asking is ImageSlider which is pascal cased, also delete the former two directories before I can merge your changes.

what I am asking is ImageSlider which is pascal cased

@purnasth
Copy link
Author

Sorry for the misunderstanding, now the camelCase has been updated to PascalCase on 5ab4cbd commit and it's ready to be merged now.

@thinkswell thinkswell closed this Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changeRequested The requested changes should be adhered to before further review or approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat]: Add an infinite image slider.
3 participants