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/change UI setting qall #4192

Merged
merged 24 commits into from
Jan 29, 2024
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
084ea2b
設定画面(通話)の項目の順番変更、文章の一部修正、ラジオボタンのレイアウト変更
damin3A3 Dec 9, 2023
00a4c44
設定画面(通話)のレイアウト、見た目微調整
damin3A3 Dec 9, 2023
cd8048e
設定画面(通話)のレイアウト変更
damin3A3 Dec 9, 2023
267ee07
文字関連の変更を一度消した
damin3A3 Dec 9, 2023
77a781f
変更を戻した
damin3A3 Dec 9, 2023
0a6646c
コードの細かい修正
damin3A3 Dec 9, 2023
fce2b7c
Merge branch 'master' into fix/change_UI_Setting_Qall
damin3A3 Dec 9, 2023
424b9c6
Merge branch 'master' into fix/change_UI_Setting_Qall
damin3A3 Dec 28, 2023
b42c4f0
指摘をいただいたところの修正
damin3A3 Dec 28, 2023
6227ea4
prettierでのフォーマット修正
damin3A3 Dec 28, 2023
fd7e59d
タグの修正
damin3A3 Dec 29, 2023
2d0d30a
行間調整
damin3A3 Dec 30, 2023
4f77a77
コードのフォーマットの修正
damin3A3 Dec 30, 2023
56f7f20
ラジオボタンの変更を元に戻した
damin3A3 Dec 30, 2023
394a67b
スライダの背景色を消去した
damin3A3 Jan 12, 2024
1846ee5
Merge branch 'master' into fix/change_UI_Setting_Qall
damin3A3 Jan 12, 2024
3f0aa29
ラベルの色の修正
damin3A3 Jan 22, 2024
70cf5ff
vue-slider-componentを使用した(色未修正)
damin3A3 Jan 23, 2024
b473e77
フォーマット修正
damin3A3 Jan 23, 2024
2fa1e4a
データ型の統一
damin3A3 Jan 23, 2024
3c61a3a
フォーマット再修正
damin3A3 Jan 23, 2024
5c9b05c
データ型の修正を試みた
damin3A3 Jan 23, 2024
4689e71
スライダの色を調整した
damin3A3 Jan 27, 2024
1942bf9
スライダーのクリック範囲を広くした
damin3A3 Jan 29, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
指摘をいただいたところの修正
  • Loading branch information
damin3A3 committed Dec 28, 2023
commit b42c4f0053dcb1d080da7207d9c2e6abda567156
87 changes: 49 additions & 38 deletions src/views/Settings/QallTab.vue
Original file line number Diff line number Diff line change
@@ -2,57 +2,59 @@
<section>
<div :class="$style.element">
<div :class="$style.enable">
<div :class="$style.header_and_content">
<h3 :class="$style.header">RTC機能</h3>
<section :class="$style.section">
<h3 :class="$style.heading">RTC機能</h3>
<p>
通話などのRTC(リアルタイムコミュニケーション)機能を有効化します。
マイクなどへのアクセス許可が必要です。
</p>
</div>
</section>
<a-toggle v-model="state.isEnabled" :class="$style.toggle" />
</div>
</div>
<template v-if="state.isEnabled">
<div :class="$style.element">
<div :class="$style.enable">
<div :class="$style.header_and_content">
<h3 :class="$style.header">メッセージの読み上げ</h3>
<section :class="$style.section">
<h3 :class="$style.heading">メッセージの読み上げ</h3>
<p>Qallしているチャンネルに投稿されたメッセージを読み上げます。</p>
</div>
</section>
<a-toggle v-model="state.isTtsEnabled" :class="$style.toggle" />
</div>
</div>
<div v-if="state.isTtsEnabled" :class="$style.element">
<div>
<form-selector
v-if="voiceOptions.length > 0"
v-model="state.voiceName"
label="読み上げボイスの種類"
:options="voiceOptions"
/>
<p v-else>読み上げ音声の声の種類が取得できませんでした。</p>
<form-input
v-model.number="state.voicePitch"
label="ピッチ"
type="number"
step="0.1"
/>
<form-input
v-model.number="state.voiceRate"
label="速度"
type="number"
step="0.1"
/>
<form-input
v-model.number="state.voiceVolume"
label="音量"
type="number"
step="0.1"
/>
</div>
<form-selector
v-if="voiceOptions.length > 0"
v-model="state.voiceName"
label="読み上げボイスの種類"
:options="voiceOptions"
:class="$style.option"
Copy link
Contributor

Choose a reason for hiding this comment

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

Figmaで要素を選択した状態でalt押すと、他の要素との距離が見れるので、それを参考にしてちょっと値変えてほしいです:eyes:
1つ目の画像のは多分form-input自体を変えることになるのですが、変えてokだと思います
Figma_qZgZWsoOtR
Figma_WJPY1tkpEU

/>
<p v-else :class="$style.content">読み上げ音声の声の種類が取得できませんでした。</p>
<form-input
v-model.number="state.voicePitch"
label="ピッチ"
type="number"
step="0.1"
:class="$style.option"
/>
<form-input
v-model.number="state.voiceRate"
label="速度"
type="number"
step="0.1"
:class="$style.option"
/>
<form-input
v-model.number="state.voiceVolume"
label="音量"
type="number"
step="0.1"
:class="$style.option"
/>
</div>
<div :class="$style.element">
<h3 :class="$style.header">入力デバイス</h3>
<h3 :class="$style.heading">入力デバイス</h3>
<div>
<form-selector
v-if="!fetchFailed && audioInputDevices.length > 0"
@@ -63,7 +65,7 @@
</div>
</div>
<div :class="$style.element">
<h3 :class="$style.header">マスターボリューム</h3>
<h3 :class="$style.heading">マスターボリューム</h3>
<form-range-with-value
Copy link
Contributor

Choose a reason for hiding this comment

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

スライダー、僕のwindows+chromeの環境だとこんな感じなのですが、そちらではどんな表示ですか?(ブラウザも教えてほしいです)
image

Copy link
Contributor Author

@damin3A3 damin3A3 Dec 28, 2023

Choose a reason for hiding this comment

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

ご確認ありがとうございます。
自分の環境だとこんな感じです。

mac+chrome モバイル表示でも大体同じ感じです。
マスターボリューム_mac+chrome

mac+safari
マスターボリューム_mac+safari

iphone+iOS PWA(Safari)
確認できなかったのですが、変更なし(開発環境でないtraQ)の場合こんな感じで、今回の変更で色などは変更していないので、これと同じ感じだと思います。
マスターボリューム_iphone+PWA

Figma上のものはchromeを想定してデザインしてくださっているのかなと思うのですが、PCやブラウザに関わらず同じデザインになるようにした方がいいのでしょうか。
よろしくお願いいたします。

Copy link
Contributor

Choose a reason for hiding this comment

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

なるほどです
ブラウザによって結構変わっちゃうのでこのままでもいい気がしますが、一応周りの灰色部分消すことができるかどうかくらいでいいので、調べてみてできそうだったらやってみてほしいです!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。わかりました。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

お疲れ様です。スライダーなのですが、FormRangeWithValue.vueを変更してしまってもいいですか?

Copy link
Contributor

Choose a reason for hiding this comment

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

リポジトリ全体に対して<form-range-with-valueで検索をかけてみると、このコンポーネントが今のところQallの設定画面でしか使われていないことが分かるので、OKです

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。確かにQallの設定画面以外では文字列がヒットしませんでした。
では、スライダーの設定のコードを変更してみます。

v-model="state.masterVolume"
max-text="100%"
@@ -74,7 +76,7 @@
/>
</div>
<div :class="$style.element">
<h3 :class="$style.header">ノイズゲート</h3>
<h3 :class="$style.heading">ノイズゲート</h3>
<p>
マイクに入力された音が指定した音量以下だった場合にミュートします。
-100dBにすると無効になります。
@@ -86,6 +88,7 @@
step="1"
max="0"
:format="formatNoiseGateThreshold"
:class="$style.content"
/>
</div>
<noise-suppression
@@ -205,7 +208,7 @@ const voiceOptions = useVoices()
</script>

<style lang="scss" module>
.header {
.heading {
margin-bottom: 8px;
}
.element {
@@ -215,11 +218,19 @@ const voiceOptions = useVoices()
display: flex;
align-items: center;
margin-bottom: 8px;
.header_and_content {
.section {
flex: 1;
}
.toggle {
margin-left: 12px;
}
}

.option {
margin: 12px 0;
font-size: 12px;
}
.content {
Copy link
Contributor

Choose a reason for hiding this comment

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

ここもFigmaだと上にだけ4pxあるので、上下に12pxから変えてほしいです!
Figma_Uobqf9RdAw

Copy link
Contributor Author

Choose a reason for hiding this comment

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

わかりました、ありがとうございます!
Figmaの方を参考に微調整します。

margin: 12px 0;
}
</style>