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

Fix line breaks in homepage links #3502

Merged
merged 1 commit into from
Jun 10, 2021
Merged

Fix line breaks in homepage links #3502

merged 1 commit into from
Jun 10, 2021

Conversation

tugbadogan
Copy link
Contributor

@roryabraham @arielgreen

Details

Fixed the line break after the privacy policy link and before licences links by putting them into a Text component. This change affects only web and mobile web.

Fixed Issues

Fixes #3100

Tests

Verified terms of service and privacy policy links are accessible via keyboard from Safari and Chrome on MacOS when VoiceOver is enabled.

QA Steps

  • Enable VoiceOver on MacOS
  • Open e.cash website on a browser
  • Verify that there is no line break after the privacy policy link and before licences links
  • Navigate on the webpage via arrow keys and check if links are clickable.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Screen.Recording.2021-06-10.at.00.11.55.mov

@tugbadogan tugbadogan requested a review from a team as a code owner June 9, 2021 23:21
@MelvinBot MelvinBot requested review from francoisl and removed request for a team June 9, 2021 23:21
@francoisl francoisl merged commit 0d3ee13 into Expensify:main Jun 10, 2021
@OSBotify
Copy link
Contributor

🚀 Deployed to staging in version: 1.0.66-13🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@pranshuchittora
Copy link
Contributor

@francoisl #3079 (comment)

@tugbadogan tugbadogan mentioned this pull request Jun 11, 2021
5 tasks
@OSBotify
Copy link
Contributor

🚀 Deployed to production in version: 1.0.68-4🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@tugbadogan tugbadogan deleted the tugbadogan-fix-textlink-regression branch July 10, 2021 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants