Skip to content

update style on priority change#349

Merged
OSPFNeighbour merged 1 commit intomaster-devfrom
lad-prioirty-redraw
Mar 21, 2026
Merged

update style on priority change#349
OSPFNeighbour merged 1 commit intomaster-devfrom
lad-prioirty-redraw

Conversation

@OSPFNeighbour
Copy link
Collaborator

This pull request improves the way job markers update their appearance in response to priority changes on the map. It ensures marker icons and related styles update live when a job's priority changes, and refactors the style update logic for clarity and maintainability.

Live priority update handling:

  • Added a subscription to job.jobPriorityType so that when a job's priority changes, the marker's icon and style are updated immediately. This ensures the map always reflects the current priority state of each job. [1] [2]

Refactoring and code organization:

  • Extracted the marker style update logic into a new syncMarkerStyle function, which handles updating the marker's icon, color, rescue status, and cluster group assignment as needed. This function also refreshes the pulse ring and cluster icons when styles change.

Minor cleanup:

  • Removed a commented-out line related to adding markers to the cluster group, clarifying the intent of the code.

@OSPFNeighbour OSPFNeighbour merged commit 6434705 into master-dev Mar 21, 2026
@OSPFNeighbour OSPFNeighbour deleted the lad-prioirty-redraw branch March 21, 2026 08:40
OSPFNeighbour added a commit that referenced this pull request Mar 21, 2026
* tiny text align fix on trackable assets under teams

* config bundle fixup

removed redundant info from config LS

* update style on priority change (#349)

* function to complete a job (#350)

* function to complete a job

* and date is needed

* Lad reduce tasking search (#348)

* first

* removed too much! put it back

* shitty theme

* Bump flatted from 3.2.7 to 3.4.2 (#346)

Bumps [flatted](https://github.com/WebReflection/flatted) from 3.2.7 to 3.4.2.
- [Commits](WebReflection/flatted@v3.2.7...v3.4.2)

---
updated-dependencies:
- dependency-name: flatted
  dependency-version: 3.4.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Lad reckless modal (#351)

* Add confirmation modal for refreshInterval changes

Prevent accidental changes to refreshInterval by tracking the last value and showing a confirmation modal whenever the observable is changed. A showRecklessModal helper is injected into the DOM (id: recklessModal) and requires the user to type "reckless" to confirm; onCancel reverts the observable to the previous value. Implementation uses bootstrap.Modal and creates/removes event handlers for confirm/cancel and the modal close button.

* gated

* Refine dark-mode styles for task list items (#353)

Adjust dark-mode CSS for vehicle popup tasking list: add styling for team badge (.veh-pop__team-title .badge.bg-light) with darker background (#444), text and border tweaks; set explicit background for tasking items (.veh-pop__tasking-item, --no-job) to #383838; set muted text color to #b0b0b0; and update hover background to #434343 to improve contrast and consistency in dark mode.

* Batch job tasking fetch and debounce (#352)

* Batch job tasking fetch and debounce

Add batched and debounced tasking fetches to reduce per-job API calls and bursts. main.js: introduce a 250ms debounced initial-load batch for newly filtered-in jobs, a periodic batch refresh (driven by config.refreshInterval) and timer management (start/restart). Job.js: remove per-job periodic refresh checks, add a SINGLE_FETCH_COOLDOWN_MS (10s) and make fetchTasking honor a force option to avoid redundant single-job requests right after batch updates. Team.js: gate fetchTasking against both last fetch time and last bulk refresh time, clear loading state when throttled, and make updateFromJson do defensive/no-op updates to avoid unnecessary observable writes. shared/BeaconClient/job.js: extend getTasking to accept single id or array of ids, build multi-JobIds parameters, and use paginated JSON fetch to return a unified Results array. Overall this reduces API traffic, prevents duplicate refreshes, and avoids unnecessary UI churn.

* Missed one type

* teams not refreshing after tasking

* sector fuckery when just wanting to see all jobs

* bad bar for sectors

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
OSPFNeighbour added a commit that referenced this pull request Mar 23, 2026
* tiny text align fix on trackable assets under teams

* config bundle fixup

removed redundant info from config LS

* update style on priority change (#349)

* function to complete a job (#350)

* function to complete a job

* and date is needed

* Lad reduce tasking search (#348)

* first

* removed too much! put it back

* shitty theme

* Bump flatted from 3.2.7 to 3.4.2 (#346)

Bumps [flatted](https://github.com/WebReflection/flatted) from 3.2.7 to 3.4.2.
- [Commits](WebReflection/flatted@v3.2.7...v3.4.2)

---
updated-dependencies:
- dependency-name: flatted
  dependency-version: 3.4.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add confirmation modal for refreshInterval changes

Prevent accidental changes to refreshInterval by tracking the last value and showing a confirmation modal whenever the observable is changed. A showRecklessModal helper is injected into the DOM (id: recklessModal) and requires the user to type "reckless" to confirm; onCancel reverts the observable to the previous value. Implementation uses bootstrap.Modal and creates/removes event handlers for confirm/cancel and the modal close button.

* gated

* lint fix

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

1 participant