add comments for anchor combinating functions #227
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
oozoofrog
added a commit
that referenced
this pull request
Aug 8, 2022
* deprecate callAsFunction for UIView * update test * update readme * update SwiftLayoutUtil.ViewPrinter to use sublayout(_:) Method * update ViewPrinterTests * add comments for anchor combinating functions (#227) * View printer congifuration (#226) * add ConfigurationPrintable * add missing double quotation marks at UIView+ConfigurationPrintable.swift accessibilityLabel propterty * Change ConfigurableProperty to a user-extensible structure * Add defaultConfigurablePropertys for UIControl / Provide a way to add other default views * add accessibilityDefaultConfigurablePropertys * Changed configurationName to fully express the output format * add uiImageViewDefaultConfigurablePropertys * add uiStackViewDefaultConfigurablePropertys & ConfigurationName extension clean up * update ConfigurableProperty to use custom getter for get value * add uiButtonDefaultConfigurablePropertys * change configurationName to configuration * update Configuration for UIImage, UIFont, UIColor * add tests for Configuration * add test for CustomConfigurablePropertys Co-authored-by: oozoofrog <oozoofrog@gmail.com> * Add combination anchor method to use CGSize (#228) * [renaming] defualt -> default (#229) * Config properties sorting (#230) - sort config properties - sort config properties printing results - update tests * Change the configuration output of UIImage, UIFont, and UIColor to multi-line comments (#231) * Better way to implement DefaultConfigurablePropertys (#232) * Crate DefaultConfigurablePropertys as a separate file * using ConfigPropertiesHandler (#233) * using ConfigPropertiesHandler * apply UILabel, UIButton * apply UIImageView, UIStackView * add default configurable properties for CustomConfigurableView * modify DefaultConfigurablePropertys to ConfigurableProperties * add unregist properties of view type * CustomConfigurablePropertys has higher priority than registed properties * separate prepared configurable properties * regist at create default instance of ConfigurableProperties * Move accessibility property to UIView's property and clean up the code * hide regist from outside of module * remove unnecessary function * readd removed test * remove unnecessary init definition from CustomConfigurableProperties * refactoring default reference view setting * remove unnecessary test code * test failed - testCustomRegistedConfigurableViewDefaultConfigurableProperties * test succeed - testCustomRegistedConfigurableViewDefaultConfigurableProperties * Change to provide defaultReferenceView unconditionally when registering propertiesHandler Co-authored-by: gmlwhdtjd <gmlwhdtjd@naver.com> Co-authored-by: oozoofrog <oozoofrog@gmail.com> * update README.md for 2.6.0 Co-authored-by: gmlwhdtjd <gmlwhdtjd@naver.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.