Skip to content

feat: map - #8931

Open
mindspank wants to merge 14 commits into
mainfrom
feat/map
Open

feat: map#8931
mindspank wants to merge 14 commits into
mainfrom
feat/map

Conversation

@mindspank

@mindspank mindspank commented Feb 26, 2026

Copy link
Copy Markdown
Contributor
  • Needs new icon
  • Needs a cleaner way to disable interactions in edit or new interaction patters in edit in general.
  • Token is public but has limited scopes. Can be refreshed via the mapbox console.
  • If we go deeper on themes I would suggest moving most of the style related settings to themes.
  • base style is perhaps a bit boring but optimized for legibility

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@mindspank
mindspank requested a review from djbarnwal February 26, 2026 08:57
@djbarnwal

djbarnwal commented Mar 17, 2026

Copy link
Copy Markdown
Member

This PR would require some refactoring. Moving to draft for now. Will pick it up from here later when prioritized.

@djbarnwal
djbarnwal marked this pull request as draft March 17, 2026 11:43
@nishantmonu51

nishantmonu51 commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

@djbarnwal : Can you please take this one up ?

@nishantmonu51 nishantmonu51 added Type:Feature New feature request Area:Dashboard Size:L Large change: 500-1,999 lines labels Jun 30, 2026
@djbarnwal
djbarnwal marked this pull request as ready for review July 24, 2026 10:31
djbarnwal and others added 3 commits August 18, 2026 17:11
@djbarnwal
Merge branch 'main' into feat/map
0d5ef06
@djbarnwal @claude
Regenerate package-lock.json against the registry
77f0695 
The previous lockfile regen ran with the repo's `.npmrc` `prefer-offline=true`,
so npm resolved the tree from the local cache: `resolved`/`integrity` fields
were stripped and platform-specific optional dependencies outside darwin were
dropped. CI's `npm ci` on linux-x64 then had no `@rolldown/binding-linux-x64-gnu`
to install, so every step loading `svelte.config.js` failed with
"Cannot find native binding".

Restored the lockfile from `main` and regenerated with `--prefer-online`, leaving
just the `mapbox-gl` entry as the diff.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@djbarnwal
prettier fix
2dd9b05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@djbarnwal djbarnwal Awaiting requested review from djbarnwal

At least 1 approving review is required to merge this pull request.

Assignees

@djbarnwal djbarnwal

Labels

Area:Dashboard Size:L Large change: 500-1,999 lines Type:Feature New feature request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mindspank @djbarnwal @nishantmonu51