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

update ハッシュタグタイムラインのタグ登録/解除ボタンを復活 #224

Merged

Conversation

lnanase
Copy link
Collaborator

@lnanase lnanase commented Jun 11, 2019

#96 の改修のお気に入りタグ登録/解除ボタンを、
現ハッシュタグタイムラインに追加

@lnanase lnanase added the ui About the User Interface label Jun 11, 2019
@lnanase lnanase requested a review from takayamaki June 11, 2019 13:03
@lnanase
Copy link
Collaborator Author

lnanase commented Jun 11, 2019

mastodon#9952 で未収載アイコンが変更になっていましたので、
未収載トゥートやお気に入りタグ設定のアイコンも変更致しました。

クリップボード02

Copy link
Member

@takayamaki takayamaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

お気に入り登録ボタン周りだけ別コンポーネントに切り出しているのは良いと思います。

未収載アイコンが変更されている件についてはこのPRの主題とは全く関係ない部分の変更ですので、別PRに切り出してください。
PRの数が多くなってしまうのが億劫に感じるかもしれませんが、見る側としては粒度の小さいPRがたくさんあるほうがそれぞれについてより短時間で見れますのでむしろ楽です。

@lnanase lnanase force-pushed the update_hashtag_timeline_to_tag_button branch from 604e936 to 45fc6d4 Compare June 16, 2019 12:18
@lnanase
Copy link
Collaborator Author

lnanase commented Jun 16, 2019

未収載アイコンが変更されている件についてはこのPRの主題とは全く関係ない部分の変更ですので、別PRに切り出してください。

仰る通りです。失礼致しました。
別PRに切り出し、本PRからコミットを削除してforce-pushしましたので、
ご確認をお願い致します。

@takayamaki takayamaki merged commit 275b058 into imas:imastodon Jun 16, 2019
@lnanase lnanase deleted the update_hashtag_timeline_to_tag_button branch April 7, 2020 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui About the User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants