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

Docs: Fix references of ASViewController/ASNavigationController (non-existent) to ASDKViewController/ASDKNavigationController #2072

Merged
merged 3 commits into from
Apr 5, 2023

Conversation

JoeFerrucci
Copy link
Contributor

@JoeFerrucci JoeFerrucci commented Dec 16, 2022

Changes documentation references of ASViewController and ASNavigationController (non-existent) to ASDKViewController and ASDKNavigationController.

As far as I can tell there's no such thing as ASViewController, and CHANGELOG says it was renamed in 3.0.0 (2020-07-15)

Same for ASNavigationController, Change was made in this CHANGELOG for 3.1.0

@CLAassistant
Copy link

CLAassistant commented Dec 16, 2022

CLA assistant check
All committers have signed the CLA.

@JoeFerrucci JoeFerrucci changed the title Fix references of ASViewController (non-existent) to ASDKViewController Docs: Fix references of ASViewController (non-existent) to ASDKViewController Dec 16, 2022
Benefits of using an `ASDKViewController`:
<ol>
<li><b>Save Memory</b>. An <code>ASDKViewController</code> that goes off screen will automatically reduce the size of the <a href="intelligent-preloading.html">fetch data</a> and <a href="intelligent-preloading.html">display ranges</a> of any of its children. This is key for memory management in large applications. </li>
<li><b><a href="asvisibility.html"><code>ASVisibility</code></a> Feature</b>. When used in an <code>ASNavigationController</code> or <code>ASTabBarController</code>, these classes know the exact number of user taps it would take to make the view controller visible.</li>
Copy link

Choose a reason for hiding this comment

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

ASDKNavigationController, not ASNavigationController

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was just for ASDKViewController but I can update it to change ASDKNavigationController as well. Thanks.

@JoeFerrucci JoeFerrucci changed the title Docs: Fix references of ASViewController (non-existent) to ASDKViewController Docs: Fix references of ASViewController/ASNavigationController (non-existent) to ASDKViewController/ASDKNavigationController Jan 23, 2023
@JoeFerrucci
Copy link
Contributor Author

Updated PR to additionally change ASNavigationController to ASDKNavigationController.

@JoeFerrucci JoeFerrucci requested a review from VAndrJ January 23, 2023 17:42
@JoeFerrucci
Copy link
Contributor Author

JoeFerrucci commented Jan 27, 2023

Does this need anything else, or can we get this approved and merge this?

@JoeFerrucci
Copy link
Contributor Author

Can we get this merged? It's been approved for almost 2 months.

@JoeFerrucci
Copy link
Contributor Author

Who maintains this repo?

Copy link
Member

@garrettmoon garrettmoon left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks for posting!

@garrettmoon garrettmoon merged commit e88783e into TextureGroup:master Apr 5, 2023
andrey-golovin-ios added a commit to andrey-golovin-ios/Texture that referenced this pull request Oct 5, 2023
* 'master' of https://github.com/TextureGroup/Texture:
  Fix build errors and a crash in xcode 15 (TextureGroup#2093)
  [ASCellNodeVisibilityEvent] Add a new event when scrolling stops (TextureGroup#2084)
  Trying to get CI to work (TextureGroup#2085)
  fix typo: ASStackLayoutElement.h (TextureGroup#2067)
  Docs: Fix references of ASViewController/ASNavigationController (non-existent) to ASDKViewController/ASDKNavigationController (TextureGroup#2072)
  [ASTextKitRenderer] Adding locking when accessing the text renderer cache (TextureGroup#2075)
  Switch UITextWritingDirection to NSWritingDirection (TextureGroup#2071)
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.

4 participants