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

Improve post form #5326

Merged
merged 2 commits into from Aug 25, 2019
Merged

Improve post form #5326

merged 2 commits into from Aug 25, 2019

Conversation

u1-liquid
Copy link
Contributor

Summary

  • リプライ時、返信元のlocalOnly属性を引き継ぐように
  • visiblity-chooserlocalOnly属性が伝わらなかったのを修正
  • 引用付きローカルのみなどの案内文にアイコン追加

など

デスクトップ版

image
image
image

モバイル版

image
image
image

@mei23
Copy link
Contributor

mei23 commented Aug 21, 2019

メモ

リプライ時、返信元のlocalOnly属性を引き継ぐように

今まではサーバーサイドにのみlocalOnlyにリプライされたらlocalOnlyにする処理があった

visiblity-chooserにlocalOnly属性が伝わらなかったのを修正

visiblity-chooserを開いた時点でどれが選択状態になっているかに影響

@acid-chicken
Copy link
Member

デザイン要検討?(備考: mei23:mei-m544346design:twista はそれぞれ別のデザインで似たものを実装しているため、そこらへんも引っ張ってくる余地はあり。)

@mei23
Copy link
Contributor

mei23 commented Aug 21, 2019

公開範囲アイコンにローカルのみの情報が含まれるなら「この投稿はローカルにのみ~」の注釈はもういらないかも

ローカルのみの公開範囲アイコンデザインは、
2つ並べがここに登場するのはちょっと不自然とか、chooserのアイコン部分にも反映させたいというのがあります。

@EbiseLutica
Copy link
Member

EbiseLutica commented Aug 22, 2019

localOnlyな公開範囲の場合、普通のアイコンの右下にバッジみたいに小さく♥が表示されるような形はどうでしょう

font-awesome の layer 機能で実現できそう...?
https://fontawesome.com/how-to-use/on-the-web/styling/layering
https://github.com/FortAwesome/vue-fontawesome#advanced

@u1-liquid
Copy link
Contributor Author

デザインに関してはわたしは詳しくないので(特に案もないし)2つ並べが不自然だと思われるなら元に戻すか、めいすきーのデザインを引っ張ってくる案も良さそうですね。

(そもそも♥=ローカルってのはわかりやすいシンボルだとは言えないので)「この投稿はローカルにのみ~」の注釈は維持した方がいいと思います。

@mei23
Copy link
Contributor

mei23 commented Aug 23, 2019

font-awesome の layer 機能

vue-fontawesome が対応していないぽい
めいすきーなどはCSSで重ね合わせ

@u1-liquid
Copy link
Contributor Author

visibility-chooserのボタンのアイコン2つ並べをrevertしました。

@syuilo syuilo merged commit 1c4e1af into misskey-dev:develop Aug 25, 2019
@syuilo
Copy link
Member

syuilo commented Aug 25, 2019

ありがとうございます🙏🙏🙏

@u1-liquid u1-liquid deleted the improve-post-form branch August 25, 2019 14:01
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.

None yet

5 participants