Skip to content
This repository has been archived by the owner on Feb 23, 2022. It is now read-only.

SpecialText #437

Open
wants to merge 74 commits into
base: master
Choose a base branch
from
Open

SpecialText #437

wants to merge 74 commits into from

Conversation

ericlin11354
Copy link
Contributor

@ericlin11354 ericlin11354 commented Jan 22, 2022

Here is a demo:
https://user-images.githubusercontent.com/50276107/150618022-1ec81aa5-4b5c-4ce0-a4f7-994dea69be35.mp4
Comparing to HighlightedText, differences include:

  • Removed isSpecial, textProps, listProps, listItemArgs, age
  • Renamed listItemsBodies to children.
  • Treated as a single word rather than a list (i.e. no opacity feature).
    -No longer passes explicitly passes props
  • No longer uses ClickableSmallText, MainTheme, TextLayoutProps, IDropdownProps, IDropdownItemProps, useEffect, useState.

Changed SpecialText to be an atomic component.

Here is a demo of the SpecialText component.
SpecialText is a highlighted SmallText with onClick functionality.
Here is a WIP demo of the SpecialText component used with Dropdown in the VUI.
Currently replacing HighlightedText with SpecialText and cleaning up the VUI. Will get a PR out for review by today evening.

@ericlin11354 ericlin11354 self-assigned this Jan 22, 2022
src/Containers/SpecialText/SpecialText.tsx Outdated Show resolved Hide resolved
src/Containers/SpecialText/SpecialText.tsx Outdated Show resolved Hide resolved
src/Containers/SpecialText/SpecialText.tsx Outdated Show resolved Hide resolved
src/Containers/SpecialText/SpecialText.tsx Outdated Show resolved Hide resolved
src/Containers/SpecialText/SpecialText.tsx Outdated Show resolved Hide resolved
@ericlin11354 ericlin11354 marked this pull request as draft January 22, 2022 18:17
@ericlin11354 ericlin11354 marked this pull request as ready for review January 28, 2022 09:01
@ericlin11354
Copy link
Contributor Author

Updated PR Description.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants