-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
Fix project details map #917
Fix project details map #917
Conversation
…e(only map visible)
…information for small screen
…tion splitted to projectOption component. projectOptions added for others bottomSheet
…nverted dom manipulation to react states
…-project-details-map
for more information, see https://pre-commit.ci
…emoved, mapLegend removed from projectOptions
…or small screen view
…3/fmtm into fix-project-details-map
<p className="fmtm-text-2xl">Map Legend</p> | ||
</div> | ||
<div className="fmtm-flex fmtm-flex-col fmtm-gap-4"> | ||
{MapDetails.map((data, index) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NSUWAL123 Can you make a seperate component for this legend ? As right now its packed with too much html elements and rendering multiple elements.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need some changes on the MapLegends create a maplist component on the same file and destructure it.
This PR contains UI adjustments made to solve problems encountered on mobile browser of 'Project Details' page due to mobile browser top bar on bottom sheet & footer. And seperating map legend from others tab.