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

User/almedina ms/phantom of the xamarin back to the future #3766

Conversation

almedina-ms
Copy link
Contributor

@almedina-ms almedina-ms commented Feb 5, 2020

This is just the merge of master to the Xamarin.Forms branch

Microsoft Reviewers: Open in CodeFlow

BalajiR and others added 30 commits June 7, 2019 12:11
* Update Web Chat hostconfig & designer css

* formatting fix for .json file
* [UWP][TestApp] Usability Improvements

* PR Feedback
* [UWP] Update test projects to use new locations

* [Samples] Sort the files under samples/Tests by version

* [UWP] Update test files to reflect new locations

* Fix .NET

* ios update

* Update datauri test files
* Add pixel widths to column

* Describe the pixel format of minHeight

* Update 1.2.0 schema descriptions

There's no actual schema validation changes, just description changes, so no need to bump the version number
Added tooling so that we can generate specs from our schema, so that our spec'ing process starts with updating the schema first, leading to less errors and mistakes in the end :)

In `schemas\src`, add your new properties/elements to the schema

![UpdateSchema](https://user-images.githubusercontent.com/13246069/59529888-0fee6b80-8e97-11e9-8925-d29264c57b4d.gif)


### 3. Generate the updated spec

If you followed the setup instructions, simply save the file and the spec will be updated! Add any necessary rendering details to the spec.

![UpdatedSpec](https://user-images.githubusercontent.com/13246069/59530207-e2ee8880-8e97-11e9-8957-de92911c2935.gif)


## Review spec

Submit a pull request for your updated spec/schema. The proposed schema in the proposal becomes outdated and should be replaced with a link to the PR.

![image](https://user-images.githubusercontent.com/13246069/59530515-a8392000-8e98-11e9-8d25-62d40d550517.png)


I'll share more details about this at Monday's meeting. Lots of details about the new proposed spec'ing process are captured here: https://github.com/microsoft/AdaptiveCards/wiki/Project-Management

###### Microsoft Reviewers: [Open in CodeFlow](http://wpcp.azurewebsites.net/CodeFlowProtocolProxy2.php?pullrequest=https://github.com/microsoft/AdaptiveCards/pull/3068)
* Fix sidebar scrolling

* Try/catch only the AC stuff
* Include version numbers on website's JS scripts

* Fix root-relative path

* Fix formatting
* Include version numbers on website's JS scripts

* Fix root-relative path

* Fix formatting

* Version CSS files too
* added adaptive cards fabric; added tests

* updated build scripts

* input validation

* reverted unintended changes

* moved jest config to package.json; moved tests into src directory

* CR feedback. updated string parsing; updated number parsing

* updated snapshots; minor cleanups
…t type (#3109)

* [Shared Model] Allow images in image set to not specify element type

* [.NET] Allow images in image set to not specify element type

* Fix spacing
* refactored fabric extension interface

* added a readme

* cleaning up fabric file names and package output structure

* API polish, minor renames, webpack tweaks, example page

* Made folders lowercase, matches the conventions in the rest of the repo
* Renamed `usedFabricComponents` to `useFabricControls`
* Changed README function call to `ACFabric.useFabricControls()` since it makes the use consistent when using either Browser/ESM usage. Devs who prefer to import just the function can do so if they choose
* Added minified version with `npm run release`
* Added example page that loads nicely!

* renamed back to `useFabricComponents` after more research

Fabric is still not externalized in webpack since I can't get the reference right. Will keep looking

* bring fabric back into buncle

* added styling to sample page

* add gif to readme, make fabric external, update peer dependency version
* [JS] Hotfix bleed regression in ColumnSet

* cleaning up sample payloads, fixing some site bugs

* site fixes, sample updates, and designer package fixes

* remove unused files

* fix script
…dImage (#3116)

* Add Deserialize from string method to Background image

* Fix formatting

* Fix extra initializer

* Remove extra qualification
* Update tests

* Update test resukts
RebeccaAnne and others added 24 commits January 8, 2020 11:48
[UWP] Fix null dereference in Media code path
* updated the doc

* Updated specs

* updated the description

* Updated specs

* updated the doc to correct wrong info

* Updated specs
* Updating adaptivecards.io homepage's roadmap links to point to the updated productboard portal

* Fixes #3577 ; Fixes #3576:

* Fixing outlook.css since that is the style sheet that is being picked up by default when you hit the explorer page

* automatic package.json changes

* Fixing color syntax
1. Including PreRequisite section around repo bootstrap linking to root readme
2. Cleaning up build commands to use lerna to be consistent with root readme and removing redundant steps
* [UWP] Remove image comparison from unit tests

* Code review feedback
* Added MaxActions limit check and replaced multiple of GetActions() call
That returns a structure by value

* added UnitTesting and Warning

* added UnitTesting and Warning

fixed comments

Co-authored-by: almedina-ms <[email protected]>
Co-authored-by: shalinijoshi19 <[email protected]>
* updating telephony detection for markdown link

* undo uneccesary changes

* updated RichTextBlockRenderer to detect Telephony link

* updated sample card

Co-authored-by: shalinijoshi19 <[email protected]>
* Fix scrolling behaviour in multiline text input

* Add test files

Co-authored-by: shalinijoshi19 <[email protected]>
* [SharedModel] Skip date/time tests outside of Pacific TZ

Fixes #3609

* Rename TEST_MODULE_INITIALIZE method

* Remove TEST_MODULE_INITIALIZE altogether

* Update macro name

Co-authored-by: shalinijoshi19 <[email protected]>
New HostConfig for Cortana Windows UWP beta, light mode. To be added to the drop down as: Cortana (Beta) - Light
* Fix to have short time format

* Change helper function

* Add unit tets

* Add sample test json

* Remove unnecessary test file
* Updated Cover FillMode Behavior

* Revew Comments Addresssed

* adding testing card

* Updated file
* Updated preprocessed text handling

* updated formatting

* Added Accesibility Tags

* updated accessibility

Co-authored-by: shalinijoshi19 <[email protected]>
* Updated iOS layout
* removed wrapper on TextBlock and RichTextBlock renderers
* updated layout views to use intrinsic contentsize
* updated property value for layout vies
* Added key value observation for hidden property to allow toggle
visibility and the new layout change to work together

* Updated views with toggle visibility to update its contentsize based on
their visibility changes
Updated padding priority for bleeding

* restores to working project file

* updated project file for xcode

* updated for CR comments

* Fixed for CR comments

Co-authored-by: shalinijoshi19 <[email protected]>
@ghost
Copy link

ghost commented Feb 5, 2020

Thanks for helping make the AdaptiveCards JS renderer + tooling better. As additional verification, once the JS build succeeds, please go to the test site to test out your website/designer changes

@almedina-ms almedina-ms merged commit 906e13c into user/almedina-ms/PhantomOfTheXamarin Feb 5, 2020
@almedina-ms almedina-ms deleted the user/almedina-ms/PhantomOfTheXamarinBackToTheFuture branch February 5, 2020 01:34
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.