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

fix: CopyDropdown protrudes from the screen issue #8734

Conversation

WNomunomu
Copy link
Contributor

@WNomunomu WNomunomu commented Apr 17, 2024

タスク

https://redmine.weseek.co.jp/issues/144765

概要

  • ページ名が長い場合にCopyDropdownがサイドバーの後ろに隠れたり、画面外にはみ出してしまう問題を解決しました。

変更点

  • DropdownItemContentsに新たにpropsを設定しました。
  • .dropdown-menu にbreakpointを設定し、新たにスタイルを設定しました。

デザインについての詳細仕様

以下の仕様は全てfumiya-sと合意済みです。
https://wsgrowi.slack.com/archives/C05HK5SA8CC/p1713347696195269

  • 「ページ名」、「ページURL」は後ろをtruncateする
  • 「ページ名とパーマリンク」は前をtruncateする
  • それ以外はtruncateしない
  • Dropdownのmax-widthはmd以上で600px、md未満で375px

スクリーンショット

image (2)
image (1)

セルフチェック

  • コンフリクト解消したか
  • 余計なコードは残っていないか
  • 適切にメモ化したか
  • 責務の問題はクリアしているか
  • CIは通っているか
  • PRの内容は適切にかけているか

Copy link

reg-suit bot commented Apr 19, 2024

reg-suit detected visual differences.

Check this report, and review them.

🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴


🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵

What do the circles mean? The number of circles represent the number of changed images.
🔴 : Changed items, ⚪ : New items, ⚫ : Deleted items, and 🔵 Passed items

How can I change the check status? If reviewers approve this PR, the reg context status will be green automatically.

Copy link
Member

@miya miya left a comment

Choose a reason for hiding this comment

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

  • 「ページ名とパーマリンク」だけ先頭が truncate されるので良いのか
  • 「マークダウン形式のリンク」は truncate されないので良いのか

@yuki-takei yuki-takei merged commit 814f7ad into master Apr 22, 2024
28 of 32 checks passed
@yuki-takei yuki-takei deleted the fix/144587-144765-CopyDropdown-protrudes-from-the-screen-issue branch April 22, 2024 08:50
@github-actions github-actions bot mentioned this pull request Apr 22, 2024
@github-actions github-actions bot mentioned this pull request Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants