Skip to content

fix: FEAT: Notification sound toggle (#1547)#1602

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

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

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 20:28
@floatpanebot floatpanebot added the bug Something isn't working label Jun 21, 2026

@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 the size/S Diff: 11–50 lines label 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: Intel(R) Xeon(R) Platinum 8370C CPU @ 2.80GHz
                           │    old.txt    │               new.txt               │
                           │    sec/op     │    sec/op      vs base              │
ParseSearchQuery_Simple-4    2.545µ ± 449%   2.846µ ± 306%       ~ (p=1.000 n=6)
ParseSearchQuery_Complex-4   6.439µ ±  31%   7.425µ ±  50%       ~ (p=0.240 n=6)
TokenizeSearchQuery-4        5.016µ ±  39%   3.726µ ±  41%       ~ (p=0.240 n=6)
geomean                      4.348µ          4.286µ         -1.42%

                           │  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        163.9µ ± 13%   145.6µ ± 17%       ~ (p=0.485 n=6)
SearchOverlayView-4   164.9µ ± 25%   160.1µ ±  7%       ~ (p=0.240 n=6)
InboxConstruction-4   916.2µ ±  9%   920.0µ ± 14%       ~ (p=0.589 n=6)
geomean               291.4µ         277.8µ        -4.68%

                    │    old.txt    │               new.txt               │
                    │     B/op      │     B/op       vs base              │
LogPanelView-4        44.67Ki ±  0%   33.23Ki ± 34%       ~ (p=0.182 n=6)
SearchOverlayView-4   33.17Ki ± 69%   56.14Ki ± 41%       ~ (p=0.097 n=6)
InboxConstruction-4   874.3Ki ±  0%   874.3Ki ±  0%       ~ (p=0.727 n=6)
geomean               109.0Ki         117.7Ki        +7.98%

                    │   old.txt   │              new.txt              │
                    │  allocs/op  │  allocs/op   vs base              │
LogPanelView-4         714.0 ± 0%    713.0 ± 0%       ~ (p=0.182 n=6)
SearchOverlayView-4    924.0 ± 0%    926.0 ± 0%       ~ (p=0.242 n=6)
InboxConstruction-4   3.478k ± 0%   3.478k ± 0%       ~ (p=1.000 n=6)
geomean               1.319k        1.319k       +0.03%

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