Controlgroup: Small fixes to entry #298
Conversation
| </desc> | ||
| <type name="String" /> | ||
| </option> | ||
| <xi:include href="../includes/widget-option-disabled.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/> | ||
| <option name="onlyVisible" default="true" example-value="false"> | ||
| <desc>Sets whether to exclude invisible children in the assignment of rounded corners. When set to false, all children of a controlgroup are taken into account when assigning rounded corners, including hidden children. Thus, if, for example, the controlgroup's first child is hidden, the controlgroup will, in effect, not have rounded corners on the top edge.</desc> | ||
| <desc>Sets whether to exclude invisible children in the assignment of rounded corners. When set to false, all children of a controlgroup are taken into account when assigning rounded corners, including hidden children. Thus, if, for example, the controlgroup's first child is hidden, the controlgroup will, in effect, not have rounded corners on the left edge.</desc> |
There was a problem hiding this comment.
Makes sense for the default direction. Could be made explicit though, something like "not have rounded corners on the left edge (for direction: "horizontal")."
There was a problem hiding this comment.
how about this (Added text in bold): "Sets whether to exclude invisible children in the assignment of rounded corners. When set to false, all children of a controlgroup are taken into account when assigning rounded corners, including hidden children. Thus, if, for example, the controlgroup's first child is hidden and the default horizontal layout is applied, the controlgroup will, in effect, not have rounded corners on the left edge."
There was a problem hiding this comment.
We could also have an additional sentence after this to say: "Likewise, if the controlgroup has a vertical layout and its first child is hidden, the controlgroup will not have rounded corners on the top edge."
There was a problem hiding this comment.
Both additions sound good to me.
|
@agcolom Can you push the updates discussed above? Thanks. |
|
@scottgonzalez done :-) |
|
Merged into 1-12. |
No description provided.