Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Add help tips to label icons [#85] #156

Merged
merged 1 commit into from
Feb 26, 2021
Merged

Conversation

nataliemt
Copy link
Contributor

#85

According with the discussion from the GitHub issue, add messaging to explain when a field is required, optional, or just controlling the appearance of other fields. The message can be displayed onmouseover (:hover) the icons.

After:
Screen Shot 2021-02-23 at 11 24 11
Screen Shot 2021-02-23 at 11 24 16
Screen Shot 2021-02-23 at 11 24 25

@nataliemt nataliemt force-pushed the feature/field-label-tooltip branch 4 times, most recently from c849eb4 to 38704f4 Compare February 23, 2021 18:53
Copy link
Contributor

@gisedaye gisedaye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nataliemt it looks great just a small comment about how I think we should handle the react components.

src/js/components/common/LabelIcon.react.js Outdated Show resolved Hide resolved
src/js/components/RulePicker.react.js Outdated Show resolved Hide resolved
@nataliemt nataliemt force-pushed the feature/field-label-tooltip branch from 38704f4 to 6b3d87e Compare February 26, 2021 13:34
@nataliemt nataliemt requested a review from gisedaye February 26, 2021 13:40
@nataliemt nataliemt force-pushed the feature/field-label-tooltip branch from 6b3d87e to 182b85f Compare February 26, 2021 13:45
Copy link
Contributor

@gisedaye gisedaye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great @nataliemt! Thanks for making the changes.

@nataliemt nataliemt merged commit cddef6f into master Feb 26, 2021
@nataliemt nataliemt deleted the feature/field-label-tooltip branch February 26, 2021 19:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants