Skip to content

Conversation

@Zaidos9
Copy link

@Zaidos9 Zaidos9 commented Aug 26, 2022

Description

Hyperlinks are no-op when added in a Bulleted List. This PR fixes this issue and enables hyperlink clicks.
Changes include adding link for hyperlink text (this was not getting added earlier), and then applying onPress for bulleted style as well.

Sample Card

Added bulletedHyperlink.json payload for testing the scenario

How Verified

Added videos for before and after.

Before:

BeforeBulletedText.mov

After:

AfterBulletedText1.mov

@Vasanth-S Vasanth-S requested a review from dan-arvi August 30, 2022 09:32
Copy link

@pragadeeshk pragadeeshk 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

@pragadeeshk pragadeeshk merged commit acd3ac3 into reactnative/release-2.2.27 Sep 7, 2022
@pragadeeshk pragadeeshk deleted the zaidos9/bulletedHyperlinkFix branch September 7, 2022 09:26
pragadeeshk added a commit that referenced this pull request Sep 13, 2022
* Fixed visualizer build issue for android (#90)

Co-authored-by: “dan-arvi” <“daniel.arvind@bigthinkcode.com”>

* Fix Hyperlink not working in a Bulleted List (#101)

* bulleted text hyperlink fix

* onpress

Co-authored-by: Mohd Zaid <zanaseem@microsoft.com>

* React native version update to 0.68.2 (#93)

* Fixed visualizer build issue for android

* react native version updated to 0.68.2

* Version updated for Picker&datetimePicker

Co-authored-by: “dan-arvi” <“daniel.arvind@bigthinkcode.com”>

* Version bump

Co-authored-by: dan-arvi <106525930+dan-arvi@users.noreply.github.com>
Co-authored-by: “dan-arvi” <“daniel.arvind@bigthinkcode.com”>
Co-authored-by: Zaid Naseem <72800429+Zaidos9@users.noreply.github.com>
Co-authored-by: Mohd Zaid <zanaseem@microsoft.com>
pragadeeshk added a commit that referenced this pull request Sep 27, 2022
* Set text as accessibility label when altText is not provided

* Added accessibility focus for image component

* Fixed visualizer build issue for android (#90)

Co-authored-by: “dan-arvi” <“daniel.arvind@bigthinkcode.com”>

* Fix Hyperlink not working in a Bulleted List (#101)

* bulleted text hyperlink fix

* onpress

Co-authored-by: Mohd Zaid <zanaseem@microsoft.com>

* React native version update to 0.68.2 (#93)

* Fixed visualizer build issue for android

* react native version updated to 0.68.2

* Version updated for Picker&datetimePicker

Co-authored-by: “dan-arvi” <“daniel.arvind@bigthinkcode.com”>

* Version bump

* Fixed alignment issue due to accessibility fix

Co-authored-by: dan-arvi <106525930+dan-arvi@users.noreply.github.com>
Co-authored-by: “dan-arvi” <“daniel.arvind@bigthinkcode.com”>
Co-authored-by: Zaid Naseem <72800429+Zaidos9@users.noreply.github.com>
Co-authored-by: Mohd Zaid <zanaseem@microsoft.com>
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.

3 participants