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

日報・DOCs・Q&Aページに「現在のページURLをコピーするボタン」を追加 #8219

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sugiwe
Copy link
Contributor

@sugiwe sugiwe commented Nov 26, 2024

Issue

概要

日報・DOCs・Q&Aページに、「現在のページURLをコピーするボタン」を追加しました。

変更確認方法

  1. {feature/add_copy_current_url_button_to_some_pages}をローカルに取り込む
    • git fetch origin pull/8219/head:feature/add_copy_current_url_button_to_some_pages
    • git switch feature/add_copy_current_url_button_to_some_pages
  2. foreman start -f Procfile.dev でローカルサーバーを立ち上げる
  3. 任意のアカウントでログイン
  4. 任意の日報ページ(こちらなど)を開き、タイトルすぐ下にある「URLコピー」ボタンをクリックし、任意の場所でペーストして見て正しくコピーされているかどうかを確認

Screenshot

変更前

1

変更後

2

コピーボタンクリック時
3

@sugiwe
Copy link
Contributor Author

sugiwe commented Nov 26, 2024

@machida
お疲れ様です!
こちらのPRですが、隣のBookmarkボタンのスタイルを踏襲しつつ仮のCSSを入れている状態です。
メンバーレビューの前にCSS関連を整えたいのですが、ご確認とCSSの調整をお願いできますでしょうか🙏

(CSSまったくあてずにデザイン依頼としてご相談することも考えたのですが、issueの方でアドバイスいただいたコメントの日時クリックによるURLコピーの機能を活用する中でis-activeの際のCopied!表示など活用をさせていただいたので、一旦僕の方で仮のCSSまで入れてみた次第です)

ご確認よろしくお願いいたします🙇‍♂️

@sugiwe
Copy link
Contributor Author

sugiwe commented Nov 26, 2024

@machida
連投になりますが、別の箇所で少しご相談です🙏

今回「URLコピー」のボタンを追加しようとしていますが、日報ページの場合はそのすぐ隣に「コピー」ボタンがあり、「URLコピー」と並んだ時に混乱を招く表示になってしまいそうだなと感じました。

日報に現在ある「コピー」は、その日報の内容をコピーした新しい日報下書きを作るので意味的には「コピー」というより「複製」ということになると思っています、なので日報に現在ある「コピー」の方の表記を変えるのが良いのではと考えています。

今回のissueとは別の話になると思うので別issueを立てるのが良いかなと思っていますがいかがでしょうか?

@machida
Copy link
Member

machida commented Nov 26, 2024

@sugiwe 提案ありがとうございます!確かに紛らわしくなってしまいますね。はい!その提案を採用させていただきたいと思います🙏Issueの作成をお願いしてもいいでしょうか?

@sugiwe
Copy link
Contributor Author

sugiwe commented Nov 26, 2024

上記の件、issue登録いたしました〜!
#8220

@sugiwe
Copy link
Contributor Author

sugiwe commented Nov 26, 2024

@machida
#8219 (comment) に書いたCSS関連の調整につきましては引き続きご確認のほどよろしくお願いいたします🙏

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