-
Notifications
You must be signed in to change notification settings - Fork 7
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 : add time-zone-filter #21
base: master
Are you sure you want to change the base?
Conversation
❌ Deploy Preview for endearing-longma-1f4440 failed.
|
src/pages/Session.vue
Outdated
} | ||
|
||
const updateInputTimeZone = (event) => { | ||
inputTimeZone.value = event.target.value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
少處理 Timezone 輸入錯誤的情境
src/pages/Session.vue
Outdated
v-show="isChangeTimeZone" | ||
placeholder="" | ||
:value="inputTimeZone" | ||
@input="updateInputTimeZone" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以使用 v-model 精簡?
const { openPopUp, removeAll } = usePopUp() | ||
const { xsOnly } = useBreakpoints() | ||
const { locale } = useI18n() | ||
|
||
const currentTimeZone = ref('') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我認為你可以把這個塞入 useSession
類似 React context 的用法
1. 將TIMEZONE_OFFSET加入useSession 2. 新增watch觀測TIMEZONE_OFFSET並重新渲染
issue in COSCUP 2023
issue 顯示且能修改時區的議程表 #13