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

【代码优化】SYSTEM:支付宝小程序兼容性:修复无法启动、tabbar与宫格组件无法点击、首页的自定义顶部导航栏与轮播图样式 #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Qiksy
Copy link

@Qiksy Qiksy commented Aug 5, 2024

修复问题:
无法启动:1. 添加mini.project.json; 2.修改@sheep/platform内容,添加alipay provider 3. wechat的provider在MP-ALIPAY情况下也要添加默认值,避免编译错误

tabbar与宫格组件无法点击: 给无法点击的自定义组件外部再包括一层view

首页样式: 轮播图修改了 style,自定义导航栏暂时屏蔽

@@ -54,7 +53,7 @@ const http = new Request({
header: {
Accept: 'text/json',
'Content-Type': 'application/json;charset=UTF-8',
platform: $platform.name,
platform: 'test',
Copy link
Contributor

Choose a reason for hiding this comment

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

这个改动,是不是不太对哈?test???

@@ -201,6 +208,8 @@
if (!isEmpty(shareInfo.value)) {
sheep.$platform.share.updateShareInfo(shareInfo.value);
}

console.log("导航栏模式",navbarMode.value,props.navbar.value)
Copy link
Contributor

Choose a reason for hiding this comment

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

这个打印,是不是去掉哈?

<s-custom-navbar
v-else-if="navbar === 'custom' && navbarMode === 'normal'"
:data="navbarStyle"
:showLeftButton="showLeftButton"
/>
<!--#endif-->

<!--#ifdef MP-ALIPAY-->
Copy link
Contributor

Choose a reason for hiding this comment

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

这里,看着是注释掉了 是不是也可以删除?

@@ -5,6 +5,7 @@
>
<view class="page-main" :style="[bgMain]">
<!-- 顶部导航栏-情况1:默认通用顶部导航栏 -->
<!--<view>{{navbar}}{{navbarMode}}</view>-->
Copy link
Contributor

Choose a reason for hiding this comment

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

这里,好像没用?

@@ -259,7 +259,7 @@
}

return {
height: height + 'rpx',
height: height + 'rpx' + ' !important',
Copy link
Contributor

Choose a reason for hiding this comment

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

这个是解决啥问题哈?

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