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

Implemented dark mode #594

Merged
merged 8 commits into from
Jun 30, 2023
Merged

Implemented dark mode #594

merged 8 commits into from
Jun 30, 2023

Conversation

Palakkgoyal
Copy link
Contributor

This PR closes issue no. #541
Screenshot 2023-06-30 133147
Screenshot 2023-06-30 133228
Screenshot 2023-06-30 133303
Screenshot 2023-06-30 133322

Palakkgoyal and others added 5 commits June 30, 2023 13:50
Improved postview page (narayan954#583)

* "fixed: changes to post-view page and image slider"

* "changes to post-view"

* "changes to post-view"

* “fixed: issues with view of comments in different screens”

* format code

---------

Co-authored-by: Narayan soni <narayansoni854@gmail.com>

working on darkmode

fix visibility of post caption that don't have images
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.

You did really good, but some of the styles are missing such as the shadow effects on logout modal
image
Signup and signin pages have also lost their cover
image

Also, I've applied background color from github dark mode
image
Is that okay??
Please fix those lost styles and I can update the difficulty level of this issue :)

index.html Outdated Show resolved Hide resolved
@Palakkgoyal
Copy link
Contributor Author

Yes, sure. I will do the required changes.

@Palakkgoyal
Copy link
Contributor Author

Palakkgoyal commented Jun 30, 2023

Did all the changes. Please review them
Screenshot (565)
Screenshot (566)
Screenshot (567)
m.

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 linked an issue Jun 30, 2023 that may be closed by this pull request
@narayan954 narayan954 merged commit bfa00b8 into narayan954:master Jun 30, 2023
@Palakkgoyal Palakkgoyal deleted the darkMode branch June 30, 2023 12:39
@Killer2-O
Copy link

@narayan954 Dont give level3 tags to such basic work. it is atmost level2

@narayan954
Copy link
Owner

narayan954 commented Jul 8, 2023

@narayan954 Dont give level3 tags to such basic work. it is atmost level2

it's level 2 only, might've assigned level2 by mistake.
Ps: label is assigned correctly in pr, that's counted

@narayan954 narayan954 added level2 and removed level3 labels Jul 8, 2023
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.

Dark mode is not implemented properly
3 participants