Skip to content

Conversation

@LSantha
Copy link
Collaborator

@LSantha LSantha commented Mar 9, 2021

Created minimal default styling for featured category list component.

Related Issue

CIF-1921

How Has This Been Tested?

Manually.

Screenshots (if appropriate):

fcl

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes and the overall coverage did not decrease.
  • All unit tests pass on CircleCi.
  • I ran all tests locally and they pass.

 * created minimal default styling for featured category list component
@LSantha LSantha added the feature New feature or request label Mar 9, 2021
@codecov
Copy link

codecov bot commented Mar 9, 2021

Codecov Report

Merging #512 (40a846a) into master (5759ca2) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #512   +/-   ##
=========================================
  Coverage     87.58%   87.58%           
  Complexity     1241     1241           
=========================================
  Files           232      232           
  Lines          5889     5889           
  Branches        858      858           
=========================================
  Hits           5158     5158           
  Misses          575      575           
  Partials        156      156           
Flag Coverage Δ Complexity Δ
integration 64.93% <ø> (ø) 0.00 <ø> (ø)
jest 83.33% <ø> (ø) 0.00 <ø> (ø)
karma 94.54% <ø> (ø) 0.00 <ø> (ø)
unittests 87.92% <ø> (ø) 0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 5759ca2...40a846a. Read the comment docs.

justify-content: center;
}

.cmp-categorylist__image {
Copy link
Member

Choose a reason for hiding this comment

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

That kind of styling is not very nice, since we use a background with an image on top. The same can be achieved on the img tag directly using https://www.w3schools.com/css/css3_object-fit.asp.

However, that's how it was built in HTL, so let's stick to it for now.

@laurentiumagureanu laurentiumagureanu merged commit 6933fa7 into master Mar 23, 2021
@laurentiumagureanu laurentiumagureanu deleted the CIF-1921 branch March 23, 2021 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants