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

Update corner-rounding.md #482

Merged
merged 3 commits into from
Aug 20, 2017
Merged

Update corner-rounding.md #482

merged 3 commits into from
Aug 20, 2017

Conversation

oferRounds
Copy link
Contributor

@oferRounds oferRounds commented Jul 28, 2017

If I understand right, both Precomposited Corners techniques uses [path clip].

If so, I think it will be better to move it to the general description of the section, as currently it seems as if only the second technique uses [path clip].

If I understand right, both Precomposited corners techniques uses `[path clip]`, right? If so, I think it will be better to move it to the general description of the section, as currently it seems as only the second option uses `[path clip]`.
@oferRounds
Copy link
Contributor Author

oferRounds commented Jul 28, 2017

Perhaps in the future, it’s also worth adding some concrete code examples, and maybe also make this section a bit more general — to include other actions which trigger offscreen rendering, such as using the Layer’s shadow and border properties (while describing also the better alternatives, including convenience methods ASDK offers).

Improved positioning.
Syntax consistency: Changed `off-screen` to `offscreen`, as both syntaxes currently appears in the text.
@oferRounds
Copy link
Contributor Author

oferRounds commented Jul 29, 2017

@appleguy - and just a question I was wondering about — why is iOS built in this way that when using the Layer’s .cornerRadius it triggers offscreen rendering?

I.e., If Apple would like to change this behaviour, can they do things different in the way that the clipping operation would not trigger the offscreen rendering, or is this thing is inherently unavoidable?

@nguyenhuy nguyenhuy requested a review from appleguy August 8, 2017 10:23
@appleguy
Copy link
Member

@oferRounds Thanks for doing this! :)

@appleguy appleguy merged commit 1204737 into TextureGroup:master Aug 20, 2017
@oferRounds oferRounds deleted the patch-3 branch August 20, 2017 08:48
bernieperez pushed a commit to AtomTickets/Texture that referenced this pull request Apr 25, 2018
* Update corner-rounding.md

If I understand right, both Precomposited corners techniques uses `[path clip]`, right? If so, I think it will be better to move it to the general description of the section, as currently it seems as only the second option uses `[path clip]`.

* Update corner-rounding.md

Improved positioning.

* Update corner-rounding.md

Syntax consistency: Changed `off-screen` to `offscreen`, as both syntaxes currently appears in the text.
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

2 participants