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.

Sign up for GitHub

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

Jump to bottom

[NETMANAGER] map visuals and responsivness fix #1772

Merged
merged 2 commits into from
Dec 16, 2023
Merged

[NETMANAGER] map visuals and responsivness fix #1772

merged 2 commits into from
Dec 16, 2023

Conversation

OchiengPaul442
Copy link
Contributor

Summary of Changes (What does this PR do?)

  • Fixed the map responsive issues on both mobile and tablet views
  • Included an offline description tag below
  • Changed the shape of the markers to distinguish between online and offline devices.

Status of maturity (all need to be checked before merging):

  • I've tested this locally
  • I consider this code done
  • This change ready to hit production in its current state

Screenshots (optional)

image

@OchiengPaul442 OchiengPaul442 self-assigned this Dec 15, 2023
@github-actions GitHub Actions
Copy link
Contributor

New netmanager changes available for preview here

@OchiengPaul442 OchiengPaul442 changed the title [NETMANAGER] map visuals and responsivness [NETMANAGER] map visuals and responsivness fix Dec 15, 2023
@Baalmart Baalmart merged commit 2279a5e into staging Dec 16, 2023
27 checks passed
@Baalmart Baalmart deleted the Net-map-visuals branch December 16, 2023 15:37
@Baalmart Baalmart mentioned this pull request Dec 16, 2023
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Baalmart Baalmart Baalmart approved these changes

@Codebmk Codebmk Awaiting requested review from Codebmk

Assignees

@OchiengPaul442 OchiengPaul442

Labels
ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@OchiengPaul442 @Baalmart