openwhyd / openwhyd-electron

💻 Install Openwhyd on your computer, play music in the background
https://openwhyd.org/download
17 stars 5 forks source link

Back navigation #8

Closed l-henri closed 6 years ago

l-henri commented 7 years ago

Hi all,

Not sure if this is an issue or a feature request. I tried the app for mac yesterday for the first time - love it! I'm running Sierra 10.12.5 When I browse the site through the app, I can not "go back". For example, if I do the following:

I can not go back directly to browsing the available playlists. I need to go back to the profile, then open the playlists again. It is a small enhancement, but it would make for a smoother navigation through the app, and it would be easier to explore the others' playlists.

adrienjoly commented 7 years ago

Thank you for posting this feature request here, @ElBurritoPodrido ! :-)

Do you have any idea / proposition on how to do this, by any chance?

It was also reported that the Ctrl-F / Cmd-F keyboard shortcut for search did not work from the electron app. Maybe we could find a way to solve those two problems at once?

l-henri commented 7 years ago

No I'm sorry, I'm clueless :-( I know more of Python and backend stuff. I am not super familiar with how to build an app like this. I'll take a look at the code though, I am curious now!

l-henri commented 7 years ago

Actually, for Ctrl F I saw this thread https://github.com/electron/electron/issues/2037 pointing toward https://electron.atom.io/docs/api/web-contents/#contentsfindinpagetext-options

And for back navigation, this document describes an Electron API for navigation https://github.com/electron/electron/blob/master/docs/api/web-contents.md

contents.goBack()

Makes the browser go back a web page.

Does this help?

adrienjoly commented 7 years ago

Thanks for sharing your findings, Henri!

Both of your solutions use Electron's API for controlling the "browser".

Concerning the "back" navigation:

Concerning in-page search:

adrienjoly commented 6 years ago

Note: window.history.back() does work.

adrienjoly commented 6 years ago

Done in v1.0.0-rc.1 => closing