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

Improved Post UI #643

Merged
merged 26 commits into from
Jul 13, 2023
Merged

Improved Post UI #643

merged 26 commits into from
Jul 13, 2023

Conversation

Tisha6661
Copy link
Contributor

@Tisha6661 Tisha6661 commented Jul 4, 2023

The post Ui is made UI , More responsive , optimized css.

##Issue Number-
#628

Screenshots

image
image

@Tisha6661
Copy link
Contributor Author

@narayan954 Is there any issue?

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.

Minor changes requested

image
can you please adjust the color of send button and No comments as well?

src/reusableComponents/ImageSlider/index.jsx Show resolved Hide resolved
src/components/Post/index.css Show resolved Hide resolved
@narayan954
Copy link
Owner

@narayan954 Is there any issue?

its fine, just minor changes

@narayan954
Copy link
Owner

Minor changes requested

image can you please adjust the color of send button and No comments as well?

you may skip fixing the color of send button and No comments if you want. Someone else might work on that

@Tisha6661
Copy link
Contributor Author

No issues I have done that! Just fixing the height.

Changes done
@Tisha6661
Copy link
Contributor Author

Tisha6661 commented Jul 5, 2023

I have changed the comment box, but now there is some issue with the site, I am unable to make changes @narayan954 -
image

@Tisha6661 Tisha6661 requested a review from narayan954 July 5, 2023 06:58
@narayan954
Copy link
Owner

narayan954 commented Jul 5, 2023

No issues I have done that! Just fixing the height.

Hi @Tisha6661 , actually the daily free limit of firebase has been exceeded due to excess database calls, so currently the db is down.
It'll be available in some time when the quota resets.
We can use another db for testing the changes.
DM me for the test db credentials in discord.

EDIT: The app is back again and you can test

@narayan954
Copy link
Owner

narayan954 commented Jul 5, 2023

No issues I have done that! Just fixing the height.

I was reviewing this, height issue is yet to be fixed. Let me know when updated :)

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.

height issue left

@Tisha6661
Copy link
Contributor Author

@narayan954 I've checked several times, but the quota exceeded is showing everytime.

@narayan954
Copy link
Owner

@narayan954 I've checked several times, but the quota exceeded is showing everytime.

oh ok then, contact me, I'll be generating some credentials for testing... you can/may also follow the steps in readme to generate your own credentials

@narayan954
Copy link
Owner

@narayan954
Copy link
Owner

Hi @Tisha6661 are you working on this?

@Tisha6661
Copy link
Contributor Author

@narayan954 I tried working through firebase, but it was my first time, so i couldn't work on it. I was waiting for the site to work.

@Tisha6661
Copy link
Contributor Author

@narayan954 If it's working, I'll work right away.

<ReadMore postId={postId}>{caption}</ReadMore>
</p>
) : (
caption &&
postHasImages && (
<p style={{ color: "var(--color)" }}>
<p style={{ color: "var(--color)", paddingInline: "16px" }}>
{caption}
Copy link
Owner

Choose a reason for hiding this comment

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

caption is repeating, please fix that as well

narayan954 and others added 3 commits July 10, 2023 00:09
Made changes in the card like image size and caption repeating issue.
@Tisha6661
Copy link
Contributor Author

@narayan954 Made the changes, please review-

image
image

@Tisha6661 Tisha6661 requested a review from narayan954 July 9, 2023 19:38
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 Tisha, responsiveness is breaking critically for mobile devices, can you please check that? rest everything seems fine.

Thank you.
I hope this is the last time I bug you on this issue XD

@narayan954
Copy link
Owner

image
image

see :/

@narayan954 narayan954 added level1 and removed level1 labels Jul 9, 2023
@narayan954 narayan954 linked an issue Jul 9, 2023 that may be closed by this pull request
@narayan954 narayan954 added level2 and removed level1 labels Jul 9, 2023
@Tisha6661
Copy link
Contributor Author

Tisha6661 commented Jul 9, 2023

@narayan954 Long captions will again make the pages uneven, because in grid mode if we'll add long caption like this, which is uncertain in all posts, some will write small caption, and some no caption at all. So it would never would in grid view or make the cards even. See-

image

Unless we put an overflow auto.

@narayan954
Copy link
Owner

@narayan954 Long captions will again make the pages uneven, because in grid mode if we'll add long caption like this, which is uncertain in all posts, some will write small caption, and some no caption at all. So it would never would in grid view or make the cards even. See-

image

Unless we put an overflow auto.

you're correct, I think either we can have a min height prop or the overflow control.... but there also please give it height enough to not look bad, as earlier it was just one line with scroll option. Sorry for delaying the pr

The cards are made responsive and the caption is made properly visible.
@Tisha6661
Copy link
Contributor Author

@narayan954 I have made the changes, here-

dummy.mp4

@narayan954
Copy link
Owner

@narayan954 I have made the changes, here-

dummy.mp4

reviewing!

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.

looks good to me!

@narayan954 narayan954 merged commit 205e8f8 into narayan954:master Jul 13, 2023
5 of 6 checks passed
@narayan954
Copy link
Owner

Thanks for your contribution!

narayan954 added a commit to Palakkgoyal/dummygram that referenced this pull request Jul 13, 2023
* issue narayan954#628  resolved

* better UI , More responsive , optimized CSS

* Changes in card

Dark mode changes done

* moved post css to post

* Comment box changes

Changes done

* Updated card

Card height updated.

* Updated the card

Made changes in the card like image size and caption repeating issue.

* removed extra lines and scrollbar for caption

* Cards made responsive

The cards are made responsive and the caption is made properly visible.

* fix syntax error in profile.jsx

---------

Co-authored-by: Narayan soni <narayansoni854@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Improve Posts Ui
2 participants