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

Fix lack of searchability in Spotlight #1930

Merged
merged 4 commits into from
May 26, 2023

Conversation

BobbyMcWho
Copy link
Contributor

This fixes the lack of searchability in spotlight however there is a nuance that I have yet to figure out (may require executable changes?) I'm not a MacOS native developer, so I'm not certain.

Scenario: Livebook has not been opened
Spotlight -> Search Livebook -> Select the app
Result: Livebook creates the icon in the menu bar, opens Livebook in the browser
Desired: Works as expected

Scenario: Livebook has been opened, but the tab was closed (icon is in menu bar)
Spotlight -> Search Livebook -> Select the app
Result: Nothing happens
Desired: A new Livebook tab is opened in your browser

@CLAassistant
Copy link

CLAassistant commented May 26, 2023

CLA assistant check
All committers have signed the CLA.

@BobbyMcWho
Copy link
Contributor Author

I'm guessing, but seems like ElixirKit.swift and Livebook.swift will need updated to handle the following events:

func applicationWillEnterForeground(_ application: UIApplication)
func applicationDidBecomeActive(_ application: UIApplication)

@BobbyMcWho
Copy link
Contributor Author

Okay, 6e1ec56 should fix the issue where selecting Livebook from Spotlight when the App is already initialized failed to do anything. Now it'll open a new tab, same as clicking the menu icon and selecting open

Copy link
Contributor

@wojtekmach wojtekmach left a comment

Choose a reason for hiding this comment

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

This is amazing. Thank you!

rel/app/macos/Sources/Livebook/Livebook.swift Outdated Show resolved Hide resolved
@wojtekmach wojtekmach merged commit 2f2c959 into livebook-dev:main May 26, 2023
6 checks 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.

None yet

3 participants