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

Add: 最近開いたプロジェクト一覧を追加 #1264

Merged
merged 12 commits into from
Apr 8, 2023

Conversation

sevenc-nanashi
Copy link
Member

内容

タイトル通り。

関連 Issue

スクリーンショット・動画など

image

その他

(なし)

@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner March 26, 2023 14:36
@sevenc-nanashi sevenc-nanashi requested review from Hiroshiba and removed request for a team March 26, 2023 14:36
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

最近保存したプロジェクトファイルも追加された方がいいかもとか思いました!

もしそうした場合、どう表示するか悩みますねぇ。
実際issue内で示されてるスクショでは「最近使用された」になってました。
開いたとか使用したとか書かず、「最近のプロジェクト」とか…?
そのまま「最近開いた」でも良いかも。

google docsとかの最近のソフトがどう書いてるかに合わせるのが良さそう。

src/components/MenuBar.vue Show resolved Hide resolved
@Hiroshiba
Copy link
Member

Hiroshiba commented Mar 28, 2023

@sevenc-nanashi こちらのコメントのアイデアどうでしょう 👀
#1264 (review)

ちなみにgoogle docsは「開く」を選んだあとに「最近使用したアイテム」がありました。
image

これ良い気がしました。
「最近開いたファイル」サブメニューを用意するのではなく、「プロジェクトを開く」のサブメニューに「最近使用したファイル」を追加すると、メニューが増えなくてユーザーフレンドリーかなと思いました。

@sevenc-nanashi
Copy link
Member Author

image

こんな感じでどうでしょう?

@Hiroshiba
Copy link
Member

良いですね!!!

「最近使用したアイテム」というヘッダーもあると、何が表示されているかわかりやすいかもとちょっと思いました。
(察しが良い慣れているユーザーは何も書かれてなくても気づくかも)

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

たぶん大多数のユーザーが開きたいのは「前回編集していたプロジェクトファイル」だと思います。
となると、保存したプロジェクトファイルもAPPEND_RECENTLY_OPENED_PROJECTされると良いのかなとちょっと思いました。

ただ保存したプロジェクトファイルが「開いた(OPENED)」なのか微妙なので、「使用した(USED)」とかのがまあ良いのかなと思いました。
が、まあ正直「開いた(OPENED)」でも全然良さそう。お好きな方で・・・!!

@sevenc-nanashi
Copy link
Member Author

(察しが良い慣れているユーザーは何も書かれてなくても気づくかも)

最初に起動したときは「最近開いたプロジェクトはありません」という表示が出るので、多分気がつくと思います。

@sevenc-nanashi
Copy link
Member Author

となると、保存したプロジェクトファイルもAPPEND_RECENTLY_OPENED_PROJECTされると良いのかなとちょっと思いました。

「開く」のサブメニューなので、開いたプロジェクトの方が想定と近そう & 保存する前に必ず開いているのであんまり変わらなそう、と思いました

@Hiroshiba
Copy link
Member

プロジェクトを保存→アプリ閉じる→アプリ起動→プロジェクトを開く
と遷移したとき、前回保存したものが表示されていてほしいな~と。

@Hiroshiba
Copy link
Member

最初に起動したときは「最近開いたプロジェクトはありません」という表示が出るので、多分気がつくと思います。

起動したユーザーが何も保存していないのにプロジェクトファイルを開こうとした場合には表示されると思いますが、まあ大多数の人はそうならないので、察しが悪い気づけないかなーみたいな・・・!

@sevenc-nanashi
Copy link
Member Author

image

こんな感じにしてみました。

@sevenc-nanashi
Copy link
Member Author

と遷移したとき、前回保存したものが表示されていてほしいな~と。

保存するように出来ました。

@wappon28dev
Copy link
Contributor

Re: #1264 (comment)

横入り失礼します. 実際の挙動と違ったら申し訳ないのですが, ”プロジェクト読み込み” をクリックすると, 最近使ったプロジェクトから選択せずに ファイルを開く のウィンドウが開くので, > の左側に仕切りを追加するのはどうでしょうか?

1

根拠としては, 話者のキャラクター選択 ( 四国めたん | > ) で,

  • 四国めたん をクリックするとデフォルトの ノーマル が選択される
  • > をクリックするとスタイルが選べる
    ...のように, > から選択しなくてもデフォルトの動作ができるのであれば, 仕切りを追加すると良いかもと思いました.

2

(補足: 右下の テキストボックス | v だと...
   ・テキストボックス をクリックするとデフォルトのスタイルでテキストボックスが作れる
   ・v をクリックすると, スタイルを選ぶメニューが表示される...
   また,
   ・図形 v を選択すると, スタイルを選ぶメニューが表示される (選択必須))

ご一考お願いします

@sevenc-nanashi
Copy link
Member Author

なるほど、かなりよさそうです。MenuItemをいじって実装してみます

@sevenc-nanashi
Copy link
Member Author

image

いじって実装してみました。

@Hiroshiba
Copy link
Member

Hiroshiba commented Apr 4, 2023

あ~~~~たしかに|で区切ったほうがわかりやすいですね!
ただキャラ選択のとことUIに統一性がなくなってしまったのが気になりました 😇
(キャラ選択はvにカーソル当てるとサブメニューが開く、ここはvのとこじゃなくてもサブメニューが開き、かつマウスホバーしたときに着色される範囲が異なる)
(あと細かいけどvの左右の余白は等幅にするとデザイン的には良さそう。)

UIの操作感が違うのは意外とユーザーに不信感を与えちゃうので、統一したいかもです。
手は3つほどありそう。

  1. vにカーソル合わせたらサブメニューが出る仕様に
    • 統一感は出るけど「最近使ったプロジェクト」機能に存在に気づけない人もいそう
  2. vにカーソルを合わせなくても出る仕様に
    • 今の形からマウスホバー時の着色範囲を変えるだけ
    • 統一感はちょっと消えるけど、まあ許容範囲かもしれない
  3. 「最近使ったプロジェクト」メニューを「プロジェクト読み込み」の下に表示
    • UIの統一感は一番保たれるけど、メニューが増える

個人的には、最近使ったプロジェクトを開く機能は普通に有用なので、3もありかなと思いました!

@sevenc-nanashi
Copy link
Member Author

3にしました。(一番簡単そうだった)

src/store/setting.ts Outdated Show resolved Hide resolved
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

いろいろ書きましたがほぼLGTMです!!

@sevenc-nanashi
Copy link
Member Author

レビュー反映しましたー。

Copy link
Member

@Hiroshiba Hiroshiba 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 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.

「最近開いたプロジェクト」メニュー
3 participants