Skip to content

refactor: map page#1750

Open
joaolopes15 wants to merge 54 commits intodevelopfrom
refactor/revamp_map
Open

refactor: map page#1750
joaolopes15 wants to merge 54 commits intodevelopfrom
refactor/revamp_map

Conversation

@joaolopes15
Copy link
Copy Markdown

@joaolopes15 joaolopes15 commented Nov 30, 2025

Closes #1666

Review checklist

  • Terms and conditions reflect the changes

View Changes

  • Description has screenshots of the UI changes.
  • Tested both in light and dark mode.
  • New text is both in portuguese (PT) and english (EN).
  • Works in different text zoom levels.
  • Works in different screen sizes.

Performance

  • No helper functions to return widgets are added. New widgets are created instead.
  • Used ListView.builder for Long Lists.
  • Controllers (TextEditingController, ...) are beeing disposed of in dispose() method.

Image displaying all floors and every marker

screenshot1

Image displaying the 1st floor

screenshot2

Image displaying the search of B002 room

screenshot3

@joaolopes15 joaolopes15 linked an issue Nov 30, 2025 that may be closed by this pull request
@joaolopes15 joaolopes15 marked this pull request as draft November 30, 2025 18:51
@pedroafmonteiro pedroafmonteiro changed the title taping off the search bar now unfocus it refactor: map page Nov 30, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 8, 2025

Codecov Report

❌ Patch coverage is 2.85714% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 7%. Comparing base (d2e3b63) to head (cd1684c).
⚠️ Report is 1 commits behind head on develop.

❌ Your project check has failed because the head coverage (7%) is below the target coverage (70%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##           develop   #1750   +/-   ##
=======================================
- Coverage        7%      7%   -0%     
=======================================
  Files           38      38           
  Lines         1886    1916   +30     
=======================================
  Hits           125     125           
- Misses        1761    1791   +30     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pedroafmonteiro pedroafmonteiro marked this pull request as ready for review March 31, 2026 13:00
@joaolopes15 joaolopes15 requested review from Togamerp, pedroafmonteiro and simaopsbarbosa and removed request for Togamerp April 15, 2026 11:02
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.

Revamp Map Page

3 participants