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

ソング:再生位置の表示形式を変更できるようにする #2306

Merged
Merged
Show file tree
Hide file tree
Changes from 21 commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
9f648aa
MBSR形式で表示するように変更、秒とMBSRどちらで表示するかを切り替えられるようにした
sigprogramming Oct 19, 2024
63336c1
拍と16分が1始まりになるように修正、コメントを追加
sigprogramming Oct 19, 2024
c41505b
修正、MBSRからMBSに変更
sigprogramming Oct 19, 2024
4563324
tickからMBSに変換する処理をdomain.tsに移動
sigprogramming Oct 19, 2024
4e9c638
変数名とclass名を変更
sigprogramming Oct 19, 2024
6c35780
小節.拍.100分に変更、MBS型からMeasuresBeats型に変更
sigprogramming Oct 22, 2024
840eab3
右クリックメニューの表記を変更
sigprogramming Oct 22, 2024
613cf65
再生ヘッド位置の表示モードが設定として保存されるようにした
sigprogramming Oct 22, 2024
725d0d0
デフォルトの表示モードを秒に変更[update snapshots]
sigprogramming Oct 22, 2024
f370ccb
displayModeからdisplayFormatに変更、リファクタリング
sigprogramming Oct 23, 2024
802a8f0
関数を削除
sigprogramming Oct 24, 2024
13365c1
Update src/components/Sing/PlayheadPositionDisplay.vue
sigprogramming Oct 24, 2024
4055aca
再生ヘッド位置の取得・設定処理をコンポーザブル化
sigprogramming Oct 24, 2024
06a8af7
SCREAMING_SNAKE_CASEに変更
sigprogramming Oct 24, 2024
3f9f201
修正
sigprogramming Oct 24, 2024
aa96997
Merge branch 'main' into display_playhead_position_in_mbsr_format
sigprogramming Oct 24, 2024
a52bd76
playheadPositionをRefにして、FrequentlyUpdatedStateとusePlayheadPositionを削除
sigprogramming Oct 26, 2024
6c3e1c2
MeasuresBeatsを移動
sigprogramming Oct 28, 2024
c6eeed2
PLAYHEAD_POSITIONの同期処理を削除
sigprogramming Oct 28, 2024
ba88ed9
displayFormatのチェックを行っているところを削除、importのところが変更できていなかったので修正
sigprogramming Oct 28, 2024
dd7c1b4
tpqnのバリデーション処理が間違っていたので修正
sigprogramming Oct 28, 2024
4d63b33
// オートスクロールの位置変更
Hiroshiba Oct 28, 2024
83fc8ba
Merge remote-tracking branch 'upstream/main' into pr/sigprogramming/2…
Hiroshiba Oct 28, 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
138 changes: 138 additions & 0 deletions src/components/Sing/PlayheadPositionDisplay.vue
Original file line number Diff line number Diff line change
@@ -0,0 +1,138 @@
<template>
<div>
<div v-if="displayFormat === 'MINUTES_SECONDS'" class="playhead-position">
<div>{{ minAndSecStr }}</div>
<div class="millisec">.{{ milliSecStr }}</div>
</div>
<div v-if="displayFormat === 'MEASURES_BEATS'" class="playhead-position">
<div>{{ measuresStr }}.</div>
<div>{{ beatsIntegerPartStr }}</div>
<div class="beats-fractional-part">.{{ beatsFractionalPartStr }}</div>
Copy link
Member

Choose a reason for hiding this comment

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

(割とただのコメントです)

ここの更新速度が早すぎて、若干目が使えるなと感じました!
ただまあ、別にこれでいい感じなのかなーとも思ってます。どうしようもない!

</div>
<ContextMenu ref="contextMenu" :menudata="contextMenuData" />
</div>
</template>

<script setup lang="ts">
import { ref, computed } from "vue";
import { useStore } from "@/store";
import ContextMenu, {
ContextMenuItemData,
} from "@/components/Menu/ContextMenu.vue";
import {
getTimeSignaturePositions,
MeasuresBeats,
ticksToMeasuresBeats,
} from "@/sing/domain";
import { useRootMiscSetting } from "@/composables/useRootMiscSetting";

const store = useStore();

const playheadTicks = computed(() => store.getters.PLAYHEAD_POSITION);
const [displayFormat, setDisplayFormat] = useRootMiscSetting(
store,
"playheadPositionDisplayFormat",
);

const timeSignatures = computed(() => {
const tpqn = store.state.tpqn;
const timeSignatures = store.state.timeSignatures;
const tsPositions = getTimeSignaturePositions(timeSignatures, tpqn);
return timeSignatures.map((value, index) => ({
...value,
position: tsPositions[index],
}));
});

const measuresBeats = computed((): MeasuresBeats => {
const tpqn = store.state.tpqn;
return ticksToMeasuresBeats(playheadTicks.value, timeSignatures.value, tpqn);
});

const measuresStr = computed(() => {
return measuresBeats.value.measures >= 0
? String(measuresBeats.value.measures).padStart(3, "0")
: String(measuresBeats.value.measures);
});

const beatsIntegerPartStr = computed(() => {
const integerPart = Math.floor(measuresBeats.value.beats);
return String(integerPart).padStart(2, "0");
});

const beatsFractionalPartStr = computed(() => {
const integerPart = Math.floor(measuresBeats.value.beats);
const fractionalPart = Math.floor(
(measuresBeats.value.beats - integerPart) * 100,
);
return String(fractionalPart).padStart(2, "0");
});

const minAndSecStr = computed(() => {
const ticks = playheadTicks.value;
const time = store.getters.TICK_TO_SECOND(ticks);
const intTime = Math.trunc(time);
const min = Math.trunc(intTime / 60);
const minStr = String(min).padStart(2, "0");
const secStr = String(intTime - min * 60).padStart(2, "0");
return `${minStr}:${secStr}`;
});

const milliSecStr = computed(() => {
const ticks = playheadTicks.value;
const time = store.getters.TICK_TO_SECOND(ticks);
const intTime = Math.trunc(time);
const milliSec = Math.trunc((time - intTime) * 1000);
const milliSecStr = String(milliSec).padStart(3, "0");
return milliSecStr;
});

const contextMenu = ref<InstanceType<typeof ContextMenu>>();
const contextMenuData = computed<ContextMenuItemData[]>(() => {
return [
{
type: "button",
label: "分:秒",
disabled: displayFormat.value === "MINUTES_SECONDS",
onClick: async () => {
contextMenu.value?.hide();
setDisplayFormat("MINUTES_SECONDS");
},
disableWhenUiLocked: false,
},
{
type: "button",
label: "小節.拍",
disabled: displayFormat.value === "MEASURES_BEATS",
onClick: async () => {
contextMenu.value?.hide();
setDisplayFormat("MEASURES_BEATS");
},
disableWhenUiLocked: false,
},
];
});
</script>

<style scoped lang="scss">
@use "@/styles/v2/variables" as vars;
@use "@/styles/colors" as colors;

.playhead-position {
align-items: center;
display: flex;
font-weight: 700;
font-size: 28px;
color: var(--scheme-color-on-surface);
}

.millisec {
font-size: 16px;
margin: 10px 0 0 2px;
}

.beats-fractional-part {
font-size: 16px;
margin: 10px 0 0 2px;
}
</style>
18 changes: 4 additions & 14 deletions src/components/Sing/ScoreSequencer.vue
Original file line number Diff line number Diff line change
Expand Up @@ -327,7 +327,7 @@ const scrollX = ref(0);
const scrollY = ref(0);

// 再生ヘッドの位置
const playheadTicks = ref(0);
const playheadTicks = computed(() => store.getters.PLAYHEAD_POSITION);
const playheadX = computed(() => {
const baseX = tickToBaseX(playheadTicks.value, tpqn.value);
return Math.floor(baseX * zoomX.value);
Expand Down Expand Up @@ -1318,9 +1318,7 @@ const onScroll = (event: Event) => {
}
};

const playheadPositionChangeListener = (position: number) => {
playheadTicks.value = position;

watch(playheadTicks, (newPlayheadPosition) => {
Hiroshiba marked this conversation as resolved.
Show resolved Hide resolved
// オートスクロール
const sequencerBodyElement = sequencerBody.value;
if (!sequencerBodyElement) {
Expand All @@ -1337,7 +1335,7 @@ const playheadPositionChangeListener = (position: number) => {
const scrollTop = sequencerBodyElement.scrollTop;
const scrollWidth = sequencerBodyElement.scrollWidth;
const clientWidth = sequencerBodyElement.clientWidth;
const playheadX = tickToBaseX(position, tpqn.value) * zoomX.value;
const playheadX = tickToBaseX(newPlayheadPosition, tpqn.value) * zoomX.value;
const tolerance = 3;
if (playheadX < scrollLeft) {
sequencerBodyElement.scrollTo(playheadX, scrollTop);
Expand All @@ -1347,7 +1345,7 @@ const playheadPositionChangeListener = (position: number) => {
) {
sequencerBodyElement.scrollTo(playheadX, scrollTop);
}
};
});

// スクロールバーの幅を取得する
onMounted(() => {
Expand Down Expand Up @@ -1394,19 +1392,11 @@ onActivated(() => {

// リスナー登録
onActivated(() => {
void store.dispatch("ADD_PLAYHEAD_POSITION_CHANGE_LISTENER", {
listener: playheadPositionChangeListener,
});

document.addEventListener("keydown", handleKeydown);
});

// リスナー解除
onDeactivated(() => {
void store.dispatch("REMOVE_PLAYHEAD_POSITION_CHANGE_LISTENER", {
listener: playheadPositionChangeListener,
});

document.removeEventListener("keydown", handleKeydown);
});

Expand Down
19 changes: 2 additions & 17 deletions src/components/Sing/SequencerRuler/Container.vue
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
</template>

<script setup lang="ts">
import { computed, onMounted, onUnmounted, ref } from "vue";
import { computed } from "vue";
import Presentation from "./Presentation.vue";
import { useStore } from "@/store";

Expand All @@ -36,27 +36,12 @@ const tpqn = computed(() => store.state.tpqn);
const timeSignatures = computed(() => store.state.timeSignatures);
const sequencerZoomX = computed(() => store.state.sequencerZoomX);

const playheadTicks = ref(0);
const playheadTicks = computed(() => store.getters.PLAYHEAD_POSITION);

const updatePlayheadTicks = (ticks: number) => {
void store.dispatch("SET_PLAYHEAD_POSITION", { position: ticks });
};

const playheadPositionChangeListener = (position: number) => {
playheadTicks.value = position;
};

onMounted(() => {
void store.dispatch("ADD_PLAYHEAD_POSITION_CHANGE_LISTENER", {
listener: playheadPositionChangeListener,
});
});
onUnmounted(() => {
void store.dispatch("REMOVE_PLAYHEAD_POSITION_CHANGE_LISTENER", {
listener: playheadPositionChangeListener,
});
});

const deselectAllNotes = () => {
void store.dispatch("DESELECT_ALL_NOTES");
};
Expand Down
62 changes: 3 additions & 59 deletions src/components/Sing/ToolBar/ToolBar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -113,12 +113,7 @@
icon="stop"
@click="stop"
/>
<div class="sing-playhead-position">
<div>{{ playheadPositionMinSecStr }}</div>
<div class="sing-playhead-position-millisec">
.{{ playHeadPositionMilliSecStr }}
</div>
</div>
<PlayheadPositionDisplay class="sing-playhead-position" />
</div>
<!-- settings for edit controls -->
<div class="sing-controls">
Expand Down Expand Up @@ -164,7 +159,8 @@
</template>

<script setup lang="ts">
import { computed, watch, ref, onMounted, onUnmounted } from "vue";
import { computed, watch, ref } from "vue";
import PlayheadPositionDisplay from "../PlayheadPositionDisplay.vue";
import EditTargetSwicher from "./EditTargetSwicher.vue";
import { useStore } from "@/store";

Expand Down Expand Up @@ -383,30 +379,6 @@ const setVolumeRangeAdjustment = () => {
});
};

const playheadTicks = ref(0);

/// 再生時間の分と秒
const playheadPositionMinSecStr = computed(() => {
const ticks = playheadTicks.value;
const time = store.getters.TICK_TO_SECOND(ticks);

const intTime = Math.trunc(time);
const min = Math.trunc(intTime / 60);
const minStr = String(min).padStart(2, "0");
const secStr = String(intTime - min * 60).padStart(2, "0");

return `${minStr}:${secStr}`;
});

const playHeadPositionMilliSecStr = computed(() => {
const ticks = playheadTicks.value;
const time = store.getters.TICK_TO_SECOND(ticks);
const intTime = Math.trunc(time);
const milliSec = Math.trunc((time - intTime) * 1000);
const milliSecStr = String(milliSec).padStart(3, "0");
return milliSecStr;
});

const nowPlaying = computed(() => store.state.nowPlaying);

const play = () => {
Expand Down Expand Up @@ -463,22 +435,6 @@ const snapTypeSelectModel = computed({
});
},
});

const playheadPositionChangeListener = (position: number) => {
playheadTicks.value = position;
};

onMounted(() => {
void store.dispatch("ADD_PLAYHEAD_POSITION_CHANGE_LISTENER", {
listener: playheadPositionChangeListener,
});
});

onUnmounted(() => {
void store.dispatch("REMOVE_PLAYHEAD_POSITION_CHANGE_LISTENER", {
listener: playheadPositionChangeListener,
});
});
</script>

<style scoped lang="scss">
Expand Down Expand Up @@ -714,19 +670,7 @@ onUnmounted(() => {
}

.sing-playhead-position {
align-items: center;
display: flex;
font-size: 28px;
font-weight: 700;
margin-left: 16px;
color: var(--scheme-color-on-surface);
}

.sing-playhead-position-millisec {
font-size: 16px;
font-weight: 700;
margin: 10px 0 0 2px;
color: var(--scheme-color-on-surface);
}

.sing-controls {
Expand Down
Loading
Loading