Skip to content

Add model for large review content in show.ejs page. #402

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

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

Suvadip-sana
Copy link
Contributor

Description

If user give the large comment for review. I add a read more button with this review. After clicking on this button it open a scrollable model in the center of the screen. it contain full length review along with profile pic, owner name, All that one review have.

Initially I thought to implement a new page for this but This model looks pretty well. And also no unnecessary page extention need. That's look though. You can check on the attached video.

  • This PR does the following:
    • Updates ... Review card to handle large review comment to display it in webpage properly.

Related Issues

Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.

Screenshots (if applicable)

Add any screenshots that help explain or visualize the changes.

rimp.1.mp4

Checklist

Make sure to check off all the items before submitting. Mark with [x] if done.

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I am working on this issue under GSSOC

@Soujanya2004 Marge it. And don't forget to add all the labels as per ISSUE #329

Copy link

vercel bot commented Nov 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wanderlust-2024 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 5, 2024 8:34am

Copy link

github-actions bot commented Nov 5, 2024

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@Soujanya2004 Soujanya2004 merged commit 3d90279 into Soujanya2004:main Nov 5, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Problem in review.
2 participants