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

DS-680 Select icon placement bug #2393

Merged
merged 15 commits into from
Dec 9, 2021

Conversation

MarcinMr
Copy link
Collaborator

Jira

https://pegadigitalit.atlassian.net/browse/DS-680

Summary

The chevrons in the select filters within the Best of Content pages were fixed.

Details

On Best of Content pages, select chevron icons were overlaping the text :

Additionally, on the Calculator - Results I found accidentally a bug related to color theming on those two colorful rectangles. I made a quick fix.

How to test

Pull the branch. Check if the select fields on the BoC pages display correctly, without overlapping the text. Confirm there're no more broken selects with chevron icons overlapping the text.

@github-actions github-actions bot added the type: bugfix List this PR in the 'Bug Fixes' section of the release notes. label Nov 29, 2021
@colbytcook colbytcook temporarily deployed to bugfix/DS-680-select-icon-placement-bug--branch-preview November 29, 2021 14:25 Inactive
@colbytcook colbytcook temporarily deployed to bugfix/DS-680-select-icon-placement-bug--branch-preview November 29, 2021 15:04 Inactive
@colbytcook colbytcook requested a deployment to bugfix/DS-680-select-icon-placement-bug--branch-preview December 1, 2021 16:20 In progress
@colbytcook colbytcook temporarily deployed to bugfix/DS-680-select-icon-placement-bug--branch-preview December 1, 2021 17:03 Inactive
@MarcinMr
Copy link
Collaborator Author

MarcinMr commented Dec 2, 2021

@colbytcook went over it again today to fix the issue mentioned in ticket DS-680

@colbytcook colbytcook temporarily deployed to bugfix/DS-680-select-icon-placement-bug--branch-preview December 2, 2021 09:18 Inactive
Copy link
Collaborator

@mikemai2awesome mikemai2awesome left a comment

Choose a reason for hiding this comment

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

This is a component bug, we shouldn't just add utility classes to the mockups but fix the component instead.

@MarcinMr
Copy link
Collaborator Author

MarcinMr commented Dec 7, 2021

This is a component bug, we shouldn't just add utility classes to the mockups but fix the component instead.

@mikemai2awesome if we cannot use utility classes like for example u-bolt-visuallyhidden and others, I added new display type for a label which uses visually hidden mixin and updated the padding in a form.scss to prevent overlapping text over the chevron

@colbytcook colbytcook temporarily deployed to bugfix/DS-680-select-icon-placement-bug--branch-preview December 7, 2021 12:52 Inactive
Copy link
Collaborator

@mikemai2awesome mikemai2awesome left a comment

Choose a reason for hiding this comment

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

@MarcinMr We can't just target all inputs, this is specific to select, so you'd need to scope it:

.c-bolt-block-label,
.c-bolt-hidden-label {
  .c-bolt-input.is-filled,
  .c-bolt-input:focus {
    padding: calc(var(--bolt-spacing-y-medium) / 2 - 1px)
      var(--bolt-spacing-x-small); // [Mai] medium/2 matches button vertical spacing.

    @include bolt-mq($until: small) {
      padding-top: calc(var(--bolt-spacing-y-medium) / 2 - 0.1875rem);
      padding-bottom: calc(var(--bolt-spacing-y-medium) / 2 - 0.25rem);
    }
  }

  .c-bolt-input--select.is-filled,
  .c-bolt-input--select.focus {
     padding-right: calc(var(--bolt-spacing-x-small) + 1.5rem);
  }
}

Also update instances of 24px to 1.5rem, there is a comment in there reminding us to update: 24px is icon size, if icon component changes, this needs to update as well..

@MarcinMr
Copy link
Collaborator Author

MarcinMr commented Dec 8, 2021

@mikemai2awesome updated 24px to 1.5rem and used fixing styles only for the select element.

@colbytcook colbytcook temporarily deployed to bugfix/DS-680-select-icon-placement-bug--aef42ccd--commit-preview December 8, 2021 10:53 Inactive
Copy link
Collaborator

@mikemai2awesome mikemai2awesome left a comment

Choose a reason for hiding this comment

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

@MarcinMr Looks good.

@colbytcook colbytcook merged commit c737d3d into master Dec 9, 2021
@colbytcook colbytcook deleted the bugfix/DS-680-select-icon-placement-bug branch December 9, 2021 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bugfix List this PR in the 'Bug Fixes' section of the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants