René's URL Explorer Experiment


Title: Replace _has variables with '_has()' fullLayout method by etpinard · Pull Request #491 · plotly/plotly.js · GitHub

Open Graph Title: Replace _has variables with '_has()' fullLayout method by etpinard · Pull Request #491 · plotly/plotly.js

X Title: Replace _has variables with '_has()' fullLayout method by etpinard · Pull Request #491 · plotly/plotly.js

Description: Open-source JavaScript charting library behind Plotly and Dash - Replace _has variables with '_has()' fullLayout method by etpinard · Pull Request #491 · plotly/plotly.js

Open Graph Description: @mdtusz @alexcjohnson Rationale Our set of hidden plot type variables _hasCartesian, _hasGL3D, _hasGeo, _hasGL2D, _hasPie and most recently _hasTernary has grown out-of-control. We need a better w...

X Description: @mdtusz @alexcjohnson Rationale Our set of hidden plot type variables _hasCartesian, _hasGL3D, _hasGeo, _hasGL2D, _hasPie and most recently _hasTernary has grown out-of-control. We need a better w...

Opengraph URL: https://github.com/plotly/plotly.js/pull/491

X: @github

direct link

Domain: github.com

route-pattern/_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format)
route-controllervoltron_pull_requests_fragments
route-actionpull_request_layout
fetch-noncev2:89f191b1-88ae-1c26-cf9b-37662eed5c62
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idC5CE:2053C6:150DE88:1E0F7ED:6A5A26EA
html-safe-noncefc61c8beb4837f15c539cf62db875bcdf57b991f458a370de50988230ca98a04
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNUNFOjIwNTNDNjoxNTBERTg4OjFFMEY3RUQ6NkE1QTI2RUEiLCJ2aXNpdG9yX2lkIjoiNDA4Mzc3MDMzNDA2NzE3MzA5OCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmac10c3a3e6a4e4934fcc5a002543bbfc875642699388df4c77346147d2c8479faa
hovercard-subject-tagpull_request:68272580
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///voltron/pull_requests_fragments/pull_request_layout
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/plotly/plotly.js/491/pull_request_layout
twitter:imagehttps://opengraph.githubassets.com/b6fdf9e6030863daf81857115ab47ac3585632c5347066ff0447a5f565d6c9ea/plotly/plotly.js/pull/491
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/b6fdf9e6030863daf81857115ab47ac3585632c5347066ff0447a5f565d6c9ea/plotly/plotly.js/pull/491
og:image:alt@mdtusz @alexcjohnson Rationale Our set of hidden plot type variables _hasCartesian, _hasGL3D, _hasGeo, _hasGL2D, _hasPie and most recently _hasTernary has grown out-of-control. We need a better w...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernameetpinard
hostnamegithub.com
expected-hostnamegithub.com
None19c67ad7579d2401ff06e88a32ef45460fbb40fab3191c7f7582d5e166f326fc
turbo-cache-controlno-cache
go-importgithub.com/plotly/plotly.js git https://github.com/plotly/plotly.js.git
octolytics-dimension-user_id5997976
octolytics-dimension-user_loginplotly
octolytics-dimension-repository_id45646037
octolytics-dimension-repository_nwoplotly/plotly.js
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id45646037
octolytics-dimension-repository_network_root_nwoplotly/plotly.js
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
releasea3dea2706161c733403beb02e41d5765d358d44e
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/plotly/plotly.js/pull/491#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fplotly%2Fplotly.js%2Fpull%2F491
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/open-source/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/open-source/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/enterprise/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://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%2F491
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
Reloadhttps://github.com/plotly/plotly.js/pull/491
Reloadhttps://github.com/plotly/plotly.js/pull/491
Reloadhttps://github.com/plotly/plotly.js/pull/491
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
plotly https://github.com/plotly
plotly.jshttps://github.com/plotly/plotly.js
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
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 773 https://github.com/plotly/plotly.js/issues
Pull requests 57 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
etpinardhttps://github.com/etpinard
masterhttps://github.com/plotly/plotly.js/tree/master
has-plot-typehttps://github.com/plotly/plotly.js/tree/has-plot-type
Conversationhttps://github.com/plotly/plotly.js/pull/491
Commits27 (27)https://github.com/plotly/plotly.js/pull/491/commits
Checkshttps://github.com/plotly/plotly.js/pull/491/checks
Files changedhttps://github.com/plotly/plotly.js/pull/491/files
Replace _has variables with '_has()' fullLayout method https://github.com/plotly/plotly.js/pull/491#top
etpinardhttps://github.com/etpinard
masterhttps://github.com/plotly/plotly.js/tree/master
has-plot-typehttps://github.com/plotly/plotly.js/tree/has-plot-type
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Apr 28, 2016https://github.com/plotly/plotly.js/pull/491#issue-151718094
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
@mdtuszhttps://github.com/mdtusz
@alexcjohnsonhttps://github.com/alexcjohnson
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
etpinardhttps://github.com/etpinard
April 27, 2016 18:14https://github.com/plotly/plotly.js/pull/491#commits-pushed-88f552a
https://github.com/etpinard
mv gd._modules -> fullLayout._moduleshttps://github.com/plotly/plotly.js/pull/491/commits/88f552a1a9ed0331f39de51352d470bc99919c1e
88f552ahttps://github.com/plotly/plotly.js/pull/491/commits/88f552a1a9ed0331f39de51352d470bc99919c1e
https://github.com/etpinard
linthttps://github.com/plotly/plotly.js/pull/491/commits/525e3c7e615858f0c190e5fe13f63b4e33ac0db2
525e3c7https://github.com/plotly/plotly.js/pull/491/commits/525e3c7e615858f0c190e5fe13f63b4e33ac0db2
https://github.com/etpinard
attach has (plot type) method to fullLayout,https://github.com/plotly/plotly.js/pull/491/commits/876ce4b01fce2c34bebcdec379f0c9819b721f11
876ce4bhttps://github.com/plotly/plotly.js/pull/491/commits/876ce4b01fce2c34bebcdec379f0c9819b721f11
https://github.com/etpinard
add tempory block _has() -> _hasCartesian etc.https://github.com/plotly/plotly.js/pull/491/commits/f45ffec98f96a718429f08d283b15f821e370a81
f45ffechttps://github.com/plotly/plotly.js/pull/491/commits/f45ffec98f96a718429f08d283b15f821e370a81
https://github.com/etpinard
rm _has??? attributeshttps://github.com/plotly/plotly.js/pull/491/commits/4e0c6dacfca75419593f0b60e90458f82514714d
4e0c6dahttps://github.com/plotly/plotly.js/pull/491/commits/4e0c6dacfca75419593f0b60e90458f82514714d
https://github.com/etpinard
make 'pie' a layout category,https://github.com/plotly/plotly.js/pull/491/commits/93493a69c16e379b36b01a9cc3827dc106ef2bb9
93493a6https://github.com/plotly/plotly.js/pull/491/commits/93493a69c16e379b36b01a9cc3827dc106ef2bb9
https://github.com/etpinard
check for hasCartesian && !hasPie for 'hovermode',https://github.com/plotly/plotly.js/pull/491/commits/e63ba4ae59f022435f8cc6d5221f6ee748391896
e63ba4ahttps://github.com/plotly/plotly.js/pull/491/commits/e63ba4ae59f022435f8cc6d5221f6ee748391896
https://github.com/etpinard
etpinardhttps://github.com/etpinard
type: maintenance https://github.com/plotly/plotly.js/issues?q=state%3Aopen%20label%3A%22type%3A%20maintenance%22
Apr 28, 2016https://github.com/plotly/plotly.js/pull/491#event-644748393
etpinardhttps://github.com/etpinard
Apr 28, 2016 https://github.com/plotly/plotly.js/pull/491#discussion-diff-61485318
View reviewed changes https://github.com/plotly/plotly.js/pull/491/files/e63ba4ae59f022435f8cc6d5221f6ee748391896#diff-bb59a2aa90fd14ef3218544010e7fe42f370e10f0c8ee702b3f85ab86887d970
src/plots/plots.jshttps://github.com/plotly/plotly.js/pull/491/files#diff-bb59a2aa90fd14ef3218544010e7fe42f370e10f0c8ee702b3f85ab86887d970
etpinardhttps://github.com/etpinard
Apr 28, 2016https://github.com/plotly/plotly.js/pull/491#discussion_r61485318
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
@mdtuszhttps://github.com/mdtusz
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
https://github.com/alexcjohnson
alexcjohnsonhttps://github.com/alexcjohnson
Apr 29, 2016https://github.com/plotly/plotly.js/pull/491#issuecomment-215656040
this blockhttps://github.com/plotly/plotly.js/blob/has-plot-type/src/plot_api/plot_api.js#L277-L283
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Apr 29, 2016https://github.com/plotly/plotly.js/pull/491#issuecomment-215734285
orphan axeshttps://github.com/plotly/plotly.js/blob/23421af6fe1ff8b5af257e11b1eeda03ad50e871/src/plots/cartesian/layout_defaults.js#L97-L99
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
https://github.com/alexcjohnson
alexcjohnsonhttps://github.com/alexcjohnson
Apr 29, 2016https://github.com/plotly/plotly.js/pull/491#issuecomment-215793132
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
etpinardhttps://github.com/etpinard
May 3, 2016 14:25https://github.com/plotly/plotly.js/pull/491#commits-pushed-b655cb0
https://github.com/etpinard
Revert "check for hasCartesian && !hasPie for 'hovermode',"https://github.com/plotly/plotly.js/pull/491/commits/b655cb059609c9cb11edf561bc2307ba50b71f45
b655cb0https://github.com/plotly/plotly.js/pull/491/commits/b655cb059609c9cb11edf561bc2307ba50b71f45
e63ba4ahttps://github.com/plotly/plotly.js/commit/e63ba4ae59f022435f8cc6d5221f6ee748391896
https://github.com/etpinard
split pie trace base plot module from cartesian:https://github.com/plotly/plotly.js/pull/491/commits/f476e05b09a7a9af1bb2689fb10f1121c9c1c72b
f476e05https://github.com/plotly/plotly.js/pull/491/commits/f476e05b09a7a9af1bb2689fb10f1121c9c1c72b
https://github.com/etpinard
rm (now useless) trace module layoutCategorieshttps://github.com/plotly/plotly.js/pull/491/commits/1c41c2e24c9426cf3886090b15d57f51de32216c
1c41c2ehttps://github.com/plotly/plotly.js/pull/491/commits/1c41c2e24c9426cf3886090b15d57f51de32216c
https://github.com/etpinard
etpinardhttps://github.com/etpinard
May 5, 2016 https://github.com/plotly/plotly.js/pull/491#ref-pullrequest-153124495
Fix relayout axis category #510 https://github.com/plotly/plotly.js/pull/510
etpinardhttps://github.com/etpinard
May 6, 2016 14:04https://github.com/plotly/plotly.js/pull/491#commits-pushed-313150e
https://github.com/etpinard
add fill unique lib function,https://github.com/plotly/plotly.js/pull/491/commits/313150eaa68ee02c09423af8b9aca7efafc61b0d
313150ehttps://github.com/plotly/plotly.js/pull/491/commits/313150eaa68ee02c09423af8b9aca7efafc61b0d
https://github.com/etpinard
fill fullLayout._basePlotModules list in default trace loop,https://github.com/plotly/plotly.js/pull/491/commits/392619c645990d78793ad8388b172b54c693d451
392619chttps://github.com/plotly/plotly.js/pull/491/commits/392619c645990d78793ad8388b172b54c693d451
https://github.com/etpinard
loop over basePlotModules instead of subplotRegistry in:https://github.com/plotly/plotly.js/pull/491/commits/3787d4734ec7269b5085b2e846e68fb015743d7d
3787d47https://github.com/plotly/plotly.js/pull/491/commits/3787d4734ec7269b5085b2e846e68fb015743d7d
https://github.com/etpinard
make sure to add cartesian base plot module to orphan axis graphs:https://github.com/plotly/plotly.js/pull/491/commits/68b12c4da0de16ab3d66d646c36ab7e44ecc3be7
68b12c4https://github.com/plotly/plotly.js/pull/491/commits/68b12c4da0de16ab3d66d646c36ab7e44ecc3be7
https://github.com/etpinard
move 3d axes init step from makePlotFrameork -> Gl3d.plothttps://github.com/plotly/plotly.js/pull/491/commits/a41427cd7500a6b19da9a4d1afab1e7918de2957
a41427chttps://github.com/plotly/plotly.js/pull/491/commits/a41427cd7500a6b19da9a4d1afab1e7918de2957
https://github.com/etpinard
check for non-cartesian/pie/ternary base plot module in restyle,https://github.com/plotly/plotly.js/pull/491/commits/a48d071e98203ec8696ba9a249d8251e52ba4cef
a48d071https://github.com/plotly/plotly.js/pull/491/commits/a48d071e98203ec8696ba9a249d8251e52ba4cef
https://github.com/etpinard
handle a few polar-only cases:https://github.com/plotly/plotly.js/pull/491/commits/a78521c438bcb0e5fde2f23590bca25a03295fc9
a78521chttps://github.com/plotly/plotly.js/pull/491/commits/a78521c438bcb0e5fde2f23590bca25a03295fc9
https://github.com/etpinard
sub has??? -> has('???')https://github.com/plotly/plotly.js/pull/491/commits/aa2ec8bd7bee480727fd1514fd354012efc950ca
aa2ec8bhttps://github.com/plotly/plotly.js/pull/491/commits/aa2ec8bd7bee480727fd1514fd354012efc950ca
https://github.com/etpinard
loop over (trace) modules in trace supplyLayoutDefaults:https://github.com/plotly/plotly.js/pull/491/commits/77fb7e20ccd97985fc834d936b1251d94741711d
77fb7e2https://github.com/plotly/plotly.js/pull/491/commits/77fb7e20ccd97985fc834d936b1251d94741711d
etpinardhttps://github.com/etpinard
May 6, 2016 14:38https://github.com/plotly/plotly.js/pull/491#commits-pushed-a141d78
https://github.com/etpinard
robustify pie clean step:https://github.com/plotly/plotly.js/pull/491/commits/a141d78cc6767769e1b88d19ae2abe9849b31fbb
a141d78https://github.com/plotly/plotly.js/pull/491/commits/a141d78cc6767769e1b88d19ae2abe9849b31fbb
https://github.com/etpinard
put ref of hasPlotType method on Plots:https://github.com/plotly/plotly.js/pull/491/commits/dc297ca1913c3ed61a559c5b679c718a7007ce08
dc297cahttps://github.com/plotly/plotly.js/pull/491/commits/dc297ca1913c3ed61a559c5b679c718a7007ce08
https://github.com/etpinard
move block of _has?? = _has('') after all defaults are filled:https://github.com/plotly/plotly.js/pull/491/commits/e3f72f092f16e67345cbabb72da742ced607f4a7
e3f72f0https://github.com/plotly/plotly.js/pull/491/commits/e3f72f092f16e67345cbabb72da742ced607f4a7
https://github.com/etpinard
update testshttps://github.com/plotly/plotly.js/pull/491/commits/c59a9607bf020198fc011259ac04937c491bfa38
c59a960https://github.com/plotly/plotly.js/pull/491/commits/c59a9607bf020198fc011259ac04937c491bfa38
https://github.com/etpinard
add info about data/layout and fullData/fullLayouthttps://github.com/plotly/plotly.js/pull/491/commits/9f94494fd4dc4c9d08c72e40c9f3ab53a9e2e96d
9f94494https://github.com/plotly/plotly.js/pull/491/commits/9f94494fd4dc4c9d08c72e40c9f3ab53a9e2e96d
https://github.com/etpinard
etpinardhttps://github.com/etpinard
status: reviewable https://github.com/plotly/plotly.js/issues?q=state%3Aopen%20label%3A%22status%3A%20reviewable%22
status: in progress https://github.com/plotly/plotly.js/issues?q=state%3Aopen%20label%3A%22status%3A%20in%20progress%22
May 6, 2016https://github.com/plotly/plotly.js/pull/491#event-653089173
https://github.com/etpinard
etpinardhttps://github.com/etpinard
May 6, 2016https://github.com/plotly/plotly.js/pull/491#issuecomment-217545704
@alexcjohnsonhttps://github.com/alexcjohnson
@mdtuszhttps://github.com/mdtusz
PR descriptionhttps://github.com/plotly/plotly.js/pull/491#issue-151718094
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
https://github.com/etpinard
Merge branch 'master' into has-plot-typehttps://github.com/plotly/plotly.js/pull/491/commits/9837d8b39bfccda75207127c3e848bf3bb4efc53
9837d8bhttps://github.com/plotly/plotly.js/pull/491/commits/9837d8b39bfccda75207127c3e848bf3bb4efc53
etpinardhttps://github.com/etpinard
May 6, 2016 https://github.com/plotly/plotly.js/pull/491#discussion-diff-62384418
View reviewed changes https://github.com/plotly/plotly.js/pull/491/files/9837d8b39bfccda75207127c3e848bf3bb4efc53#diff-4cc342865f02fea8acedf92e7c0855b4c805c9824b4b4aaf0f587dba2506c9e7
src/plot_api/plot_api.jshttps://github.com/plotly/plotly.js/pull/491/files#diff-4cc342865f02fea8acedf92e7c0855b4c805c9824b4b4aaf0f587dba2506c9e7
etpinardhttps://github.com/etpinard
May 6, 2016https://github.com/plotly/plotly.js/pull/491#discussion_r62384418
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
etpinardhttps://github.com/etpinard
May 9, 2016 17:16https://github.com/plotly/plotly.js/pull/491#commits-pushed-0c7b2f8
https://github.com/etpinard
rename Lib.fillUnique -> Lib.pushUniquehttps://github.com/plotly/plotly.js/pull/491/commits/0c7b2f800ca21e0ddbc3f7e72189231150b00bf7
0c7b2f8https://github.com/plotly/plotly.js/pull/491/commits/0c7b2f800ca21e0ddbc3f7e72189231150b00bf7
https://github.com/etpinard
rm unnecessary boolean type coercionhttps://github.com/plotly/plotly.js/pull/491/commits/5e6759b10d47fe7d8dd11e641b019bb0169930da
5e6759bhttps://github.com/plotly/plotly.js/pull/491/commits/5e6759b10d47fe7d8dd11e641b019bb0169930da
https://github.com/mdtusz
mdtuszhttps://github.com/mdtusz
May 9, 2016https://github.com/plotly/plotly.js/pull/491#issuecomment-217996514
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
etpinardhttps://github.com/etpinard
May 11, 2016 https://github.com/plotly/plotly.js/pull/491#discussion-diff-62896838
View reviewed changes https://github.com/plotly/plotly.js/pull/491/files/5e6759b10d47fe7d8dd11e641b019bb0169930da#diff-91cf4d62b1e2b6b1ba56098e7cfdc6a3ec8cf275c6fd2b6f4b9a6f5bfca058d8
src/plots/cartesian/layout_defaults.jshttps://github.com/plotly/plotly.js/pull/491/files#diff-91cf4d62b1e2b6b1ba56098e7cfdc6a3ec8cf275c6fd2b6f4b9a6f5bfca058d8
etpinardhttps://github.com/etpinard
May 11, 2016https://github.com/plotly/plotly.js/pull/491#discussion_r62896838
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
https://github.com/etpinard
etpinardhttps://github.com/etpinard
1903061https://github.com/plotly/plotly.js/commit/19030614649428a0341d28402178bdd32fc75357
May 11, 2016https://github.com/plotly/plotly.js/pull/491#event-657576146
https://github.com/etpinard
etpinardhttps://github.com/etpinard
May 11, 2016 18:16https://github.com/plotly/plotly.js/pull/491#event-657576176
https://github.com/etpinard
etpinardhttps://github.com/etpinard
May 19, 2016 https://github.com/plotly/plotly.js/pull/491#ref-pullrequest-155816404
Seamless subplots #554 https://github.com/plotly/plotly.js/pull/554
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Jun 10, 2016 https://github.com/plotly/plotly.js/pull/491#ref-pullrequest-159713886
Introducing plotly.js + Mapbox GL #626 https://github.com/plotly/plotly.js/pull/626
https://github.co/hiddenchars
https://github.com/plotly/plotly.js/pull/{{ revealButtonHref }}
Sign up for freehttps://github.com/join?source=comment-repo
Sign in to commenthttps://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fplotly%2Fplotly.js%2Fpull%2F491
Please reload this pagehttps://github.com/plotly/plotly.js/pull/491
https://github.com/etpinard
https://github.com/alexcjohnson
https://github.com/mdtusz
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width

URLs of crawlers that visited me.