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

read more feature #125

Merged
merged 5 commits into from
Dec 14, 2022
Merged

read more feature #125

merged 5 commits into from
Dec 14, 2022

Conversation

MaYaNkKashyap681
Copy link
Contributor

1). Added Box Shadow Around Post Box.
2). Added Read More Feature in the post -> If the caption is long, the read more button will be displayed and if the user clicks on the read more button, the whole caption will be displayed.
3). Added Comment Icon.
4). On hovering the user profile, the circle turns black and the image scales up a little.

Screenshot (113)
Screenshot (114)

Screenshot (115)
Screenshot (116)

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hi there, Thanks a lot for your first pull request, we'll be reviewing it asap :)

Copy link
Owner

@narayan954 narayan954 left a comment

Choose a reason for hiding this comment

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

Hi @MaYaNkKashyap681 Some older changes have also been changed incorrectly in this new commit, can you please fix them? :) If you face any problem lmk

src/components/Post.jsx Outdated Show resolved Hide resolved
src/components/Post.jsx Show resolved Hide resolved
src/components/Post.jsx Show resolved Hide resolved
src/components/Post.jsx Show resolved Hide resolved
@MaYaNkKashyap681
Copy link
Contributor Author

I have made a new commit if still there is some issue then let me know!

src/components/Post.jsx Outdated Show resolved Hide resolved
src/components/Post.jsx Outdated Show resolved Hide resolved
Copy link
Owner

@narayan954 narayan954 left a comment

Choose a reason for hiding this comment

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

lgtm!

@narayan954 narayan954 merged commit 214c825 into narayan954:master Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants