user.js版を生成する #95
Merged
user.js版を生成する #95
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
|
ホワイトスペースの差分を消すと見やすいかもです https://github.com/inajob/toICN/pull/95/files?diff=split&w=1 |
inajob
reviewed
Sep 11, 2021
| if(specifiedKey.keyNo != -1){isAutoKeyDetection = false;} | ||
| function waitElement(cb) { | ||
| let selector; | ||
| if (webSiteName === "ufret") { |
There was a problem hiding this comment.
グローバル変数への依存を減らしたいのでwebSiteNameは waitElementの引数にしてほしいです。
ちょっとどこかから探してくるか、自分で適当に設えてリンク貼る形で準備しようと思います。別のPRなりで追加させてください 🙏 |
Co-authored-by: inajob <[email protected]>
https://gakufu.gakki.me/m/index.php?p=DT10646&k=m2#rp など Co-authored-by: inajob <[email protected]>
inajob
reviewed
Sep 11, 2021
toICN.bookmarklet
Outdated
| -n javascript:(function(){ |
There was a problem hiding this comment.
おや、-n が効いてないですね・・ echoのバージョンが違うようですね
この辺ですかね？ 試してもらえると助かります。
There was a problem hiding this comment.
情報ありがとうございます！bash gen.shで回避できました。
inajob
approved these changes
Sep 11, 2021
|
PRありがとうございました。ブックマークレット以外の利用方法提供の第一歩となりました！ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
#85 を眺めていて、ひとまずuser.jsを作っておくと良いのではということで生成出来るようにしたので、PRにしました。
インストール方法というかメジャーなブラウザだとこれらを使うことで有効にできます。（READMEなどに足した方が良さそうなら足します）
Chromeは開発者モードになっていると
chrome://extensions/からローカルに一度落としたuser.jsをインストール出来ます。表題と関係ないのですが、
confirmが出る時にキャンセルを押すとnullが返ってきてエラーになるのを回避するようにしています。