Skip to content
This repository has been archived by the owner on Jul 17, 2024. It is now read-only.

多言語選択メニューのユーザビリティ向上 #225

Merged
merged 6 commits into from
May 21, 2020

Conversation

Morichan
Copy link

👏 解決する issue / Resolved Issues

close #208

📝 関連する issue / Related Issues

#94

⛏ 変更内容 / Details of Changes

  • 多言語選択メニューの上部移動
  • やさしい日本語ページのリンク追加
    • 「こどものみなさん」リンクを追加します。

📸 スクリーンショット / Screenshots

image

多言語選択メニューを上部に移動する。

下記プルリクを参考にした。

Tokyo-Metro-Gov#1866
やさしい日本語ページのリンクを追加する。
アイコンは、下記を参考にした。

https://vuetifyjs.com/ja/components/icons/

https://materialdesignicons.com/icon/human-child
@korosuke613
Copy link
Member

ありがとうございます。
「子供の皆さん」が他のメニューと同じ位置にあるのがちょっと気になりますね...

@Morichan
Copy link
Author

「子供の皆さん」が他のメニューと同じ位置にあるのがちょっと気になりますね...

私も最初は「多言語選択メニュー」上部に配置していました。
パソコンに詳しくない知合いに外観を見てもらった際(トップ画面を見せ、わかりやすいページに飛ぶためにはどうすればいいと思うか、という質問をした際)、「県内の最新感染動向」項目より下のところしか見ておらず、やさしい日本語ページに辿りつかなかったという経緯があります。
「県内の最新感染動向」がリンクとして認識されていなかったのが原因でした。

そこで、メニュー項目の1つとして配置を変えてみました。
やっぱり見づらいですかね?

@korosuke613
Copy link
Member

個人的にはこのプルリクエストにおける「多言語選択メニュー」の真下の配置がしっくりきますね...
最新感染項目と同じ区域に言語に関するボタンがあるとノイズになる気がします。

@Morichan
Copy link
Author

@korosuke613

個人的にはこのプルリクエストにおける「多言語選択メニュー」の真下の配置がしっくりきますね

とりあえず、「多言語選択メニュー」真下の状態だと、下記のようになります。
こちらの方が良いですか?

image

@korosuke613
Copy link
Member

あ、すみません。
僕のイメージでは、多言語対応選択メニューと子どものみなさんの間に線はありませんでした。

@Morichan
Copy link
Author

@korosuke613

僕のイメージでは、多言語対応選択メニューと子どものみなさんの間に線はありませんでした。

線を消しました。
また、間隔も狭めました。
いかがでしょう?

image

@korosuke613
Copy link
Member

僕は良いと思います!

Copy link
Member

@korosuke613 korosuke613 left a comment

Choose a reason for hiding this comment

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

LGTM

下記レビュー指摘より、次の修正をする。

- 「こどものみなさん」リンクを「多言語選択メニュー」下に移動
- SideNavigation-Language下部の間隔縮減
- MenuList上部のボーダーライン除去

covid19-miyazaki#225 (comment)

covid19-miyazaki#225 (comment)
@Morichan
Copy link
Author

@korosuke613

「多言語選択メニュー」と「こどものみなさん」リンクの間を、若干開けました。(0px -> 5px)

image

また、コードを書換えました。
レビューお願いします。

Copy link
Member

@TakaShinoda TakaShinoda left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@korosuke613 korosuke613 left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

多言語対応選択メニューの位置を変更する
3 participants