| route-pattern | /_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format) |
| route-controller | voltron_pull_requests_fragments |
| route-action | pull_request_layout |
| fetch-nonce | v2:9976ed5b-3046-3ef6-062e-7702508caa18 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | C456:3FE40B:194C542:2300071:6A55E131 |
| html-safe-nonce | e35e6bafdaffd2f4b9f3620fde5667eda4c0b2e7638f9581b6c68614bce8d22e |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNDU2OjNGRTQwQjoxOTRDNTQyOjIzMDAwNzE6NkE1NUUxMzEiLCJ2aXNpdG9yX2lkIjoiNzU1NTUwNDU3NTM3OTUzMDAzMyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 2e5ee4974bd5046df2ffdfa8884dbca44665ad56462f287b62e35a40b7c81f75 |
| hovercard-subject-tag | pull_request:81020099 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///voltron/pull_requests_fragments/pull_request_layout |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/plotly/plotly.js/845/pull_request_layout |
| twitter:image | https://opengraph.githubassets.com/f39bd796c0a6db33f88ef8e28c2fcfa77f8701857817de72d7fbc3b4030140c0/plotly/plotly.js/pull/845 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/f39bd796c0a6db33f88ef8e28c2fcfa77f8701857817de72d7fbc3b4030140c0/plotly/plotly.js/pull/845 |
| og:image:alt | This PR introduces a new registry, this one for component modules. For those keeping track, we now have registries for trace, plot (aka subplot aka base plot), transform and component modules.
This... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | etpinard |
| hostname | github.com |
| expected-hostname | github.com |
| None | b5665c84831ed9ac4fb79519c16c9c5580ba8092fb8bb6e3e72972ec7197348e |
| turbo-cache-control | no-cache |
| go-import | github.com/plotly/plotly.js git https://github.com/plotly/plotly.js.git |
| octolytics-dimension-user_id | 5997976 |
| octolytics-dimension-user_login | plotly |
| octolytics-dimension-repository_id | 45646037 |
| octolytics-dimension-repository_nwo | plotly/plotly.js |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 45646037 |
| octolytics-dimension-repository_network_root_nwo | plotly/plotly.js |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 3725c7f9b2a6e3a27ef57c5aed13428c0f65e045 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://github.com/plotly/plotly.js/pull/845#start-of-content |
|
| https://github.com/ |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fplotly%2Fplotly.js%2Fpull%2F845 |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub Copilot appDirect agents from issue to merge | https://github.com/features/ai/github-app |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| View all resources | https://github.com/resources |
| GitHub SponsorsFund open source developers | https://github.com/open-source/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/open-source/accelerator |
| GitHub Stars | https://stars.github.com |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/enterprise/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fplotly%2Fplotly.js%2Fpull%2F845 |
|
Sign up
| https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=plotly%2Fplotly.js |
| Reload | https://github.com/plotly/plotly.js/pull/845 |
| Reload | https://github.com/plotly/plotly.js/pull/845 |
| Reload | https://github.com/plotly/plotly.js/pull/845 |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
|
plotly
| https://github.com/plotly |
| plotly.js | https://github.com/plotly/plotly.js |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
|
Notifications
| https://github.com/login?return_to=%2Fplotly%2Fplotly.js |
|
Fork
2k
| https://github.com/login?return_to=%2Fplotly%2Fplotly.js |
|
Star
18.3k
| https://github.com/login?return_to=%2Fplotly%2Fplotly.js |
|
Code
| https://github.com/plotly/plotly.js |
|
Issues
771
| https://github.com/plotly/plotly.js/issues |
|
Pull requests
56
| https://github.com/plotly/plotly.js/pulls |
|
Actions
| https://github.com/plotly/plotly.js/actions |
|
Security and quality
0
| https://github.com/plotly/plotly.js/security |
|
Insights
| https://github.com/plotly/plotly.js/pulse |
|
Code
| https://github.com/plotly/plotly.js |
|
Issues
| https://github.com/plotly/plotly.js/issues |
|
Pull requests
| https://github.com/plotly/plotly.js/pulls |
|
Actions
| https://github.com/plotly/plotly.js/actions |
|
Security and quality
| https://github.com/plotly/plotly.js/security |
|
Insights
| https://github.com/plotly/plotly.js/pulse |
| etpinard | https://github.com/etpinard |
| master | https://github.com/plotly/plotly.js/tree/master |
| registry-use-component | https://github.com/plotly/plotly.js/tree/registry-use-component |
| Conversation | https://github.com/plotly/plotly.js/pull/845 |
| Commits18 (18) | https://github.com/plotly/plotly.js/pull/845/commits |
| Checks | https://github.com/plotly/plotly.js/pull/845/checks |
| Files changed | https://github.com/plotly/plotly.js/pull/845/files |
| Add registry of component modules | https://github.com/plotly/plotly.js/pull/845#top |
| etpinard | https://github.com/etpinard |
| master | https://github.com/plotly/plotly.js/tree/master |
| registry-use-component | https://github.com/plotly/plotly.js/tree/registry-use-component |
|
| https://github.com/etpinard |
| etpinard | https://github.com/etpinard |
| Aug 11, 2016 | https://github.com/plotly/plotly.js/pull/845#issue-170731643 |
| v2.0.0 | https://github.com/plotly/plotly.js/issues/420 |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
| etpinard | https://github.com/etpinard |
| August 10, 2016 17:04 | https://github.com/plotly/plotly.js/pull/845#commits-pushed-bf28ad9 |
|
| https://github.com/etpinard |
| move Plotly.register in plot_api/register.js | https://github.com/plotly/plotly.js/pull/845/commits/bf28ad9db8b7f8a299e0da1c9375503c1a354cfe |
| bf28ad9 | https://github.com/plotly/plotly.js/pull/845/commits/bf28ad9db8b7f8a299e0da1c9375503c1a354cfe |
|
| https://github.com/etpinard |
| lint: split registry switch block into routines | https://github.com/plotly/plotly.js/pull/845/commits/00536f43b5da075a8147e24965a5218cca011ad3 |
| 00536f4 | https://github.com/plotly/plotly.js/pull/845/commits/00536f43b5da075a8147e24965a5218cca011ad3 |
|
| https://github.com/etpinard |
| move registry objects and helper method to src/registry.js | https://github.com/plotly/plotly.js/pull/845/commits/910e17c756062d95d50f3698008520802009663f |
| 910e17c | https://github.com/plotly/plotly.js/pull/845/commits/910e17c756062d95d50f3698008520802009663f |
|
| https://github.com/etpinard |
| move Registry tests, plots_test.js -> register_test.js | https://github.com/plotly/plotly.js/pull/845/commits/11f475eadb8e364b9a9350cc59714453f83b19cf |
| 11f475e | https://github.com/plotly/plotly.js/pull/845/commits/11f475eadb8e364b9a9350cc59714453f83b19cf |
|
| https://github.com/etpinard |
| rm some unnecessary deps on Plots | https://github.com/plotly/plotly.js/pull/845/commits/4a3f2540822a080e5577308deb0f095f768f08e6 |
| 4a3f254 | https://github.com/plotly/plotly.js/pull/845/commits/4a3f2540822a080e5577308deb0f095f768f08e6 |
|
| https://github.com/etpinard |
| sub Plots[method] -> Registry[method] | https://github.com/plotly/plotly.js/pull/845/commits/161af7ec98d8cf84d7294a16186a9096f4d5b494 |
| 161af7e | https://github.com/plotly/plotly.js/pull/845/commits/161af7ec98d8cf84d7294a16186a9096f4d5b494 |
|
| https://github.com/etpinard |
| rm a few exported objects in src/plotly.js: | https://github.com/plotly/plotly.js/pull/845/commits/2382c0456574e624be620cde2eaadde971aa3b15 |
| 2382c04 | https://github.com/plotly/plotly.js/pull/845/commits/2382c0456574e624be620cde2eaadde971aa3b15 |
|
| https://github.com/etpinard |
| rm circular dep in plot_image/to_image.js | https://github.com/plotly/plotly.js/pull/845/commits/710f3122e1959028dc0f9a92d54b21bd432bfc6c |
| 710f312 | https://github.com/plotly/plotly.js/pull/845/commits/710f3122e1959028dc0f9a92d54b21bd432bfc6c |
|
| https://github.com/etpinard |
| register: add support for component modules | https://github.com/plotly/plotly.js/pull/845/commits/c6aa0fb7f1d749cb00b8667718049cfc04d470d5 |
| c6aa0fb | https://github.com/plotly/plotly.js/pull/845/commits/c6aa0fb7f1d749cb00b8667718049cfc04d470d5 |
|
| https://github.com/etpinard |
| register components modules | https://github.com/plotly/plotly.js/pull/845/commits/68268e3e0ce4dec10400b4e03573edfd2613f0d8 |
| 68268e3 | https://github.com/plotly/plotly.js/pull/845/commits/68268e3e0ce4dec10400b4e03573edfd2613f0d8 |
|
| https://github.com/etpinard |
| lib: add Lib.noop method | https://github.com/plotly/plotly.js/pull/845/commits/4f754aab88ff2e457cc0caabf6a0f54cea197c55 |
| 4f754aa | https://github.com/plotly/plotly.js/pull/845/commits/4f754aab88ff2e457cc0caabf6a0f54cea197c55 |
|
| https://github.com/etpinard |
| add Registry.getComponentMethod + use it in plot_api / plots | https://github.com/plotly/plotly.js/pull/845/commits/0aa87c498d7b5538f3daaa276a6aaf2957c62b70 |
| 0aa87c4 | https://github.com/plotly/plotly.js/pull/845/commits/0aa87c498d7b5538f3daaa276a6aaf2957c62b70 |
|
| https://github.com/etpinard |
| require only range slider/selector default handlers cartesian defaults | https://github.com/plotly/plotly.js/pull/845/commits/65ffb525e123aca1a91b70fc53397fcd5b207643 |
| 65ffb52 | https://github.com/plotly/plotly.js/pull/845/commits/65ffb525e123aca1a91b70fc53397fcd5b207643 |
|
| https://github.com/etpinard |
| lint plot_schema.js | https://github.com/plotly/plotly.js/pull/845/commits/d11c8f912833887b0a34d93515fbc27deb56e433 |
| d11c8f9 | https://github.com/plotly/plotly.js/pull/845/commits/d11c8f912833887b0a34d93515fbc27deb56e433 |
|
| https://github.com/etpinard |
| use componentsRegistry in plot schema | https://github.com/plotly/plotly.js/pull/845/commits/a1ea3658de01e04795ef3497f05325b97e907bd7 |
| a1ea365 | https://github.com/plotly/plotly.js/pull/845/commits/a1ea3658de01e04795ef3497f05325b97e907bd7 |
| https://github.com/etpinard |
| etpinard | https://github.com/etpinard |
|
type: maintenance
| https://github.com/plotly/plotly.js/issues?q=state%3Aopen%20label%3A%22type%3A%20maintenance%22 |
| Aug 11, 2016 | https://github.com/plotly/plotly.js/pull/845#event-753177991 |
| https://github.com/etpinard |
| etpinard | https://github.com/etpinard |
| Decircularise src dependencies | https://github.com/plotly/plotly.js/milestone/9 |
| Aug 11, 2016 | https://github.com/plotly/plotly.js/pull/845#event-753177994 |
| etpinard | https://github.com/etpinard |
|
Aug 11, 2016
| https://github.com/plotly/plotly.js/pull/845#discussion-diff-74492153 |
| View reviewed changes
| https://github.com/plotly/plotly.js/pull/845/files/a1ea3658de01e04795ef3497f05325b97e907bd7#diff-4cc342865f02fea8acedf92e7c0855b4c805c9824b4b4aaf0f587dba2506c9e7 |
| src/plot_api/plot_api.js | https://github.com/plotly/plotly.js/pull/845/files#diff-4cc342865f02fea8acedf92e7c0855b4c805c9824b4b4aaf0f587dba2506c9e7 |
| etpinard | https://github.com/etpinard |
| Aug 11, 2016 | https://github.com/plotly/plotly.js/pull/845#discussion_r74492153 |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
| https://github.com/etpinard |
| etpinard | https://github.com/etpinard |
|
Aug 11, 2016
| https://github.com/plotly/plotly.js/pull/845#ref-issue-148094498 |
|
wishlist for potential breaking changes since v1
#420
| https://github.com/plotly/plotly.js/issues/420 |
| etpinard | https://github.com/etpinard |
|
Aug 11, 2016
| https://github.com/plotly/plotly.js/pull/845#discussion-diff-74493247 |
| View reviewed changes
| https://github.com/plotly/plotly.js/pull/845/files/a1ea3658de01e04795ef3497f05325b97e907bd7#diff-413348d17b0e186a1ad0c0651756719f679d488a3e5dad1cb5ea6e62db824845 |
| src/plots/cartesian/layout_attributes.js | https://github.com/plotly/plotly.js/pull/845/files#diff-413348d17b0e186a1ad0c0651756719f679d488a3e5dad1cb5ea6e62db824845 |
| etpinard | https://github.com/etpinard |
| Aug 11, 2016 | https://github.com/plotly/plotly.js/pull/845#discussion_r74493247 |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
| https://github.com/mdtusz |
| mdtusz | https://github.com/mdtusz |
| Aug 18, 2016 | https://github.com/plotly/plotly.js/pull/845#issuecomment-240816927 |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
|
| https://github.com/etpinard |
| Merge branch 'master' into registry-use-component | https://github.com/plotly/plotly.js/pull/845/commits/2802d22b9dfaa01a012207f11a57da63df1f103c |
| 2802d22 | https://github.com/plotly/plotly.js/pull/845/commits/2802d22b9dfaa01a012207f11a57da63df1f103c |
| https://github.co/hiddenchars |
| https://github.com/plotly/plotly.js/pull/{{ revealButtonHref }} |
| Sign up for free | https://github.com/join?source=comment-repo |
| Sign in to comment | https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fplotly%2Fplotly.js%2Fpull%2F845 |
|
Decircularise src dependencies
| https://github.com/plotly/plotly.js/milestone/9 |
| Please reload this page | https://github.com/plotly/plotly.js/pull/845 |
|
| https://github.com/etpinard |
|
| https://github.com/mdtusz |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |