Skip to content

ROU-12504: Performance improvements for Markers and MarkerCluster#244

Merged
os-davidlourenco merged 7 commits intodevfrom
ROU-12504
Feb 12, 2026
Merged

ROU-12504: Performance improvements for Markers and MarkerCluster#244
os-davidlourenco merged 7 commits intodevfrom
ROU-12504

Conversation

@os-davidlourenco
Copy link
Contributor

@os-davidlourenco os-davidlourenco commented Feb 10, 2026

This PR is to improve the general performance for Markers and Marker Clusters

What was happening

  • The performance of the Map was degraded when a large number of markers were displayed (500+), causing the browser to stop and the overall experience to be degraded

What was done

  • Some corrections on the storage of the Markers were done as well as the retrieval and clening of those markers
  • A change was made to not draw every time a marker was added, even though the repaint is always called

Screen Recording 2026-02-10 at 11 19 20

Test Steps

  1. Open the sample application
  2. Click on the button to add the Markers
  3. Validate that the markers should appear almost immediately
  4. Click on the button to remove the Markers
  5. Validate that the marker should disappear almost immediately

Screenshots

Screen Recording 2026-02-10 at 11 19 02

Checklist

  • tested locally
  • documented the code
  • clean all warnings and errors of eslint
  • requires changes in OutSystems (if so, provide a module with changes)
  • requires new sample page in OutSystems (if so, provide a module with changes)

@os-davidlourenco os-davidlourenco requested a review from a team as a code owner February 10, 2026 11:34
@os-davidlourenco os-davidlourenco changed the title Performance improvements for Markers and MarkerCluster ROU-12504: Performance improvements for Markers and MarkerCluster Feb 10, 2026
Copy link
Contributor

@rugoncalves rugoncalves left a comment

Choose a reason for hiding this comment

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

Overall there are problems when dealing with the Ids of the markers. The current approach is not considering the Ids (marker or map), can be different, depending in the situation the code is being invoked from. Please review this.

gnbm
gnbm previously approved these changes Feb 11, 2026
@sonarqubecloud
Copy link

@os-davidlourenco os-davidlourenco dismissed rugoncalves’s stale review February 12, 2026 16:33

Giuliana reviewed instead

@os-davidlourenco os-davidlourenco merged commit ab53035 into dev Feb 12, 2026
16 checks passed
@os-davidlourenco os-davidlourenco deleted the ROU-12504 branch February 12, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants