René's URL Explorer Experiment


Title: [WIP] Fix issue #384 (invisible legends when y is negative) by n-riesco · Pull Request #410 · plotly/plotly.js · GitHub

Open Graph Title: [WIP] Fix issue #384 (invisible legends when y is negative) by n-riesco · Pull Request #410 · plotly/plotly.js

X Title: [WIP] Fix issue #384 (invisible legends when y is negative) by n-riesco · Pull Request #410 · plotly/plotly.js

Description: Open-source JavaScript charting library behind Plotly and Dash - [WIP] Fix issue #384 (invisible legends when y is negative) by n-riesco · Pull Request #410 · plotly/plotly.js

Open Graph Description: TODO Rename Plot.autoMarginVertical() to something more sensible (any suggestions?) Add missing select('rect') Debug test failure: pie hovering event data should contain the correct fie...

X Description: TODO Rename Plot.autoMarginVertical() to something more sensible (any suggestions?) Add missing select('rect') Debug test failure: pie hovering event data should contain the cor...

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

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:f9e2008b-6532-7a67-44ce-84cec3c26a63
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-id9C22:39E76C:951C25:CD02BF:6A53946C
html-safe-noncea5cf371b69d738c08a486fca59ea878733f73d41c9a527e557e69e6b04701bac
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5QzIyOjM5RTc2Qzo5NTFDMjU6Q0QwMkJGOjZBNTM5NDZDIiwidmlzaXRvcl9pZCI6IjI5MzExMzQ4OTQ2MzE1Mzc3MiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmacbd8eb199105679397cd19d764add606dd2a41e75df25515cb3ff8eb8eb7f3832
hovercard-subject-tagpull_request:65881553
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/410/pull_request_layout
twitter:imagehttps://opengraph.githubassets.com/fe72c5bc84fc2272e5ea84be9707575467009f67bc124fb36f2b8526ca91ae37/plotly/plotly.js/pull/410
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/fe72c5bc84fc2272e5ea84be9707575467009f67bc124fb36f2b8526ca91ae37/plotly/plotly.js/pull/410
og:image:altTODO Rename Plot.autoMarginVertical() to something more sensible (any suggestions?) Add missing select('rect') Debug test failure: pie hovering event data should contain the correct fie...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernamen-riesco
hostnamegithub.com
expected-hostnamegithub.com
Noneb9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb
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
release07a982c1d40157c619b364352b704c3ce66bb332
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/plotly/plotly.js/pull/410#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fplotly%2Fplotly.js%2Fpull%2F410
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%2F410
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/410
Reloadhttps://github.com/plotly/plotly.js/pull/410
Reloadhttps://github.com/plotly/plotly.js/pull/410
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
plotly https://github.com/plotly
plotly.jshttps://github.com/plotly/plotly.js
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
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 60 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
n-riescohttps://github.com/n-riesco
plotly:masterhttps://github.com/plotly/plotly.js/tree/master
n-riesco:issue-384-legend-negative-yhttps://github.com/n-riesco/plotly.js/tree/issue-384-legend-negative-y
Conversationhttps://github.com/plotly/plotly.js/pull/410
Commits4 (4)https://github.com/plotly/plotly.js/pull/410/commits
Checkshttps://github.com/plotly/plotly.js/pull/410/checks
Files changedhttps://github.com/plotly/plotly.js/pull/410/files
[WIP] Fix issue #384 (invisible legends when y is negative)https://github.com/plotly/plotly.js/pull/410#top
n-riescohttps://github.com/n-riesco
plotly:masterhttps://github.com/plotly/plotly.js/tree/master
n-riesco:issue-384-legend-negative-yhttps://github.com/n-riesco/plotly.js/tree/issue-384-legend-negative-y
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
Apr 9, 2016https://github.com/plotly/plotly.js/pull/410#issue-147130966
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/n-riesco
Remove unnecessary code in repositionLegend()https://github.com/plotly/plotly.js/pull/410/commits/5ef69b38e3759edba66b2d21fa8508221a0b0dff
5ef69b3https://github.com/plotly/plotly.js/pull/410/commits/5ef69b38e3759edba66b2d21fa8508221a0b0dff
n-riescohttps://github.com/n-riesco
Apr 10, 2016 https://github.com/plotly/plotly.js/pull/410#discussion-diff-59130045
View reviewed changes https://github.com/plotly/plotly.js/pull/410/files/2388e2b36da8d1075a8a1a5e41da5dbb13b4ee5a#diff-d8edc6534e1e0d8e389e68825f8ecdcdc8a607d22ab022006515ad1077182578
src/components/legend/draw.jshttps://github.com/plotly/plotly.js/pull/410/files#diff-d8edc6534e1e0d8e389e68825f8ecdcdc8a607d22ab022006515ad1077182578
n-riescohttps://github.com/n-riesco
Apr 10, 2016https://github.com/plotly/plotly.js/pull/410#discussion_r59130045
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
force-pushedhttps://github.com/plotly/plotly.js/compare/2388e2b36da8d1075a8a1a5e41da5dbb13b4ee5a..45d5eb127043ef05c01da629dda504ee508a5a4e
2388e2bhttps://github.com/plotly/plotly.js/commit/2388e2b36da8d1075a8a1a5e41da5dbb13b4ee5a
45d5eb1https://github.com/plotly/plotly.js/commit/45d5eb127043ef05c01da629dda504ee508a5a4e
Compare https://github.com/plotly/plotly.js/compare/2388e2b36da8d1075a8a1a5e41da5dbb13b4ee5a..45d5eb127043ef05c01da629dda504ee508a5a4e
April 11, 2016 08:53https://github.com/plotly/plotly.js/pull/410#event-622114214
n-riescohttps://github.com/n-riesco
April 11, 2016 10:45https://github.com/plotly/plotly.js/pull/410#commits-pushed-c68ed58
https://github.com/n-riesco
Legend.draw: Add missing select('rect')https://github.com/plotly/plotly.js/pull/410/commits/c68ed58d95b3949a90c8efa00ee413434d271976
c68ed58https://github.com/plotly/plotly.js/pull/410/commits/c68ed58d95b3949a90c8efa00ee413434d271976
https://github.com/n-riesco
Fix invisible legends with negative yhttps://github.com/plotly/plotly.js/pull/410/commits/680dcd31d9e6bf58d996abee085dd025beaacbd1
680dcd3https://github.com/plotly/plotly.js/pull/410/commits/680dcd31d9e6bf58d996abee085dd025beaacbd1
plotly#384https://github.com/plotly/plotly.js/issues/384
https://github.com/n-riesco
Do not round off the legend locationhttps://github.com/plotly/plotly.js/pull/410/commits/ca23f6c0496c388bbe0a437b74e061c081c3b66f
ca23f6chttps://github.com/plotly/plotly.js/pull/410/commits/ca23f6c0496c388bbe0a437b74e061c081c3b66f
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
force-pushedhttps://github.com/plotly/plotly.js/compare/45d5eb127043ef05c01da629dda504ee508a5a4e..ca23f6c0496c388bbe0a437b74e061c081c3b66f
45d5eb1https://github.com/plotly/plotly.js/commit/45d5eb127043ef05c01da629dda504ee508a5a4e
ca23f6chttps://github.com/plotly/plotly.js/commit/ca23f6c0496c388bbe0a437b74e061c081c3b66f
Compare https://github.com/plotly/plotly.js/compare/45d5eb127043ef05c01da629dda504ee508a5a4e..ca23f6c0496c388bbe0a437b74e061c081c3b66f
April 11, 2016 11:06https://github.com/plotly/plotly.js/pull/410#event-622278303
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
Apr 11, 2016https://github.com/plotly/plotly.js/pull/410#issuecomment-208314741
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
etpinardhttps://github.com/etpinard
Apr 11, 2016 https://github.com/plotly/plotly.js/pull/410#discussion-diff-59215847
View reviewed changes https://github.com/plotly/plotly.js/pull/410/files/ca23f6c0496c388bbe0a437b74e061c081c3b66f#diff-bb59a2aa90fd14ef3218544010e7fe42f370e10f0c8ee702b3f85ab86887d970
src/plots/plots.jshttps://github.com/plotly/plotly.js/pull/410/files#diff-bb59a2aa90fd14ef3218544010e7fe42f370e10f0c8ee702b3f85ab86887d970
etpinardhttps://github.com/etpinard
Apr 11, 2016https://github.com/plotly/plotly.js/pull/410#discussion_r59215847
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Apr 11, 2016https://github.com/plotly/plotly.js/pull/410#issuecomment-208381963
#243https://github.com/plotly/plotly.js/pull/243
#365https://github.com/plotly/plotly.js/pull/365
#243https://github.com/plotly/plotly.js/pull/243
#384https://github.com/plotly/plotly.js/issues/384
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
Apr 11, 2016https://github.com/plotly/plotly.js/pull/410#issuecomment-208417252
@etpinardhttps://github.com/etpinard
#243https://github.com/plotly/plotly.js/pull/243
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Apr 11, 2016https://github.com/plotly/plotly.js/pull/410#issuecomment-208467363
@n-riescohttps://github.com/n-riesco
#243https://github.com/plotly/plotly.js/pull/243
greater than zerohttps://github.com/plotly/plotly.js/blob/f692714952568346b6a92e0ff59dc8458cdfe5ac/src/plots/layout_attributes.js#L75-L103
not bigger than the plot width and/or heighthttps://github.com/plotly/plotly.js/blob/f692714952568346b6a92e0ff59dc8458cdfe5ac/src/plots/plots.js#L818-L844
Plots.autoMarginhttps://github.com/plotly/plotly.js/blob/f692714952568346b6a92e0ff59dc8458cdfe5ac/src/plots/plots.js#L851
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Apr 11, 2016https://github.com/plotly/plotly.js/pull/410#issuecomment-208469628
@n-riescohttps://github.com/n-riesco
#243https://github.com/plotly/plotly.js/pull/243
greater than zerohttps://github.com/plotly/plotly.js/blob/f692714952568346b6a92e0ff59dc8458cdfe5ac/src/plots/layout_attributes.js#L75-L103
not bigger than the plot width and/or heighthttps://github.com/plotly/plotly.js/blob/f692714952568346b6a92e0ff59dc8458cdfe5ac/src/plots/plots.js#L818-L844
Plots.autoMarginhttps://github.com/plotly/plotly.js/blob/f692714952568346b6a92e0ff59dc8458cdfe5ac/src/plots/plots.js#L851
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
Apr 13, 2016https://github.com/plotly/plotly.js/pull/410#issuecomment-209405665
#417https://github.com/plotly/plotly.js/pull/417
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
Apr 13, 2016https://github.com/plotly/plotly.js/pull/410#event-625675464
https://github.com/n-riesco
n-riescohttps://github.com/n-riesco
April 15, 2016 12:02https://github.com/plotly/plotly.js/pull/410#event-628821583
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%2F410
Please reload this pagehttps://github.com/plotly/plotly.js/pull/410
https://github.com/n-riesco
https://github.com/etpinard
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.