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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FEAT: Notification sound toggle

2 participants