Skip to content

Navigation Menu

Appearance settings

GitHub Copilot
Write better code with AI

GitHub Spark New
Build and deploy intelligent apps

GitHub Models New
Manage and compare prompts

GitHub Advanced Security
Find and fix vulnerabilities

Actions
Automate any workflow
Codespaces
Instant dev environments

Issues
Plan and track work

Code Review
Manage code changes

Discussions
Collaborate outside of code

Code Search
Find more, search less
Explore

Why GitHub

Documentation

GitHub Skills

Blog
Integrations

GitHub Marketplace

MCP Registry
View all features
By company size

Enterprises

Small and medium teams

Startups

Nonprofits
By use case

App Modernization

DevSecOps

DevOps

CI/CD

View all use cases
By industry

Healthcare

Financial services

Manufacturing

Government

View all industries
View all solutions
Topics

AI

DevOps

Security

Software Development

View all
Explore

Learning Pathways

Events & Webinars

Ebooks & Whitepapers

Customer Stories

Partners

Executive Insights
GitHub Sponsors
Fund open source developers
The ReadME Project
GitHub community articles
Repositories

Topics

Trending

Collections
Enterprise platform
AI-powered developer platform
Available add-ons

GitHub Advanced Security
Enterprise-grade security features

Copilot for business
Enterprise-grade AI features

Premium Support
Enterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

plotly / plotly.js Public

Notifications You must be signed in to change notification settings
Fork 1.9k
Star 17.9k

Code
Issues 702
Pull requests 43
Actions
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Security
Insights

Break up legend module#329

Merged

etpinard merged 22 commits intomasterplotly/plotly.js:masterfrom

break-up-legendplotly/plotly.js:break-up-legendCopy head branch name to clipboard

Mar 16, 2016

Conversation Commits22 (22)Checks Files changed

Merged

Break up legend module#329
etpinard merged 22 commits intomasterfrom
break-up-legend

Commits

Commits on Mar 14, 2016

factor out legend helper into separate file + add tests
etpinard
committed
move auto-anchor logic to own module + add tests
etpinard
committed
move legend defaults to own module
etpinard
committed
move legend style step to own module:
etpinard
committed
move legend-data logic to own module
etpinard
committed
move legend draw step to own module:
etpinard
committed
lint
etpinard
committed

Commits on Mar 15, 2016

fix typo (function name)
etpinard
committed
lint (mostly infix spacing)
etpinard
committed
make legend element a <g> instead of nestead <svg>
etpinard
committed
cropped legend element using clip-path on legend <g>
etpinard
committed
rm useless .exit().remove() :
etpinard
committed
robustify top-paper <defs>:
etpinard
committed
merge top-paper <defs> into main svg on to-svg step
etpinard
committed
add in-house getBBox test asset:
etpinard
committed
add legend and clip path count test,
etpinard
committed
update baseline:
etpinard
committed
add fallback for FF in getBBox test asset:
etpinard
committed

Commits on Mar 16, 2016

lint (define variable in block)
etpinard
committed
change top-paper <defs> id to 'topdefs' + uid:
etpinard
committed
make copy of top-paper childNodes array-like:
etpinard
committed
make user-select-none CSS class:
etpinard
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.