This repository has been archived by the owner on Jun 12, 2020. It is now read-only.
미디어 컨트롤 연동, 우클릭 메뉴, 자동 실행, Dock 에서 보기 추가 #9
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
풀리퀘 이후에 한 커밋도 이전 풀리퀘에 합쳐지는 것 같네요. 메뉴바 아이콘을 우클릭 했을 때 메뉴가 나오게 수정했고, 메뉴에서 시작시 자동 실행, 미디어키 사용 토글, Dock 에서 보기 토글 등을 추가하였습니다. 로그아웃 등도 넣어두고 싶은데, 로그인 관련 기능이 이상하게 작동해서 넣지 않았습니다. |
mskims
suggested changes
Jul 5, 2018
| @@ -11,5 +11,15 @@ if (location.pathname === '/player/fPlayer') { | |||
| genie.on('track-changed', (track) => { | |||
| ipc.send('track-changed', track) | |||
| }) | |||
|
|
|||
| ipc.on('track-playpause', (e) => { | |||
| document.getElementsByClassName('fp-playbtn')[0].click() | |||
There was a problem hiding this comment.
중복 접속 팝업, 성인인증 등 재생을 중단시키는 팝업에 대한 처리가 필요할 것 같습니다.
Notification API 를 사용해서 사용자에게 인지 시키는건 어떨까요?
There was a problem hiding this comment.
대충 코드를 봤는데, Notification 으로 보내기엔 애매한 부분들이 좀 있네요...
그러한 경우에 플레이어를 보이게 하면 어떨까 싶습니다
|
일부 팝업에 대해선 플레이어가 열리도록(player.show()), alert 에 대해선 Notification 으로 처리하도록 수정하였습니다. |
mskims
approved these changes
Jul 6, 2018
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
미디어 단축키(재생/일시정지, 이전, 다음) 제어를 추가하였습니다.
추가로, 패키지와 관계없는 .DS_Store 파일도 .gitignore 에 추가하고 소스에서 제거하였습니다.