Skip to content

fix(pf4): Select styling.#87

Merged
rvsia merged 1 commit into
masterfrom
pf4-single-select-placeholder
Aug 15, 2019
Merged

fix(pf4): Select styling.#87
rvsia merged 1 commit into
masterfrom
pf4-single-select-placeholder

Conversation

@Hyperkid123

@Hyperkid123 Hyperkid123 commented Aug 15, 2019

Copy link
Copy Markdown
Member

Padding on single select was off

Before:
screenshot-data-driven-forms surge sh-2019 08 15-10-04-55

After:
screenshot-localhost-8080-2019 08 15-10-05-17

Fixed clear able button margin:
screenshot-localhost-8080-2019 08 15-10-06-35

@Hyperkid123 Hyperkid123 force-pushed the pf4-single-select-placeholder branch from d10e9ea to ccfa72b Compare August 15, 2019 08:03
@Hyperkid123 Hyperkid123 requested a review from rvsia August 15, 2019 08:05
@Hyperkid123 Hyperkid123 added bug Something isn't working PF4 PF4 pull request labels Aug 15, 2019
@codecov-io

codecov-io commented Aug 15, 2019

Copy link
Copy Markdown

Codecov Report

Merging #87 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #87   +/-   ##
=======================================
  Coverage   83.48%   83.48%           
=======================================
  Files          81       81           
  Lines        1344     1344           
  Branches      318      318           
=======================================
  Hits         1122     1122           
  Misses        191      191           
  Partials       31       31
Impacted Files Coverage Δ
...-component-mapper/src/form-fields/select/select.js 92.5% <ø> (ø) ⬆️
...t-mapper/src/form-fields/select/clear-indicator.js 80% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2531ce8...e877dd4. Read the comment docs.

@Hyperkid123 Hyperkid123 force-pushed the pf4-single-select-placeholder branch from ccfa72b to 25f57cd Compare August 15, 2019 08:11
@rvsia

rvsia commented Aug 15, 2019

Copy link
Copy Markdown
Contributor

The styling does make some strange things if you have a lot of items selected

image

Clearable + searchable

image

Only clearable

image

@Hyperkid123 If this is not a serious issue, then it's good to merge.

@Hyperkid123 Hyperkid123 force-pushed the pf4-single-select-placeholder branch from 25f57cd to e877dd4 Compare August 15, 2019 09:17

@rvsia rvsia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🐠 🏆

@rvsia rvsia merged commit 5edf336 into master Aug 15, 2019
@Hyperkid123 Hyperkid123 deleted the pf4-single-select-placeholder branch August 15, 2019 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working PF4 PF4 pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants