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

Dyanmic thumbnails generated according to the title of the video , notion , folder(current week) #791

Merged
merged 1 commit into from
Jun 16, 2024

Conversation

kethesainikhil
Copy link
Contributor

PR Fixes:

  • 1 Introuduced the dynamic content generated (card component) so it changes the content inside based on the title of the folder , video , notion accordingly.
  • 2 To not to disturb the design of the previous static images I have designed the same card like template so it changes the content inside the card and also the icons will be dynamically changed.

Resolves #790 .
Here are the proofs of how this works and my design , I am providing both the screenshots as well as screencast so in screencast you can able to see the responsiveness of the card component
screenshot of how video and notion looks like
Screenshot from 2024-06-15 22-41-21
screenshot of how folder looks like
Screenshot from 2024-06-15 22-56-20
screencast of all together with mobile responsiveness has been shown here
Screencast from 2024-06-16 00-08-27.webm

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

@kethesainikhil

This comment was marked as resolved.

@hkirat hkirat requested a review from siinghd June 16, 2024 12:57
@hkirat
Copy link
Contributor

hkirat commented Jun 16, 2024

very cool PR

@@ -95,11 +95,7 @@ export const CertificateComponent = ({
</Button>
<div className="flex items-center justify-center">
<Button
<<<<<<< tailwind-prettier
Copy link
Contributor

Choose a reason for hiding this comment

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

where did this get in?
@siinghd

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was a merge conflit so I have allowed a incoming one

Copy link
Collaborator

Choose a reason for hiding this comment

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

@hkirat tailwind prettier was here #770

@kethesainikhil
Copy link
Contributor Author

kethesainikhil commented Jun 16, 2024

very cool PR

Thanks @hkirat if the PR eligible for bounty could you please mention that for motivation purpose

@siinghd
Copy link
Collaborator

siinghd commented Jun 16, 2024

testing it RIght now

@siinghd
Copy link
Collaborator

siinghd commented Jun 16, 2024

Merging, you will get the bounty for sure

@siinghd siinghd merged commit f66a0bf into code100x:main Jun 16, 2024
1 check passed
@kethesainikhil
Copy link
Contributor Author

Merging, you will get the bounty for sure

Thank you @siinghd

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.

bug: Folder , Video , Notion doesnot have dynamic content & thumbnail.
3 participants