-
Notifications
You must be signed in to change notification settings - Fork 8
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
Aインスタンスでトゥートを追加ボタンを押してからBインスタンスに変更すると、プレビューとURLが整合的でなくなる #116
Comments
これってそういう仕様なんじゃないんですか?同じTooTが2つ出るのが正しいという。 #55 を私はそう解釈しているんですが。
|
目指す方向性は #55で良いと思いますが、右側のプレビューの内容が書き換わっていないので、プレビューとURLで不整合を起こします。 あるべき姿は、URLの内容に従って過去プレビューを書き直すか、途中でのインスタンスの変更を禁止するか、ではないかなと思います。 |
同じサーバーの URL しか受け付けないのは仕様ですが、別サーバーの URL を混在させると動いちゃうのはやはり気になります。 UI の改善 #102 に貼り付けたモックの画像( #102 (comment) )で編集中のドメインという ReadOnly のフィールドを入れたのもそこらんの意図がありました。 |
に 👍 。まぁ、一発目のトゥートのドメインから変わったらエラーでいいんじゃないですかね? |
ちなみに、なぜこれを気にし始めたかというと、@useridを含むURLを認めるからでした。 |
インスタンス変更というか、UI的には一度入れたら書き換え不能になって、変更にはボタンが必要くらいでよさそう |
<再現手順>
1.インスタンス名= https://qiitadon.com でID=2でプレビュー表示してトゥートを追加
2.インスタンス名= https://mstdn.jp に変更してID=2でプレビュー表示してトゥートを追加
こうすると、別のトゥートが画面上のプレビューに登録されますが、プレビューURLで指定されるインスタンスはmstdn.jpなので、表示が不整合を起こします。
これと本質的に同じ事象としては、
・Aインスタンスでトゥートを登録してから、Bインスタンスのまとめをインポート
・プレビュー表示してからインスタンスを変更して何もせずにトゥートを追加
などがあります。
※あまりそういう事をする人はいない気もしますが、一応
The text was updated successfully, but these errors were encountered: