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

feat: improve design of about page #290

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

shootermv
Copy link
Contributor

@shootermv shootermv commented Dec 12, 2023

Description

add some improvements to about page styling
closes #288

screenshots

image

@shootermv shootermv requested a review from NoamGaash as a code owner December 12, 2023 10:03
@shootermv shootermv force-pushed the improve-design-of-about-page branch from d2436ed to 562334d Compare December 12, 2023 10:06
@NoamGaash
Copy link
Member

it looks so much better! 👏 👏 👏

Copy link
Member

@NoamGaash NoamGaash left a comment

Choose a reason for hiding this comment

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

perfect!

}

li img {
width: 5%;
Copy link
Member

Choose a reason for hiding this comment

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

did you check that on different screen sizes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is there something wrong you can see now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed img width to 1.9em
image

Copy link
Member

Choose a reason for hiding this comment

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

I didn't check how it looks, only the code and screenshot.
consider adding margin-right and bullets to list

@shootermv shootermv force-pushed the improve-design-of-about-page branch from 562334d to 6a6e2b4 Compare December 12, 2023 11:37
@shootermv shootermv merged commit 146b281 into hasadna:main Dec 12, 2023
9 checks passed
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.

about page can be styled better using Widget util
2 participants