This repository has been archived by the owner on May 1, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 477
FormsCommunityToolkit: Added Animations Sample #39
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
Some fixes in Animations Library Updated Xamarin.Forms NuGet version of sample projects
Depechie
suggested changes
Jun 29, 2017
public AnimationsPage() | ||
{ | ||
InitializeComponent(); | ||
var test = new BeginAnimationBehavior(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this test variable being declared here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooops. It was a test that has totally escaped!
|
||
protected override async void Invoke(VisualElement sender) | ||
{ | ||
if (Animation != null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know, it's hard ;) But please only positive checks. if( == null ) return;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, no problem ;)
Depechie
approved these changes
Jun 29, 2017
Depechie
approved these changes
Jul 6, 2017
jamesmontemagno
added a commit
that referenced
this pull request
Apr 4, 2018
* Initial Commit * Renaming folders * Sample app renaming projects and namespace * Folder renaming * Italic Placeholder Effect Created Effect that makes the placeholder of an Entry in Xamarin.Forms italicized. A comment on StackOverflow said Font.PointSize isn't exactly the size of the text, but I've found it's close enough. * Updated Sample app Updated the Sample app to include a demonstration of the Italic Placeholder Effect. I forgot the RoutingEffect so had to make that. * Add tests and rename * Add visibility * Tabs to spaces * Add the ValueConversionAttribute that is used by tools such as the WPF Designer or MFractor to verify conversion type-safety in Xaml #2 Also, annotate all existing classes with ValueConversionAttribute. * Fix references for pcl * Add xml file to release mode * Add sample project * Add SelectAllTextEntryEffect for Android and iOS, with Example * Add support for changing the true color for a toggle switch * First implementation SelectAllTextEntryEffect for UWP * More Organon renaming * PR feedback on unspecified * removed unused variables * Added SelectAllTextEntryEffect to UWP * [UWP] Add SearchBarSuggestionEffect * Add ChangePickerColorEffect for Android * Clean * Add ChangeColorPickerEffect also for iOS, with sample. SelectAllTextEntryEffect corrected… * Re-organization/Naming/Appveyor Scripts/More Gettings ready for builds in appveyor. Moved to a standard that I use in all projects. * Mass renaming. remove _, remove private, and more style guidelines * Initial Commit of Behaviors * Added ChangeColorPickerEffect for UWP * Initial Control Commit * Initial Gesture Commit * Initial Animation Commit * Update nugget & rd.xml file * Add platform specific projects * Add platform specific projects * Add platform specific projects * Add Entry Behaviors * Add New Behaviors EventToCommand, AddEffect, and NumericValidation * Add projects to solution * Update solution with projects. * Add projects to solution * Rename rd file * rename rd file * Fixes for building Effects lib * Fixing clear entry effect * Remove arrow pattern Fixes #31 * Else not needed on if - return pattern * Spaces instead of Tabs * Bye tabs * Try saving the Visual Studio settings! * Revert "Try saving the Visual Studio settings!" This reverts commit 22229ce356f8890e72a838ffb2f78d14ad53543d. * Converting tabs to spaces * Fix for PR27 * Tabs to spaces * Adding unstaged project file Someone got lost * Fixed filename and tts * Build fixes * Some tabs slipped in * Removing magic strings from Effects ctors * Removing orphaned old sample files * Moving filtered Converters back to /src * Moving filtered Behaviors back to /src * Moving filtered Gestures back to /src * Moving filtered Effects back to /src * Moving filtered Controls back to /src * Moving filtered Animations back to /src * Restructure the codebase and add CI! * Update cake * Fix package locations * cleanup slns * fix gestures sln * Update README.md * Update README.md * Adding editorconfig * Update Contributing.md * Refactor and rename * Adjustments * Build fix * Added allias * Fixing NuGet monnikers * Update build.cake * Updating nuspecs to unified repo * Unify package descriptions * Fix for #25 This code fixes #25. It is a bug of built-in behaviour in Xamarin.iOS, as seen on their [bug tracker](https://bugzilla.xamarin.com/show_bug.cgi?id=33666). After each change of certain properties (TextColor, Font, Text or FormattedText) the effect is reapplied. * Code formatting Reformatted some code for better readability * Toggled event and value is being eaten by the effect implementation! Need to apply the value to the Xamarin forms Control Fixes #31 * First stab at adding samples project * Changes to address issues during code review * I blame NUGET * Fix #35 * Added Switch change color effect sample * Added Forms Community Toolkit animations code * Changed namespaces declaration position * Changed project error message from Spanish to English. * BindableProperty nameof * Added Animations Sample * Revert "BindableProperty nameof" This reverts commit d0749a5. * Revert "Revert "BindableProperty nameof"" This reverts commit 30d41f0. * Deleted animations sample (left for another PR). * Change condition to positive checks * if (Animation == null) Change condition * Added Animations Sample Some fixes in Animations Library Updated Xamarin.Forms NuGet version of sample projects * Fixed wrong Spanish error project message * More error messages fixes * Changes related to PR #39 Feedback * Update README.md * Effect sample Entry disable auto correct * Effect sample Entry italic placeholder * Review adjustments * Added all current entry effect samples * Added Size Font To Fit label effect sample * Remove file level xaml compilation attribute * trying to fix cake * Target newer Cake * Try specific Cake filehelpers version * Image View blur effect sample #HacktoberFest * Updating AppVeyor badge * Adding missing tempkey * Fixed link to sample app Fixes #29 * Added link to documentation Fixes #33 * Add converter * Add tests * Add doc * Move MockConverter to separate file * Update explanation of methods * Update docs * Added ColorAnimation Updated Animations Sample Added Animations nuspec * Fixed ColorAnimation usings Changed Animations nuspec Xamarin.Forms version * Make it easy to spot esp on mobile * Updated Xamarin.Forms NuGet packages to v2.5 * Update README.md toolkit name change. * Update Contributing.md name change * Update LICENSE name change, update year * Updated more projects to Net Standard Renamed namespaces from FormsCommunityToolkit to XamarinCommunityToolkit Updated some nuspecs * Changed Sample namespaces More changes in nuspecs * Fixed Effects.Android namespace error * Fixed incorrect project references * Updated nuspecs * Updated nuspecs (PR feedback) * Multi-target! * Rename to Xamarin.Toolkit Aligns with UWP Community Toolkit * Fix tests * Fix android build * Final fix for android * Finalize new SLN * update readme * Final tweaks to csproj files * Add netstandard 1.0 and docs * Refactor Build Props and Targets Nothing for targets right now. * Cleanup samples * Update android resources :) * Remove readthedocs link
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Added Animations Sample that allows to test all the possibilities of the animations library.