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

no scrolling on live map preview #650

Closed
OpenBusBot opened this issue Apr 3, 2024 · 10 comments
Closed

no scrolling on live map preview #650

OpenBusBot opened this issue Apr 3, 2024 · 10 comments
Assignees
Labels
CSS related to the design, without affecting business logic good first issue Good for newcomers REPORTED-BY-USER usability

Comments

@OpenBusBot
Copy link
Collaborator

Contact Name: דורון זמיר
Contact Email: [email protected]
Description:
במסך 'מפה בזמן אמת' לא מופיעים פסי גלילה ולא ניתן לראות את המפה. מצרף צילום מסך
Environment: ווינדוס, דפדפן edge
Expected Behavior:
האתר המוצג גדול מגודל המסך, צפוי שיופיעו פסי גלילה שמאפשרים גלילת המסך מטה ותצוגה של כולו
Actual Behavior:
לא מופיעים פסי גלילה ולא ניתן לגלול מטה וגם כאשר לוחצים על 'מפה במצב מלא' משהו לא מסתדר בתצוגה
Reproducibility: כל פעם

@YuvalMasada
Copy link
Collaborator

The map located at the end of the page which really seems odd and not so comfortable to look at.
@NoamGaash , it would be easier on the eye to add bottom margin and then it would be easy to see that the map is not overflowing on the Y-axis.
On cellphone the map ratio is better, but also looks like it overflow on the Y-axis.

tell me what you think

@NoamGaash
Copy link
Member

@YuvalMasada I think a simple

.leaflet-container {
    min-height: calc(100vh - 88px);
}

will solve the issue

@NoamGaash NoamGaash added good first issue Good for newcomers CSS related to the design, without affecting business logic usability labels Apr 17, 2024
@YuvalMasada
Copy link
Collaborator

YuvalMasada commented Apr 17, 2024

I've added the css you wrote in your comment. I'm adding a screenshot.

without margin

@NoamGaash do you think it needs some gap from the bottom of the page?
It might be something that only bothering me :D

@NoamGaash
Copy link
Member

@YuvalMasada as you wish - you know better :)
I believe margin-bottom might help you here

@zushar
Copy link
Collaborator

zushar commented Apr 30, 2024

fixing issue #721 can really fix it
@NoamGaash

@NoamGaash
Copy link
Member

@YuvalMasada
Copy link
Collaborator

I've committed two css changes in https://github.com/hasadna/open-bus-map-search/pull/724

@YuvalMasada YuvalMasada self-assigned this May 1, 2024
@NoamGaash
Copy link
Member

NoamGaash commented May 1, 2024

solved in #724

Good Job @YuvalMasada ! 👏 🏅

@NoamGaash
Copy link
Member

@all-contributors please add @YuvalMasada for code

Copy link
Contributor

@NoamGaash

I've put up a pull request to add @YuvalMasada! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSS related to the design, without affecting business logic good first issue Good for newcomers REPORTED-BY-USER usability
Projects
None yet
Development

No branches or pull requests

4 participants