GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — chore: Remove deprecated Mapbox traces by camdecoster · Pull Request #7860 · plotly/plotly.js

Conversation

@camdecoster

@camdecoster camdecoster commented Jun 22, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Description

Remove Mapbox traces and related references, files.

Part of #7174.

Changes

  • Remove Mapbox traces
  • Remove old Mapbox baseline images
  • Remove Mapbox references
  • Update docs
  • Update types and schema
  • Update build process to remove Mapbox, use MapLibre for partial bundle
  • Update tests to remove Mapbox references

Testing

  • Check the CI results

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
camdecoster added 10 commits June 22, 2026 15:13
@camdecoster
Remove dead code
bf61c08
@camdecoster
Fix links in comments
0593154
@camdecoster
Remove Mapbox trace related files
90d120c
@camdecoster
Remove references to Mapbox
ce0683c
@camdecoster
Update tests to remove Mapbox references
3847b3d
@camdecoster
Remove leftover Mapbox baseline images
12e2347
@camdecoster
Remove unnecessary test script
750492d
@camdecoster
Update build process to remove Mapbox references
4bcb09a
@camdecoster
Update types and schema
ab74b33
@camdecoster
Add draftlog
7d048dd
@camdecoster camdecoster added this to the v4.0.0 milestone Jun 23, 2026
@camdecoster
camdecoster marked this pull request as ready for review June 23, 2026 16:24
@camdecoster
Merge remote-tracking branch 'origin/v4.0' into cam/7174/remove-mapbox
5013ac3
emilykl
emilykl reviewed Jun 26, 2026
View reviewed changes
Comment thread tasks/util/constants.js
'carpet',
'choropleth',
'choroplethmapbox',
'choroplethmap',

@emilykl emilykl Jun 26, 2026

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Huh, I think these should have been updated when we first introduced the *map traces, but better late than never! Good catch.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
emilykl
emilykl approved these changes Jun 26, 2026
View reviewed changes

@emilykl emilykl left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@camdecoster Merge conflicts will need to be resolved, but LGTM. 🚀

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
camdecoster added 3 commits June 26, 2026 15:36
@camdecoster
Merge remote-tracking branch 'origin/v4.0' into cam/7174/remove-mapbox
ce4a55e
@camdecoster
Update types after merge
a71c645
@camdecoster
Update trace count in docs
61cbad1
@camdecoster
camdecoster merged commit df7ca4b into v4.0 Jun 26, 2026
84 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@camdecoster
camdecoster deleted the cam/7174/remove-mapbox branch June 26, 2026 22:20
@jqnatividad jqnatividad mentioned this pull request Oct 28, 2026
Upgrade bundled plotly.js to from 3.x to 4.x; releasing 0.15.0 plotly/plotly.rs#437
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@emilykl emilykl emilykl approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v4.0.0

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@camdecoster @emilykl
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.