8000 优化移动端UI by dinphy · Pull Request #186 · kingwrcy/moments · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

优化移动端UI #186

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Sep 4, 2024
Merged

优化移动端UI #186

merged 3 commits into from
Sep 4, 2024

Conversation

dinphy
Copy link
Contributor
@dinphy dinphy commented Aug 25, 2024
  • 侧边栏菜单调整至从底部滑出
  • 分离置顶、编辑和删除操作，移至memo详情页面，点击置顶图标旁的图标从底部滑出
  • 更换了几个图标

首页移动端菜单
image
详情页置顶、编辑和删除
image
其他页面移动端菜单
image

@dinphy
优化移动端UI
5736cbd 
- 侧边栏菜单调整至从底部滑出
- 分离置顶、编辑和删除操作，移至memo详情页面，点击置顶图标旁的图标从底部滑出
- 更换了几个图标
@kingwrcy
Copy link
Owner

最近没时间开发,这些功能你都本地完整的测试过吧?

@dinphy
Copy link
Contributor Author
dinphy commented Aug 26, 2024

最近没时间开发,这些功能你都本地完整的测试过吧?

测试好了我才提交的。就是原来的这段代码：

const setPinned = async (id: number) => {
  await useMyFetch('/memo/setPinned?id=' + id)
  toast.success("操作成功!")
  showToolbar.value = false
  memoReloadEvent.emit()
}

memoReloadEvent.emit() 这个重载没有生效。

我做了操作成功后跳转到首页

const setPinned = async (id: number) => {
  await useMyFetch('/memo/setPinned?id=' + id)
  toast.success("操作成功!")
  if (isDetailPage.value) {
    await navigateTo('/')
  } else {
    memoReloadEvent.emit()
  }
  moreToolbar.value = false
}

其他的我都测试了，没有问题，有一个小的底部遮挡，我也修复了提交了PR

@kingwrcy kingwrcy merged commit 5f00252 into kingwrcy:dev Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dinphy @kingwrcy
0