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

返信に表示される "通知をオフ" を削除する機能の追加 #41

Merged
merged 13 commits into from
Jan 27, 2024

Conversation

AioiLight
Copy link
Contributor

@AioiLight AioiLight commented Jan 25, 2024

通知から返信しようとしたら間違えて相手の通知をオフにしてしまう忌まわしき「通知をオフ」アクションを削除する機能を提供します。

Before
image

After
image

残っている実装

  • オプション化
  • 複数言語対応

@AioiLight
Copy link
Contributor Author

デフォルトでオンとオフ、どっちが良いでしょうか……。

@Chipppppppppp
Copy link
Owner

LINE 内に設定を埋め込むのに対応させておきました

AioiLight and others added 2 commits January 27, 2024 21:27
# Conflicts:
#	app/src/main/java/io/github/chipppppppppp/lime/Main.java
#	app/src/main/java/io/github/chipppppppppp/lime/SettingsActivity.java
#	app/src/main/res/layout/settings_activity.xml
@Chipppppppppp Chipppppppppp merged commit c1cfd76 into Chipppppppppp:master Jan 27, 2024
@AioiLight
Copy link
Contributor Author

あっちょうどのタイミングでマージ作業してしまった
ありがとうございます 🙇‍♂️

@Chipppppppppp
Copy link
Owner

これ、アイコンラベルを消すときにアイコンを大きくするコードが、設定を読み込む前に実行されちゃうみたいで設定が反映されないですね...
リソースを変えるのではなく別の方法でできれば解決するんですが...

@AioiLight
Copy link
Contributor Author

public void handleLoadPackage(XC_LoadPackage.LoadPackageParam lparam) throws Throwable {
        if (!lparam.packageName.equals(PACKAGE)) return;

        Class hookTarget;

        XSharedPreferences xPref = new XSharedPreferences(PACKAGE, "io.github.chipppppppppp.lime-options");
        for (int i = 0; i < limeOptions.size; ++i) {
            LimeOptions.LimeOption option = limeOptions.getByIndex(i);
            option.checked = xPref.getBoolean(option.name, option.checked);
        }

これはどうでしょうか?

@Chipppppppppp
Copy link
Owner

XSharedPreferencesは読み取り専用だったと思うんですよね...だから LINE アプリに埋め込めない気がします

@AioiLight
Copy link
Contributor Author

LSPatch環境では試してないですけど、
Prefsの読み込みは読み取り専用のXSharedPreferencesで、書き込みはContextから呼び出せるSharedPreferencesというような実装でもちゃんと動いてそうな感じがします。

Screenshot_20240127-230505_LINE_1.png

Screenshot_20240127-230524_LINE_1.png

@AioiLight
Copy link
Contributor Author

Screenshot_20240127_235330_LINE
LSPatchでもいけてるっぽいです

Familyye pushed a commit to Familyye/LIME that referenced this pull request Jun 17, 2024
返信に表示される "通知をオフ" を削除する機能の追加
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants