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

1. Fix a bug on IView. 2. Add '-' usage on 'ratioWidth/ratioHeight' #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TonyJR
Copy link

@TonyJR TonyJR commented Oct 16, 2015

1.Fix a bug on IView.
The error code like this:

2.Add a usage of '-' on 'ratioWidth/ratioHeight'
eg.

It's mean the height is 25% of the div width,in order to constraint ratio.

@ideawu
Copy link
Owner

ideawu commented Oct 19, 2015

Hi, 能否详细介绍下这个bug? 另外, 你对代码格式和缩进做了不符合规范的改动. 另外, 建议把这个 PR 拆分成多个小的 PR, 每一个只做一个改动.

@TonyJR
Copy link
Author

TonyJR commented Oct 19, 2015

Hi,这个bug是这样的。我希望实现的场景是这样的——在一个未知宽度的div中加入了4个子div,每个div的宽度是25%,于是我写了如下一段xml

<style> .btnContent { width : 25%; height : 100%; margin : 2 1; }
    .btn {
    float : center;
    valign: middle;
    }
</style>
<div style="float: center; clear: both; width : 100% ;" >
    <img id="banner" src="1_4.png" style="float : center ; clear:both;width:100%;height:100"/>
</div>
<div style="clear: both;width:100%;height:100;>
    <div name="div1" class="btnContent">
        <img class="btn" src="home_btn1"/>
    </div>
    <div name="div2" class="btnContent">
        <img class="btn" src="home_btn2"/>
    </div>
    <div name="div3" class="btnContent">
        <img class="btn" src="home_btn3"/>
    </div>
    <div name="div4" class="btnContent">
        <img class="btn" src="home_btn4"/>
    </div>
</div>

结果发现第一个div是正确的,占用25%空间。但是从第二个div开始,占用的宽度并不是父级div总宽度的25%,剩余空白空间的25%。
修复这个bug我只修改了IFlowLayout.m文件

另外缩进的问题可能是因为我使用新版本xcode导致的问题,每次拷贝代码都会产生变化。

by: 秦涛
mail: [email protected]
phone: 13375313826

在 2015年10月19日,上午11:08,ideawu [email protected] 写道:

Hi, 能否详细介绍下这个bug? 另外, 你对代码格式和缩进做了不符合规范的改动. 另外, 建议把这个 PR 拆分成多个小的 PR, 每一个只做一个改动.


Reply to this email directly or view it on GitHub #14 (comment).

@ideawu
Copy link
Owner

ideawu commented Oct 19, 2015

这不是bug, 设计如此, 具体看文档: http://www.cocoaui.com/docs/flowlayout

CocoaUI 的宽度百分比和 Web 浏览器中的百分比意义有差异, CocoaUI 是指相对当前空间的百分比, 不是相对父元素. 所以要实现等宽, 第一列是 1/n, 第二列是 1/(n-1)... 最后一列是 1/1, 即 100%.

@ideawu
Copy link
Owner

ideawu commented Oct 19, 2015

关于缩进问题, 你必须修改你的编辑器, 使用和 CocoaUI 风格统一的缩进方式, 即用 TAB 来缩进, 永远不要用空格来缩进.

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