-
Notifications
You must be signed in to change notification settings - Fork 125
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
feat: form messages adoption from styles 4.0 #1827
Conversation
cb4f519
to
6183a10
Compare
Deploy preview for fundamental-ngx ready! Built with commit cb4f519 |
Deploy preview for fundamental-ngx ready! Built with commit a1536a6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good Job @stefanoScalzo, please solve issues from comments and we can merge it.
libs/core/src/lib/form/form-input-message-group/form-input-message-group.component.ts
Outdated
Show resolved
Hide resolved
libs/core/src/lib/form/form-input-message-group/form-input-message-group.component.ts
Outdated
Show resolved
Hide resolved
libs/core/src/lib/form/form-input-message-group/form-input-message-group.component.ts
Show resolved
Hide resolved
58c9d93
to
792cfa2
Compare
765ce2c
to
f55d021
Compare
f55d021
to
5dcd266
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I included some changes, new inputs for popover
control and documentation fixes.
* mmake messages popover for angular * add text area * blank * remove scss popover * resolve comments * remove ps * Apply PR comments fixes, add more message options, change comments * Minor import fix * Add new component to api files Co-authored-by: jmarkowski <jdrekmarkowski@gmail.com>
Please provide a link to the associated issue.
#1811
Please provide a brief summary of this pull request.
Adding the new form-input-message-group component which uses the fd-form-input-message class.
Added input for the fd-popover for the popover container since the div might need extra classes added for whatever reasons. In this case I needed it for the width of the inputs.
Note: there are styling errors in text area which will be fixed in a pr in fundamental styles such as the z-index for text areas and the margins for both inputs and textareas
Please check whether the PR fulfills the following requirements
https://github.com/SAP/fundamental-ngx/blob/master/CONTRIBUTING.md
Documentation checklist:
README.md