techmatters / terraso-mobile-client

A React Native project for Terraso’s LandPKS mobile application.
https://landpks.terraso.org
GNU Affero General Public License v3.0
2 stars 0 forks source link

feat: tooltip component design updates #2499

Closed shrouxm closed 1 day ago

shrouxm commented 3 days ago

Description

Updates the Tooltip component to match the new design spec. Waiting on confirmation of a couple things in this Figma thread before fully ready to merge.

Checklist

Related Issues

Related to #1776

Verification steps

Tooltip component should look like the design. I believe the only example right now is on the ? on the create site screen.

paulschreiber commented 3 days ago

@shrouxm do we want to use this as an opportunity to get rid of NativeBase in tooltips?

shrouxm commented 2 days ago

react-native-paper doesn't offer a tooltip component which is able to match the spec, so it would increase the effort required to find and vet another library and/or implement it ourselves. this doesn't further entrench us in nativebase and it at least gets us compliant with the spec, so i don't see any reason to couple that work together.