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

XAddress 组件有一条多余的上边线 #2371

Closed
hooraygith opened this issue Jan 3, 2018 · 9 comments
Closed

XAddress 组件有一条多余的上边线 #2371

hooraygith opened this issue Jan 3, 2018 · 9 comments

Comments

@hooraygith
Copy link

如图所示:

20180103191548

当在 XAddress 组件是 Group 组件的第一的子组件时,应该没有边线

@wg5945
Copy link
Collaborator

wg5945 commented Jan 3, 2018

你指的是.vux-cell-box:before这个样式么,观察很细致

@hooraygith
Copy link
Author

就是.vux-cell-box:before这个样式。

这样修改不够吧,vux-cell-box 外层还有个空div:

20180104095842

@wg5945
Copy link
Collaborator

wg5945 commented Jan 4, 2018

这个没什么影响吧,只是个空节点

现在address仅仅是对popup-picker做了一层封装,这里是留个根div是为了方便以后扩展其他内容?

@hooraygith
Copy link
Author

image

x-address 不是 group 第一个子组件时, 这个样式还是有问题,多套了层div:

image

需要去掉 components/x-address/index.vue 最外层的 div

@lichunqiang
Copy link
Collaborator

不需要吧?我印象中vue好像需要要一个根dom包裹。况且不会有什么影响

@hooraygith
Copy link
Author

hooraygith commented Feb 6, 2018

vue好像需要要一个根dom包裹,是指vue模板template标签里只能有一个子元素吧

所有的weui-cells的直接子元素都应该有weui-cell这个class,用这个 class 控制这个边线。这样就可以去掉x-address自己的边线

@wg5945
Copy link
Collaborator

wg5945 commented Feb 7, 2018

恩,是这样,之前没注意 - -

wg5945 pushed a commit that referenced this issue Feb 7, 2018
@WilliamLin1990
Copy link

WilliamLin1990 commented Feb 17, 2018

image
我现在这样使用的话就少了上边线了 @wg5945
<cell title="产品类型" :value="type2" is-link @click.native="showPopup=true" value-align="left"></cell> <x-address title="省份城市" v-model="province_city" raw-value :list="addressData" value-text-align="left"></x-address>

@anselwong
Copy link

anselwong commented Mar 12, 2018

@wg5945 @WilliamLin1990 +1

暂时解决方式是去掉源码的外层div

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

No branches or pull requests

5 participants