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

fix: form messages margin fix for textarea #591

Merged
merged 6 commits into from Jan 20, 2020

Conversation

stefanoScalzo
Copy link
Contributor

Related Issue

Closes #590

Description

Putting margins to make form messages and shadow fiori 3 compliant. Shadow doesn't have the same visuals as UI5 BUT is right according to specs. Did not include input screenshots because since it is behind the border they won't be visible

Screenshots

NOTE: If you've made any style changes, please provide appropriate screenshots (before and after) to help reviewers.

To see examples of which screenshots to include, go to Screenshot Examples.

Before:

Screen Shot 2020-01-16 at 4 27 36 PM

After:

Screen Shot 2020-01-16 at 4 34 16 PM

@stefanoScalzo stefanoScalzo changed the title fix z index and text area margins fix: form messages Jan 16, 2020
@netlify
Copy link

netlify bot commented Jan 16, 2020

Deploy preview for fundamental-styles ready!

Built with commit 58b465d

https://deploy-preview-591--fundamental-styles.netlify.com

@@ -156,27 +156,28 @@ The state of the input field can reflect validity of the data entered, whether t
Along with Invalid and Warning, error messages should be displayed below the field so the user can correct the error and move forward.

{% capture inputs %}
<div class="fd-form-item fd-has-margin-none fd-popover">
<div class="fd-form-item fd-popover">
Copy link
Contributor

Choose a reason for hiding this comment

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

You actually don't need fd-popover. This example doesn't have a message

@InnaAtanasova
Copy link
Contributor

@stefanoScalzo is off Monday so I pushed some changes into his PR (removed classes that we don't need, add cursor: text for textarea.
I think it looks good. @JKMarkowski could you also take a look at it and if it's ok you can merge it.

@droshev droshev added this to the Sprint 28 - Sydney milestone Jan 19, 2020
Copy link
Contributor

@JKMarkowski JKMarkowski left a comment

Choose a reason for hiding this comment

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

It looks good for me

@InnaAtanasova InnaAtanasova changed the title fix: form messages fix: form messages margin fix for textarea Jan 20, 2020
@InnaAtanasova InnaAtanasova merged commit 50c19e9 into master Jan 20, 2020
@InnaAtanasova InnaAtanasova deleted the fix/input-message-groups branch January 20, 2020 13:19
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.

form messages not placed properly
5 participants