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

[保留中] windows7 の AeroSnapに対応させる #107

Closed
berryzplus opened this issue Jun 12, 2018 · 6 comments
Closed

[保留中] windows7 の AeroSnapに対応させる #107

berryzplus opened this issue Jun 12, 2018 · 6 comments
Assignees
Labels
🐛bug🦋 ■バグ修正(Something isn't working) enhancement ■機能追加

Comments

@berryzplus
Copy link
Contributor

berryzplus commented Jun 12, 2018

目的

windows7 の AeroSnapに対応させる。

経緯

タブを表示中のサクラエディタをエアロスナップさせた状態で、
別のタブに切り替えるとエアロスナップが解除されてしまう。

関連するissues
#45, #47, #65

当初は、過去の貢献者が残した暫定パッチを取り込むつもりのPRを作成したが、
何も手を加えずに取り込むことが難しいことを認識したので方針を転換し、
わたし ( @berryzplus ) が代理コミットについて一旦レビューを行い、
修正コミットを積んだうえで再度 PR を行うこととしました。

対応予定について

少し時間をかけてレビュアーと仮想レビュイーの自作自演のやり取りを行った後、
最終的にberryzplusがコミットできると判断した時点でタイトルのWIPを外します。
今週いっぱいくらいかけるつもりです。
 ↓
思考過程を紹介できるほど余裕がなかったので、
単純にぼくが修正した内容についてコメントしただけのPRになってしまいました。
ようやくレビュー可能な状態(=ぼくがOK出せる状態)にできたので、
このissueはそちらが完了次第閉じようと思います。

よろしくお願いします。

@berryzplus berryzplus added 🐛bug🦋 ■バグ修正(Something isn't working) enhancement ■機能追加 labels Jun 12, 2018
@kobake
Copy link
Member

kobake commented Jun 12, 2018

Issue に WIP 付ける慣習ってありましたっけ。
一般的には PR のほうに WIP 付けるものだと思っています。

@berryzplus
Copy link
Contributor Author

Issue に WIP 付ける慣習ってありましたっけ。

了解です。今回は両方につけます。
よろしくお願いします。

@berryzplus berryzplus changed the title [WIP 着手済み] windows7 の AeroSnapに対応させる [着手済み] windows7 の AeroSnapに対応させる Jun 18, 2018
@berryzplus berryzplus changed the title [着手済み] windows7 の AeroSnapに対応させる [保留中] windows7 の AeroSnapに対応させる Jul 28, 2018
@beru
Copy link
Contributor

beru commented Sep 9, 2018

こちらもう PR 作成されていますか?
WIP でも良いので忘れないうちに上げておいていただけると機能が入る可能性が高まると思うのでありがたいです。

@berryzplus
Copy link
Contributor Author

作成してPRして、誰もレビュしてくれないもんでwin10でしばらく動作確認してたら、想定しない現象が起きました。

で、よくよく調べてみたらその現象はwindowsバージョンの違いによって起きているらしいことがわかりました。さらに、どうも起動遅延の原因の1つが、エアロスナップ対応にも影響を与えてそうなことが分かりました。

対応を完遂するには、大規模変更に寛容な空気作りが必須と考えています。あとはコードじゃなくアプリ仕様を把握できるような何かが必要です。

とりあえず、不具合系の対応で最優先と考えてるのは桁数取得マクロの話で、不具合対応を口実に心臓部をx64 readyにしてしまいたいと考えております。

@beru
Copy link
Contributor

beru commented Sep 9, 2018

あ、 #120 がそうでしたか。すでにPR作られてたんですね。失礼しました。

@beru beru added 🐛bug🦋 ■バグ修正(Something isn't working) enhancement ■機能追加 labels Sep 18, 2018
@berryzplus
Copy link
Contributor Author

モチベーションが尽きたので閉じてしまいます。 #1394

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working) enhancement ■機能追加
Projects
None yet
Development

No branches or pull requests

3 participants