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

PER-9446-form input #358

Merged
merged 8 commits into from
Feb 22, 2024
Merged

PER-9446-form input #358

merged 8 commits into from
Feb 22, 2024

Conversation

crisnicandrei
Copy link
Contributor

Implementation of the form input component from the new component library.

I have kept the old implementation and added a way to also use it without a form control, also using a debounce + style changes.

This pr also includes the new fonts for the new design system.

@cecilia-donnelly
Copy link
Member

Looks like the linting is failing, probably because of the conflicts @crisnicandrei -- can you rebase and fix any lint errors? Thanks!

@codecov-commenter
Copy link

Codecov Report

Attention: 28 lines in your changes are missing coverage. Please review.

Comparison is base (b3e0fe9) 35.59% compared to head (07bde88) 35.97%.
Report is 13 commits behind head on main.

Files Patch % Lines
...rary/components/form-input/form-input.component.ts 61.76% 23 Missing and 3 partials ⚠️
src/app/component-library/components.module.ts 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #358      +/-   ##
==========================================
+ Coverage   35.59%   35.97%   +0.38%     
==========================================
  Files         299      303       +4     
  Lines       10379    10480     +101     
  Branches     1719     1742      +23     
==========================================
+ Hits         3694     3770      +76     
- Misses       6540     6561      +21     
- Partials      145      149       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Implementation of the form input component from the new component library.

I have kept the old implementation and added a way to also use it without a form control, also using a debounce + style changes
@crisnicandrei crisnicandrei merged commit 249d510 into main Feb 22, 2024
2 checks passed
@crisnicandrei crisnicandrei deleted the PER-9446-form-input branch February 22, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants