Repository navigation
feat(qcm): one Cameras list for the view or near a chosen place (QQA-16a) - #42
Merged
Merged
Conversation
Ported from headlessmode-archive #790 (commit 3e807b4f) onto master; conflicts in main.js and layer-registry.test.mjs resolved against master's groupHint and screen/names render maps.
… (QQA-18) The Layers panel was headed by provider and rendering role. It now has ten task headings (Map style, Places & routes, Land & access, Terrain & landscape, Weather & conditions, Night sky, Cameras, Place facts, Explore counties, Print options); every layer id has exactly one home. Search also matches group purpose words and opens matching groups, restoring them when cleared. An Active on map list shows each layer that is on with Remove and Retry. Print-only pages are not counted as on the map. Presentation only: ids, input ids, radio/checkbox roles, URL state and what each layer draws are unchanged. Ported from headlessmode-archive #794 (commit 7f665250). Adapted to the shipped catalogue: the eleven layers added since get a task home, and the browser tests account for the default-on names and roads layers in the Active list.
…16a) Under the Cameras group, a list built from loaded camera records, not pins: "In this view" (from zoom 7) or "Near selected place" (25 km, any zoom), with purpose filters over the five camera layer ids. - camera-layer.js: cameraList, cameraListStatus, cameraRowText, listLoadState, listPlace, PURPOSE_OF_LAYER, kmBetween, reachBox. - main.js: builds and redraws the list (150 ms debounce, rows rebuilt only on change); the near scope loads the camera files around the place on demand. The latest place pick wins: finder, pin or places list, else the drawn area's centre. - Counts are honest: "Listing the nearest 50 of 350" and the 300-pin cap is stated, never silent. Loading, zoom-in, nothing near, unavailable and partly-unavailable states are distinct, and a near-scope failure is judged on the near files alone. - The Active on map list counts only layer rows, not the list controls. - Tests: camera-list (11) and camera-list-browser J10, J10b-e (6). Ported from headlessmode-archive #798 (commit 212364d9), stacked on the QQA-18 grouping PR.
Deploying headlessmode-current with
|
| Latest commit: |
b40fd6d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://21b84b9d.headlessmode-current.pages.dev |
| Branch Preview URL: | https://qqa-16a-camera-list.headlessmode-current.pages.dev |
# Conflicts: # content/quiet-corners-maps.md # scripts/qcm/README.md # scripts/qcm/context-overlays-browser.test.mjs # scripts/qcm/layer-count-browser.test.mjs # themes/headlessmode/static/js/quiet-corners-maps/main.js
# Conflicts: # scripts/qcm/site-coupling.json # themes/headlessmode/static/js/quiet-corners-maps/main.js
# Conflicts: # scripts/qcm/README.md # themes/headlessmode/static/js/quiet-corners-maps/main.js
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ported from headlessmode-archive #798 (commit 212364d9), QC U13 camera chain (QC-04), plan row U13. Base
qqa-18-groups(PR for #794), stacked. Closed in the archive, not merged there; this is the port.What changes: one Cameras list for the current view or for a chosen place, with a purpose filter, in the Layers panel.
Adapted to the shipped catalogue: master's place finder is "Search places" (not "Search names"), so the no-place message and the radius text in
main.jsandcamera-layer.js, and one md phrase, use that name.control-register.jsongainscamera-scopeandcamera-purpose;site-coupling.jsonis regenerated.Tests (through
bin/pytest-slot.sh, logs unpiped):