8000 Implement legendrank attribute in traces by archmoj · Pull Request #5591 · plotly/plotly.js · GitHub
[go: up one dir, main page]
Skip to content

Implement legendrank attribute in traces #5591

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 20 commits into from
Jun 16, 2021
Merged

Implement legendrank attribute in traces #5591

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
3cee835
implement legendrank
archmoj Apr 15, 2021
a60a382
keep track of initial order - to handle chrome unstable sort in old v…
archmoj Apr 15, 2021
5813717
remove some empty lines
archmoj Apr 16, 2021
5136cc0
add legend calc jasmine test with legendrank
archmoj Apr 16, 2021
598d60c
add image test - pie ranking looks strange
archmoj Apr 16, 2021
4ffb06c
describe why dflt 1000 is used - mentioning ranking works within each…
archmoj Apr 16, 2021
6e61bd8
refactor - less variable and use push
archmoj Apr 19, 2021
737478e
refactor - add sort step
archmoj Apr 19, 2021
274db06
refactor - bring down the sort function
archmoj Apr 19, 2021
915c67a
refactor - simplify logic
archmoj Apr 19, 2021
69c0b29
update image test
archmoj Apr 19, 2021
e1e9a74
simplify and fixup legend sort
archmoj Apr 19, 2021
f232f42
no more need for _initID now that sort and collapse are fixed
archmoj Apr 19, 2021
c00d56e
more refactor
archmoj Apr 19, 2021
69bf7eb
remove extra loop - update comments
archmoj Apr 19, 2021
d87ee12
update legendrank description
archmoj Apr 22, 2021
d35ff08
add fallback to have stable sort in Chrome < 70
archmoj Jun 15, 2021
a1f87b9
would be > are
archmoj Jun 15, 2021
32a890c
rank groups based on minimum legendranks within each group
archmoj Jun 16, 2021
a1abb19
add legendrank2 mock
archmoj Jun 16, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
update legendrank description
  • Loading branch information
@archmoj
archmoj committed Apr 22, 2021
commit d87ee126e88997e4e0cc44d62dc1bf9a646eebe8
4 changes: 2 additions & 2 deletions src/plots/attributes.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,8 @@ module.exports = {
dflt: 1000,
editType: 'style',
description: [
'Sets the legend rank for this trace in each `legendgroup`.',
'Items with smaller ranks would be presented on top/left side while',
'Sets the legend rank for this trace.',
'Items and groups with smaller ranks would be presented on top/left side while',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace "would be" with "are" please, throughout :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in a1f87b9

'with `*reversed* `legend.traceorder` they would be on bottom/right side.',
'The default legendrank is 1000,',
'so that you can use ranks less than 1000 to place certain items before all unranked items,',
Expand Down
0