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

Adobe/dev to master #948

Closed
wants to merge 29 commits into from
Closed

Adobe/dev to master #948

wants to merge 29 commits into from

Conversation

barshat7
Copy link
Contributor

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes and the overall coverage did not decrease.
  • All unit tests pass on CircleCi.
  • I ran all tests locally and they pass.

ravise5 and others added 29 commits October 3, 2023 18:00
* Update af-core version

* Fixed test cases
* FORMS-3696 adding rich text support for labels

FORMS-3696 adding rich text support for labels

Co-authored-by: Shivam Agarwal <[email protected]>

* FORMS-3696 adding test for deprecated method

* FORMS-3696 adding inline editor logic

* FORMS-3696 addressed review comments

* FORMS-3696 sorting options constraint

* FORMS-3696 adding some more test cases

---------

Co-authored-by: Shivam Agarwal <[email protected]>
* fix multiple form embedded issue in site page

* add repeatability test

---------

Co-authored-by: Amit Kumar <[email protected]>
* FORMS-11284 fixing enum name display on hide

* FORMS-11284 adding test case

---------

Co-authored-by: Shivam Agarwal <[email protected]>
* Terms and Conditions

Terms and Conditions Component

Terms and Conditions Component

Terms and Conditions component

Terms and conditions: code formatting and readme

Terms and conditions tooltip

Terms and conditions review comments

Terms and conditions review comments

Terms and Conditions Test Case Fix

Terms and Conditions added new tests

Terms and Conditions review comments

Terms and Conditions component

* Terms and Conditions

* Terms and Conditions

* Terms and conditions: Rich Text Support

* Terms and conditions: bump model version

---------

Co-authored-by: barshatr <[email protected]>
* Added Switch component and resolved PR comments

* Resolved PR comments

Abstracted nitialize common logic for switch and checkbox
Added test cases
Fixed bug with enableUncheckedValue sync after save
updated js docs and java docs
fixed bem
updated abstraction logic for switch and checkbox

* skipped temporarily tests due to dialog bugs

* Refactored code. ternery operator instead of if else
fixed bem
fixed flaky tests
bumped package-info version

* FORMS-11284 fixing enum name display on hide (#911)

* FORMS-11284 fixing enum name display on hide

* FORMS-11284 adding test case

Co-authored-by: Shivam Agarwal <[email protected]>

---------

Co-authored-by: Gulshan Mishra <[email protected]>
Co-authored-by: Shivam Agarwal <[email protected]>
Co-authored-by: Shivam Agarwal <[email protected]>
* FORMS-10078: updated default pattern to align with v1

* FORMS-10078: resolved PR comments

---------

Co-authored-by: Gulshan Mishra <[email protected]>
* Supporting qualified name for accordion, wizard, panel etc

* Supporting more core components

* Tests for qualified name

* Supporting qualified name for accordion, wizard, panel etc

* Supporting more core components

* Tests for qualified name

* Removing jquery from qualified name hook

* Enabling other tests for qualified name

* Button component support

* Alternate way of handling async call to allow sites use case to work

* Unclean - tried to re-use sites tests

* Tests with sites

* Fixing flaky tests

* Updated aem-forms-sdk-api version after public release

* Fixed typo in test case

* Updated test with addon check
Example Canvas Themes for CheckbBox and TermsAndConditions
… 'setFocus' is triggered on the Text Component. (#920)

* FORMS-11276 : Added scroll feature after setFocus

* FORMS-11276 : new file added for testcase

* FORMS-11276 : changes due to test cases failure

* FORMS-11276 : form added
* all the components were getting scrolled when clicked, fix

* setfocus fix on view layer for checkboxgroup and radio buttons

---------

Co-authored-by: barshatr <[email protected]>
@barshat7
Copy link
Contributor Author

Closing for now, waiting for some fixes in dev

@barshat7 barshat7 closed this Oct 25, 2023
@adobe-bot
Copy link

Lighthouse scores (desktop)

Performance Accessibility Best-Practices SEO
Scores 100 100 100 73

@adobe-bot
Copy link

Lighthouse scores (mobile)

Performance Accessibility Best-Practices SEO
Scores 92 100 100 73

@adobe-bot
Copy link

Accessibility Violations Found

Id Impact
label-title-only serious
landmark-one-main moderate
region moderate

@barshat7 barshat7 deleted the adobe/devToMaster branch October 25, 2023 11:05
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.