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

#1846 ノートのエラー時にツールチップ表示 #1893

Merged
merged 4 commits into from
Mar 16, 2024

Conversation

romot-co
Copy link
Contributor

@romot-co romot-co commented Mar 3, 2024

内容

  • ノートの重なりエラー時にホバーでツールチップ表示
  • ノートのフレーズエラー時にホバーでツールチップ表示

いったん動作しますが、可能であればホバー表示条件のリファクタをおこなればと思います
(来週火曜まで時間とれるか不明なため、プルリクとして作成)

関連 Issue

ref #1864
close #1864

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

スクリーンショット 2024-03-03 14 58 34 スクリーンショット 2024-03-03 14 55 19

その他

※ 色は仮です・別途別Issue

@romot-co romot-co requested a review from a team as a code owner March 3, 2024 05:49
@romot-co romot-co requested review from y-chan and removed request for a team March 3, 2024 05:49
@romot-co romot-co force-pushed the feature/1864_display_error_reason_in_notes branch from 9a472a9 to 9ba9055 Compare March 3, 2024 05:53
@Hiroshiba Hiroshiba requested review from Hiroshiba and removed request for y-chan March 5, 2024 16:43
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!!!

とても良いなと思いました!!
ちょっとデザイン面、というよりライトモードでの配色で選択色が分かりづらくなっているかもなので、一旦次のリリース(今日明日ぐらいの)からは申し訳ないのですが省かせていただこうかなと思います!!

src/components/Sing/SequencerNote.vue Outdated Show resolved Hide resolved
src/components/Sing/SequencerNote.vue Outdated Show resolved Hide resolved
src/components/Sing/SequencerNote.vue Outdated Show resolved Hide resolved
@romot-co
Copy link
Contributor Author

romot-co commented Mar 6, 2024

こちら色調整別途行います…!

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.

マージします!!

色調整お願いできるととても助かります・・・!!
リリース時のタイミングでまだ調整されていなさそうだったら、ちょっと一旦こっちで調整させていただきます!
ちなみに来週土曜にハミング対応キャラクター追加リリースがある予定です。その前に月~水あたりにまた1回お試しリリースする・・・かもみたいな感じです!

@Hiroshiba Hiroshiba merged commit f5053ba into main Mar 16, 2024
18 checks passed
@Hiroshiba Hiroshiba deleted the feature/1864_display_error_reason_in_notes branch March 16, 2024 06:55
@Hiroshiba
Copy link
Member

あっ、今更気付いたんですが #1864 にリンクされてるけどリンク先もしかしたらこれじゃないかも・・・?

こっちかも。だと思うので #1846 の方をクローズさせていただきます!

@Hiroshiba Hiroshiba changed the title #1864 ノートのエラー時にツールチップ表示 #1846 ノートのエラー時にツールチップ表示 Mar 16, 2024
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