update request - Githubissues

hibiyasleep commented 7 years ago

Comments for will-updated layout:

'Set' label of button isn't fit with it's usage. How about Open...?
'Reload overlay' button is too narrow, and isn't enough for ja_JP locale text (オーバーレイをリロード). so, may layout changed like this:
Width of 주소 또는 파일 is docked to parent.
All button has >24px of height.
Width of 개발자용 and 정렬 is 280px.

this is rough edit, so it doesn't need to match pixel-by-pixel.

lalafellsleep commented 7 years ago

It's good advice but...

2016-12-20 12 Can't give fit size. (anchor & dock)

Main TabControl fix I tried... (hmm)

hibiyasleep commented 7 years ago

this may work: Merge "Tab Name", "주소 또는 파일" and "스크린샷 저장 폴더" as one GroupBox named "일반" (as RainbowMage did before), make it inserted as first, and set Dock=Top.

- or -

put them all in TableLayout?

lalafellsleep commented 7 years ago

It's...

now change this..

lalafellsleep commented 7 years ago

upper addon page screenshot already using TableLayout in Docking GroupBox in Textbox Control (Anchored)

lalafellsleep commented 7 years ago

now progress

lalafellsleep commented 7 years ago

-- ToDo

fileOpen (for Overlay Timeline) sendFileList sendDirectoryList callTTS

hibiyasleep commented 7 years ago

does changes in screenshot were uploaded? I fetched your repository to local, but no notable change here...

PS: What is MiniParse Extended? does it have uncompatible change then MiniParse? PS2: 영어 어려우시면 한국어로 쓰셔도 돼요 영어권 유저들이 볼까봐 영어로 쓰는거라...

hibiyasleep commented 7 years ago

@laiglinne-ff

MiniParseExt가 반드시 별도의 바이너리로, 폼으로 분리되어야 하는 이유가 있나요? 바이너리는 모르겠지만, 적어도 설정 UI의 전체적인 통일성은 지켜졌으면 합니다.
지금 배포되고 있는 플러그인의 버전을 받아서 조금 알아보았습니다만... console.log를 캡쳐해서 스크린샷을 찍는 등의 황당한 구현은 자제하여 주셨으면 합니다. HtmlRenderer/BuiltinFunctionHandler.cs에서 하듯이, 브라우저 내에서 사용할 수 있는 함수로 구현해주세요. 이유는:

숨어있지 않습니다. Object로 존재하기 때문에, 해당 기능을 모르는 개발자도 API를 찾아낼 수 있게 됩니다.
확장 가능합니다. 예를 들어, OverlayPluginApi.takeScreenshot({ filename: 'asdf.png' })와 같은 사용법이 가능해질 수 있죠.
적어도 상식적입니다. console.log로 API를 불러내는 생각을 해낼 수 있는 사람은 몇 없을겁니다.

Zoom은 브라우저 레벨에서 이루어질 일입니다. CSS에 zoom이라는 속성도 있고... Ctrl+스크롤의 그 확대/축소가 필요하다면 1번의 제안과 같이 깔끔하게 처리해주세요. 지금의 구현은 7가지의 확대값밖에 사용할 수 없게 되어있습니다.
개발 중의 버전을 정식 버전 넘버와 함께 배포하지 말아주셨으면 (ex: 0.3.3.12) 합니다. 같은 버전넘버를 가진 파일이 여러 버전을 가지면 버전의 의미가 불확실해지고, 배포 관리가 힘들어집니다. 0.3.3.12-pre1 등의 숫자를 사용해주세요.

lalafellsleep commented 7 years ago

https://github.com/cefsharp/CefSharp/wiki/Frequently-asked-questions#JSEvent :(

지원하지 않습니다.

lalafellsleep commented 7 years ago

지원해주신다면 그 "황당한" 구현을 그만둘 수 있는데 말이죠

hibiyasleep commented 7 years ago

이 프로젝트는 CefGlue를 사용하고 / window.OverlayPluginApi가 이미 존재하며 / body에 이벤트를 dispatch하고 그걸 캐치하라는 의도가 아니었습니다.

hibiyasleep commented 7 years ago

우선 공격적이었던 어투에 대해서 사과드립니다. 제가 조금 많이 예민하게 반응한 것 같네요... 커밋 26d2ce5a066733c9902c6afbf691a68c0f523086 의 구현을 확인해주셨으면 합니다.

hibiyasleep commented 7 years ago

OverlayPlugin 버리시는 거 같으니 닫겠습니다

hibiyasleep / OverlayPlugin

update request #3