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

添加搜索框自定义高度 #1113

Merged
merged 4 commits into from
Apr 10, 2017

Conversation

yezongyang
Copy link

在默认主题中加入搜索框自定义高度

@mention-bot
Copy link

@yezongyang, thanks for your PR! By analyzing the history of the files in this pull request, we identified @pingan1927, @afc163 and @warmhug to be potential reviewers.

@codecov
Copy link

codecov bot commented Apr 9, 2017

Codecov Report

Merging #1113 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1113      +/-   ##
==========================================
+ Coverage   69.84%   69.86%   +0.01%     
==========================================
  Files         218      218              
  Lines        4079     4081       +2     
  Branches     1212     1214       +2     
==========================================
+ Hits         2849     2851       +2     
  Misses       1229     1229              
  Partials        1        1
Flag Coverage Δ
#rn 72.03% <100%> (ø) ⬆️
#web 67.57% <ø> (+0.03%) ⬆️
Impacted Files Coverage Δ
components/style/themes/default.tsx 100% <ø> (ø) ⬆️
components/search-bar/index.tsx 79.62% <100%> (ø) ⬆️
components/grid/index.web.tsx 100% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0e1f0a...e0cea8f. Read the comment docs.

@paranoidjk paranoidjk self-requested a review April 9, 2017 08:12
Copy link
Contributor

@paranoidjk paranoidjk left a comment

Choose a reason for hiding this comment

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

RN 这边也统一加下吧,注意 RN 的尺寸是 web 的一半

https://github.com/ant-design/ant-design-mobile/blob/master/components/style/themes/default.tsx#L159

@silentcloud
Copy link
Contributor

silentcloud commented Apr 10, 2017

@paranoidjk 这个也需要设计师敲定一下的,变量添加需要设计师一起拍

@paranoidjk
Copy link
Contributor

paranoidjk commented Apr 10, 2017

@silentcloud review RN 那块的样式修改有没有问题,没问题就合掉这个。其他变量的增加后续我来加,设计还没定稿。

@silentcloud
Copy link
Contributor

@paranoidjk 代码 ok 的,你再 npm run ios/android 看看,没问题你直接合

@@ -39,7 +39,7 @@ export default StyleSheet.create({
tintColor: variables.input_color_icon,
position: 'absolute',
left: variables.h_spacing_md + 8,
top: (variables.search_bar_input_height - variables.icon_size_xxs) / 2 + 8,
top: (variables.search_bar_height - variables.icon_size_xxs) / 2,
Copy link
Contributor

Choose a reason for hiding this comment

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

top: (variables.search_bar_input_height - variables.icon_size_xxs) / 2,

@yezongyang @silentcloud

Copy link
Contributor

Choose a reason for hiding this comment

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

@yezongyang 你可以按开发文档 跑起来看下,是有问题的

Copy link
Author

@yezongyang yezongyang Apr 10, 2017

Choose a reason for hiding this comment

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

alignItems 好像对 TextInput 不起作用

@peter0011
Copy link

增加一个组件变量,@search-bar-height

@paranoidjk paranoidjk merged commit 4447270 into ant-design:master Apr 10, 2017
@paranoidjk
Copy link
Contributor

@yezongyang 👍
Antd-Mobile 在 React-Native 组件这方面还有很多可以做的工作,请问你有意愿成为 Collaborator
帮我们一起完善吗? 😄
#1068

@yezongyang yezongyang deleted the fix/searchbar-height-custom branch April 10, 2017 15:04
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.

None yet

5 participants