GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Change default doubleClickDelay to 500ms (from 300ms) by emilykl · Pull Request #8014 · plotly/plotly.js

Conversation

@emilykl

@emilykl emilykl commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Closes #8013 (see also original discussion in #6538)

Increase Plotly's default double-click delay to 500ms, to align with the most common OS defaults.

Steps for testing

  1. Run the devtools dashboard and open any mock with a legend, for example legend_horizontal
  2. Experiment with how fast you need to double-click a legend item in order to hide all other traces. If you double-click too slow, your clicks will be interpreted as single clicks and the clicked item will simply hide and then show again quickly
  3. Do the same experimentation on main. Notice that the double-click speed threshold is noticeably slower on this branch compared to main.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@emilykl
change default doubleClickDelay to 500ms (from 300ms)
f6630c1
@emilykl
emilykl requested review from KoolADE85 and camdecoster September 3, 2026 12:58
@emilykl
add draftlog
e1c1ec3
camdecoster
camdecoster approved these changes Sep 3, 2026
View reviewed changes
@emilykl
emilykl merged commit 8070ed5 into main Sep 3, 2026
410 of 414 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@emilykl
emilykl deleted the double-click-delay-500ms branch September 3, 2026 18:18
@oscarrutt

oscarrutt commented Sep 3, 2026

Copy link
Copy Markdown

Thanks for at last for looking into this. However, as emilykl mentioned on another thread, on macOS the current setting for double-click delay is found by

% defaults read -g com.apple.mouse.doubleClickThreshold
0.5

So why can't Javascript run this command and read the output?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@emilykl

emilykl commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

@oscarrutt That's a shell command, so unfortunately there's no way to run it from JavaScript in the browser.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@oscarrutt

oscarrutt commented Sep 3, 2026

Copy link
Copy Markdown

@emilykl Thanks. I didn't know that. Most languages have some facility for that but I suppose JavaScript limits this for security. Thanks for your help on this little annoyance in Plotly.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@emilykl

emilykl commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

@oscarrutt Yeah, if we were running as a desktop app I think we'd be able to get this info but for security/privacy reasons, what's available in the browser is pretty limited.

Ultimately I think the most correct approach would be a refactor to use double-click event handlers, so we'll look into whether that's possible and feasible.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@MingoDynasty MingoDynasty mentioned this pull request Oct 5, 2026
chore(deps): first refresh from the playbook, with the minimum package age on MingoDynasty/Corporate-Serf-Dashboard#347
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@camdecoster camdecoster camdecoster approved these changes
@KoolADE85 KoolADE85 Awaiting requested review from KoolADE85

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Increase default plotly.js double-click delay to 500ms (from current 300)

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@emilykl @oscarrutt @camdecoster
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.