Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

IBX-4233: Form Inputs - space between fields #796

Merged
merged 5 commits into from
Jun 14, 2023

Conversation

@GrabowskiM GrabowskiM force-pushed the IBX-4233-form-inputs-margins branch from d101e94 to b992475 Compare May 16, 2023 10:35
Co-authored-by: Łukasz Ostafin <ostafin.lucas@gmail.com>
Copy link
Contributor

@katarzynazawada katarzynazawada left a comment

Choose a reason for hiding this comment

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

@GrabowskiM I found few places that need fix

  1. Form builder - validation tab
Screenshot 2023-05-26 at 11 25 33
  1. User account field
Screenshot 2023-05-26 at 11 27 37
  1. Headers e.g.
Screenshot 2023-05-26 at 13 21 03 Screenshot 2023-05-26 at 13 24 05
  1. Missing empty space at the bottom of the form
Screenshot 2023-05-26 at 13 27 26
  1. PB field settings https://recordit.co/zkueyOJ3MS

@GrabowskiM
Copy link
Contributor Author

@GrabowskiM I found few places that need fix

  1. Form builder - validation tab
Screenshot 2023-05-26 at 11 25 33 2. User account field Screenshot 2023-05-26 at 11 27 37 3. Headers e.g.

Screenshot 2023-05-26 at 13 21 03 Screenshot 2023-05-26 at 13 24 05
4. Missing empty space at the bottom of the form

Screenshot 2023-05-26 at 13 27 26 5. PB field settings https://recordit.co/zkueyOJ3MS

Fixed

@katarzynazawada
Copy link
Contributor

@dew326, should this now go to 4.5 as 4.4 is not supported(EOM in May)?

@dew326
Copy link
Contributor

dew326 commented Jun 2, 2023

I think we should do one last release for 4.4 to include all fixes that were merged in May. But this is more to @webhdx

Copy link
Contributor

@micszo micszo left a comment

Choose a reason for hiding this comment

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

Issues reported by Kasia have been corrected. 👍🏻

One thing to double-check, amount of empty space in create forms:

Screenshot 2023-06-13 at 12 31 02
Screenshot 2023-06-13 at 12 33 29
Screenshot 2023-06-13 at 12 37 10
Screenshot 2023-06-13 at 12 41 47
Screenshot 2023-06-13 at 12 49 50
Screenshot 2023-06-13 at 12 50 45

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
1.9% 1.9% Duplication

@dew326 dew326 merged commit 1c82c06 into 4.4 Jun 14, 2023
@dew326 dew326 deleted the IBX-4233-form-inputs-margins branch June 14, 2023 10:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants