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

style: singleLineMap component #723

Merged
merged 3 commits into from
May 3, 2024
Merged

style: singleLineMap component #723

merged 3 commits into from
May 3, 2024

Conversation

zushar
Copy link
Collaborator

@zushar zushar commented May 1, 2024

Description

screenshots

Screenshot 2024-05-01 120754

@zushar zushar requested a review from NoamGaash as a code owner May 1, 2024 09:11
@NoamGaash NoamGaash changed the title reafctor: Refactor singleLineMap component refactor: singleLineMap component May 1, 2024
@NoamGaash NoamGaash changed the title refactor: singleLineMap component style: singleLineMap component May 1, 2024
@zushar
Copy link
Collaborator Author

zushar commented May 3, 2024

this is a screenshot of youtubModel which includes a description for the page

Screenshot 2024-05-03 105611
@NoamGaash

@zushar zushar mentioned this pull request May 3, 2024
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.

Thank you! that's a good improvement, feel free to merge when you'll feel it's ready

@@ -123,79 +121,70 @@ const SingleLineMapPage = () => {
{t('singleline_map_page_title')}
<InfoYoutubeModal
label={t('open_video_about_this_page')}
title={t('youtube_modal_info_title')}
title={t('realtime_map_page_description')}
Copy link
Member

Choose a reason for hiding this comment

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

I think it's not the description of the right page, and perhaps we should consider includith both title and paragraph in the modal

@zushar zushar merged commit 27a2bf6 into main May 3, 2024
17 checks passed
@zushar zushar deleted the RedesignSingleLineMapPage branch May 3, 2024 09:15
@zushar zushar restored the RedesignSingleLineMapPage branch May 3, 2024 10:24
zushar added a commit that referenced this pull request May 3, 2024
@NoamGaash
Copy link
Member

@all-contributors please add @zushar as code contributor

Thanks for the great work! the page does look much better 🥳

Copy link
Contributor

@NoamGaash

@zushar already contributed before to code

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

@NoamGaash NoamGaash deleted the RedesignSingleLineMapPage branch May 6, 2024 05:28
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.

None yet

2 participants