Skip to content

docs: add screenshots to the docs site - #1714

Open
volard wants to merge 2 commits into
floatpane:masterfrom
volard:docs/add-views-screenshots
Open

docs: add screenshots to the docs site#1714
volard wants to merge 2 commits into
floatpane:masterfrom
volard:docs/add-views-screenshots

Conversation

@volard

@volard volard commented Sep 7, 2026

Copy link
Copy Markdown

What?

  • new calendar tape and screenshot picture
  • pipeline job update to use that calendar tape
  • usage in docs/ already existed screenshots from screenshots/

Why?

@volard
volard requested a review from a team as a code owner September 7, 2026 19:22

@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 @volard! Please fix the following issues with your PR:

  • Title: Is too long (57 characters). The PR title must be strictly under 40 characters.

@floatpanebot floatpanebot added ci CI / build pipeline area/docs Docs site / README documentation Documentation changes size/M Diff: 51–200 lines labels Sep 7, 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    3.004µ ± 35%   3.111µ ± 149%       ~ (p=0.818 n=6)
ParseSearchQuery_Complex-4   7.322µ ± 28%   7.947µ ±  32%       ~ (p=0.699 n=6)
TokenizeSearchQuery-4        4.577µ ± 26%   4.437µ ±  20%       ~ (p=0.937 n=6)
geomean                      4.652µ         4.787µ         +2.90%

                           │  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        180.0µ ± 10%    162.4µ ±  9%       ~ (p=0.065 n=6)
SearchOverlayView-4   177.6µ ± 25%    189.1µ ± 24%       ~ (p=0.699 n=6)
InboxConstruction-4   949.4µ ± 11%   1037.5µ ± 12%       ~ (p=0.180 n=6)
geomean               311.9µ          317.0µ        +1.64%

                    │    old.txt    │               new.txt               │
                    │     B/op      │     B/op       vs base              │
LogPanelView-4        44.67Ki ± 51%   33.23Ki ± 34%       ~ (p=0.545 n=6)
SearchOverlayView-4   56.14Ki ± 41%   56.14Ki ± 41%       ~ (p=0.355 n=6)
InboxConstruction-4   874.2Ki ±  0%   874.3Ki ±  0%       ~ (p=0.145 n=6)
geomean               129.9Ki         117.7Ki        -9.39%

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

auto-generated by benchmarks.yml

@volard volard changed the title docs: add screenshots for each main view to the docs site docs: add views screenshots to the docs site Sep 7, 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 @volard! Please fix the following issues with your PR:

  • Title: Is too long (44 characters). The PR title must be strictly under 40 characters.

@volard volard changed the title docs: add views screenshots to the docs site docs: add screenshots to the docs site Sep 7, 2026
@floatpanebot
floatpanebot dismissed stale reviews from themself September 7, 2026 19:26

Formatting issues have been resolved. Thank you!

@volard

volard commented Sep 7, 2026

Copy link
Copy Markdown
Author

@andrinoff should I fix security related things right here or make another branch and stack that one? I just don't think it's relatable to the issue topic, but I'm not sure anyway

@andrinoff

Copy link
Copy Markdown
Member

this should be a separate PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Docs site / README ci CI / build pipeline documentation Documentation changes size/M Diff: 51–200 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants