René's URL Explorer Experiment


Title: Fix/legend resize by timelyportfolio · Pull Request #356 · plotly/plotly.js · GitHub

Open Graph Title: Fix/legend resize by timelyportfolio · Pull Request #356 · plotly/plotly.js

X Title: Fix/legend resize by timelyportfolio · Pull Request #356 · plotly/plotly.js

Description: Open-source JavaScript charting library behind Plotly and Dash - Fix/legend resize by timelyportfolio · Pull Request #356 · plotly/plotly.js

Open Graph Description: This is my first attempt at addressing #348 and plotly/plotly.R#525. In summary, I moved some code out of the if(firstRender==true block as suggested in @etpinard issue comment fixed clipPath ent...

X Description: This is my first attempt at addressing #348 and plotly/plotly.R#525. In summary, I moved some code out of the if(firstRender==true block as suggested in @etpinard issue comment fixed clipPath ent...

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

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:c4d7110b-7f30-0b4b-1d91-c08a71e59fde
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idAD1E:27E2F7:5F9F7E:7F821E:6A52FDE9
html-safe-nonce107488f629f56c1394e200ddad33888e6d04fad1c214b9a2d183577c15fc25f8
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBRDFFOjI3RTJGNzo1RjlGN0U6N0Y4MjFFOjZBNTJGREU5IiwidmlzaXRvcl9pZCI6IjQ2MTQ0NDczODI0MDkzNzQ0OSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmacc631966de68cf0de93e68fed16e6795491d7141334bfadf3169d1d13a7305571
hovercard-subject-tagpull_request:64074000
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/356/pull_request_layout
twitter:imagehttps://opengraph.githubassets.com/5215449c1d8eed500be0f26f515bb9d25fa89c84c3813d2eacc9360106df65ca/plotly/plotly.js/pull/356
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/5215449c1d8eed500be0f26f515bb9d25fa89c84c3813d2eacc9360106df65ca/plotly/plotly.js/pull/356
og:image:altThis is my first attempt at addressing #348 and plotly/plotly.R#525. In summary, I moved some code out of the if(firstRender==true block as suggested in @etpinard issue comment fixed clipPath ent...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernametimelyportfolio
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/356#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fplotly%2Fplotly.js%2Fpull%2F356
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%2F356
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/356
Reloadhttps://github.com/plotly/plotly.js/pull/356
Reloadhttps://github.com/plotly/plotly.js/pull/356
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
plotly https://github.com/plotly
plotly.jshttps://github.com/plotly/plotly.js
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
Notifications https://github.com/login?return_to=%2Fplotly%2Fplotly.js
Fork 2k https://github.com/login?return_to=%2Fplotly%2Fplotly.js
Star 18.2k 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
timelyportfoliohttps://github.com/timelyportfolio
plotly:masterhttps://github.com/plotly/plotly.js/tree/master
timelyportfolio:fix/legend-resizehttps://github.com/timelyportfolio/plotly.js/tree/fix/legend-resize
Conversationhttps://github.com/plotly/plotly.js/pull/356
Commits4 (4)https://github.com/plotly/plotly.js/pull/356/commits
Checkshttps://github.com/plotly/plotly.js/pull/356/checks
Files changedhttps://github.com/plotly/plotly.js/pull/356/files
Fix/legend resizehttps://github.com/plotly/plotly.js/pull/356#top
timelyportfoliohttps://github.com/timelyportfolio
plotly:masterhttps://github.com/plotly/plotly.js/tree/master
timelyportfolio:fix/legend-resizehttps://github.com/timelyportfolio/plotly.js/tree/fix/legend-resize
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
Mar 24, 2016https://github.com/plotly/plotly.js/pull/356#issue-143317505
#348https://github.com/plotly/plotly.js/issues/348
plotly/plotly.R#525https://github.com/plotly/plotly.R/issues/525
@etpinard issue commenthttps://github.com/plotly/plotly.js/issues/348#issuecomment-200411524
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
etpinardhttps://github.com/etpinard
March 23, 2016 15:02https://github.com/plotly/plotly.js/pull/356#commits-pushed-52f1a3c
https://github.com/etpinard
https://github.com/timelyportfolio
incremental progress toward working scrolling legend after resizehttps://github.com/plotly/plotly.js/pull/356/commits/52f1a3c686daea20e3ebd798a84416201d7808b2
52f1a3chttps://github.com/plotly/plotly.js/pull/356/commits/52f1a3c686daea20e3ebd798a84416201d7808b2
https://github.com/timelyportfolio
change legend scrollbar drag to d3 drag behaviorhttps://github.com/plotly/plotly.js/pull/356/commits/197f7fa1cf4bb903fade6e44a2c9322546ee9eff
197f7fahttps://github.com/plotly/plotly.js/pull/356/commits/197f7fa1cf4bb903fade6e44a2c9322546ee9eff
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
Mar 24, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-200953513
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/timelyportfolio
linthttps://github.com/plotly/plotly.js/pull/356/commits/94bd5fffce7b0b4055ea7f7604a5569cf794e1ee
94bd5ffhttps://github.com/plotly/plotly.js/pull/356/commits/94bd5fffce7b0b4055ea7f7604a5569cf794e1ee
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
force-pushedhttps://github.com/plotly/plotly.js/compare/fd2351cbfdc4ccdb79211776791595b44b66f46d..94bd5fffce7b0b4055ea7f7604a5569cf794e1ee
fd2351chttps://github.com/plotly/plotly.js/commit/fd2351cbfdc4ccdb79211776791595b44b66f46d
94bd5ffhttps://github.com/plotly/plotly.js/commit/94bd5fffce7b0b4055ea7f7604a5569cf794e1ee
Compare https://github.com/plotly/plotly.js/compare/fd2351cbfdc4ccdb79211776791595b44b66f46d..94bd5fffce7b0b4055ea7f7604a5569cf794e1ee
March 24, 2016 18:31https://github.com/plotly/plotly.js/pull/356#event-602158869
mdtuszhttps://github.com/mdtusz
Mar 24, 2016 https://github.com/plotly/plotly.js/pull/356#discussion-diff-57395611
View reviewed changes https://github.com/plotly/plotly.js/pull/356/files/94bd5fffce7b0b4055ea7f7604a5569cf794e1ee#diff-d8edc6534e1e0d8e389e68825f8ecdcdc8a607d22ab022006515ad1077182578
src/components/legend/draw.jshttps://github.com/plotly/plotly.js/pull/356/files#diff-d8edc6534e1e0d8e389e68825f8ecdcdc8a607d22ab022006515ad1077182578
mdtuszhttps://github.com/mdtusz
Mar 24, 2016https://github.com/plotly/plotly.js/pull/356#discussion_r57395611
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
timelyportfoliohttps://github.com/timelyportfolio
Mar 24, 2016https://github.com/plotly/plotly.js/pull/356#discussion_r57396058
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/mdtusz
mdtuszhttps://github.com/mdtusz
Mar 24, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-201045176
@etpinardhttps://github.com/etpinard
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Mar 25, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-201337454
d3-selectionhttps://github.com/d3/d3-selection#handling-events
@timelyportfoliohttps://github.com/timelyportfolio
casehttps://github.com/plotly/plotly.js/blob/bbb8205903cdd34ae019d5cd6a2c197e9a2550e7/test/jasmine/tests/legend_scroll_test.js#L80
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Mar 28, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-202576909
@timelyportfoliohttps://github.com/timelyportfolio
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
Mar 29, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-202868225
@etpinardhttps://github.com/etpinard
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Mar 29, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-202889052
@timelyportfoliohttps://github.com/timelyportfolio
examplehttp://community.plot.ly/t/default-showlegend-changed-with-update/708/2
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
Mar 29, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-202908582
@etpinardhttps://github.com/etpinard
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
force-pushedhttps://github.com/plotly/plotly.js/compare/c012d741b761a419bdfe446e6f70b8c5fe766b25..43d430c7c1889fa3ef5d0b7722d88698752359a0
c012d74https://github.com/plotly/plotly.js/commit/c012d741b761a419bdfe446e6f70b8c5fe766b25
43d430chttps://github.com/plotly/plotly.js/commit/43d430c7c1889fa3ef5d0b7722d88698752359a0
Compare https://github.com/plotly/plotly.js/compare/c012d741b761a419bdfe446e6f70b8c5fe766b25..43d430c7c1889fa3ef5d0b7722d88698752359a0
March 29, 2016 21:22https://github.com/plotly/plotly.js/pull/356#event-607357074
mdtuszhttps://github.com/mdtusz
Mar 29, 2016 https://github.com/plotly/plotly.js/pull/356#discussion-diff-57804561
View reviewed changes https://github.com/plotly/plotly.js/pull/356/files/94bd5fffce7b0b4055ea7f7604a5569cf794e1ee..43d430c7c1889fa3ef5d0b7722d88698752359a0#diff-1c9011cecef7c5fa1a9ae135b7282735601013c5e703ee0da2f47853c0f462ba
test/jasmine/tests/legend_scroll_test.jshttps://github.com/plotly/plotly.js/pull/356/files#diff-1c9011cecef7c5fa1a9ae135b7282735601013c5e703ee0da2f47853c0f462ba
mdtuszhttps://github.com/mdtusz
Mar 29, 2016https://github.com/plotly/plotly.js/pull/356#discussion_r57804561
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
timelyportfoliohttps://github.com/timelyportfolio
Mar 29, 2016https://github.com/plotly/plotly.js/pull/356#discussion_r57805289
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
force-pushedhttps://github.com/plotly/plotly.js/compare/a565dc691c946403425da92f68c702f0c6a2586b..3e6b949354b2f9e6f906d9879f716699d9d27221
a565dc6https://github.com/plotly/plotly.js/commit/a565dc691c946403425da92f68c702f0c6a2586b
3e6b949https://github.com/plotly/plotly.js/commit/3e6b949354b2f9e6f906d9879f716699d9d27221
Compare https://github.com/plotly/plotly.js/compare/a565dc691c946403425da92f68c702f0c6a2586b..3e6b949354b2f9e6f906d9879f716699d9d27221
March 30, 2016 03:00https://github.com/plotly/plotly.js/pull/356#event-607640758
https://github.com/timelyportfolio
add tests for legend resize and this forum [post](https://github.com/plotly/plotly.js/pull/356/commits/3e6b949354b2f9e6f906d9879f716699d9d27221
http://community.pl…http://community.plot.ly/t/default-showlegend-changed-with-update/708
3e6b949https://github.com/plotly/plotly.js/pull/356/commits/3e6b949354b2f9e6f906d9879f716699d9d27221
…ot.ly/t/default-showlegend-changed-with-update/708http://community.plot.ly/t/default-showlegend-changed-with-update/708
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
Mar 30, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-203227143
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Mar 30, 2016 https://github.com/plotly/plotly.js/pull/356#ref-pullrequest-144596594
Legend fixups [fixes #348] #365 https://github.com/plotly/plotly.js/pull/365
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Mar 30, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-203438699
@timelyportfoliohttps://github.com/timelyportfolio
#365https://github.com/plotly/plotly.js/pull/365
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/etpinard
etpinardhttps://github.com/etpinard
Mar 30, 2016https://github.com/plotly/plotly.js/pull/356#event-608277669
https://github.com/timelyportfolio
timelyportfoliohttps://github.com/timelyportfolio
Mar 30, 2016https://github.com/plotly/plotly.js/pull/356#issuecomment-203449128
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
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%2F356
Please reload this pagehttps://github.com/plotly/plotly.js/pull/356
https://github.com/timelyportfolio
https://github.com/mdtusz
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.