Skip to content

fix: FEAT: Notification sound toggle (#1547) - #1606

Open
BWM0223 wants to merge 1 commit into
floatpane:masterfrom
BWM0223:fix-1547-1782080508
Open

fix: FEAT: Notification sound toggle (#1547)#1606
BWM0223 wants to merge 1 commit into
floatpane:masterfrom
BWM0223:fix-1547-1782080508

Conversation

@BWM0223

@BWM0223 BWM0223 commented Jun 21, 2026

Copy link
Copy Markdown

Fixes #1547

Automated by REAPR

@BWM0223
BWM0223 requested a review from a team as a code owner June 21, 2026 22:21

@floatpanebot floatpanebot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @BWM0223! Please fix the following issues with your PR:

  • Title: Is too long (44 characters). The PR title must be strictly under 40 characters.
  • Body: Missing the ## What? or ## Why? headings required by the PR template.

@floatpanebot floatpanebot added bug Something isn't working size/S Diff: 11–50 lines labels Jun 21, 2026
@floatpanebot

Copy link
Copy Markdown
Member

Benchmark report — no significant change

Metrics worse: 0 · better: 0 (threshold: ±3%).

benchstat output
goos: linux
goarch: amd64
pkg: github.com/floatpane/matcha/backend
cpu: AMD EPYC 7763 64-Core Processor                
                           │    old.txt    │               new.txt               │
                           │    sec/op     │    sec/op     vs base               │
ParseSearchQuery_Simple-4    2.663µ ± 450%   2.687µ ± 38%        ~ (p=0.937 n=6)
ParseSearchQuery_Complex-4   6.502µ ±  27%   8.194µ ± 39%        ~ (p=0.240 n=6)
TokenizeSearchQuery-4        4.037µ ±  36%   4.909µ ± 24%        ~ (p=0.240 n=6)
geomean                      4.119µ          4.763µ        +15.62%

                           │  old.txt   │              new.txt               │
                           │    B/op    │    B/op     vs base                │
ParseSearchQuery_Simple-4    26.00 ± 0%   26.00 ± 0%       ~ (p=1.000 n=6) ¹
ParseSearchQuery_Complex-4   762.0 ± 0%   762.0 ± 0%       ~ (p=1.000 n=6) ¹
TokenizeSearchQuery-4        176.0 ± 0%   176.0 ± 0%       ~ (p=1.000 n=6) ¹
geomean                      151.6        151.6       +0.00%
¹ all samples are equal

                           │  old.txt   │              new.txt               │
                           │ allocs/op  │ allocs/op   vs base                │
ParseSearchQuery_Simple-4    2.000 ± 0%   2.000 ± 0%       ~ (p=1.000 n=6) ¹
ParseSearchQuery_Complex-4   23.00 ± 0%   23.00 ± 0%       ~ (p=1.000 n=6) ¹
TokenizeSearchQuery-4        9.000 ± 0%   9.000 ± 0%       ~ (p=1.000 n=6) ¹
geomean                      7.453        7.453       +0.00%
¹ all samples are equal

pkg: github.com/floatpane/matcha/tui
                    │   old.txt    │              new.txt               │
                    │    sec/op    │    sec/op     vs base              │
LogPanelView-4        155.1µ ± 10%   170.3µ ± 11%       ~ (p=0.132 n=6)
SearchOverlayView-4   178.3µ ± 18%   180.7µ ± 14%       ~ (p=0.589 n=6)
InboxConstruction-4   866.4µ ±  9%   933.6µ ±  6%  +7.76% (p=0.041 n=6)
geomean               288.3µ         306.3µ        +6.24%

                    │    old.txt    │               new.txt                │
                    │     B/op      │     B/op       vs base               │
LogPanelView-4        44.67Ki ± 51%   33.23Ki ± 34%        ~ (p=1.000 n=6)
SearchOverlayView-4   56.14Ki ± 41%   44.66Ki ± 26%        ~ (p=0.784 n=6)
InboxConstruction-4   874.2Ki ±  0%   874.3Ki ±  0%        ~ (p=0.851 n=6)
geomean               129.9Ki         109.1Ki        -16.04%

                    │   old.txt   │              new.txt              │
                    │  allocs/op  │  allocs/op   vs base              │
LogPanelView-4         714.0 ± 0%    713.0 ± 0%       ~ (p=1.000 n=6)
SearchOverlayView-4    926.0 ± 0%    925.0 ± 0%       ~ (p=0.500 n=6)
InboxConstruction-4   3.478k ± 0%   3.478k ± 0%       ~ (p=1.000 n=6)
geomean               1.320k        1.319k       -0.08%

auto-generated by benchmarks.yml

@floatpanebot

Copy link
Copy Markdown
Member

This has had no activity for 45 days. It will be closed in 21 days unless updated. Comment or remove the stale label to keep it open.

@floatpanebot floatpanebot added the stale No activity for extended period label Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size/S Diff: 11–50 lines stale No activity for extended period

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FEAT: Notification sound toggle

2 participants