Skip to content

[feat]: Shortcut for Show All Data#1057

Closed
RubenHalman wants to merge 1 commit into
tprouvot:releaseCandidatefrom
RubenHalman:917
Closed

[feat]: Shortcut for Show All Data#1057
RubenHalman wants to merge 1 commit into
tprouvot:releaseCandidatefrom
RubenHalman:917

Conversation

@RubenHalman
Copy link
Copy Markdown
Contributor

Describe your changes

This PR attempts to resolve Issue #917

Issue ticket number and link

#917

Checklist before requesting a review

  • I have read and understand the Contributions section
  • Target branch is releaseCandidate and not master
  • [] I have performed a self-review of my code
  • I ran the unit tests and my PR does not break any tests
  • I documented the changes I've made on the CHANGES.md and followed actual conventions
  • I added a new section on how-to.md (optional)

@RubenHalman RubenHalman changed the title [feat] Dedicated shortcut for Show All Data [feat]: Shortcut for Show All Data Jan 17, 2026
@RubenHalman
Copy link
Copy Markdown
Contributor Author

RubenHalman commented Jan 19, 2026

@tprouvot I figured if this works well(currently unfinished), I might include or combine it with #1058

@Garywoo
Copy link
Copy Markdown
Contributor

Garywoo commented Feb 24, 2026

I would love to be able to open the Show All Data page directly with one keyboard shortcut that could be configured in the extension keyboard shortcuts settings, as described in #633. Does this PR enable this?

@tprouvot
Copy link
Copy Markdown
Owner

@Garywoo in the idea yes, but I need to see with @RubenHalman if it can be done without replicating the logic from background js and popup into button.js

@RubenHalman
Copy link
Copy Markdown
Contributor Author

RubenHalman commented May 3, 2026

@Garywoo Closing in favour of #1192, rebased onto latest releaseCandidate and addressing the feedback to avoid duplicating logic from background.js/popup.js into button.js.

@RubenHalman RubenHalman closed this May 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants