Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Bug] Entry is wrongly selected when opening #941

Closed
2 tasks done
wenfangdu opened this issue Nov 2, 2024 · 18 comments
Closed
2 tasks done

[Bug] Entry is wrongly selected when opening #941

wenfangdu opened this issue Nov 2, 2024 · 18 comments
Labels
enhancement New feature or request

Comments

@wenfangdu
Copy link

Before Submitting Your Feature Request

  • Check that there isn't already a similar feature request to avoid creating a duplicate.
  • I have seen the FAQ.

Problem

First of all, thank you for making this great app! Quite frequently, maccy popup shows up while the cursor is hovering on a maccy entry, then that entry got selected, when I press enter, that entry is pasted, which is unintended.

Solution

Have an option to disable hovering to select, in this case, an entry can only be selected via keyboard.

@wenfangdu wenfangdu added the enhancement New feature or request label Nov 2, 2024
@p0deje
Copy link
Owner

p0deje commented Nov 2, 2024

please share the details of running defaults read org.p0deje.Maccy in Terminal.app

@wenfangdu
Copy link
Author

@p0deje

$ defaults read org.p0deje.Maccy
2024-11-03 11:34:46.674 defaults[70591:13299712] 
Domain /Users/x/Library/Containers/org.p0deje.Maccy/Data/Library/Preferences/org.p0deje.Maccy does not exist

@stephen714
Copy link

This happens for me too, regularly. I second OP's request 😊 Great app though; absolute life saver!!

@p0deje p0deje closed this as completed in 6d3450e Nov 22, 2024
@wenfangdu
Copy link
Author

@p0deje Is it released in 2.2.0? Still reproducible in 2.2.0 though, please reopen.
Notice "exodus" is selected which it shouldn't, the cursor isn't moved, but that entry got selected:

iShot_2024-11-23_11.35.29.mov

@p0deje
Copy link
Owner

p0deje commented Nov 23, 2024

@wenfangdu I cannot reproduce this error, see how the first item remains selected when I popup

Screen.Recording.2024-11-22.at.21.35.59.mov

@wenfangdu
Copy link
Author

@p0deje Strange, what macOS version are you on? I'm on macOS 15.1.1, it's not 100% reproducible, maybe 25% chance, you can try different ways to reproduce, e.g. different app, different popup position, different cursor position, with/without clicking, etc.

With all due respect, this issue asks for an option to disable the hovering to select entirely, this could fix it once and for all. The closing commit didn't add such option, hence, this issue should be reopened.

@750
Copy link

750 commented Nov 23, 2024

Not sure if this is the same issue, but here is a video of something similar:

Screen+Recording+2024-11-23+at+12.56.11.mov

I'm on sonoma 14.5 though

@p0deje
Copy link
Owner

p0deje commented Nov 23, 2024

@p0deje Strange, what macOS version are you on? I'm on macOS 15.1.1, it's not 100% reproducible, maybe 25% chance, you can try different ways to reproduce, e.g. different app, different popup position, different cursor position, with/without clicking, etc.

I'm on macOS 15.2, will try to reproduce more.

With all due respect, this issue asks for an option to disable the hovering to select entirely, this could fix it once and for all. The closing commit didn't add such option, hence, this issue should be reopened.

I don't plan to include an option to disable hovering at all. This would ruin a native user experience where Maccy behaves as the context menu, so you could select items with arrow keys and/or a mouse. If there is an issue when cursors hijacks selection with enter, it's just a bug that needs to be fixed.

@p0deje
Copy link
Owner

p0deje commented Nov 23, 2024

@wenfangdu Please try 2.2.1, it should be a bit better. If still an issue, please provide steps how to reproduce it reliably.

@wenfangdu
Copy link
Author

@p0deje Yes, it's a bit better, but unfortunately still reproducible:

iShot_2024-11-26_11.11.51.mp4

@wenfangdu wenfangdu changed the title [FR] An option to disable hovering to select [Bug] Entry is wrongly selected when opening Nov 26, 2024
@wenfangdu
Copy link
Author

@p0deje I've edited the issue title, please reopen until it gets fixed 🙏.

@p0deje
Copy link
Owner

p0deje commented Nov 26, 2024

@wenfangdu please share your settings by running the following in Terminal.app

defaults read org.p0deje.Maccy

@p0deje p0deje reopened this Nov 26, 2024
@wenfangdu
Copy link
Author

@p0deje You've asked before though.

$ defaults read org.p0deje.Maccy
2024-11-26 22:26:00.814 defaults[49208:3729202] 
Domain /Users/x/Library/Containers/org.p0deje.Maccy/Data/Library/Preferences/org.p0deje.Maccy does not exist

@p0deje
Copy link
Owner

p0deje commented Nov 26, 2024

@wenfangdu This is really strange and bothers me because I cannot confirm an integrity of your Maccy version. Have you ever changed any settings in Maccy?

@wenfangdu
Copy link
Author

@p0deje Sorry, my bad, my privacy settings blocked the terminal from reading the config:

{
    "KeyboardShortcuts_delete" = "{\\"carbonKeyCode\\":51,\\"carbonModifiers\\":2048}";
    "KeyboardShortcuts_pin" = "{\\"carbonModifiers\\":2048,\\"carbonKeyCode\\":35}";
    "KeyboardShortcuts_popup" = "{\\"carbonModifiers\\":4096,\\"carbonKeyCode\\":9}";
    "LaunchAtLogin__hasMigrated" = 1;
    "NSStatusItem Preferred Position Item-0" = 540;
    "NSStatusItem Visible Item-1" = 0;
    "NSWindow Frame SUStatusFrame" = "788 691 400 134 0 0 1920 1055 ";
    "NSWindow Frame SUUpdateAlert" = "677 492 620 398 0 0 1920 1055 ";
    "NSWindow Frame com.sindresorhus.Preferences.FrameAutosaveName" = "1108 113 549 661 0 0 1920 1055 ";
    "NSWindow Frame com.sindresorhus.Settings.FrameAutosaveName" = "732 448 510 474 0 0 1920 1055 ";
    SUEnableAutomaticChecks = 1;
    SUHasLaunchedBefore = 1;
    SULastCheckTime = "2024-11-26 06:10:54 +0000";
    SUSendProfileInfo = 0;
    SUUpdateGroupIdentifier = 2169978086;
    WebKitDefaultFontSize = 13;
    WebKitJavaScriptEnabled = 0;
    WebKitPluginsEnabled = 0;
    WebKitStandardFont = "-apple-system-font";
    WebKitUserStyleSheetEnabledPreferenceKey = 1;
    WebKitUserStyleSheetLocationPreferenceKey = "/Applications/Maccy.app/Contents/Frameworks/Sparkle.framework/Resources/ReleaseNotesColorStyle.css";
    avoidTakingFocus = 0;
    enabledPasteboardTypes =     (
        "public.html",
        "public.utf8-plain-text",
        "public.png",
        "public.rtf",
        "public.tiff"
    );
    historySize = 500;
    ignoredPasteboardTypes =     (
        "Pasteboard generator type",
        "com.typeit4me.clipping",
        "com.agilebits.onepassword",
        "de.petermaurer.TransientPasteboardType",
        "net.antelle.keeweb"
    );
    imageMaxHeight = 40;
    migrations =     {
        "2024-07-01-version-2" = 1;
    };
    pasteByDefault = 1;
    pinTo = top;
    popupPosition = window;
    removeFormattingByDefault = 0;
    showFooter = 1;
    showRecentCopyInMenuBar = 0;
    showSearch = 1;
    showTitle = 0;
    windowSize = "[450,800]";
}

@p0deje p0deje closed this as completed in cfe57c6 Nov 28, 2024
@wenfangdu
Copy link
Author

@p0deje Thanks for the fix, when will it be released?

@p0deje
Copy link
Owner

p0deje commented Dec 11, 2024

@wenfangdu 2.3.0 has just been released

@wenfangdu
Copy link
Author

@p0deje Thanks, it works great!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants