search

p0deje / Maccy

Lightweight clipboard manager for macOS
https://maccy.app
MIT License
12.68k stars 533 forks source link

Optimize ignore apps list view #758

Closed phlpsong closed 6 months ago

phlpsong commented 6 months ago
  1. Some app's name is different in Applications with Ignore app list(VidHub named MediaCenter in ignore app)
  2. Chose the same app multiple times will have duplicate items
  3. Icon view may shown cached image if the app is removed(last item in below screenshot)

SCR-20240327-ujze

2024-03-27 23 42 35
phlpsong commented 6 months ago

It looks like have an issue with Swift Lint check, but I can't find it in the build info. Could you kindly help with this?

p0deje commented 6 months ago

It looks like have an issue with Swift Lint check, but I can't find it in the build info. Could you kindly help with this?

Not sure why it's not showing in CI logs, but here what I see locally:

Maccy/Extensions/Bundle+ApplicationName.swift:16:1: warning: Trailing Newline Violation: Files should have a single trailing newline (trailing_newline)
phlpsong commented 6 months ago

It looks like have an issue with Swift Lint check, but I can't find it in the build info. Could you kindly help with this?

Not sure why it's not showing in CI logs, but here what I see locally:

Maccy/Extensions/Bundle+ApplicationName.swift:16:1: warning: Trailing Newline Violation: Files should have a single trailing newline (trailing_newline)

Much appreciate it, I left an extra blank line at the bottom of the file. This should be fixed in the latest commit.

p0deje commented 6 months ago

Thank you for the contribution!