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

[V2.x] Move popover instead of hide and show #819

Open
2 tasks
weisJ opened this issue Jul 24, 2024 · 11 comments
Open
2 tasks

[V2.x] Move popover instead of hide and show #819

weisJ opened this issue Jul 24, 2024 · 11 comments
Labels
enhancement New feature or request

Comments

@weisJ
Copy link
Contributor

weisJ commented Jul 24, 2024

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

I think the hide and immediate reshow of the popover when the selection changes is a bit unwieldy. I think it would be beneficial to transition the position of the popover in the case where the selection moves to an adjacent item. Thought?

Solution

No response

@weisJ weisJ added the enhancement New feature or request label Jul 24, 2024
@p0deje
Copy link
Owner

p0deje commented Jul 25, 2024

I would not mind implementing this, but I don't think it's possible in SwiftUI without implementing custom views. A much simpler solution would probably be a complete disabling of popover show/hide animation, but I haven't found a way to do this in SwiftUI too.

@weisJ
Copy link
Contributor Author

weisJ commented Jul 25, 2024

I noticed this too today. The API is pretty locked down. And various versions to implement it using NSViewRepresentable seem to not reliably produce the same behaviour (the positioning seems to be a bit off).

@p0deje
Copy link
Owner

p0deje commented Jul 29, 2024

I did some smaller improvements in f1f3ae7, maybe this approach can be used for moving popover too.

@weisJ
Copy link
Contributor Author

weisJ commented Aug 4, 2024

I haven't found a way to utilise this for moving the popover, however I have experimented with having a "detached" popover without an arrow, that updates the content if the selection changes.

I also added a shortcut for opening the popover (⌘I by default) for manual control instead of the automatic delay.

Here is how it looks like:

Screen.Recording.2024-08-04.at.12.15.49.mov

What do you think about this? Would it be worth while to add (some of) the following options:

  • Toggle visibility manually with keyboard shortcut. (The automatic delay still being an option)
    • A selection change could then either dismiss the popover or have the current open/close behaviour.
  • Have a "global" popover as in the video without an anchor. Maybe as an alternative option, though I'm not sure whether it is a good idea to bloat the settings in this regard.

@p0deje
Copy link
Owner

p0deje commented Aug 5, 2024

I think it looks great overall, can you share a rough implementation how it works? I am not sure it makes sense to change this feature in 2.0, maybe postpone until 2.1 or consider a completely different approach to previews. I was thinking about potentially adding a slideout right sidebar with preview similar to https://cindori.com/developer/expandable-floating-panel but that's something for Maccy 3.x.

@weisJ
Copy link
Contributor Author

weisJ commented Aug 5, 2024

The code can be found here: https://github.com/weisJ/Maccy/tree/custom/

There are some other changes in there too, but it should be possible to discern what is related to the popup. It’s also not very cleaned up currently.

I like the idea of the slideout, though some caution needs to be taken to ensure it is always visible i.e if the window is too close to the right screen edge the slideout needs to be on the left etc.

@p0deje
Copy link
Owner

p0deje commented Aug 5, 2024

Let's park the idea for now, I'd like to focus on releasing 2.0 without any new big features, but make sure the app behaves well with increased storage size to 9999 (memory utilization, search, etc). If I can make it happen, it's already a big win and worth its own major release.

@weisJ
Copy link
Contributor Author

weisJ commented Aug 5, 2024

Sure thing 👍🏻 getting out a first release for 2.0 really is the priority here.

@o000u
Copy link

o000u commented Aug 15, 2024

I haven't found a way to utilise this for moving the popover, however I have experimented with having a "detached" popover without an arrow, that updates the content if the selection changes.

I also added a shortcut for opening the popover (⌘I by default) for manual control instead of the automatic delay.

Here is how it looks like:

Screen.Recording.2024-08-04.at.12.15.49.mov

What do you think about this? Would it be worth while to add (some of) the following options:

  • Toggle visibility manually with keyboard shortcut. (The automatic delay still being an option)

    • A selection change could then either dismiss the popover or have the current open/close behaviour.
  • Have a "global" popover as in the video without an anchor. Maybe as an alternative option, though I'm not sure whether it is a good idea to bloat the settings in this regard.

I strongly agree with these two suggestions and hope that they can be realized as soon as possible.
Compared to automatically delaying the display of the preview window, I really think it is much easier and faster to switch between displaying/hiding the preview window using a shortcut key (such as the spacebar) @p0deje

@p0deje
Copy link
Owner

p0deje commented Sep 17, 2024

Per https://www.reddit.com/r/macapps/comments/1fi6gw8/comment/lnmbpn8/

One big annoyance: The list of clips is hidden by the preview of the currently selected clip because it is no longer shown to the side of the list as in version 1.

https://youtu.be/p2235zT6ST8

@weisJ
Copy link
Contributor Author

weisJ commented Oct 2, 2024

I have rebased https://github.com/weisJ/Maccy/tree/custom/ onto master. Maybe now that 2.0 has landed we can reopen the discussion for what the best approach to the info popup is.

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

No branches or pull requests

3 participants