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 category tile presentation #551

Merged
merged 1 commit into from
Nov 27, 2018
Merged

Fix category tile presentation #551

merged 1 commit into from
Nov 27, 2018

Conversation

jimbo
Copy link
Contributor

@jimbo jimbo commented Nov 27, 2018

This PR is a:

  • New feature
  • Enhancement/Optimization
  • Refactor
  • Bugfix
  • Test for existing code
  • Documentation

Summary

When this pull request is merged, it will fix the presentation of the category tiles.

Additional information

Tiles should use background-image, so that we can use CSS to control the presentation and make use of imperfect images.

max-width: 100%;
width: auto;
height: 100%;
opacity: 0;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hide the image, but not from screen readers. It should be there, and if you right click the tile, the context menu should give you actions related to the image element (e.g., View image).

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 17.777% when pulling 0f304fd on jimbo/category-tiles into df340e1 on release/2.0.

@@ -6,22 +6,21 @@
}

.imageWrapper {
background-image: var(--venia-image);
background-position: 50% 50%;
background-size: cover;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is what we want.

@jimbo jimbo requested a review from zetlen November 27, 2018 18:47
Copy link
Contributor

@zetlen zetlen left a comment

Choose a reason for hiding this comment

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

👍 lgtm

@zetlen zetlen merged commit 3f3a48a into release/2.0 Nov 27, 2018
@supernova-at supernova-at deleted the jimbo/category-tiles branch July 25, 2019 15:17
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.

3 participants