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: redesigning home page #395

Closed
wants to merge 13 commits into from
Closed

Conversation

BinaHovav
Copy link
Collaborator

@BinaHovav BinaHovav commented Jan 15, 2024

This will be an ongoing PR for designing the new homepage.

@BinaHovav BinaHovav requested a review from NoamGaash as a code owner January 15, 2024 20:47
@BinaHovav BinaHovav changed the title Add home page Add home page fundamentals Jan 15, 2024
@BinaHovav BinaHovav changed the title Add home page fundamentals add: home page fundamentals Jan 15, 2024
@BinaHovav BinaHovav changed the title add: home page fundamentals feat: home page fundamentals Jan 15, 2024
@BinaHovav BinaHovav changed the title feat: home page fundamentals feat: redesigning home page Jan 16, 2024
@BinaHovav BinaHovav marked this pull request as draft January 16, 2024 08:06
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.

great start!
but let's postpone the navbar changes until the page is ready
see here - #388
how I hide pages from the menu

<Title level={3}>{t('rides_history')}</Title>
<Space direction="vertical" size="middle" style={{ marginBottom: '22px' }}>
<Alert message={t('timestamp_list')} type="info" />
</Space>

main
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
main

<Route path="release" element={<iframe src="https://noam-gaash.co.il/databus/" />} />
<Route path="*" element={<RedirectToDashboard />} key="back" />
<Route path="*" element={<RedirectToHome />} key="back" />
main
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
main

@NoamGaash NoamGaash changed the base branch from main to main2 January 16, 2024 11:50
@NoamGaash NoamGaash changed the base branch from main2 to main January 16, 2024 11:51
@BinaHovav BinaHovav marked this pull request as ready for review January 16, 2024 12:41
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.

you're doing great, thanks for merging with main

Copy link
Member

Choose a reason for hiding this comment

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

an this file be removed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

why removed? i need it for the styles

Copy link
Member

Choose a reason for hiding this comment

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

it's an empty file

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

not for long :)

@BinaHovav BinaHovav closed this Jan 18, 2024
@BinaHovav BinaHovav deleted the add-home-page branch January 18, 2024 07:14
@BinaHovav
Copy link
Collaborator Author

I'm closing this PR and opening a cleaner one.

Copy link
Member

Choose a reason for hiding this comment

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

if we're going to use that, please add an attribution section in bottom of the about page

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.

3 participants