Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

When preview a big picture, the pic popup covers all the panel #893

Closed
2 tasks done
chenxiccc opened this issue Sep 22, 2024 · 4 comments
Closed
2 tasks done

When preview a big picture, the pic popup covers all the panel #893

chenxiccc opened this issue Sep 22, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@chenxiccc
Copy link

Before Submitting Your Bug Report

  • I have verified that there isn't already an issue reporting the same bug to prevent duplication.
  • I have seen the FAQ.

Maccy Version (see 'About' window)

2.0.1

macOS Version

15.0 (24A335)

Maccy Settings

{
    "KeyboardShortcuts_delete" = "{\\"carbonKeyCode\\":51,\\"carbonModifiers\\":2048}";
    "KeyboardShortcuts_pin" = "{\\"carbonKeyCode\\":35,\\"carbonModifiers\\":2048}";
    "KeyboardShortcuts_popup" = "{\\"carbonModifiers\\":4096,\\"carbonKeyCode\\":9}";
    "LaunchAtLogin__hasMigrated" = 1;
    "NSStatusItem Preferred Position Item-0" = 5506;
    "NSStatusItem Visible Item-1" = 0;
    "NSWindow Frame com.sindresorhus.Settings.FrameAutosaveName" = "608 427 510 473 0 0 1680 1025 ";
    SUAutomaticallyUpdate = 1;
    SUEnableAutomaticChecks = 1;
    SUHasLaunchedBefore = 1;
    SULastCheckTime = "2024-09-22 05:45:54 +0000";
    SUSendProfileInfo = 0;
    SUUpdateGroupIdentifier = 2417714185;
    avoidTakingFocus = 0;
    enabledPasteboardTypes =     (
        "public.html",
        "public.tiff",
        "public.utf8-plain-text",
        "public.png",
        "public.rtf"
    );
    hideFooter = 1;
    historySize = 50;
    ignoredPasteboardTypes =     (
        "net.antelle.keeweb",
        "com.typeit4me.clipping",
        "Pasteboard generator type",
        "com.agilebits.onepassword",
        "de.petermaurer.TransientPasteboardType"
    );
    imageMaxHeight = 20;
    maxMenuItems = 15;
    menuIcon = clipboard;
    migrations =     {
        "2020-04-25-allow-custom-ignored-types" = 1;
        "2020-06-19-use-keyboardshortcuts" = 1;
        "2020-09-01-ignore-keeweb" = 1;
        "2021-02-20-allow-to-customize-supported-types" = 1;
        "2021-06-28-add-title-to-history-item" = 1;
        "2021-10-16-remove-dynamic-pasteboard-types" = 1;
        "2022-08-01-rename-suppress-clear-alert" = 1;
        "2022-11-14-add-html-rtf-to-supported-types" = 1;
        "2023-01-22-add-regexp-search-mode" = 1;
        "2024-07-01-version-2" = 1;
    };
    pasteByDefault = 1;
    pinTo = bottom;
    popupPosition = cursor;
    previewDelay = 1500;
    removeFormattingByDefault = 0;
    showFooter = 0;
    showInStatusBar = 1;
    showRecentCopyInMenuBar = 0;
    showSearch = 1;
    showTitle = 1;
    windowSize = "[450,800]";
}

Description

When copy a big screen shot, such as a whole screen, when preview the pic in the panel, the preview popup covers the maccy panel, and cannot select other items.
The preview popup should appears on the left or right side of the panel, should not appears in front of the panel.
image

Steps to Reproduce

No response

@chenxiccc chenxiccc added the bug Something isn't working label Sep 22, 2024
@chenxiccc
Copy link
Author

The right position:
image

The wrong position:
image

@p0deje p0deje closed this as completed in 748f662 Oct 2, 2024
@chenxiccc
Copy link
Author

chenxiccc commented Oct 9, 2024

@p0deje I've tested 2.1.0 release, this is NOT fixed.

The popup covers the maccy panel. Not show at the left or right of the panel. Not only "height / 0.8"
image

Where I think is the right position.
image

@p0deje
Copy link
Owner

p0deje commented Oct 9, 2024

That's the default behavior whenever there is not enough space to the left / right of the window to show the preview.

@chenxiccc
Copy link
Author

That's the default behavior whenever there is not enough space to the left / right of the window to show the preview.

Oh, I see. Thanks.
But when there is not enough space to the left / right of the window to show the preview, reduce the preview width to make the preview on the left/right may be better?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants