Skip to content

Conversation

@lowjoel
Copy link
Contributor

@lowjoel lowjoel commented Mar 23, 2025

Depends on #57. Please review that changeset first.

This adds the number of keys loaded in the tooltip, and displays a balloon when a key is used to sign.

@lowjoel lowjoel force-pushed the update-tray-tooltips branch from 372fe10 to ce3bd24 Compare March 23, 2025 02:36
0,
0,
)
systray.Quit()
Copy link
Owner

Choose a reason for hiding this comment

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

A build error occurs here.

PS > wails build
Wails CLI v2.8.2


# Build Options

Platform(s)        | windows/amd64
Compiler           | C:\Program Files\Go\bin\go.exe
Skip Bindings      | false
Build Mode         | production
Devtools           | false
Frontend Directory | C:\Users\ck000\git\OmniSSHAgent\frontend
Obfuscated         | false
Skip Frontend      | false
Compress           | false
Package            | true
Clean Bin Dir      | false
LDFlags            |
Tags               | []
Race Detector      | false

# Building target: windows/amd64

  • Generating bindings:   ERROR
          # github.com/masahide/OmniSSHAgent/pkg/wintray
          pkg\wintray\wintray.go:398:2: undefined: systray

          exit status 1

  ERROR
          # github.com/masahide/OmniSSHAgent/pkg/wintray
          pkg\wintray\wintray.go:398:2: undefined: systray

          exit status 1
 ♥   If Wails is useful to you or your company, please consider sponsoring the project:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

d'oh, had a bunch of dirty changes I didn't fully commit. Fixed in #57, will rebase #58 now.

@lowjoel lowjoel force-pushed the update-tray-tooltips branch from 6dbb03f to 2e91fe4 Compare April 5, 2025 11:40
@lowjoel
Copy link
Contributor Author

lowjoel commented Apr 5, 2025

@masahide just rebased it, have a look.

@masahide
Copy link
Owner

masahide commented Apr 5, 2025

LGTM

@masahide masahide merged commit 4b5a994 into masahide:main Apr 5, 2025
1 check passed
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.

2 participants