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/55 website footer #191

Merged
merged 14 commits into from
Dec 6, 2023
Merged

Feat/55 website footer #191

merged 14 commits into from
Dec 6, 2023

Conversation

williamlines
Copy link
Contributor

@williamlines williamlines commented Dec 4, 2023

Adds new component WebsiteFooter and storybook docs

closes: #55

Screen.Recording.2023-12-05.at.15.20.34.mov

Copy link

@review-bot-for-github review-bot-for-github bot 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! No comments.

Copy link

@nherment nherment left a comment

Choose a reason for hiding this comment

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

The code looks good., The behavior is incorrect when the page is resized.
Also, it's ok to demo the mobile version but I expect the design to be responsive and switch automatically to the mobile version when the screen size is too small.

Screen.Recording.2023-12-05.at.09.18.04.mov

@Ademsk1 Ademsk1 self-requested a review December 5, 2023 09:59
Ademsk1
Ademsk1 previously requested changes Dec 5, 2023
Copy link
Contributor

@Ademsk1 Ademsk1 left a comment

Choose a reason for hiding this comment

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

Could you try pulling from main? I'm getting issues running this locally

@Ademsk1 Ademsk1 self-requested a review December 5, 2023 10:11
@williamlines williamlines requested a review from nherment December 5, 2023 10:33
src/assets/build/subtract.icon.tsx Outdated Show resolved Hide resolved
src/assets/build/subtract.icon.tsx Outdated Show resolved Hide resolved
@williamlines williamlines requested a review from Ademsk1 December 5, 2023 16:01
@williamlines williamlines merged commit 9b71994 into main Dec 6, 2023
3 checks passed
@williamlines williamlines deleted the feat/55-website-footer branch December 6, 2023 09:27
@github-actions github-actions bot mentioned this pull request Dec 7, 2023
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.

[Component] - Website footer
3 participants