Skip to content
Snippets Groups Projects

only allow a single boost via hotkey - fixes #467 #468

Merged dakkar requested to merge feature/468-fix-boost-hotkey into develop
All threads resolved!

What does this PR do?

As usual, have to write the same thing four times

The parentheses around q tell the hotkey/keymap code to ignore auto-repeat events (which works fine in Chrome but not in Firefox, I reported the bug https://bugzilla.mozilla.org/show_bug.cgi?id=1900397 )

In addition, I've added a guard variable that is true while calling the backend to boost/renote, and false otherwise. This way, even in Firefox we don't spam-boost. Unboosting is still only with the mouse, I have not added that functionality.

Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
  • I have made sure to test this pull request

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hazelnoot
  • dakkar added 1 commit

    added 1 commit

    • 38892677 - I meant `finally`, not `then`, thanks fEmber

    Compare with previous version

  • dakkar added 1 commit

    added 1 commit

    • 079abfd7 - rate limit note/reply/boost creation more tightly

    Compare with previous version

  • Hazelnoot resolved all threads

    resolved all threads

  • Hazelnoot approved this merge request

    approved this merge request

  • Marie approved this merge request

    approved this merge request

  • merged

  • dakkar mentioned in commit 10631021

    mentioned in commit 10631021

  • Please register or sign in to reply
    Loading