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.

Sign up for GitHub

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

Jump to bottom

Add helper text slot and API #170

Closed
juuso-vaadin opened this issue Apr 15, 2020 · 1 comment
Closed

Add helper text slot and API #170

juuso-vaadin opened this issue Apr 15, 2020 · 1 comment
Assignees
@DiegoCardoso
Labels
enhancement New feature or request

Comments

@juuso-vaadin
Copy link

Add slot for displaying text or component related to field.
Baseline implementation has been done for vaadin-text-field. See vaadin/vaadin-text-field#410
All implementations should follow the baseline implementation to achieve the same functionality.

Requirements

  • Add helper-text part which contains slot helper
  • Add observer to add has-helper attribute when helperText property is set
  • Add slotchange event handler to add has-helper attribute when content placed in helper slot
  • Add helper text to aria-describedby
  • Add observer for error message to add has-error-message attribute (used to hide helper in material theme when error message is set)

Visual specifications

image
@juuso-vaadin juuso-vaadin added the enhancement New feature or request label Apr 15, 2020
@DiegoCardoso DiegoCardoso self-assigned this May 27, 2020
DiegoCardoso added a commit that referenced this issue May 27, 2020
@DiegoCardoso
feat: add helper text
f4f34cf 
- Add property and named slot
- Add styles for supported themes

Part of #170
@DiegoCardoso DiegoCardoso mentioned this issue May 27, 2020
Merged
DiegoCardoso added a commit that referenced this issue May 28, 2020
@DiegoCardoso
feat: add helper text API (#174)
97105f7 
- Add property and named slot
- Add styles for supported themes
- Add demo for helper text API
- Add unit and visual tests for helper text

Part of #170
@DiegoCardoso
Copy link
Contributor

Fixed by #174

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DiegoCardoso DiegoCardoso

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DiegoCardoso @juuso-vaadin