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

FolderSelectDialogが空白になる #478

Open
poppingmoon opened this issue Dec 4, 2023 · 3 comments
Open

FolderSelectDialogが空白になる #478

poppingmoon opened this issue Dec 4, 2023 · 3 comments
Milestone

Comments

@poppingmoon
Copy link
Contributor

PushableListViewに広告が表示されるようになったが広告は Account.of(context) を呼んでいるためAccountScopeで囲まれていない場合 Exception: has not ancestor を発生させるようになった

FolderSelectDialogではPushableListViewがAccountScopeに囲まれていないため設定のインポートとエクスポートができなくなっている

@poppingmoon
Copy link
Contributor Author

poppingmoon commented Dec 4, 2023

他にもAccountScopeに囲まれていないPushableListViewが存在するかもしれませんが確認していません

同様の問題は以前にもありましたし (#129, #130)、今後も発生することが予想されます
この原因はInheritedWidgetを使ったデータの受け渡しではどのウィジェットが囲まれていてどれがそうでないかわかりにくいことにあります
やみくもにテストケースを増やすのもいいですが、AccountScopeを用いたデータの受け渡しを MisskeyNote の中など限られた場所に限定して基本的に引数による受け渡しを使うようにするべきだと思います

@shiosyakeyakini-info
Copy link
Owner

ネストが深くなるところまで引数のリレーをしたくない、アカウント情報はいろんなところでキーになっていて必要、同じ画面でも複数のアカウント情報が必要になることがあってその文脈がProviderで解決できないの3つからInheritedWidgetにしたと思う

@poppingmoon
Copy link
Contributor Author

それはわかるけれどInheritedWidgetの恩恵が少ないところでも使っているのをやめるべきだと思う

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

No branches or pull requests

2 participants