-
Notifications
You must be signed in to change notification settings - Fork 28
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
[ G2 ] WP6.6かつG2使用時のメニュー名にunderlineが入るようになってしまったのを修正 #1172
Conversation
もしかしたら 現象を確認しておらず憶測ですみませんが… |
@sysbird |
# Conflicts: # _g2/design-skin/origin2/css/style.css
内容的に text-decoration:none 指定はあっても良いのでマージします(・w・ |
@mtdkei G2 変更の場合は同様に Lightning Pro の方も変更よろしくお願いいたします ( ̄人 ̄) |
@kurudrive |
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
WP6.6 かつ Lightning G2 おいて、Header Navigation と Mobile Navigation のメニュー名にunderlineが入るようになってしまったため。
どういう変更をしたか?
G2使用時ののメニュー名にunderlineが入らないようにするため、元々a:hoverなどに書いてあった
text-decoration: none;
をaに移動しました。スクリーンショットまたは動画
変更前 Before
Header Navigation
Mobile Navigation
変更後 After
Header Navigation
Mobile Navigation
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
→CSSのみのためスキップ
変更内容について何を確認したか、どういう方法で確認をしたかなど
Lightning G2の状態で以下を確認しました。
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
実装者と同じ確認を行ってください。
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。